On 11.03.2025 22:10, Andrew Cooper wrote: > --- a/xen/arch/x86/domain.c > +++ b/xen/arch/x86/domain.c > @@ -1211,10 +1211,10 @@ int arch_set_info_guest( > v->arch.user_regs.rflags = c.nat->user_regs.rflags; > v->arch.user_regs.rsp = c.nat->user_regs.rsp; > v->arch.user_regs.ss = c.nat->user_regs.ss; > - v->arch.user_regs.es = c.nat->user_regs.es; > - v->arch.user_regs.ds = c.nat->user_regs.ds; > - v->arch.user_regs.fs = c.nat->user_regs.fs; > - v->arch.user_regs.gs = c.nat->user_regs.gs; > + v->arch.pv.es = c.nat->user_regs.es; > + v->arch.pv.ds = c.nat->user_regs.ds; > + v->arch.pv.fs = c.nat->user_regs.fs; > + v->arch.pv.gs = c.nat->user_regs.gs; > > if ( is_pv_domain(d) ) > memcpy(v->arch.pv.trap_ctxt, c.nat->trap_ctxt, > @@ -1238,10 +1238,10 @@ int arch_set_info_guest( > v->arch.user_regs.eflags = c.cmp->user_regs.eflags; > v->arch.user_regs.esp = c.cmp->user_regs.esp; > v->arch.user_regs.ss = c.cmp->user_regs.ss; > - v->arch.user_regs.es = c.cmp->user_regs.es; > - v->arch.user_regs.ds = c.cmp->user_regs.ds; > - v->arch.user_regs.fs = c.cmp->user_regs.fs; > - v->arch.user_regs.gs = c.cmp->user_regs.gs; > + v->arch.pv.es = c.nat->user_regs.es; > + v->arch.pv.ds = c.nat->user_regs.ds; > + v->arch.pv.fs = c.nat->user_regs.fs; > + v->arch.pv.gs = c.nat->user_regs.gs;
I assume you mean c.cmp-> here. Then: Reviewed-by: Jan Beulich <jbeul...@suse.com> Jan
