On 08/28/2012 11:39 AM, Yu Mingfei wrote:
Signed-off-by: Yu Mingfei<[email protected]>
---
  client/virt/subtests.cfg.sample |   63
+++++++++++++++++++++++++++++++++++++++
  1 files changed, 63 insertions(+), 0 deletions(-)

diff --git a/client/virt/subtests.cfg.sample
b/client/virt/subtests.cfg.sample
index 8e73c0e..1e43299 100644
--- a/client/virt/subtests.cfg.sample
+++ b/client/virt/subtests.cfg.sample
@@ -361,6 +361,69 @@ variants:
                  addition_status_error = "yes"
                  libvirtd = "off"

+    - virsh_setmem:
+        type = virsh_setmem
+        kill_vm = yes
+        kill_vm_on_error = yes
+        options = ""

How about 'virsh_setmem_options' ?

+        options_prefix = ""
+        options_suffix = ""

'virh_setmem_options_prefix' ?

+        setmem_vm = "%s"
+        variants:
+            - with_valid_vm:
+                vm_status_error = no
+                variants:
+                    - domid:
+                        vm_ref = "domid"
+                    - domname:
+                        vm_ref = "domname"
+                    - domuuid:
+                        vm_ref = "domuuid"
+            - with_invalid_vm:
+                vm_status_error = yes
+                variants:
+                    - none:
+                        vm_ref = ""
+                    - hex_domid:
+                        vm_ref = "domid"
+                        setmem_vm = "%x"
+                    - invalid_domid:
+                        vm_ref = "9999"
+                    - invalid_domuuid:
+                        vm_ref = "99999999-9999-9999-9999-999999999999"
+        variants:
+            - with_valid_mem:
+                mem_status_error = no
+                options_ref = "default_half"
+            - with_invalid_mem:
+                mem_status_error = yes
+                variants:
+                    - none:
+                        options_ref = ""
+                    - zero:
+                        options_ref = "0"
+                    - memory1:
+                        options_ref = "8388608"
+        variants:
+            - normal_test:
+            - with_vm_state:
+                only with_valid_mem.with_valid_vm.domname
+                variants:
+                    - paused:
+                        addition_status_error = no
+                        paused_after_start_vm = yes
+                    - shut_off:
+                        addition_status_error = yes
+                        start_vm = no
+            - with_libvirtd_stop:
+                only with_valid_mem.with_valid_vm.domname
+                addition_status_error = yes
+                libvirt = "off"
+            - with_addtion_args:
+                only with_valid_mem.with_valid_vm.domname
+                addition_status_error = yes
+                options_suffix = "xyz"
+
      - module_probe:
          type = module_probe
          # You can specify your own module list, though it is not
needed usually.
-- 1.7.1

--
Best Regards
Yu Mingfei


Otherwise this looks good to me.

--
Chris Evich, RHCA, RHCE, RHCDS, RHCSS
Quality Assurance Engineer
e-mail: cevich + `@' + redhat.com o: 1-888-RED-HAT1 x44214

_______________________________________________
Autotest-kernel mailing list
[email protected]
https://www.redhat.com/mailman/listinfo/autotest-kernel

Reply via email to