Re: [Autotest] [PATCH] KVM-test: Add hdparm subtest

2011-08-03 Thread Lucas Meneghel Rodrigues
It does look sane to me too. However, I find this to work only under virtio_blk. (ide hard drive that qemu has doesn't have the needed ioctls for hdparm to work) Amos and Feng, I'll trust that this somehow should pass on non-virtio, so I won't restrict it to virtio_blk. I made some minor changes t

[PATCH] KVM test: Add hdparm subtest v2

2011-08-03 Thread Lucas Meneghel Rodrigues
From: Amos Kong This test uses 'hdparm' to set disk device to low/high performance status, and compare the reading speed. The emulated device should pass all the tests. Changes from v1: * Little changes in message wording * Better logging of hdparm messages (better looking logs) * Proper prin

Re: [Autotest] [PATCH] KVM-test: Add hdparm subtest

2011-08-03 Thread Lukáš Doktor
Reviewed, looks sane. Lukáš Dne 2.8.2011 05:39, Amos Kong napsal(a): This test uses 'hdparm' to set disk device to low/high performance status, and compare the reading speed. The emulated device should pass all the tests. Signed-off-by: Feng Yang Signed-off-by: Amos Kong --- client/tests/kvm

[PATCH] KVM-test: Add hdparm subtest

2011-08-01 Thread Amos Kong
This test uses 'hdparm' to set disk device to low/high performance status, and compare the reading speed. The emulated device should pass all the tests. Signed-off-by: Feng Yang Signed-off-by: Amos Kong --- client/tests/kvm/tests/hdparm.py | 84 client/t