[libvirt] [PATCH 3/6] qemu: Add support for parsing iotune group setting

Martin Kletzander mkletzan at redhat.com
Wed Nov 2 14:34:37 UTC 2016


On Wed, Nov 02, 2016 at 03:24:10PM +0100, Martin Kletzander wrote:
>On Mon, Oct 31, 2016 at 05:22:59PM -0400, John Ferlan wrote:
>>Add support to read/parse the iotune group setting for qemu.
>>
>>Signed-off-by: John Ferlan <jferlan at redhat.com>
>>---
>> include/libvirt/libvirt-domain.h |  8 +++++
>> src/conf/domain_conf.c           |  1 +
>> src/conf/domain_conf.h           |  1 +
>> src/qemu/qemu_driver.c           | 50 ++++++++++++++++++++++++---
>> src/qemu/qemu_monitor.c          |  2 ++
>> src/qemu/qemu_monitor.h          |  1 +
>> src/qemu/qemu_monitor_json.c     | 21 ++++++++++--
>> src/qemu/qemu_monitor_json.h     |  1 +
>> tests/qemumonitorjsontest.c      | 74 +++++++++++++++++++++++++++++++---------
>> 9 files changed, 134 insertions(+), 25 deletions(-)
>>
>>diff --git a/tests/qemumonitorjsontest.c b/tests/qemumonitorjsontest.c
>>index d8fd958..19bfc2a 100644
>>--- a/tests/qemumonitorjsontest.c
>>+++ b/tests/qemumonitorjsontest.c
>>@@ -2002,7 +2003,9 @@ testQemuMonitorJSONqemuMonitorJSONSetBlockIoThrottle(const void *data)
>>     if (!test)
>>         return -1;
>>
>>-    expectedInfo = (virDomainBlockIoTuneInfo) {1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14, 15, 16, 17, 18, 19};
>>+    expectedInfo = (virDomainBlockIoTuneInfo) {1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, NULL, 15, 16, 17, 18, 19, 20};

Also I believe the numbers are different just to check that they won't
be swapped or something.  You can keep them, it doesn't have to be the
number of the parameter.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 801 bytes
Desc: Digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20161102/5a196442/attachment-0001.sig>


More information about the libvir-list mailing list