On Thu, May 16, 2019 at 05:35:31PM +0200, Andrea Bolognani wrote: > Up until now we've probed machine type properties, along with > properties for other types, in virQEMUCapsProbeQMPDevices(), but > soon we're going to need some logic that is specific to machine > types and as such wouldn't quite fit into that function. > > Signed-off-by: Andrea Bolognani <abologna@xxxxxxxxxx> > --- > src/qemu/qemu_capabilities.c | 52 ++++++++++++++++++++++++++++++------ > 1 file changed, 44 insertions(+), 8 deletions(-) [...] > +static int > +virQEMUCapsProbeQMPMachineProps(virQEMUCapsPtr qemuCaps, > + qemuMonitorPtr mon) > +{ > + char **values; > + int nvalues; > + size_t i; > + > + /* We need qom-list-properties for this to work */ Useless comment, the code says the same thing. > + if (!virQEMUCapsGet(qemuCaps, QEMU_CAPS_QOM_LIST_PROPERTIES)) > + return 0; Reviewed-by: Pavel Hrdina <phrdina@xxxxxxxxxx>
Attachment:
signature.asc
Description: PGP signature
-- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list