[libvirt] [PATCH v2 11/31] qemu: Flatten qemuMonitorCPUDefs.cpus
Ján Tomko
jtomko at redhat.com
Wed Oct 16 15:51:33 UTC 2019
On Wed, Oct 16, 2019 at 05:46:24PM +0200, Jiri Denemark wrote:
>On Wed, Oct 16, 2019 at 13:15:19 +0200, Ján Tomko wrote:
>> On Tue, Oct 15, 2019 at 05:34:47PM +0200, Jiri Denemark wrote:
>> >Let's store qemuMonitorCPUDefInfo directly in the array of CPUs in
>> >qemuMonitorCPUDefs rather then using an array of pointers.
>> >
>> >Signed-off-by: Jiri Denemark <jdenemar at redhat.com>
>> >Reviewed-by: Ján Tomko <jtomko at redhat.com>
>> >---
>> >
>> >Notes:
>> > Version 2:
>> > - trivial rebase
>> >
>> > src/qemu/qemu_capabilities.c | 14 +++++++-------
>> > src/qemu/qemu_monitor.c | 5 ++---
>> > src/qemu/qemu_monitor.h | 2 +-
>> > src/qemu/qemu_monitor_json.c | 7 +------
>> > tests/qemumonitorjsontest.c | 8 ++++----
>> > 5 files changed, 15 insertions(+), 21 deletions(-)
>> >
>>
>> Reviewed-by: Ján Tomko <jtomko at redhat.com>
>>
>> @@ -3573,7 +3572,7 @@ qemuMonitorCPUDefsNew(size_t count)
>> g_autoptr(qemuMonitorCPUDefs) defs = NULL;
>>
>> defs = g_new0(qemuMonitorCPUDefs, 1);
>> - defs->cpus = g_new0(qemuMonitorCPUDefInfoPtr, count);
>> + defs->cpus = g_new0(qemuMonitorCPUDefInfo, count);
>> defs->ncpus = count;
>> return g_steal_pointer(&defs);
>> }
>
>Looks like a change in something that doesn't exist yet :-)
>
Yes, it's exactly what it looks like! :)
Jano
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 488 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20191016/9853cf8e/attachment-0001.sig>
More information about the libvir-list
mailing list