[v2,16/18] kvm: convert "-machine kvm_shadow_mem" to an accelerator property
diff mbox series

Message ID 1575903705-12925-17-git-send-email-pbonzini@redhat.com
State New
Headers show
Series
  • Complete the implementation of -accel
Related show

Commit Message

Paolo Bonzini Dec. 9, 2019, 3:01 p.m. UTC
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
---
 accel/kvm/kvm-all.c | 43 +++++++++++++++++++++++++++++++++++++++++++
 hw/core/machine.c   | 39 ---------------------------------------
 include/hw/boards.h |  2 --
 qemu-options.hx     |  6 +++---
 target/i386/kvm.c   |  2 +-
 vl.c                |  4 ++++
 6 files changed, 51 insertions(+), 45 deletions(-)

Comments

Philippe Mathieu-Daudé Dec. 10, 2019, 12:01 p.m. UTC | #1
On 12/9/19 4:01 PM, Paolo Bonzini wrote:
> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
> ---
>   accel/kvm/kvm-all.c | 43 +++++++++++++++++++++++++++++++++++++++++++
>   hw/core/machine.c   | 39 ---------------------------------------
>   include/hw/boards.h |  2 --
>   qemu-options.hx     |  6 +++---
>   target/i386/kvm.c   |  2 +-
>   vl.c                |  4 ++++
>   6 files changed, 51 insertions(+), 45 deletions(-)
> 
> diff --git a/accel/kvm/kvm-all.c b/accel/kvm/kvm-all.c
> index ca00daa..f0b9294 100644
> --- a/accel/kvm/kvm-all.c
> +++ b/accel/kvm/kvm-all.c
> @@ -41,6 +41,7 @@
>   #include "hw/irq.h"
>   #include "sysemu/sev.h"
>   #include "sysemu/balloon.h"
> +#include "qapi/visitor.h"
>   
>   #include "hw/boards.h"
>   
> @@ -92,6 +93,7 @@ struct KVMState
>       int max_nested_state_len;
>       int many_ioeventfds;
>       int intx_set_mask;
> +    int kvm_shadow_mem;
>       bool sync_mmu;
>       bool manual_dirty_log_protect;
>       /* The man page (and posix) say ioctl numbers are signed int, but
> @@ -2940,6 +2942,40 @@ static bool kvm_accel_has_memory(MachineState *ms, AddressSpace *as,
>       return false;
>   }
>   
> +static void kvm_get_kvm_shadow_mem(Object *obj, Visitor *v,
> +                                   const char *name, void *opaque,
> +                                   Error **errp)
> +{
> +    KVMState *s = KVM_STATE(obj);
> +    int64_t value = s->kvm_shadow_mem;
> +
> +    visit_type_int(v, name, &value, errp);
> +}
> +
> +static void kvm_set_kvm_shadow_mem(Object *obj, Visitor *v,
> +                                   const char *name, void *opaque,
> +                                   Error **errp)
> +{
> +    KVMState *s = KVM_STATE(obj);
> +    Error *error = NULL;
> +    int64_t value;
> +
> +    visit_type_int(v, name, &value, &error);
> +    if (error) {
> +        error_propagate(errp, error);
> +        return;
> +    }
> +
> +    s->kvm_shadow_mem = value;
> +}
> +
> +static void kvm_accel_instance_init(Object *obj)
> +{
> +    KVMState *s = KVM_STATE(obj);
> +
> +    s->kvm_shadow_mem = -1;
> +}
> +
>   static void kvm_accel_class_init(ObjectClass *oc, void *data)
>   {
>       AccelClass *ac = ACCEL_CLASS(oc);
> @@ -2947,11 +2983,18 @@ static void kvm_accel_class_init(ObjectClass *oc, void *data)
>       ac->init_machine = kvm_init;
>       ac->has_memory = kvm_accel_has_memory;
>       ac->allowed = &kvm_allowed;
> +
> +    object_class_property_add(oc, "kvm-shadow-mem", "int",
> +        kvm_get_kvm_shadow_mem, kvm_set_kvm_shadow_mem,
> +        NULL, NULL, &error_abort);
> +    object_class_property_set_description(oc, "kvm-shadow-mem",
> +        "KVM shadow MMU size", &error_abort);
>   }
>   
>   static const TypeInfo kvm_accel_type = {
>       .name = TYPE_KVM_ACCEL,
>       .parent = TYPE_ACCEL,
> +    .instance_init = kvm_accel_instance_init,
>       .class_init = kvm_accel_class_init,
>       .instance_size = sizeof(KVMState),
>   };
> diff --git a/hw/core/machine.c b/hw/core/machine.c
> index d7a0356..bfb5f6f 100644
> --- a/hw/core/machine.c
> +++ b/hw/core/machine.c
> @@ -211,33 +211,6 @@ static void machine_set_kernel_irqchip(Object *obj, Visitor *v,
>       }
>   }
>   
> -static void machine_get_kvm_shadow_mem(Object *obj, Visitor *v,
> -                                       const char *name, void *opaque,
> -                                       Error **errp)
> -{
> -    MachineState *ms = MACHINE(obj);
> -    int64_t value = ms->kvm_shadow_mem;
> -
> -    visit_type_int(v, name, &value, errp);
> -}
> -
> -static void machine_set_kvm_shadow_mem(Object *obj, Visitor *v,
> -                                       const char *name, void *opaque,
> -                                       Error **errp)
> -{
> -    MachineState *ms = MACHINE(obj);
> -    Error *error = NULL;
> -    int64_t value;
> -
> -    visit_type_int(v, name, &value, &error);
> -    if (error) {
> -        error_propagate(errp, error);
> -        return;
> -    }
> -
> -    ms->kvm_shadow_mem = value;
> -}
> -
>   static char *machine_get_kernel(Object *obj, Error **errp)
>   {
>       MachineState *ms = MACHINE(obj);
> @@ -785,12 +758,6 @@ static void machine_class_init(ObjectClass *oc, void *data)
>       object_class_property_set_description(oc, "kernel-irqchip",
>           "Configure KVM in-kernel irqchip", &error_abort);
>   
> -    object_class_property_add(oc, "kvm-shadow-mem", "int",
> -        machine_get_kvm_shadow_mem, machine_set_kvm_shadow_mem,
> -        NULL, NULL, &error_abort);
> -    object_class_property_set_description(oc, "kvm-shadow-mem",
> -        "KVM shadow MMU size", &error_abort);
> -
>       object_class_property_add_str(oc, "kernel",
>           machine_get_kernel, machine_set_kernel, &error_abort);
>       object_class_property_set_description(oc, "kernel",
> @@ -892,7 +859,6 @@ static void machine_initfn(Object *obj)
>   
>       ms->kernel_irqchip_allowed = true;
>       ms->kernel_irqchip_split = mc->default_kernel_irqchip_split;
> -    ms->kvm_shadow_mem = -1;
>       ms->dump_guest_core = true;
>       ms->mem_merge = true;
>       ms->enable_graphics = true;
> @@ -963,11 +929,6 @@ bool machine_kernel_irqchip_split(MachineState *machine)
>       return machine->kernel_irqchip_split;
>   }
>   
> -int machine_kvm_shadow_mem(MachineState *machine)
> -{
> -    return machine->kvm_shadow_mem;
> -}
> -
>   int machine_phandle_start(MachineState *machine)
>   {
>       return machine->phandle_start;
> diff --git a/include/hw/boards.h b/include/hw/boards.h
> index cdcf481..46119eb 100644
> --- a/include/hw/boards.h
> +++ b/include/hw/boards.h
> @@ -66,7 +66,6 @@ bool machine_usb(MachineState *machine);
>   bool machine_kernel_irqchip_allowed(MachineState *machine);
>   bool machine_kernel_irqchip_required(MachineState *machine);
>   bool machine_kernel_irqchip_split(MachineState *machine);
> -int machine_kvm_shadow_mem(MachineState *machine);
>   int machine_phandle_start(MachineState *machine);
>   bool machine_dump_guest_core(MachineState *machine);
>   bool machine_mem_merge(MachineState *machine);
> @@ -278,7 +277,6 @@ struct MachineState {
>       bool kernel_irqchip_allowed;
>       bool kernel_irqchip_required;
>       bool kernel_irqchip_split;
> -    int kvm_shadow_mem;
>       char *dtb;
>       char *dumpdtb;
>       int phandle_start;
> diff --git a/qemu-options.hx b/qemu-options.hx
> index 6f12b31..779c8af 100644
> --- a/qemu-options.hx
> +++ b/qemu-options.hx
> @@ -34,7 +34,6 @@ DEF("machine", HAS_ARG, QEMU_OPTION_machine, \
>       "                supported accelerators are kvm, xen, hax, hvf, whpx or tcg (default: tcg)\n"
>       "                kernel_irqchip=on|off|split controls accelerated irqchip support (default=off)\n"
>       "                vmport=on|off|auto controls emulation of vmport (default: auto)\n"
> -    "                kvm_shadow_mem=size of KVM shadow MMU in bytes\n"
>       "                dump-guest-core=on|off include guest memory in a core dump (default=on)\n"
>       "                mem-merge=on|off controls memory merge support (default: on)\n"
>       "                aes-key-wrap=on|off controls support for AES key wrapping (default=on)\n"
> @@ -74,8 +73,6 @@ Controls in-kernel irqchip support for the chosen accelerator when available.
>   Enables emulation of VMWare IO port, for vmmouse etc. auto says to select the
>   value based on accel. For accel=xen the default is off otherwise the default
>   is on.
> -@item kvm_shadow_mem=size
> -Defines the size of the KVM shadow MMU.
>   @item dump-guest-core=on|off
>   Include guest memory in a core dump. The default is on.
>   @item mem-merge=on|off
> @@ -118,6 +115,7 @@ DEF("accel", HAS_ARG, QEMU_OPTION_accel,
>       "-accel [accel=]accelerator[,prop[=value][,...]]\n"
>       "                select accelerator (kvm, xen, hax, hvf, whpx or tcg; use 'help' for a list)\n"
>       "                igd-passthru=on|off (enable Xen integrated Intel graphics passthrough, default=off)\n"
> +    "                kvm-shadow-mem=size of KVM shadow MMU in bytes\n"
>       "                tb-size=n (TCG translation block cache size)\n"
>       "                thread=single|multi (enable multi-threaded TCG)\n", QEMU_ARCH_ALL)
>   STEXI
> @@ -131,6 +129,8 @@ fails to initialize.
>   @item igd-passthru=on|off
>   When Xen is in use, this option controls whether Intel integrated graphics
>   devices can be passed through to the guest (default=off)
> +@item kvm-shadow-mem=size
> +Defines the size of the KVM shadow MMU.
>   @item tb-size=@var{n}
>   Controls the size (in MiB) of the TCG translation block cache.
>   @item thread=single|multi
> diff --git a/target/i386/kvm.c b/target/i386/kvm.c
> index 1d10046..62ce681 100644
> --- a/target/i386/kvm.c
> +++ b/target/i386/kvm.c
> @@ -2163,7 +2163,7 @@ int kvm_arch_init(MachineState *ms, KVMState *s)
>       }
>       qemu_register_reset(kvm_unpoison_all, NULL);
>   
> -    shadow_mem = machine_kvm_shadow_mem(ms);
> +    shadow_mem = object_property_get_int(OBJECT(s), "kvm-shadow-mem", &error_abort);
>       if (shadow_mem != -1) {
>           shadow_mem /= 4096;
>           ret = kvm_vm_ioctl(s, KVM_SET_NR_MMU_PAGES, shadow_mem);
> diff --git a/vl.c b/vl.c
> index ee872f2..3678522 100644
> --- a/vl.c
> +++ b/vl.c
> @@ -2639,6 +2639,10 @@ static int machine_set_property(void *opaque,
>           object_register_sugar_prop(ACCEL_CLASS_NAME("xen"), qom_name, value);
>           return 0;
>       }
> +    if (g_str_equal(qom_name, "kvm-shadow-mem")) {
> +        object_register_sugar_prop(ACCEL_CLASS_NAME("kvm"), qom_name, value);
> +        return 0;
> +    }
>   
>       return object_parse_property_opt(opaque, name, value, "type", errp);
>   }
> 

Nice.

Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>

Patch
diff mbox series

diff --git a/accel/kvm/kvm-all.c b/accel/kvm/kvm-all.c
index ca00daa..f0b9294 100644
--- a/accel/kvm/kvm-all.c
+++ b/accel/kvm/kvm-all.c
@@ -41,6 +41,7 @@ 
 #include "hw/irq.h"
 #include "sysemu/sev.h"
 #include "sysemu/balloon.h"
+#include "qapi/visitor.h"
 
 #include "hw/boards.h"
 
@@ -92,6 +93,7 @@  struct KVMState
     int max_nested_state_len;
     int many_ioeventfds;
     int intx_set_mask;
+    int kvm_shadow_mem;
     bool sync_mmu;
     bool manual_dirty_log_protect;
     /* The man page (and posix) say ioctl numbers are signed int, but
@@ -2940,6 +2942,40 @@  static bool kvm_accel_has_memory(MachineState *ms, AddressSpace *as,
     return false;
 }
 
+static void kvm_get_kvm_shadow_mem(Object *obj, Visitor *v,
+                                   const char *name, void *opaque,
+                                   Error **errp)
+{
+    KVMState *s = KVM_STATE(obj);
+    int64_t value = s->kvm_shadow_mem;
+
+    visit_type_int(v, name, &value, errp);
+}
+
+static void kvm_set_kvm_shadow_mem(Object *obj, Visitor *v,
+                                   const char *name, void *opaque,
+                                   Error **errp)
+{
+    KVMState *s = KVM_STATE(obj);
+    Error *error = NULL;
+    int64_t value;
+
+    visit_type_int(v, name, &value, &error);
+    if (error) {
+        error_propagate(errp, error);
+        return;
+    }
+
+    s->kvm_shadow_mem = value;
+}
+
+static void kvm_accel_instance_init(Object *obj)
+{
+    KVMState *s = KVM_STATE(obj);
+
+    s->kvm_shadow_mem = -1;
+}
+
 static void kvm_accel_class_init(ObjectClass *oc, void *data)
 {
     AccelClass *ac = ACCEL_CLASS(oc);
@@ -2947,11 +2983,18 @@  static void kvm_accel_class_init(ObjectClass *oc, void *data)
     ac->init_machine = kvm_init;
     ac->has_memory = kvm_accel_has_memory;
     ac->allowed = &kvm_allowed;
+
+    object_class_property_add(oc, "kvm-shadow-mem", "int",
+        kvm_get_kvm_shadow_mem, kvm_set_kvm_shadow_mem,
+        NULL, NULL, &error_abort);
+    object_class_property_set_description(oc, "kvm-shadow-mem",
+        "KVM shadow MMU size", &error_abort);
 }
 
 static const TypeInfo kvm_accel_type = {
     .name = TYPE_KVM_ACCEL,
     .parent = TYPE_ACCEL,
+    .instance_init = kvm_accel_instance_init,
     .class_init = kvm_accel_class_init,
     .instance_size = sizeof(KVMState),
 };
diff --git a/hw/core/machine.c b/hw/core/machine.c
index d7a0356..bfb5f6f 100644
--- a/hw/core/machine.c
+++ b/hw/core/machine.c
@@ -211,33 +211,6 @@  static void machine_set_kernel_irqchip(Object *obj, Visitor *v,
     }
 }
 
-static void machine_get_kvm_shadow_mem(Object *obj, Visitor *v,
-                                       const char *name, void *opaque,
-                                       Error **errp)
-{
-    MachineState *ms = MACHINE(obj);
-    int64_t value = ms->kvm_shadow_mem;
-
-    visit_type_int(v, name, &value, errp);
-}
-
-static void machine_set_kvm_shadow_mem(Object *obj, Visitor *v,
-                                       const char *name, void *opaque,
-                                       Error **errp)
-{
-    MachineState *ms = MACHINE(obj);
-    Error *error = NULL;
-    int64_t value;
-
-    visit_type_int(v, name, &value, &error);
-    if (error) {
-        error_propagate(errp, error);
-        return;
-    }
-
-    ms->kvm_shadow_mem = value;
-}
-
 static char *machine_get_kernel(Object *obj, Error **errp)
 {
     MachineState *ms = MACHINE(obj);
@@ -785,12 +758,6 @@  static void machine_class_init(ObjectClass *oc, void *data)
     object_class_property_set_description(oc, "kernel-irqchip",
         "Configure KVM in-kernel irqchip", &error_abort);
 
-    object_class_property_add(oc, "kvm-shadow-mem", "int",
-        machine_get_kvm_shadow_mem, machine_set_kvm_shadow_mem,
-        NULL, NULL, &error_abort);
-    object_class_property_set_description(oc, "kvm-shadow-mem",
-        "KVM shadow MMU size", &error_abort);
-
     object_class_property_add_str(oc, "kernel",
         machine_get_kernel, machine_set_kernel, &error_abort);
     object_class_property_set_description(oc, "kernel",
@@ -892,7 +859,6 @@  static void machine_initfn(Object *obj)
 
     ms->kernel_irqchip_allowed = true;
     ms->kernel_irqchip_split = mc->default_kernel_irqchip_split;
-    ms->kvm_shadow_mem = -1;
     ms->dump_guest_core = true;
     ms->mem_merge = true;
     ms->enable_graphics = true;
@@ -963,11 +929,6 @@  bool machine_kernel_irqchip_split(MachineState *machine)
     return machine->kernel_irqchip_split;
 }
 
-int machine_kvm_shadow_mem(MachineState *machine)
-{
-    return machine->kvm_shadow_mem;
-}
-
 int machine_phandle_start(MachineState *machine)
 {
     return machine->phandle_start;
diff --git a/include/hw/boards.h b/include/hw/boards.h
index cdcf481..46119eb 100644
--- a/include/hw/boards.h
+++ b/include/hw/boards.h
@@ -66,7 +66,6 @@  bool machine_usb(MachineState *machine);
 bool machine_kernel_irqchip_allowed(MachineState *machine);
 bool machine_kernel_irqchip_required(MachineState *machine);
 bool machine_kernel_irqchip_split(MachineState *machine);
-int machine_kvm_shadow_mem(MachineState *machine);
 int machine_phandle_start(MachineState *machine);
 bool machine_dump_guest_core(MachineState *machine);
 bool machine_mem_merge(MachineState *machine);
@@ -278,7 +277,6 @@  struct MachineState {
     bool kernel_irqchip_allowed;
     bool kernel_irqchip_required;
     bool kernel_irqchip_split;
-    int kvm_shadow_mem;
     char *dtb;
     char *dumpdtb;
     int phandle_start;
diff --git a/qemu-options.hx b/qemu-options.hx
index 6f12b31..779c8af 100644
--- a/qemu-options.hx
+++ b/qemu-options.hx
@@ -34,7 +34,6 @@  DEF("machine", HAS_ARG, QEMU_OPTION_machine, \
     "                supported accelerators are kvm, xen, hax, hvf, whpx or tcg (default: tcg)\n"
     "                kernel_irqchip=on|off|split controls accelerated irqchip support (default=off)\n"
     "                vmport=on|off|auto controls emulation of vmport (default: auto)\n"
-    "                kvm_shadow_mem=size of KVM shadow MMU in bytes\n"
     "                dump-guest-core=on|off include guest memory in a core dump (default=on)\n"
     "                mem-merge=on|off controls memory merge support (default: on)\n"
     "                aes-key-wrap=on|off controls support for AES key wrapping (default=on)\n"
@@ -74,8 +73,6 @@  Controls in-kernel irqchip support for the chosen accelerator when available.
 Enables emulation of VMWare IO port, for vmmouse etc. auto says to select the
 value based on accel. For accel=xen the default is off otherwise the default
 is on.
-@item kvm_shadow_mem=size
-Defines the size of the KVM shadow MMU.
 @item dump-guest-core=on|off
 Include guest memory in a core dump. The default is on.
 @item mem-merge=on|off
@@ -118,6 +115,7 @@  DEF("accel", HAS_ARG, QEMU_OPTION_accel,
     "-accel [accel=]accelerator[,prop[=value][,...]]\n"
     "                select accelerator (kvm, xen, hax, hvf, whpx or tcg; use 'help' for a list)\n"
     "                igd-passthru=on|off (enable Xen integrated Intel graphics passthrough, default=off)\n"
+    "                kvm-shadow-mem=size of KVM shadow MMU in bytes\n"
     "                tb-size=n (TCG translation block cache size)\n"
     "                thread=single|multi (enable multi-threaded TCG)\n", QEMU_ARCH_ALL)
 STEXI
@@ -131,6 +129,8 @@  fails to initialize.
 @item igd-passthru=on|off
 When Xen is in use, this option controls whether Intel integrated graphics
 devices can be passed through to the guest (default=off)
+@item kvm-shadow-mem=size
+Defines the size of the KVM shadow MMU.
 @item tb-size=@var{n}
 Controls the size (in MiB) of the TCG translation block cache.
 @item thread=single|multi
diff --git a/target/i386/kvm.c b/target/i386/kvm.c
index 1d10046..62ce681 100644
--- a/target/i386/kvm.c
+++ b/target/i386/kvm.c
@@ -2163,7 +2163,7 @@  int kvm_arch_init(MachineState *ms, KVMState *s)
     }
     qemu_register_reset(kvm_unpoison_all, NULL);
 
-    shadow_mem = machine_kvm_shadow_mem(ms);
+    shadow_mem = object_property_get_int(OBJECT(s), "kvm-shadow-mem", &error_abort);
     if (shadow_mem != -1) {
         shadow_mem /= 4096;
         ret = kvm_vm_ioctl(s, KVM_SET_NR_MMU_PAGES, shadow_mem);
diff --git a/vl.c b/vl.c
index ee872f2..3678522 100644
--- a/vl.c
+++ b/vl.c
@@ -2639,6 +2639,10 @@  static int machine_set_property(void *opaque,
         object_register_sugar_prop(ACCEL_CLASS_NAME("xen"), qom_name, value);
         return 0;
     }
+    if (g_str_equal(qom_name, "kvm-shadow-mem")) {
+        object_register_sugar_prop(ACCEL_CLASS_NAME("kvm"), qom_name, value);
+        return 0;
+    }
 
     return object_parse_property_opt(opaque, name, value, "type", errp);
 }