Re: [PATCH v4 1/3] KVM: SVM: use vmsave/vmload for saving/restoring additional host state

[Paolo Bonzini]

On 03/02/21 01:38, Sean Christopherson wrote:
> >   
> > +static inline void vmload(hpa_t pa)
> This needs to be 'unsigned long', using 'hpa_t' in vmsave() is wrong as the
> instructions consume rAX based on effective address.  I wrote the function
> comment for the vmsave() fix so that it applies to both VMSAVE and VMLOAD,
> so this can be a simple fixup on application (assuming v5 isn't needed for
> other reasons).
>
> https://lkml.kernel.org/r/20210202223416.2702336-1-seanjc@xxxxxxxxxx

Yup, fixed.

Paolo