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

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

 



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

Attachment: signature.asc
Description: Digital signature

--
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]