On Fri, Aug 17, 2018 at 04:12:24PM +0100, Wei Liu wrote: > The one in context_switch is fixed by annotating with __maybe_unused > because I want to keep prevd around. > > The other is fixed by eliminating v. > > Signed-off-by: Wei Liu <wei.l...@citrix.com>
Reviewed-by: Roger Pau Monné <roger....@citrix.com> > --- > xen/arch/x86/domain.c | 3 ++- > xen/arch/x86/mm/shadow/common.c | 3 +-- > 2 files changed, 3 insertions(+), 3 deletions(-) > > diff --git a/xen/arch/x86/domain.c b/xen/arch/x86/domain.c > index 5bb900e..574bdf0 100644 > --- a/xen/arch/x86/domain.c > +++ b/xen/arch/x86/domain.c > @@ -1671,7 +1671,8 @@ static void __context_switch(void) > void context_switch(struct vcpu *prev, struct vcpu *next) > { > unsigned int cpu = smp_processor_id(); > - const struct domain *prevd = prev->domain, *nextd = next->domain; > + const struct domain * __maybe_unused prevd = prev->domain, > + *nextd = next->domain; Could you align *nextd to prevd? Roger. _______________________________________________ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel