Let's avoid such open-coding. There's also no need to use guest_cpu_user_regs(), when the function has a suitable parameter.
Signed-off-by: Jan Beulich <jbeul...@suse.com> --- a/xen/arch/x86/hvm/vmx/vvmx.c +++ b/xen/arch/x86/hvm/vmx/vvmx.c @@ -2675,7 +2675,7 @@ int nvmx_n2_vmexit_handler(struct cpu_us { case VMX_CR_ACCESS_TYPE_MOV_TO_CR: { - val = *decode_gpr(guest_cpu_user_regs(), qual.gpr); + val = reg_read(regs, qual.gpr); if ( qual.cr == 0 ) {