Re: [Xen-devel] [PATCH] xen/arm: alternative: Make it possible to patch outside of the hypervisor

[Julien Grall <julien.grall@xxxxxxx>]

Hi Konrad,

On 07/09/2016 04:06, Konrad Rzeszutek Wilk wrote:
> > > ---
> > >  xen/arch/arm/alternative.c | 58 +++++++++++++++++++++++++---------------------
> > >  1 file changed, 31 insertions(+), 27 deletions(-)
> > >
> > > diff --git a/xen/arch/arm/alternative.c b/xen/arch/arm/alternative.c
> > > index 8ee5a11..db4bd0f 100644
> > > --- a/xen/arch/arm/alternative.c
> > > +++ b/xen/arch/arm/alternative.c
> > > @@ -97,26 +97,11 @@ static u32 get_alt_insn(const struct alt_instr *alt,
> > >  static int __apply_alternatives(const struct alt_region *region)
> > >  {
> > >      const struct alt_instr *alt;
> > > -    const u32 *origptr, *replptr;
> > > -    u32 *writeptr, *writemap;
> > > -    mfn_t text_mfn = _mfn(virt_to_mfn(_stext));
> > > -    unsigned int text_order = get_order_from_bytes(_end - _start);
> > > +    const u32 *replptr;
> > > +    u32 *origptr;
> > >
> > >      printk(XENLOG_INFO "alternatives: Patching kernel code\n");
> This is still there. Should it be "Patching %p->%p code" instead?
I will replace by
printk(XENLOG_INFO "alternatives: alt table %p -> %p", region->begin, 
region->end);
Regards,

--
Julien Grall

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
https://lists.xen.org/xen-devel