Message ID | 20250430165655.605595-1-tabba@google.com |
---|---|
Headers | show |
Series | KVM: Mapping guest_memfd backed memory at the host for software protected VMs | expand |
Fuad Tabba wrote: > The option KVM_PRIVATE_MEM enables guest_memfd in general. Subsequent > patches add shared memory support to guest_memfd. Therefore, rename it > to KVM_GMEM to make its purpose clearer. > > Co-developed-by: David Hildenbrand <david@redhat.com> > Signed-off-by: David Hildenbrand <david@redhat.com> > Signed-off-by: Fuad Tabba <tabba@google.com> Reviewed-by: Ira Weiny <ira.weiny@intel.com>
Fuad Tabba wrote: > The function kvm_arch_has_private_mem() is used to indicate whether > guest_memfd is supported by the architecture, which until now implies > that its private. To decouple guest_memfd support from whether the > memory is private, rename this function to kvm_arch_supports_gmem(). > > Co-developed-by: David Hildenbrand <david@redhat.com> > Signed-off-by: David Hildenbrand <david@redhat.com> > Signed-off-by: Fuad Tabba <tabba@google.com> Reviewed-by: Ira Weiny <ira.weiny@intel.com> [snip]
Fuad Tabba wrote: > The function kvm_slot_can_be_private() is used to check whether a memory > slot is backed by guest_memfd. Rename it to kvm_slot_has_gmem() to make > that clearer and to decouple memory being private from guest_memfd. > > Co-developed-by: David Hildenbrand <david@redhat.com> > Signed-off-by: David Hildenbrand <david@redhat.com> > Signed-off-by: Fuad Tabba <tabba@google.com> Reviewed-by: Ira Weiny <ira.weiny@intel.com> [snip]
Fuad Tabba wrote: > Fix comments so that they refer to slots_lock instead of slots_locks > (remove trailing s). > > Signed-off-by: Fuad Tabba <tabba@google.com> Reviewed-by: Ira Weiny <ira.weiny@intel.com> [snip]
On Thu, 1 May 2025 at 22:43, Ira Weiny <ira.weiny@intel.com> wrote: > > Fuad Tabba wrote: > > Fix comments so that they refer to slots_lock instead of slots_locks > > (remove trailing s). > > > > Signed-off-by: Fuad Tabba <tabba@google.com> > > Reviewed-by: Ira Weiny <ira.weiny@intel.com> Thank you for the reviews! /fuad > [snip]