lkml.org 
[lkml]   [2023]   [Jul]   [21]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    SubjectRe: [RFC PATCH v11 06/29] KVM: Introduce KVM_SET_USER_MEMORY_REGION2
    From
    On 7/19/23 01:44, Sean Christopherson wrote:
    > Cc: Jarkko Sakkinen <jarkko@kernel.org>
    > Signed-off-by: Sean Christopherson <seanjc@google.com>
    > ---
    > arch/x86/kvm/x86.c | 2 +-
    > include/linux/kvm_host.h | 4 ++--
    > include/uapi/linux/kvm.h | 13 +++++++++++++
    > virt/kvm/kvm_main.c | 38 ++++++++++++++++++++++++++++++--------
    > 4 files changed, 46 insertions(+), 11 deletions(-)
    >
    > diff --git a/arch/x86/kvm/x86.c b/arch/x86/kvm/x86.c
    > index a6b9bea62fb8..92e77afd3ffd 100644
    > --- a/arch/x86/kvm/x86.c
    > +++ b/arch/x86/kvm/x86.c
    > @@ -12420,7 +12420,7 @@ void __user * __x86_set_memory_region(struct kvm *kvm, int id, gpa_t gpa,
    > }
    >
    > for (i = 0; i < KVM_ADDRESS_SPACE_NUM; i++) {
    > - struct kvm_userspace_memory_region m;
    > + struct kvm_userspace_memory_region2 m;
    >
    > m.slot = id | (i << 16);
    > m.flags = 0;
    > diff --git a/include/linux/kvm_host.h b/include/linux/kvm_host.h
    > index d2d3e083ec7f..e9ca49d451f3 100644
    > --- a/include/linux/kvm_host.h
    > +++ b/include/linux/kvm_host.h
    > @@ -1130,9 +1130,9 @@ enum kvm_mr_change {
    > };
    >
    > int kvm_set_memory_region(struct kvm *kvm,
    > - const struct kvm_userspace_memory_region *mem);
    > + const struct kvm_userspace_memory_region2 *mem);
    > int __kvm_set_memory_region(struct kvm *kvm,
    > - const struct kvm_userspace_memory_region *mem);
    > + const struct kvm_userspace_memory_region2 *mem);
    > void kvm_arch_free_memslot(struct kvm *kvm, struct kvm_memory_slot *slot);
    > void kvm_arch_memslots_updated(struct kvm *kvm, u64 gen);
    > int kvm_arch_prepare_memory_region(struct kvm *kvm,
    > diff --git a/include/uapi/linux/kvm.h b/include/uapi/linux/kvm.h
    > index f089ab290978..4d4b3de8ac55 100644
    > --- a/include/uapi/linux/kvm.h
    > +++ b/include/uapi/linux/kvm.h
    > @@ -95,6 +95,16 @@ struct kvm_userspace_memory_region {
    > __u64 userspace_addr; /* start of the userspace allocated memory */
    > };
    >
    > +/* for KVM_SET_USER_MEMORY_REGION2 */
    > +struct kvm_userspace_memory_region2 {
    > + __u32 slot;
    > + __u32 flags;
    > + __u64 guest_phys_addr;
    > + __u64 memory_size;
    > + __u64 userspace_addr;
    > + __u64 pad[16];
    > +};
    > +
    > /*
    > * The bit 0 ~ bit 15 of kvm_userspace_memory_region::flags are visible for
    > * userspace, other bits are reserved for kvm internal use which are defined
    > @@ -1192,6 +1202,7 @@ struct kvm_ppc_resize_hpt {
    > #define KVM_CAP_COUNTER_OFFSET 227
    > #define KVM_CAP_ARM_EAGER_SPLIT_CHUNK_SIZE 228
    > #define KVM_CAP_ARM_SUPPORTED_BLOCK_SIZES 229
    > +#define KVM_CAP_USER_MEMORY2 230
    >
    > #ifdef KVM_CAP_IRQ_ROUTING
    >
    > @@ -1466,6 +1477,8 @@ struct kvm_vfio_spapr_tce {
    > struct kvm_userspace_memory_region)
    > #define KVM_SET_TSS_ADDR _IO(KVMIO, 0x47)
    > #define KVM_SET_IDENTITY_MAP_ADDR _IOW(KVMIO, 0x48, __u64)
    > +#define KVM_SET_USER_MEMORY_REGION2 _IOW(KVMIO, 0x49, \
    > + struct kvm_userspace_memory_region2)
    >
    > /* enable ucontrol for s390 */
    > struct kvm_s390_ucas_mapping {
    > diff --git a/virt/kvm/kvm_main.c b/virt/kvm/kvm_main.c
    > index 53346bc2902a..c14adf93daec 100644
    > --- a/virt/kvm/kvm_main.c
    > +++ b/virt/kvm/kvm_main.c
    > @@ -1549,7 +1549,7 @@ static void kvm_replace_memslot(struct kvm *kvm,
    > }
    > }
    >
    > -static int check_memory_region_flags(const struct kvm_userspace_memory_region *mem)
    > +static int check_memory_region_flags(const struct kvm_userspace_memory_region2 *mem)
    > {
    > u32 valid_flags = KVM_MEM_LOG_DIRTY_PAGES;
    >
    > @@ -1951,7 +1951,7 @@ static bool kvm_check_memslot_overlap(struct kvm_memslots *slots, int id,
    > * Must be called holding kvm->slots_lock for write.
    > */
    > int __kvm_set_memory_region(struct kvm *kvm,
    > - const struct kvm_userspace_memory_region *mem)
    > + const struct kvm_userspace_memory_region2 *mem)
    > {
    > struct kvm_memory_slot *old, *new;
    > struct kvm_memslots *slots;
    > @@ -2055,7 +2055,7 @@ int __kvm_set_memory_region(struct kvm *kvm,
    > EXPORT_SYMBOL_GPL(__kvm_set_memory_region);
    >
    > int kvm_set_memory_region(struct kvm *kvm,
    > - const struct kvm_userspace_memory_region *mem)
    > + const struct kvm_userspace_memory_region2 *mem)
    > {
    > int r;
    >
    > @@ -2067,7 +2067,7 @@ int kvm_set_memory_region(struct kvm *kvm,
    > EXPORT_SYMBOL_GPL(kvm_set_memory_region);
    >
    > static int kvm_vm_ioctl_set_memory_region(struct kvm *kvm,
    > - struct kvm_userspace_memory_region *mem)
    > + struct kvm_userspace_memory_region2 *mem)
    > {
    > if ((u16)mem->slot >= KVM_USER_MEM_SLOTS)
    > return -EINVAL;
    > @@ -4514,6 +4514,7 @@ static int kvm_vm_ioctl_check_extension_generic(struct kvm *kvm, long arg)
    > {
    > switch (arg) {
    > case KVM_CAP_USER_MEMORY:
    > + case KVM_CAP_USER_MEMORY2:
    > case KVM_CAP_DESTROY_MEMORY_REGION_WORKS:
    > case KVM_CAP_JOIN_MEMORY_REGIONS_WORKS:
    > case KVM_CAP_INTERNAL_ERROR_DATA:
    > @@ -4757,6 +4758,14 @@ static int kvm_vm_ioctl_get_stats_fd(struct kvm *kvm)
    > return fd;
    > }
    >
    > +#define SANITY_CHECK_MEM_REGION_FIELD(field) \
    > +do { \
    > + BUILD_BUG_ON(offsetof(struct kvm_userspace_memory_region, field) != \
    > + offsetof(struct kvm_userspace_memory_region2, field)); \
    > + BUILD_BUG_ON(sizeof_field(struct kvm_userspace_memory_region, field) != \
    > + sizeof_field(struct kvm_userspace_memory_region2, field)); \
    > +} while (0)
    > +
    > static long kvm_vm_ioctl(struct file *filp,
    > unsigned int ioctl, unsigned long arg)
    > {
    > @@ -4779,15 +4788,28 @@ static long kvm_vm_ioctl(struct file *filp,
    > r = kvm_vm_ioctl_enable_cap_generic(kvm, &cap);
    > break;
    > }
    > + case KVM_SET_USER_MEMORY_REGION2:
    > case KVM_SET_USER_MEMORY_REGION: {
    > - struct kvm_userspace_memory_region kvm_userspace_mem;
    > + struct kvm_userspace_memory_region2 mem;
    > + unsigned long size;
    > +
    > + if (ioctl == KVM_SET_USER_MEMORY_REGION)
    > + size = sizeof(struct kvm_userspace_memory_region);
    > + else
    > + size = sizeof(struct kvm_userspace_memory_region2);
    > +
    > + /* Ensure the common parts of the two structs are identical. */
    > + SANITY_CHECK_MEM_REGION_FIELD(slot);
    > + SANITY_CHECK_MEM_REGION_FIELD(flags);
    > + SANITY_CHECK_MEM_REGION_FIELD(guest_phys_addr);
    > + SANITY_CHECK_MEM_REGION_FIELD(memory_size);
    > + SANITY_CHECK_MEM_REGION_FIELD(userspace_addr);
    >
    > r = -EFAULT;
    > - if (copy_from_user(&kvm_userspace_mem, argp,
    > - sizeof(kvm_userspace_mem)))
    > + if (copy_from_user(&mem, argp, size))
    > goto out;
    >
    > - r = kvm_vm_ioctl_set_memory_region(kvm, &kvm_userspace_mem);
    > + r = kvm_vm_ioctl_set_memory_region(kvm, &mem);
    > break;
    > }
    > case KVM_GET_DIRTY_LOG: {

    Reviewed-by: Paolo Bonzini <pbonzini@redhat.com>

    \
     
     \ /
      Last update: 2023-07-21 11:05    [W:4.028 / U:0.132 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site