On Fri, Jul 16, 2021, Vitaly Kuznetsov wrote: > diff --git a/arch/x86/kvm/svm/nested.c b/arch/x86/kvm/svm/nested.c > index 3bd09c50c98b..8493592b63b4 100644 > --- a/arch/x86/kvm/svm/nested.c > +++ b/arch/x86/kvm/svm/nested.c > @@ -722,7 +722,7 @@ void svm_copy_vmrun_state(struct vmcb_save_area *from_save, > to_save->cpl = 0; > } > > -void nested_svm_vmloadsave(struct vmcb *from_vmcb, struct vmcb *to_vmcb) > +void svm_copy_vmloadsave_state(struct vmcb *from_vmcb, struct vmcb *to_vmcb) And swap the parameter order for both functions in a follow-up patch? I.e. have the destination first to match memcpy().