Re: [Qemu-devel] [PATCH 15/15] qtest: add rtc-test test-case

2012-01-11 Thread Paolo Bonzini
On 01/11/2012 06:06 PM, Anthony Liguori wrote: The right thing to do would be to run the test with vm_clock for the rtc_clock, add a way for the qtest machine to bump the vm_clock to the next event, I actually was looking at this yesterday. Just bumping to the next event is not enough, you w

Re: [Qemu-devel] [PATCH 15/15] qtest: add rtc-test test-case

2012-01-11 Thread Anthony Liguori
On 01/10/2012 01:59 PM, Paolo Bonzini wrote: On 01/10/2012 08:10 PM, Anthony Liguori wrote: + sec = cmos_read(0x00); + min = cmos_read(0x02); + hour = cmos_read(0x04); + mday = cmos_read(0x07); + mon = cmos_read(0x08); + year = cmos_read(0x09); Please use identifiers for register numbers. +

Re: [Qemu-devel] [PATCH 15/15] qtest: add rtc-test test-case

2012-01-11 Thread Paolo Bonzini
On 01/10/2012 08:10 PM, Anthony Liguori wrote: +sec = cmos_read(0x00); +min = cmos_read(0x02); +hour = cmos_read(0x04); +mday = cmos_read(0x07); +mon = cmos_read(0x08); +year = cmos_read(0x09); Please use identifiers for register numbers. +/* + * This check ass

[Qemu-devel] [PATCH 15/15] qtest: add rtc-test test-case

2012-01-10 Thread Anthony Liguori
Signed-off-by: Anthony Liguori --- tests/Makefile |2 +- tests/rtc-test.c | 222 ++ 2 files changed, 223 insertions(+), 1 deletions(-) create mode 100644 tests/rtc-test.c diff --git a/tests/Makefile b/tests/Makefile index 63b835d..a90ea