On 11/12/2014 04:16 PM, John Ferlan wrote:
The recent commit to add support for block_set_io_throttle
parameters
from version 1.7 of qemu did not add any tests - this adds the tests
Signed-off-by: John Ferlan <jferlan(a)redhat.com>
---
.../qemuxml2argv-blkdeviotune-max.args | 12 +++++
.../qemuxml2argv-blkdeviotune-max.xml | 51 ++++++++++++++++++++++
tests/qemuxml2argvtest.c | 3 ++
3 files changed, 66 insertions(+)
create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-blkdeviotune-max.args
create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-blkdeviotune-max.xml
+ <source dev='/dev/HostVG/QEMUGuest2'/>
+ <target dev='hdb' bus='ide'/>
+ <iotune>
+ <read_bytes_sec>5000</read_bytes_sec>
+ <write_bytes_sec>5000</write_bytes_sec>
+ <read_iops_sec>3500</read_iops_sec>
+ <write_iops_sec>3500</write_iops_sec>
+ <read_bytes_sec_max>5000</read_bytes_sec_max>
+ <write_bytes_sec_max>5000</write_bytes_sec_max>
+ <read_iops_sec_max>3500</read_iops_sec_max>
+ <write_iops_sec_max>3500</write_iops_sec_max>
It would be nicer to use different values for different options, but coverity
already found the copy and paste error :)
Jan
+ <size_iops_sec>2000</size_iops_sec>
+ </iotune>
+ <address type='drive' controller='0' bus='0'
target='0' unit='1'/>
+ </disk>
+ <controller type='usb' index='0'/>
+ <controller type='ide' index='0'/>
+ <controller type='pci' index='0' model='pci-root'/>
+ <memballoon model='virtio'/>
+ </devices>
+</domain>