[KVM-AUTOTEST PATCH] KVM test: support 'cache' and 'serial' params to the qemu -drive option

[KVM-AUTOTEST PATCH] KVM test: support 'cache' and 'serial' params to the qemu -drive option

[Michael Goldish]

Support 'cache' and 'serial'.
Both can be controlled directly from the config file, e.g.:
drive_cache = writeback

or for a specific image 'image1':
drive_cache_image1 = writeback

or for 'image1' of VM 'vm1':
drive_cache_image1_vm1 = writeback

The cache options are listed in the QEMU documentation.
(currently "none", "writeback", or "writethrough" are supported.)

Signed-off-by: Michael Goldish <mgoldish@redhat.com>
---
 client/tests/kvm/kvm_vm.py |    4 ++++
 1 files changed, 4 insertions(+), 0 deletions(-)

diff --git a/client/tests/kvm/kvm_vm.py b/client/tests/kvm/kvm_vm.py
index 9aea3fb..e98ffd7 100644
--- a/client/tests/kvm/kvm_vm.py
+++ b/client/tests/kvm/kvm_vm.py
@@ -226,6 +226,10 @@ class VM:
                                                                image_dir)
             if image_params.get("drive_format"):
                 qemu_cmd += ",if=%s" % image_params.get("drive_format")
+            if image_params.get("drive_cache"):
+                qemu_cmd += ",cache=%s" % image_params.get("drive_cache")
+            if image_params.get("drive_serial"):
+                qemu_cmd += ",serial=%s" % image_params.get("drive_serial")
             if image_params.get("image_snapshot") == "yes":
                 qemu_cmd += ",snapshot=on"
             if image_params.get("image_boot") == "yes":
-- 
1.5.4.1

Re: [KVM-AUTOTEST PATCH] KVM test: support 'cache' and 'serial' params to the qemu -drive option

[Lucas Meneghel Rodrigues]

On Wed, 2009-06-17 at 18:24 +0300, Michael Goldish wrote:
> Support 'cache' and 'serial'.
> Both can be controlled directly from the config file, e.g.:
> drive_cache = writeback
> 
> or for a specific image 'image1':
> drive_cache_image1 = writeback
> 
> or for 'image1' of VM 'vm1':
> drive_cache_image1_vm1 = writeback
> 
> The cache options are listed in the QEMU documentation.
> (currently "none", "writeback", or "writethrough" are supported.)

Applied, thanks!

> Signed-off-by: Michael Goldish <mgoldish@redhat.com>
-- 
Lucas Meneghel Rodrigues
Software Engineer (QE)
Red Hat - Emerging Technologies