[libvirt] [PATCHv3 0/3] query-cpu-model-baseline QMP command
Chris Venteicher
cventeic at redhat.com
Tue May 1 20:17:22 UTC 2018
Quoting John Ferlan (2018-05-01 08:57:08)
>
>
> On 04/30/2018 10:55 PM, Chris Venteicher wrote:
> > This is part of resolution of: https://bugzilla.redhat.com/show_bug.cgi?id=1511999
> >
> > Signed-off-by: Chris Venteicher <cventeic at redhat.com>
> >
> > diff to v1:
> > - Replaced c++ style comments with c style
> > - qemuMonitorJSONGetCPUModelInfo{ToJSON,FromJSON} use To/From form
> > - qemuMonitorJSONGetCPUModelInfo{ToJSON,FromJSON} return pointers not integers
> > - VIR_STEAL_PTR form used
> > - switch statement uses virReportEnumRangeError
> > - qemuMonitorJSONHasError(reply, "GenericError") treated as no info, not fatal error
> > - virJSONValueFree(cpu_props) during error case
> >
> > diff to v2:
> > - qemuMonitorJSONGetCPUModelInfo{ToJSON,FromJSON}
> > works on {"name": "IvyBridge", "props": {}}
> > rather than {"model": {"name": "IvyBridge", "props": {}}}
> > - additional cleanups and fixes
> >
> > Chris Venteicher (3):
> > qemu_monitor_json: Populate CPUModelInfo struct from json
> > qemu_monitor_json: Build Json CPU Model Info
> > qemu_monitor: query-cpu-model-baseline QMP command
> >
> > src/qemu/qemu_monitor.c | 13 ++++
> > src/qemu/qemu_monitor.h | 5 ++
> > src/qemu/qemu_monitor_json.c | 179 +++++++++++++++++++++++++++++++++++++------
> > src/qemu/qemu_monitor_json.h | 7 ++
> > 4 files changed, 182 insertions(+), 22 deletions(-)
> >
>
> Changes look good to me,
>
> Reviewed-by: John Ferlan <jferlan at redhat.com>
>
> I have to wait for 4.4.0 to open before pushing, but it would also be
> good to know what the plans are for code that will use the new
> functions. Do you plan on posting something for 4.4.0?
I plan on posting something yet this week that uses this qemu_monitor interface.
It will likely take some rounds of feedback before the new patch set is ready to
be accepted. Not sure how this maps to 4.4.0 window.
However, I don't think there is much that could happen with the new patch set
that would change the need for the qemu_monitor interface exposed by this patch
set.
Thanks, Chris
>
> John
>
More information about the libvir-list
mailing list