>>> On 15.08.16 at 01:07, <[email protected]> wrote: > That way common code can use the same macro to access > the most common attributes without much #ifdef. > > Take advantage of it right away in the livepatch code. > > Signed-off-by: Konrad Rzeszutek Wilk <[email protected]>
x86 part Acked-by: Jan Beulich <[email protected]> with one adjustment: > --- a/xen/include/asm-x86/alternative.h > +++ b/xen/include/asm-x86/alternative.h > @@ -23,6 +23,10 @@ struct alt_instr { > u8 replacementlen; /* length of new instruction, <= instrlen */ > }; > > +#define __ALT_PTR(a,f) (u8 *)((void *)&(a)->f + (a)->f) This needs another set of parentheses around the entire expression (also in the ARM variant). And I'd like to also note that from an x86 perspective the "Move" in the title isn't really correct. Jan _______________________________________________ Xen-devel mailing list [email protected] https://lists.xen.org/xen-devel
