Re: [PATCHv3 0/3] query-cpu-model-baseline QMP command

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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@xxxxxxxxxx>
> > 
> > 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@xxxxxxxxxx>
> 
> 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
> 

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list



[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux