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(a)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.