On Wed, Nov 8, 2023 at 12:10 PM Sean Christopherson <seanjc@xxxxxxxxxx> wrote: > > On Wed, Nov 08, 2023, Peter Xu wrote: > > On Wed, Nov 08, 2023 at 08:56:22AM -0800, David Matlack wrote: > > > Thanks for the longer explanation. Yes kvm_read_guest() eventually > > > calls __copy_from_user() which will trigger a page fault and > > > UserfaultFD will notify userspace and wait for the page to become > > > present. In the KVM-specific proposal I outlined, calling > > > kvm_read_guest() will ultimately result in a check of the VM's present > > > bitmap and KVM will nnotify userspace and wait for the page to become > > > present if it's not, before calling __copy_from_user(). So I don't > > > expect a KVM-specific solution to have any increased maintenance > > > burden for VGIC (or any other widgets). > > > > The question is how to support modules that do not use kvm apis at all, > > like vhost. I raised the question in my initial reply, too. > > > > I think if vhost is going to support gmemfd, it'll need new apis so maybe > > there'll be a chance to take that into account, but I'm not 100% sure it'll > > be the same complexity, also not sure if that's the plan even for CoCo. > > > > Or is anything like vhost not considered to be supported for gmemfd at all? > > vhost shouldn't require new APIs. To support vhost, guest_memfd would first need > to support virtio for host userspace, i.e. would need to support .mmap(). At that > point, all of the uaccess and gup() stuff in vhost should work without modification. Does this imply the need for some case-specific annotations for the proposed KVM demand paging implementation to "catch" these accesses? IIUC this was one of the larger downsides to our internal implementation, and something David had hoped to avoid in his RFC proposal. Whereas, I think this is a point in UFFD's favor, where all accesses "just work" with one centralized check in the fault handler path.
