[PATCH] [Autotest PATCH v2] KVM-test: Add a subtest 'qemu_img'

2010-03-31 Thread Yolkfull Chow
is not supported currently. Others runs good from my side. Signed-off-by: Yolkfull Chow yz...@redhat.com --- client/tests/kvm/tests/qemu_img.py | 280 client/tests/kvm/tests_base.cfg.sample | 40 + 2 files changed, 320 insertions(+), 0 deletions(-) create

[PATCH] KVM test: Put os.kill in kvm_stat into try block to avoid traceback

2010-03-29 Thread Yolkfull Chow
Sometimes it tried to kill an already terminated process which can cause a traceback. This patch fixes the problem. Signed-off-by: Yolkfull Chow yz...@redhat.com --- client/profilers/kvm_stat/kvm_stat.py |5 - 1 files changed, 4 insertions(+), 1 deletions(-) diff --git a/client

Re: [Autotest] [Autotest PATCH] KVM-test: Add a subtest 'qemu_img'

2010-03-29 Thread Yolkfull Chow
) --- But I don't think it's a good idea. What do you think? On Fri, Jan 29, 2010 at 4:00 AM, Yolkfull Chow yz...@redhat.com wrote: This is designed to test all subcommands of 'qemu-img' however so far 'commit' is not implemented. * For 'check' subcommand test, it will 'dd' to create a file

[PATCH] KVM-test: SR-IOV: Fix a bug that wrongly check VFs count

2010-03-10 Thread Yolkfull Chow
-off-by: Yolkfull Chow yz...@redhat.com --- client/tests/kvm/kvm_utils.py | 19 +-- 1 files changed, 13 insertions(+), 6 deletions(-) diff --git a/client/tests/kvm/kvm_utils.py b/client/tests/kvm/kvm_utils.py index 4565dc1..1813ed1 100644 --- a/client/tests/kvm/kvm_utils.py +++ b

Re: How to disable KVM start at boot

2010-02-25 Thread Yolkfull Chow
On Thu, Feb 25, 2010 at 05:23:15PM +0800, sati...@pacific.net.hk wrote: Quoting Yolkfull Chow yz...@redhat.com: $ lsmod | grep kvm kvm_amd38452 0 kvm 163952 1 kvm_amd # chkconfig --level 35 kvm off error reading information on service kvm

Re: How to disable KVM start at boot

2010-02-24 Thread Yolkfull Chow
On Thu, Feb 25, 2010 at 12:25:07PM +0800, sati...@pacific.net.hk wrote: Quoting Hao, Xudong xudong@intel.com: That's right. I run it on RHEL system, F12 should be same, you can have a try. Hi Hao, $ lsmod | grep kvm kvm_amd38452 0 kvm 163952 1

[PATCH] KVM-test: SR-IOV: fix a bug that misplaced parameters location

2010-02-08 Thread Yolkfull Chow
Reverse both parameters of utils.open_write_close() that was misplaced. Signed-off-by: Yolkfull Chow yz...@redhat.com --- client/tests/kvm/kvm_utils.py |5 ++--- 1 files changed, 2 insertions(+), 3 deletions(-) diff --git a/client/tests/kvm/kvm_utils.py b/client/tests/kvm/kvm_utils.py index

[PATCH] KVM-test: Log the output of failed executed command 'pre/post-command'

2010-02-03 Thread Yolkfull Chow
Sometimes we need the output of failed command 'pre/post-command' when raising error.TestError. Take an example that using post-command to check images of qcow2 format, it will simply tell command failed rather than show '%d errors were found on the image'. Signed-off-by: Yolkfull Chow yz

Re: [KVM-AUTOTEST PATCH 1/1] KVM test: kvm_vm.py: shorten VM.destroy()

2010-02-03 Thread Yolkfull Chow
On Wed, Feb 03, 2010 at 01:21:12PM +0200, Michael Goldish wrote: Call self.pci_assignable.release_devs() in the finally block. Looks good for me. Thanks, Michael, for this cleanup. Signed-off-by: Michael Goldish mgold...@redhat.com --- client/tests/kvm/kvm_vm.py | 11 ++- 1

Re: [Autotest] [RFC] KVM test: Ship rss.exe and finish.exe binaries with KVM test

2010-02-02 Thread Yolkfull Chow
On Tue, Feb 02, 2010 at 09:48:34AM -0200, Lucas Meneghel Rodrigues wrote: Hi folks: We're on an effort of streamlining the KVM test experience, by choosing sane defaults and helper scripts that can overcome the initial barrier with getting the KVM test running. On one of the conversations

Re: [Autotest] [Autotest PATCH] KVM-test: Add a subtest 'qemu_img'

2010-01-28 Thread Yolkfull Chow
On Tue, Jan 26, 2010 at 03:11:34PM -0200, Lucas Meneghel Rodrigues wrote: On Tue, 2010-01-26 at 11:25 +0800, Yolkfull Chow wrote: This is designed to test all subcommands of 'qemu-img' however so far 'commit' is not implemented. Hi Yolkful, this is very good! Seeing this test made me think

[Autotest PATCH] KVM-test: Add a subtest 'qemu_img'

2010-01-28 Thread Yolkfull Chow
to have the same guest visible content as before. To achieve this, old and new backing file are compared and, if necessary, data is copied from the old backing file into the COW image. Signed-off-by: Yolkfull Chow yz...@redhat.com --- client/tests/kvm/tests/qemu_img.py | 235

Re: [Autotest PATCH] KVM-test: Add a subtest 'qemu_img'

2010-01-28 Thread Yolkfull Chow
On Fri, Jan 29, 2010 at 03:00:09PM +0800, Yolkfull Chow wrote: This is designed to test all subcommands of 'qemu-img' however so far 'commit' is not implemented. * For 'check' subcommand test, it will 'dd' to create a file with specified size and see whether it's supported to be checked

Re: [Autotest] [Autotest PATCH] KVM-test: Add a subtest 'qemu_img'

2010-01-27 Thread Yolkfull Chow
On Tue, Jan 26, 2010 at 03:11:34PM -0200, Lucas Meneghel Rodrigues wrote: On Tue, 2010-01-26 at 11:25 +0800, Yolkfull Chow wrote: This is designed to test all subcommands of 'qemu-img' however so far 'commit' is not implemented. Hi Yolkful, this is very good! Seeing this test made me think

Re: [Autotest] [Autotest PATCH] KVM-test: Add a subtest 'qemu_img'

2010-01-27 Thread Yolkfull Chow
On Wed, Jan 27, 2010 at 07:37:46AM -0500, Michael Goldish wrote: - Yolkfull Chow yz...@redhat.com wrote: On Tue, Jan 26, 2010 at 03:11:34PM -0200, Lucas Meneghel Rodrigues wrote: On Tue, 2010-01-26 at 11:25 +0800, Yolkfull Chow wrote: This is designed to test all subcommands

[Autotest PATCH] KVM-test: Add a subtest 'qemu_img'

2010-01-25 Thread Yolkfull Chow
if no errors found. * For 'info' subcommand test, it simply get output from specified image file. Signed-off-by: Yolkfull Chow yz...@redhat.com --- client/tests/kvm/tests/qemu_img.py | 155 client/tests/kvm/tests_base.cfg.sample | 36 2 files changed

Re: [Autotest] [AUTOTEST PATCH 1/2] KVM-test: Fix a bug that about list slice in scan_results.py

2010-01-24 Thread Yolkfull Chow
. Thanks for your analysis. :) - Yolkfull Chow yz...@redhat.com wrote: If 'info_list' is empty slice operation can result in traceback: ... info_list[-1] = parts[5] IndexError: list assignment index out of range Signed-off-by: Yolkfull Chow yz...@redhat.com

Re: PCI Passthrough Problem

2010-01-21 Thread Yolkfull Chow
On Thu, Jan 21, 2010 at 09:24:36PM -0800, Aaron Clausen wrote: I'm trying once again to get PCI passthrough working (KVM 84 on Ubuntu 9.10), and I'm getting this error : LC_ALL=C PATH=/usr/local/sbin:/usr/local/bin:/sbin:/bin:/usr/sbin:/usr/bin /usr/bin/kvm -S -M pc-0.11 -m 4096 -smp 4 -name

[Autotest PATCH] KVM-test: subtest migrate: Use 'wait_for_login' to log into migrated guest

2010-01-20 Thread Yolkfull Chow
Using 'wait_for' for logging into migrated guest repeats the work of 'wait_for_login' which exists already. We just need to change the name of 'dest_vm'. Signed-off-by: Yolkfull Chow yz...@redhat.com --- client/tests/kvm/kvm_test_utils.py |1 + client/tests/kvm/tests/migration.py |7

[Autotest PATCH] KVM-test: Fix a bug that pci_assignable type name mismatch

2010-01-18 Thread Yolkfull Chow
The pci_assignable type name is nic_vf/nic_pf in kvm_utils.py whereas in kvm_vm.py they are vf/pf. Weird that why I tested it pass last week. Hope that it will not bring in any trouble before this patch applied. Signed-off-by: Yolkfull Chow yz...@redhat.com --- client/tests/kvm/kvm_utils.py

Re: [PATCH] KVM test: Add PCI device assignment support

2010-01-12 Thread Yolkfull Chow
On Sun, Dec 27, 2009 at 09:55:56PM -0200, Lucas Meneghel Rodrigues wrote: Add support to PCI device assignment on the kvm test. It supports both SR-IOV virtual functions and physical NIC card device assignment. Single Root I/O Virtualization (SR-IOV) allows a single PCI device to be shared

Re: [PATCH] KVM test: Add PCI device assignment support

2010-01-12 Thread Yolkfull Chow
. * For physical NIC card pass through, we should specify the device name(s). Looks good for me. Lucas, thank you so much for improving this patch. :-) Signed-off-by: Yolkfull Chow yz...@redhat.com Signed-off-by: Lucas Meneghel Rodrigues l...@redhat.com --- client/tests/kvm/kvm_utils.py

Re: [AUTOTEST PATCH 2/2] KVM-test: Move two 'remote_login' out of try block in kvm_vm.py

2010-01-12 Thread Yolkfull Chow
On Thu, Dec 31, 2009 at 05:03:21AM -0500, Michael Goldish wrote: - Yolkfull Chow yz...@redhat.com wrote: If vm.remote_login failed 'session.close()' can result in exception in finally clause. This patch fix the problem. Signed-off-by: Yolkfull Chow yz...@redhat.com

Re: [Autotest] [AUTOTEST PATCH 2/2] KVM-test: Move two 'remote_login' out of try block in kvm_vm.py

2010-01-12 Thread Yolkfull Chow
On Tue, Jan 12, 2010 at 08:37:25PM -0200, Lucas Meneghel Rodrigues wrote: On Thu, Dec 31, 2009 at 8:03 AM, Michael Goldish mgold...@redhat.com wrote: - Yolkfull Chow yz...@redhat.com wrote: If vm.remote_login failed 'session.close()' can result in exception in finally clause

[Autotest PATCH 1/2] KVM-test: Move two 'remote_login' out of try block in kvm_vm.py

2010-01-12 Thread Yolkfull Chow
'self.remote_login()' should be outside of try block. And as suggested by Michael, we need fix the problem that if self.remote_login() fails, session will be None. Signed-off-by: Yolkfull Chow yz...@redhat.com --- client/tests/kvm/kvm_vm.py | 49 ++- 1

[Autotest PATCH 2/2] KVM-test: subtest guest_s4: Add check of if there's enough space left for S4

2010-01-12 Thread Yolkfull Chow
If disk has no enough space left, check S4 support will fail. Also use 'TestNAError' as error type if guest really doesn't support S4. (Thanks Jason for pointing this out. :) Signed-off-by: Yolkfull Chow yz...@redhat.com --- client/tests/kvm/tests/guest_s4.py | 10 +- 1 files changed

[Autotest PATCH 1/2] KVM-test: linux_s3 subtest: Tune up timeout for suspend command

2010-01-05 Thread Yolkfull Chow
As suggested by Jason, timeout value can be various if different guest CPU number specified. This patch fixs the problem. Signed-off-by: Yolkfull Chow yz...@redhat.com --- client/tests/kvm/tests/linux_s3.py |3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/client/tests

[Autotest PATCH 2/2] KVM-test: guest_s4 subtest: Tune up timeout value for `set_s4_cmd' command

2010-01-05 Thread Yolkfull Chow
Signed-off-by: Yolkfull Chow yz...@redhat.com --- client/tests/kvm/tests/guest_s4.py |3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/client/tests/kvm/tests/guest_s4.py b/client/tests/kvm/tests/guest_s4.py index f08b9d2..82572f6 100644 --- a/client/tests/kvm/tests

[Autotest PATCH] KVM-test: Add a subtest image_copy

2010-01-05 Thread Yolkfull Chow
Add image_copy subtest for convenient KVM functional testing. The target image will be copied into the linked directory if link 'images' is created, and copied to the directory specified in config file otherwise. Signed-off-by: Yolkfull Chow yz...@redhat.com --- client/tests/kvm/kvm_utils.py

[Autotest PATCH] KVM-test: Add a subtest image_copy

2010-01-04 Thread Yolkfull Chow
Add image_copy subtest for convenient KVM functional testing. The target image will be copied into the linked directory if link 'images' is created, and copied to the directory specified in config file otherwise. Signed-off-by: Yolkfull Chow yz...@redhat.com --- client/tests/kvm/kvm_utils.py

Re: [Autotest] [Autotest PATCH] KVM-test: Add a subtest image_copy

2010-01-04 Thread Yolkfull Chow
On Mon, Jan 04, 2010 at 10:52:13PM +0800, Amos Kong wrote: On Mon, Jan 04, 2010 at 05:30:21PM +0800, Yolkfull Chow wrote: Add image_copy subtest for convenient KVM functional testing. The target image will be copied into the linked directory if link 'images' is created, and copied

[AUTOTEST PATCH 1/2] KVM-test: Fix a bug that about list slice in scan_results.py

2009-12-30 Thread Yolkfull Chow
If 'info_list' is empty slice operation can result in traceback: ... info_list[-1] = parts[5] IndexError: list assignment index out of range Signed-off-by: Yolkfull Chow yz...@redhat.com --- client/tests/kvm/scan_results.py |1 + 1 files changed, 1 insertions(+), 0 deletions

Re: [Autotest] [Autotest PATCH] KVM test: No need close session when login timeout

2009-12-28 Thread Yolkfull Chow
On Sat, Dec 26, 2009 at 10:07:58AM -0500, Michael Goldish wrote: - Amos Kong ak...@redhat.com wrote: On Fri, Dec 25, 2009 at 08:28:18AM -0500, Michael Goldish wrote: - Amos Kong ak...@redhat.com wrote: If login timeout, wait_for() returned 'None' and assigned to

[Autotest PATCH] KVM test: Fixup memory size shown in 'GB' in get_memory_size

2009-12-24 Thread Yolkfull Chow
In guest RHEL-3.9 $mem_chk_cmd will catch memory size in GB which will be computed wrongly in get_memory_size. This patch fix the problem. Thanks akong for pointing this out. Signed-off-by: Yolkfull Chow yz...@redhat.com --- client/tests/kvm/kvm_vm.py |6 +- 1 files changed, 5

[Autotest PATCH] KVM test: physical_resources_check subtest:Fixup `mem_chk_cmd' for rhel3.9

2009-12-23 Thread Yolkfull Chow
Signed-off-by: Yolkfull Chow yz...@redhat.com --- client/tests/kvm/tests_base.cfg.sample |4 1 files changed, 4 insertions(+), 0 deletions(-) diff --git a/client/tests/kvm/tests_base.cfg.sample b/client/tests/kvm/tests_base.cfg.sample index a403399..f5a55a0 100644 --- a/client/tests

[Autotest PATCH] KVM test: Add a subtest vnc via which interacts with guest

2009-12-17 Thread Yolkfull Chow
Signed-off-by: Yolkfull Chow yz...@redhat.com --- client/tests/kvm/tests/vnc.py | 24 client/tests/kvm/tests_base.cfg.sample |3 +++ 2 files changed, 27 insertions(+), 0 deletions(-) create mode 100644 client/tests/kvm/tests/vnc.py diff --git a/client

[Autotest] [PATCH] KVM test: subtest stress_boot: Fix a bug that cloned VMs are not screendumped

2009-12-10 Thread Yolkfull Chow
We just used vm.create() to create those cloned VMs whereas ignored catching screendumps of them. This patch fix this problem. Signed-off-by: Yolkfull Chow yz...@redhat.com --- client/tests/kvm/tests/stress_boot.py |8 +++- 1 files changed, 3 insertions(+), 5 deletions(-) diff --git

[AUTOTEST PATCH 1/2 - V3] Add a server-side test - kvm_migration

2009-12-08 Thread Yolkfull Chow
' to 'mig_port'. I also rename 'rem_host' to be 'dest_host'. Signed-off-by: Yolkfull Chow yz...@redhat.com --- client/tests/kvm/kvm_migration.py | 170 client/tests/kvm/kvm_test_utils.py | 27 +++--- client/tests/kvm/kvm_tests.cfg.sample |2 + client/tests

[AUTOTEST PATCH 2/2] KVM test: subtest migration: Add rem_host and rem_port for migrate()

2009-12-08 Thread Yolkfull Chow
Since kvm_test_utils.migrate() adds two arguments to adopt server-side migration. This client side test also needs update. Signed-off-by: Yolkfull Chow yz...@redhat.com --- client/tests/kvm/tests/migration.py |3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/client/tests

[AUTOTEST PATCH] KVM test: subtest block_hotplug: Fixup pci_test_cmd in config file

2009-12-07 Thread Yolkfull Chow
RHEL-4.8 is still using 'hd[a-z]' as harddisk device name. This patch adds 'h' to regular expression in command `pci_test_cmd'. Signed-off-by: Yolkfull Chow yz...@redhat.com --- client/tests/kvm/kvm_tests.cfg.sample |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/client

Re: [Autotest] [PATCH] Add a server-side test - kvm_migration

2009-12-07 Thread Yolkfull Chow
, Yolkfull Chow yz...@redhat.com wrote: This patch will add a server-side test namely kvm_migration. Currently, it will use existing KVM client test framework and add a new file kvm_migration.py to help judge executing routine: source machine or dest machine. * One thing need

[PATCH V2] Add a server-side test - kvm_migration

2009-12-04 Thread Yolkfull Chow
, and then add 'start_vm_for_migration' into dict cloned from original test dict for dest machine. * In order to run this test so far, we need to setup NFS for both source and dest machines. Signed-off-by: Yolkfull Chow yz...@redhat.com --- client/tests/kvm/kvm_migration.py | 165

[PATCH 1/2] Add a server-side test - kvm_migration

2009-12-04 Thread Yolkfull Chow
cloned from original test dict for dest machine. * In order to run this test so far, we need to setup NFS for both source and dest machines. Signed-off-by: Yolkfull Chow yz...@redhat.com --- client/tests/kvm/kvm_migration.py | 165 client/tests/kvm

[PATCH 2/2] KVM test: subtest migration: Add rem_host and rem_port for migrate().

2009-12-04 Thread Yolkfull Chow
Since kvm_test_utils.migrate() adds two arguments to adopt server-side migration. This client side test also needs update. Signed-off-by: Yolkfull Chow yz...@redhat.com --- client/tests/kvm/tests/migration.py |3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/client/tests

Re: [PATCH 1/2] Adds a test to verify resources inside a VM

2009-12-04 Thread Yolkfull Chow
On Sun, Nov 29, 2009 at 11:04:55AM +0200, Yaniv Kaul wrote: On 11/29/2009 9:20 AM, Yolkfull Chow wrote: On Wed, Nov 25, 2009 at 11:35:02AM +0530, sudhir kumar wrote: This patch adds a test for verifying whether the number of cpus and amount of memory as seen inside a guest is same as allocated

Re: [Autotest] [PATCH 1/2] Adds a test to verify resources inside a VM

2009-12-01 Thread Yolkfull Chow
for me. Thanks Lucas for improving this test. Sudhir, what do you think about this? :) Cheers, Yolkfull Cheers, On Sun, Nov 29, 2009 at 8:40 AM, Yolkfull Chow yz...@redhat.com wrote: On Sun, Nov 29, 2009 at 02:22:40PM +0530, sudhir kumar wrote: On Sun, Nov 29, 2009 at 12:50 PM, Yolkfull

[PATCH] Add a server-side test: kvm_migration

2009-12-01 Thread Yolkfull Chow
/migration_control.srv new file mode 100644 index 000..d8b87f1 --- /dev/null +++ b/server/tests/kvm/migration_control.srv @@ -0,0 +1,98 @@ +AUTHOR = Yolkfull Chow yz...@redhat.com +TIME = SHORT +NAME = Migration across Multi-machine +TEST_CATEGORY = Functional +TEST_CLASS = 'Virtualization' +TEST_TYPE

Re: [PATCH] KVM test: Add PCI device assignment support

2009-11-30 Thread Yolkfull Chow
have some ideas about devices_requested parameter, please see comments below: Signed-off-by: Yolkfull Chow yz...@redhat.com Signed-off-by: Lucas Meneghel Rodrigues l...@redhat.com --- client/tests/kvm/kvm_tests.cfg.sample | 20 +++- client/tests/kvm/kvm_utils.py | 278

About implementation of KVM server-side migration in autotest

2009-11-30 Thread Yolkfull Chow
migrate. Initial version of control.srv for server-migration: - AUTHOR = Yolkfull Chow yz...@redhat.com TIME = SHORT NAME = Migration across Multi-machine TEST_CATEGORY = Functional TEST_CLASS = 'Virtualization' TEST_TYPE = Server DOC = Migrate KVM guest between two hosts. Arguments to run_test

Re: [PATCH 1/2] Adds a test to verify resources inside a VM

2009-11-29 Thread Yolkfull Chow
On Sun, Nov 29, 2009 at 02:22:40PM +0530, sudhir kumar wrote: On Sun, Nov 29, 2009 at 12:50 PM, Yolkfull Chow yz...@redhat.com wrote: On Wed, Nov 25, 2009 at 11:35:02AM +0530, sudhir kumar wrote: This patch adds a test for verifying whether the number of cpus and amount of memory as seen

[PATCH] KVM test: Add a subtest params_verify

2009-11-29 Thread Yolkfull Chow
This patch will test following parameters of a VM: 1) count of CPU, hard disks and NICs 2) memory size 3) model of hard disks and NICs 4) NICs' mac address 5) UUID and serial number (if defined the command in config file) Signed-off-by: Yolkfull Chow yz...@redhat.com --- client/bin

Re: [PATCH 1/2] Adds a test to verify resources inside a VM

2009-11-28 Thread Yolkfull Chow
On Wed, Nov 25, 2009 at 11:35:02AM +0530, sudhir kumar wrote: This patch adds a test for verifying whether the number of cpus and amount of memory as seen inside a guest is same as allocated to it on the qemu command line. Hello Sudhir, Please see embedded comments as below:

[PATCH] KVM test: Fix two typos in config file

2009-11-18 Thread Yolkfull Chow
Signed-off-by: Yolkfull Chow yz...@redhat.com --- client/tests/kvm/kvm_tests.cfg.sample |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/client/tests/kvm/kvm_tests.cfg.sample b/client/tests/kvm/kvm_tests.cfg.sample index ac9ef66..7f37994 100644 --- a/client/tests/kvm

Re: [Autotest] [KVM-AUTOTEST PATCH 5/7] KVM test: minor pci_hotplug fixes

2009-11-05 Thread Yolkfull Chow
On Thu, Nov 05, 2009 at 12:01:10PM +0200, Michael Goldish wrote: - Put the PCI device removal code in a finally clause. Hi Michael, I have a little concern with the removal procedure. Thinking about if pci_add failed, the output will not contain right information including PCI ID. The slice

Re: [Autotest] [KVM-AUTOTEST PATCH 7/7] KVM test: remove monitor socket file when destroying a VM

2009-11-05 Thread Yolkfull Chow
On Thu, Nov 05, 2009 at 12:01:12PM +0200, Michael Goldish wrote: This should slow the rate of accumulation of monitor files in /tmp. Hi Michael, I recommend we use TCP as monitor dev of VM. Two reasons: 1) we don't need to add extra code to remove monitor files 2) it's necessary for some

Re: [Autotest] [PATCH] [RFC] KVM test: Major control file cleanup

2009-11-03 Thread Yolkfull Chow
On Wed, Oct 28, 2009 at 02:04:59PM -0400, Michael Goldish wrote: - Lucas Meneghel Rodrigues l...@redhat.com wrote: On Wed, Oct 28, 2009 at 1:43 PM, Michael Goldish mgold...@redhat.com wrote: Sounds great, except it won't allow you to debug your configuration using kvm_config.py.

[PATCH] Seperate smp from extra_params and add into default VM params

2009-11-02 Thread Yolkfull Chow
We may need leave smp as standalone parameter of VM. Reasons I can proposal: 1) memory is a standalone parameter, so is smp 2) smp parameter is needed in some test case, say VM params_verify Signed-off-by: Yolkfull Chow yz...@redhat.com --- client/tests/kvm/kvm_tests.cfg.sample |3

Re: GDB Debugging

2009-10-24 Thread Yolkfull Chow
On Fri, Oct 23, 2009 at 09:19:40AM -0700, Saksena, Abhishek wrote: Hi Guys, Any help will be appreciated on following issue. I have been struggling on this for quite some time... -Abhishek -Original Message- From: Saksena, Abhishek Sent: Tuesday, October 20, 2009

[PATCH] Add 'downscript=no' into kvm command line

2009-10-20 Thread Yolkfull Chow
If no downscript is assigned, add 'downscript=no' to avoid error: /etc/qemu-ifdown: could not launch network script Signed-off-by: Yolkfull Chow yz...@redhat.com --- client/tests/kvm/kvm_vm.py |2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/client/tests/kvm/kvm_vm.py b

Re: [PATCH] KVM test: Add PCI pass through test

2009-10-15 Thread Yolkfull Chow
-by: Yolkfull Chow yz...@redhat.com --- client/tests/kvm/kvm_tests.cfg.sample | 11 ++- client/tests/kvm/kvm_utils.py | 278 + client/tests/kvm/kvm_vm.py| 72 + 3 files changed, 360 insertions(+), 1 deletions(-) diff --git

Re: [Autotest] [PATCH] Add pass through feature test (support SR-IOV)

2009-10-15 Thread Yolkfull Chow
. Thanks! Lucas, thank you very much for adding a detailed explanation and improving for this test. I had reviewed the new patch and some new consideration came to my mind. I had added them on the email, please reviewed. :) On Mon, Sep 14, 2009 at 11:20 PM, Yolkfull Chow yz...@redhat.com

Re: [Autotest] [PATCH] Add a kvm test guest_s4 which supports both Linux and Windows platform

2009-10-14 Thread Yolkfull Chow
On Wed, Oct 14, 2009 at 06:58:01AM -0300, Lucas Meneghel Rodrigues wrote: On Tue, Oct 13, 2009 at 11:54 PM, Yolkfull Chow yz...@redhat.com wrote: On Tue, Oct 13, 2009 at 05:29:40PM -0300, Lucas Meneghel Rodrigues wrote: Hi Yolkfull and Chen: Thanks for your test! I have some comments

[PATCH] Little bug fix in pci_hotplug.py

2009-10-13 Thread Yolkfull Chow
If command executed timeout, the return value of status could be None, which is missed in judge statement: if s: ... Thanks Jason Wang for pointing this out. Signed-off-by: Yolkfull Chow yz...@redhat.com --- client/tests/kvm/tests/pci_hotplug.py |2 +- 1 files changed, 1 insertions

Re: [Autotest] [PATCH] Add a kvm test guest_s4 which supports both Linux and Windows platform

2009-10-13 Thread Yolkfull Chow
, Yolkfull Chow yz...@redhat.com wrote: For this case, Ken Cao wrote the linux part previously and I did extensive modifications on Windows platform support. Signed-off-by: Ken Cao k...@redhat.com Signed-off-by: Yolkfull Chow yz...@redhat.com ---  client/tests/kvm/kvm_tests.cfg.sample

Re: [Autotest] [PATCH] Fix a bug in function create in kvm_vm

2009-10-10 Thread Yolkfull Chow
, thanks for checking. On Mon, Sep 28, 2009 at 4:45 AM, Yolkfull Chow yz...@redhat.com wrote: Signed-off-by: Yolkfull Chow yz...@redhat.com ---  client/tests/kvm/kvm_vm.py |    2 +-  1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/client/tests/kvm/kvm_vm.py b/client

Re: [Autotest] [PATCH] Fix a bug in function create in kvm_vm

2009-10-10 Thread Yolkfull Chow
On Sat, Oct 10, 2009 at 04:24:45PM +0800, Yolkfull Chow wrote: On Mon, Oct 05, 2009 at 04:03:22PM -0300, Lucas Meneghel Rodrigues wrote: Hi Yolkfull! I've checked your patch, but it turns out that the comma is valid syntax for the logging module. By any chance you actually had an error

[PATCH] Add two parameters for wait_for_login

2009-09-30 Thread Yolkfull Chow
Sometimes we need login to guest using different start_time and step_time. Signed-off-by: Yolkfull Chow yz...@redhat.com --- client/tests/kvm/kvm_test_utils.py |6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/client/tests/kvm/kvm_test_utils.py b/client/tests/kvm

[PATCH] Fix a bug in function create in kvm_vm

2009-09-28 Thread Yolkfull Chow
Signed-off-by: Yolkfull Chow yz...@redhat.com --- client/tests/kvm/kvm_vm.py |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/client/tests/kvm/kvm_vm.py b/client/tests/kvm/kvm_vm.py index 55220f9..8ae 100755 --- a/client/tests/kvm/kvm_vm.py +++ b/client/tests/kvm

[PATCH] Add a kvm test guest_s4 which supports both Linux and Windows platform

2009-09-27 Thread Yolkfull Chow
For this case, Ken Cao wrote the linux part previously and I did extensive modifications on Windows platform support. Signed-off-by: Ken Cao k...@redhat.com Signed-off-by: Yolkfull Chow yz...@redhat.com --- client/tests/kvm/kvm_tests.cfg.sample | 14 +++ client/tests/kvm/tests/guest_s4.py

[PATCH] Change log message of VM login

2009-09-27 Thread Yolkfull Chow
We may use this function 'wait_for_login' for several times in a case, only the first time login could be Waiting guest to be up. Signed-off-by: Yolkfull Chow yz...@redhat.com --- client/tests/kvm/kvm_test_utils.py |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/client

Re: [Autotest] [KVM-AUTOTEST PATCH 2/4] KVM test: rss.cpp: send characters to the console window rather than directly to STDIN

2009-09-22 Thread Yolkfull Chow
On Mon, Sep 21, 2009 at 08:30:26AM -0400, Michael Goldish wrote: - Yolkfull Chow yz...@redhat.com wrote: On Sun, Sep 20, 2009 at 06:16:28PM +0300, Michael Goldish wrote: Some Windows programs behave badly when their STDIN is redirected to a pipe (most notably wmic). Therefore

Re: [Autotest] [KVM-AUTOTEST PATCH 2/4] KVM test: rss.cpp: send characters to the console window rather than directly to STDIN

2009-09-21 Thread Yolkfull Chow
On Sun, Sep 20, 2009 at 06:16:28PM +0300, Michael Goldish wrote: Some Windows programs behave badly when their STDIN is redirected to a pipe (most notably wmic). Therefore, keep STDIN unredirected, and send input to the console window as a series of WM_CHAR messages. Hi Michael, I just

Re: [Autotest] [KVM-AUTOTEST PATCH 0/7] KVM test: support for the new remote shell server for Windows

2009-09-18 Thread Yolkfull Chow
On Thu, Sep 17, 2009 at 11:40:46AM -0400, Michael Goldish wrote: - Michael Goldish mgold...@redhat.com wrote: - Yolkfull Chow yz...@redhat.com wrote: On Tue, Aug 18, 2009 at 06:30:14PM -0400, Michael Goldish wrote: - Lucas Meneghel Rodrigues l...@redhat.com wrote

Re: [Autotest] [PATCH 12/19] KVM test: Add new module kvm_test_utils.py

2009-09-14 Thread Yolkfull Chow
On Mon, Sep 14, 2009 at 10:58:01AM +0300, Uri Lublin wrote: On 09/14/2009 08:26 AM, Yolkfull Chow wrote: On Wed, Sep 09, 2009 at 09:12:05PM +0300, Michael Goldish wrote: This module is meant to reduce code size by performing common test procedures. Generally, code here should look like test

Re: [Autotest] [KVM-AUTOTEST PATCH 0/7] KVM test: support for the new remote shell server for Windows

2009-09-14 Thread Yolkfull Chow
On Tue, Aug 18, 2009 at 06:30:14PM -0400, Michael Goldish wrote: - Lucas Meneghel Rodrigues l...@redhat.com wrote: On Tue, Aug 18, 2009 at 7:15 AM, Michael Goldishmgold...@redhat.com wrote: - Lucas Meneghel Rodrigues l...@redhat.com wrote: Ok, very good, similarly to

[PATCH] Add pass through feature test (support SR-IOV)

2009-09-14 Thread Yolkfull Chow
It supports both SR-IOV virtual functions' and physical NIC card pass through. * For SR-IOV virtual functions passthrough, we could specify the module parameter 'max_vfs' in config file. * For physical NIC card pass through, we should specify the device name(s). Signed-off-by: Yolkfull

Re: [Autotest] [PATCH 12/19] KVM test: Add new module kvm_test_utils.py

2009-09-13 Thread Yolkfull Chow
On Wed, Sep 09, 2009 at 09:12:05PM +0300, Michael Goldish wrote: This module is meant to reduce code size by performing common test procedures. Generally, code here should look like test code. More specifically: - Functions in this module should raise exceptions if things go wrong

Re: [Autotest] [KVM-AUTOTEST PATCH v2 1/3] KVM test: add AutoIt test

2009-08-11 Thread Yolkfull Chow
On Tue, Aug 11, 2009 at 03:10:42PM +0300, Michael Goldish wrote: Currently the test only logs in, runs a given script and fails if the script takes too long to exit or if its exit status is nonzero. The test expects these parameters: autoit_binary: Path to AutoIt binary in the guest.

Re: [Autotest] [KVM-AUTOTEST PATCH v2 1/3] KVM test: add AutoIt test

2009-08-11 Thread Yolkfull Chow
On Tue, Aug 11, 2009 at 09:27:17AM -0400, Michael Goldish wrote: - Yolkfull Chow yz...@redhat.com wrote: On Tue, Aug 11, 2009 at 03:10:42PM +0300, Michael Goldish wrote: Currently the test only logs in, runs a given script and fails if the script takes too long to exit or if its

Re: [KVM-AUTOTEST PATCH 12/12] KVM test: make stress_boot work properly with TAP networking

2009-08-04 Thread Yolkfull Chow
On Mon, Aug 03, 2009 at 04:00:45AM -0400, Michael Goldish wrote: - Yolkfull Chow yz...@redhat.com wrote: On Mon, Aug 03, 2009 at 02:45:23AM -0400, Michael Goldish wrote: - Yolkfull Chow yz...@redhat.com wrote: Hi Michael, I just have some comments on what you changed

Re: [PATCH] Add a subtest pci_hotplug in kvm test

2009-08-04 Thread Yolkfull Chow
.nic_hotplug.nic_virtio GOOD 99 completed successfully Win2008.32.block_hotplug.fmt_qcow2.block_scsi GOOD 90 completed successfully GOOD 953 Signed-off-by: Yolkfull Chow yz...@redhat.com

Re: [PATCH] Add a subtest pci_hotplug in kvm test

2009-08-04 Thread Yolkfull Chow
GOOD 953 Signed-off-by: Yolkfull Chow yz...@redhat.com --- client/tests/kvm/kvm.py |1 + client/tests/kvm/kvm_tests.cfg.sample | 67 + client/tests/kvm/kvm_tests.py | 105 + client/tests/kvm/kvm_vm.py

Re: [KVM-AUTOTEST PATCH 12/12] KVM test: make stress_boot work properly with TAP networking

2009-08-03 Thread Yolkfull Chow
On Mon, Aug 03, 2009 at 02:45:23AM -0400, Michael Goldish wrote: - Yolkfull Chow yz...@redhat.com wrote: Hi Michael, I just have some comments on what you changed on stress_boot. :-) On Mon, Aug 03, 2009 at 02:58:21AM +0300, Michael Goldish wrote: Take an additional

Re: [PATCH] Add a subtest pci_hotplug in kvm test

2009-08-03 Thread Yolkfull Chow
On 06/30/2009 09:58 PM, Dor Laor wrote: On 06/30/2009 02:11 PM, Yolkfull Chow wrote: Signed-off-by: Yolkfull Chowyz...@redhat.com --- client/tests/kvm/kvm.py |1 + client/tests/kvm/kvm_tests.cfg.sample | 56 client/tests/kvm/kvm_tests.py

Re: [PATCH] Add a subtest pci_hotplug in kvm test

2009-08-03 Thread Yolkfull Chow
On Mon, Aug 03, 2009 at 02:37:29PM +0300, Dor Laor wrote: On 08/03/2009 12:19 PM, Yolkfull Chow wrote: On 06/30/2009 09:58 PM, Dor Laor wrote: On 06/30/2009 02:11 PM, Yolkfull Chow wrote: Signed-off-by: Yolkfull Chowyz...@redhat.com --- client/tests/kvm/kvm.py | 1 + client/tests/kvm

Re: [KVM-AUTOTEST PATCH 12/12] KVM test: make stress_boot work properly with TAP networking

2009-08-02 Thread Yolkfull Chow
Hi Michael, I just have some comments on what you changed on stress_boot. :-) On Mon, Aug 03, 2009 at 02:58:21AM +0300, Michael Goldish wrote: Take an additional parameter 'clone_address_index_base' which indicates the initial value for 'address_index' for the cloned VMs. This value is

Re: [PATCH] Specify the system UUID for VM

2009-07-29 Thread Yolkfull Chow
On Wed, Jul 29, 2009 at 03:18:51PM +0300, Avi Kivity wrote: On 07/16/2009 01:26 PM, Yolkfull Chow wrote: Signed-off-by: Yolkfull Chowyz...@redhat.com --- client/tests/kvm/kvm_vm.py | 11 +++ 1 files changed, 11 insertions(+), 0 deletions(-) diff --git a/client/tests/kvm

Re: [PATCH] Specify the system UUID for VM

2009-07-29 Thread Yolkfull Chow
On Wed, Jul 29, 2009 at 09:06:25PM +0800, Yolkfull Chow wrote: On Wed, Jul 29, 2009 at 03:18:51PM +0300, Avi Kivity wrote: On 07/16/2009 01:26 PM, Yolkfull Chow wrote: Signed-off-by: Yolkfull Chowyz...@redhat.com --- client/tests/kvm/kvm_vm.py | 11 +++ 1 files changed, 11

Re: [Autotest] [PATCH] Add a kvm subtest -- pci_hotplug, which supports both Windows OS and Linux OS.

2009-07-28 Thread Yolkfull Chow
On Tue, Jul 28, 2009 at 02:03:10AM -0300, Lucas Meneghel Rodrigues wrote: On Thu, Jul 23, 2009 at 4:18 AM, Yolkfull Chowyz...@redhat.com wrote: Hi Yaniv, following is the output from Windows guest: --- Microsoft DiskPart version 6.0.6001 Copyright (C) 1999-2007 Microsoft Corporation.

Re: [PATCH] Add a kvm subtest -- pci_hotplug, which supports both Windows OS and Linux OS.

2009-07-23 Thread Yolkfull Chow
On Tue, Jul 21, 2009 at 10:45:31AM +0300, Yaniv Kaul wrote: On 7/21/2009 9:11 AM, Yolkfull Chow wrote: SNIP Previously, I used 'create partition primary' to verify whether the disk could be formatted but always got an error: --- diskpart has encountered an error

Re: [PATCH] Add a kvm subtest -- pci_hotplug, which supports both Windows OS and Linux OS.

2009-07-21 Thread Yolkfull Chow
On Tue, Jul 21, 2009 at 10:45:31AM +0300, Yaniv Kaul wrote: On 7/21/2009 9:11 AM, Yolkfull Chow wrote: SNIP Previously, I used 'create partition primary' to verify whether the disk could be formatted but always got an error: --- diskpart has encountered an error

Re: [PATCH] Add a kvm subtest -- pci_hotplug, which supports both Windows OS and Linux OS.

2009-07-21 Thread Yolkfull Chow
On Tue, Jul 21, 2009 at 10:45:31AM +0300, Yaniv Kaul wrote: On 7/21/2009 9:11 AM, Yolkfull Chow wrote: SNIP Previously, I used 'create partition primary' to verify whether the disk could be formatted but always got an error: --- diskpart has encountered an error

Re: [Autotest] [KVM-AUTOTEST PATCH 12/17] KVM test: add simple timedrift test (mainly for Windows)

2009-07-21 Thread Yolkfull Chow
On Mon, Jul 20, 2009 at 06:07:19PM +0300, Michael Goldish wrote: 1) Log into a guest. 2) Take a time reading from the guest and host. 3) Run load on the guest and host. 4) Take a second time reading. 5) Stop the load and rest for a while. 6) Take a third time reading. 7) If the drift

Re: [Autotest] [KVM-AUTOTEST PATCH 12/17] KVM test: add simple timedrift test (mainly for Windows)

2009-07-21 Thread Yolkfull Chow
On Tue, Jul 21, 2009 at 11:29:56AM -0400, Michael Goldish wrote: - Yolkfull Chow yz...@redhat.com wrote: On Mon, Jul 20, 2009 at 06:07:19PM +0300, Michael Goldish wrote: 1) Log into a guest. 2) Take a time reading from the guest and host. 3) Run load on the guest and host. 4

[PATCH] Add UUID option into kvm command line

2009-07-17 Thread Yolkfull Chow
Signed-off-by: Yolkfull Chow yz...@redhat.com --- client/tests/kvm/kvm_vm.py | 24 1 files changed, 24 insertions(+), 0 deletions(-) diff --git a/client/tests/kvm/kvm_vm.py b/client/tests/kvm/kvm_vm.py index 503f636..48f2916 100644 --- a/client/tests/kvm/kvm_vm.py

Re: [PATCH] Specify the system UUID for VM

2009-07-16 Thread Yolkfull Chow
On Thu, Jul 16, 2009 at 06:26:46PM +0800, Yolkfull Chow wrote: Signed-off-by: Yolkfull Chow yz...@redhat.com --- client/tests/kvm/kvm_vm.py | 11 +++ 1 files changed, 11 insertions(+), 0 deletions(-) diff --git a/client/tests/kvm/kvm_vm.py b/client/tests/kvm/kvm_vm.py index

[PATCH] Specify the system UUID for VM

2009-07-16 Thread Yolkfull Chow
Signed-off-by: Yolkfull Chow yz...@redhat.com --- client/tests/kvm/kvm_vm.py | 12 1 files changed, 12 insertions(+), 0 deletions(-) diff --git a/client/tests/kvm/kvm_vm.py b/client/tests/kvm/kvm_vm.py index 503f636..5f81965 100644 --- a/client/tests/kvm/kvm_vm.py +++ b/client

Re: [Autotest] [PATCH] Assign an UUID for each VM in kvm command line

2009-07-15 Thread Yolkfull Chow
On 07/15/2009 09:36 PM, Dor Laor wrote: On 07/15/2009 12:12 PM, Yolkfull Chow wrote: Would submit this patch which is from our internal kvm-autotest patches submitted by Jason. So that we could go on test case about parameters verification(UUID, DMI data etc). Signed-off-by: Yolkfull Chowyz

Re: [Autotest] [PATCH] Add a subtest pci_hotplug in kvm test

2009-07-14 Thread Yolkfull Chow
:57 PM, Yolkfull Chow wrote: Signed-off-by: Yolkfull Chowyz...@redhat.com --- client/tests/kvm/kvm.py |1 + client/tests/kvm/kvm_tests.cfg.sample | 65 ++- client/tests/kvm/kvm_tests.py | 94 + client

[PATCH] Add a kvm subtest -- pci_hotplug, which supports both Windows OS and Linux OS.

2009-07-14 Thread Yolkfull Chow
successfully Win2008.32.block_hotplug.fmt_qcow2.block_scsi GOOD186 completed successfully Win2008.32.block_hotplug.fmt_raw.block_scsi GOOD71 completed successfully Signed-off-by: Yolkfull Chow yz...@redhat.com --- client/tests/kvm/kvm.py

  1   2   >