From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([140.186.70.92]:38786) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1RWXit-0006zQ-Te for qemu-devel@nongnu.org; Fri, 02 Dec 2011 13:20:36 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1RWXis-0000uh-RN for qemu-devel@nongnu.org; Fri, 02 Dec 2011 13:20:35 -0500 Received: from mx1.redhat.com ([209.132.183.28]:35069) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1RWXis-0000ud-Kq for qemu-devel@nongnu.org; Fri, 02 Dec 2011 13:20:34 -0500 Date: Fri, 2 Dec 2011 16:20:29 -0200 From: Luiz Capitulino Message-ID: <20111202162029.700f8687@doriath> In-Reply-To: <4ED90EB7.1080300@redhat.com> References: <1322765012-3164-1-git-send-email-aliguori@us.ibm.com> <1322765012-3164-7-git-send-email-aliguori@us.ibm.com> <4ED90A02.8080901@redhat.com> <4ED90A55.4010405@codemonkey.ws> <4ED90EB7.1080300@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [RFC v2 6/6] Add C version of rtc-test List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Kevin Wolf Cc: qemu-devel@nongnu.org On Fri, 02 Dec 2011 18:45:27 +0100 Kevin Wolf wrote: > Am 02.12.2011 18:26, schrieb Anthony Liguori: > > On 12/02/2011 11:25 AM, Kevin Wolf wrote: > >> Am 01.12.2011 19:43, schrieb Anthony Liguori: > >>> --- > >>> Makefile | 4 + > >>> rtc-test.c | 201 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ > >>> 2 files changed, 205 insertions(+), 0 deletions(-) > >>> create mode 100644 rtc-test.c > >>> > >>> diff --git a/Makefile b/Makefile > >>> index 301c75e..838cb01 100644 > >>> --- a/Makefile > >>> +++ b/Makefile > >>> @@ -215,6 +215,10 @@ $(qga-obj-y) qemu-ga.o: $(QGALIB_GEN) $(GENERATED_HEADERS) > >>> > >>> qemu-ga$(EXESUF): qemu-ga.o $(qga-obj-y) $(qapi-obj-y) $(tools-obj-y) $(qobject-obj-y) $(version-obj-y) $(QGALIB_OBJ) > >>> > >>> +libqtest.o: libqtest.c > >> > >> Did you forget to commit libqtest.c? > > > > Probably, sorry about that. > > > >> I think your series is a good start, but the examples only use port I/O. > >> Most other tests would probably need to access RAM (not sure if it's > >> convenient to do explicit read/write commands for that), interrupts and > >> some even QMP. > > > > libqtest.h has: > > > > bool qtest_get_irq(QTestState *s, int num); > > > > void qtest_outb(QTestState *s, uint16_t addr, uint8_t value); > > > > void qtest_outw(QTestState *s, uint16_t addr, uint16_t value); > > > > void qtest_outl(QTestState *s, uint16_t addr, uint32_t value); > > > > uint8_t qtest_inb(QTestState *s, uint16_t addr); > > > > uint16_t qtest_inw(QTestState *s, uint16_t addr); > > > > uint32_t qtest_inl(QTestState *s, uint16_t addr); > > > > void qtest_memread(QTestState *s, uint64_t addr, void *data, size_t size); > > > > void qtest_memwrite(QTestState *s, uint64_t addr, const void *data, size_t size); > > > > So that's how you read/write memory. Likewise, for IRQs, you can poll the > > status of a given IRQ. I thought about doing some sort of signal magic around > > but when writing tests, polling the IRQ seems easier to deal with. > > Okay, polling interrupts should be good enough for tests. > > I guess the test still needs to do everything that a guest OS would have > to do, for example send an EOI to the PIC? We'll probably want to have a > library for such things then, but we can add it with the first test that > uses interrupts. > > >> Should the framework and the tests live in the tests/ directory? > > > > Probably, except that tests/ has it's own Makefile which is sort of awkward. > > Any objections to moving tests/* to tests/tcg-test and then moving all of the > > various gtest/check tests to tests/ along with qtest? > > No objection, I think this is exactly what we should do. I fully agree.