I think I made a typo here. It should be:
+# @plugged-memory: size *of* memory that can be hot-unplugged. This field
+# is omitted if target does *not* support memory hotplug
+# (i.e. CONFIG_MEM_HOTPLUG not defined on build time).
On Thu, Sep 14, 2017 at 12:26 PM, Dr. David Alan Gilbert <
dgilb...@redhat.com> wrote:
> * Igor Mammedov (imamm...@redhat.com) wrote:
> > On Tue, 29 Aug 2017 17:30:21 +0200
> > Vadim Galitsyn <vadim.galit...@profitbricks.com> wrote:
> >
> > > Add a new query-memory-size-summary command which provides the
> > > following memory information in bytes:
> > >
> > > * base-memory - size of "base" memory specified with command line
> option -m.
> > >
> > > * plugged-memory - amount of memory that was hot-plugged.
> > > If target does not have CONFIG_MEM_HOTPLUG enabled, no
> > > value is reported.
> > >
> > > Signed-off-by: Vasilis Liaskovitis <vasilis.liaskovitis@
> profitbricks.com>
> > > Signed-off-by: Mohammed Gamal <mohammed.ga...@profitbricks.com>
> > > Signed-off-by: Eduardo Otubo <eduardo.ot...@profitbricks.com>
> > > Signed-off-by: Vadim Galitsyn <vadim.galit...@profitbricks.com>
> > > Reviewed-by: Eugene Crosser <evgenii.cherkas...@profitbricks.com>
> > > Cc: Dr. David Alan Gilbert <dgilb...@redhat.com>
> > > Cc: Markus Armbruster <arm...@redhat.com>
> > > Cc: Igor Mammedov <imamm...@redhat.com>
> > > Cc: Eric Blake <ebl...@redhat.com>
> > > Cc: qemu-devel@nongnu.org
> > > ---
> > > qapi-schema.json | 32
> ++++++++++++++++++++++
> > > include/hw/mem/pc-dimm.h | 1 +
> > > hw/mem/pc-dimm.c | 5 ++++
> > > qmp.c | 13 +++++++++
> > > stubs/{qmp_pc_dimm_device_list.c => qmp_pc_dimm.c} | 5 ++++
> > > stubs/Makefile.objs | 2 +-
> > > 6 files changed, 57 insertions(+), 1 deletion(-)
> > > rename stubs/{qmp_pc_dimm_device_list.c => qmp_pc_dimm.c} (68%)
> > >
> > > diff --git a/qapi-schema.json b/qapi-schema.json
> > > index 802ea53d00..9402ac3b3a 100644
> > > --- a/qapi-schema.json
> > > +++ b/qapi-schema.json
> > > @@ -4407,6 +4407,38 @@
> > > 'data': { 'name': 'str', '*migration-safe': 'bool', 'static':
> 'bool',
> > > '*unavailable-features': [ 'str' ], 'typename': 'str' } }
> > >
> > > +##
> > > +# @MemoryInfo:
> > > +#
> > > +# Actual memory information in bytes.
> > > +#
> > > +# @base-memory: size of "base" memory specified with command line
> > > +# option -m.
> > > +#
> > > +# @plugged-memory: size memory that can be hot-unplugged. This field
> > > +# is omitted if target does support memory hotplug
> > > +# (i.e. CONFIG_MEM_HOTPLUG not defined on build
> time).
> > field description doesn't match what's actually reported.
> > s/cat be/is/
>
> Are you sure about that? That would read:
> size memory that is hot-unplugged
>
> which doesn't sound right, since it's not be hot-unplugged
> yet.
>
> Dave
>
> > s/does/doesn't/
> >
> > > +#
> > > +# Since: 2.11.0
> > > +##
> > > +{ 'struct': 'MemoryInfo',
> > > + 'data' : { 'base-memory': 'size', '*plugged-memory': 'size' } }
> > > +
> > > +##
> > > +# @query-memory-size-summary:
> > > +#
> > > +# Return the amount of initially allocated and hot-plugged (if
> > > +# enabled) memory in bytes.
> > it could count dimm's on CLI, so not only hotplugged
> >
> > s/hot-plugged/present hotpluggable/
> >
> > > +#
> > > +# Example:
> > > +#
> > > +# -> { "execute": "query-memory-size-summary" }
> > > +# <- { "return": { "base-memory": 4294967296, "plugged-memory": 0 } }
> > > +#
> > > +# Since: 2.11.0
> > > +##
> > > +{ 'command': 'query-memory-size-summary', 'returns': 'MemoryInfo' }
> > > +
> > > ##
> > > # @query-cpu-definitions:
> > > #
> > > diff --git a/include/hw/mem/pc-dimm.h b/include/hw/mem/pc-dimm.h
> > > index 6f8c3eb1b3..d83b957829 100644
> > > --- a/include/hw/mem/pc-dimm.h
> > > +++ b/include/hw/mem/pc-dimm.h
> > > @@ -95,6 +95,7 @@ int pc_dimm_get_free_slot(const int *hint, int
> max_slots, Error **errp);
> > >
> > > int qmp_pc_dimm_device_list(Object *obj, void *opaque);
> > > uint64_t pc_existing_dimms_capacity(Error **errp);
> > > +uint64_t get_plugged_memory_size(void);
> > > void pc_dimm_memory_plug(DeviceState *dev, MemoryHotplugState *hpms,
> > > MemoryRegion *mr, uint64_t align, Error
> **errp);
> > > void pc_dimm_memory_unplug(DeviceState *dev, MemoryHotplugState
> *hpms,
> > > diff --git a/hw/mem/pc-dimm.c b/hw/mem/pc-dimm.c
> > > index bdf6649083..66eace5a5c 100644
> > > --- a/hw/mem/pc-dimm.c
> > > +++ b/hw/mem/pc-dimm.c
> > > @@ -159,6 +159,11 @@ uint64_t pc_existing_dimms_capacity(Error **errp)
> > > return cap.size;
> > > }
> > >
> > > +uint64_t get_plugged_memory_size(void)
> > > +{
> > > + return pc_existing_dimms_capacity(&error_abort);
> > > +}
> > > +
> > > int qmp_pc_dimm_device_list(Object *obj, void *opaque)
> > > {
> > > MemoryDeviceInfoList ***prev = opaque;
> > > diff --git a/qmp.c b/qmp.c
> > > index b86201e349..e8c303116a 100644
> > > --- a/qmp.c
> > > +++ b/qmp.c
> > > @@ -709,3 +709,16 @@ ACPIOSTInfoList *qmp_query_acpi_ospm_status(Error
> **errp)
> > >
> > > return head;
> > > }
> > > +
> > > +MemoryInfo *qmp_query_memory_size_summary(Error **errp)
> > > +{
> > > + MemoryInfo *mem_info = g_malloc0(sizeof(MemoryInfo));
> > > +
> > > + mem_info->base_memory = ram_size;
> > > +
> > > + mem_info->plugged_memory = get_plugged_memory_size();
> > > + mem_info->has_plugged_memory =
> > > + mem_info->plugged_memory != (uint64_t)-1;
> > > +
> > > + return mem_info;
> > > +}
> > > diff --git a/stubs/qmp_pc_dimm_device_list.c b/stubs/qmp_pc_dimm.c
> > > similarity index 68%
> > > rename from stubs/qmp_pc_dimm_device_list.c
> > > rename to stubs/qmp_pc_dimm.c
> > > index def211564d..9ddc4f619a 100644
> > > --- a/stubs/qmp_pc_dimm_device_list.c
> > > +++ b/stubs/qmp_pc_dimm.c
> > > @@ -6,3 +6,8 @@ int qmp_pc_dimm_device_list(Object *obj, void *opaque)
> > > {
> > > return 0;
> > > }
> > > +
> > > +uint64_t get_plugged_memory_size(void)
> > > +{
> > > + return (uint64_t)-1;
> > > +}
> > > diff --git a/stubs/Makefile.objs b/stubs/Makefile.objs
> > > index e69c217aff..f5f139f310 100644
> > > --- a/stubs/Makefile.objs
> > > +++ b/stubs/Makefile.objs
> > > @@ -33,7 +33,7 @@ stub-obj-y += uuid.o
> > > stub-obj-y += vm-stop.o
> > > stub-obj-y += vmstate.o
> > > stub-obj-$(CONFIG_WIN32) += fd-register.o
> > > -stub-obj-y += qmp_pc_dimm_device_list.o
> > > +stub-obj-y += qmp_pc_dimm.o
> > > stub-obj-y += target-monitor-defs.o
> > > stub-obj-y += target-get-monitor-def.o
> > > stub-obj-y += pc_madt_cpu_entry.o
> >
> --
> Dr. David Alan Gilbert / dgilb...@redhat.com / Manchester, UK
>
