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 <[email protected]>
> ---
>  .../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>


Attachment: signature.asc
Description: OpenPGP digital signature

--
libvir-list mailing list
[email protected]
https://www.redhat.com/mailman/listinfo/libvir-list

Reply via email to