On 17/03/2025 11:58 am, Jan Beulich wrote: > 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 <[email protected]>
Oops, yes I do. Thanks. ~Andrew
