Re: [PATCH v11 10/13] KVM: x86: Introduce new KVM_FEATURE_SEV_LIVE_MIGRATION feature & Custom MSR.

[Paolo Bonzini]

On 06/04/21 15:26, Ashish Kalra wrote:
> > It's a little unintuitive to see KVM_MSR_RET_FILTERED here, since
> > userspace can make this happen on its own without having an entry in
> > this switch statement (by setting it in the msr filter bitmaps). When
> > using MSR filters, I would only expect to get MSR filter exits for
> > MSRs I specifically asked for.
> >
> > Not a huge deal, just a little unintuitive. I'm not sure other options
> > are much better (you could put KVM_MSR_RET_INVALID, or you could just
> > not have these entries in svm_{get,set}_msr).
> Actually KVM_MSR_RET_FILTERED seems more logical to use, especially in
> comparison with KVM_MSR_RET_INVALID.
>
> Also, hooking this msr in svm_{get,set}_msr allows some in-kernel error
> pre-processsing before doing the pass-through to userspace.

I agree that it should be up to userspace to set up the filter since we 
now have that functionality.

Let me read the whole threads for the past versions to see what the 
objections were...

Paolo