On Mon, Oct 30, 2023, Paolo Bonzini wrote:
> On 10/27/23 20:21, Sean Christopherson wrote:
> > @@ -635,6 +635,13 @@ static __always_inline kvm_mn_ret_t 
> > __kvm_handle_hva_range(struct kvm *kvm,
> >                      * the second or later invocation of the handler).
> >                      */
> >                     gfn_range.arg = range->arg;
> > +
> > +                   /*
> > +                    * HVA-based notifications aren't relevant to private
> > +                    * mappings as they don't have a userspace mapping.
> 
> It's confusing who "they" is.  Maybe
> 
>        * HVA-based notifications provide a userspace address,
>        * and as such are only relevant for shared mappings.

Works for me.

Reply via email to