[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v27 8/8] target/avr: Add tests
From: |
Michael Rolnik |
Subject: |
Re: [Qemu-devel] [PATCH v27 8/8] target/avr: Add tests |
Date: |
Fri, 19 Jul 2019 18:05:13 +0300 |
thanks.
On Fri, Jul 19, 2019 at 4:26 PM Philippe Mathieu-Daudé <address@hidden>
wrote:
> On 7/19/19 10:26 AM, Michael Rolnik wrote:
> > 1. Avocado test
> > The test is based on
> >
> https://github.com/seharris/qemu-avr-tests/tree/master/free-rtos/Demo
> > demo which, If working correctly, prints 'ABCDEFGHIJKLMNOPQRSTUVWX'
> out
>
> Note, that test also demonstrates the timer and irq are working :)
>
> Enabling DB_PRINT:
>
> avr_timer16_interrupt: CTC OCRA overflow
> avr_timer16_interrupt: interrupt, cnt = 253
> avr_timer16_interrupt: CTC OCRA overflow
> avr_timer16_interrupt: interrupt, cnt = 269
> avr_timer16_interrupt: CTC OCRA overflow
> avr_timer16_interrupt: interrupt, cnt = 269
>
> (There is also GPIO accesses but you don't implement them yet).
>
> >
> > 2. Boot serial test
> > Prinit out 'T' through serial port
>
> Excellent!
>
> >
> > Signed-off-by: Michael Rolnik <address@hidden>
> > ---
> > tests/Makefile.include | 2 ++
> > tests/acceptance/machine_avr6.py | 36 ++++++++++++++++++++++++++++++++
> > tests/boot-serial-test.c | 12 +++++++++++
> > 3 files changed, 50 insertions(+)
> > create mode 100644 tests/acceptance/machine_avr6.py
> >
> > diff --git a/tests/Makefile.include b/tests/Makefile.include
> > index fd7fdb8658..8a186dcc23 100644
> > --- a/tests/Makefile.include
> > +++ b/tests/Makefile.include
> > @@ -201,6 +201,8 @@ check-qtest-i386-y +=
> tests/test-x86-cpuid-compat$(EXESUF)
> > check-qtest-i386-y += tests/numa-test$(EXESUF)
> > check-qtest-x86_64-y += $(check-qtest-i386-y)
> >
> > +check-qtest-avr-y += tests/boot-serial-test$(EXESUF)
> > +
> > check-qtest-alpha-y += tests/boot-serial-test$(EXESUF)
> > check-qtest-alpha-$(CONFIG_VGA) += tests/display-vga-test$(EXESUF)
> >
> > diff --git a/tests/acceptance/machine_avr6.py
> b/tests/acceptance/machine_avr6.py
> > new file mode 100644
> > index 0000000000..0601080b01
> > --- /dev/null
> > +++ b/tests/acceptance/machine_avr6.py
> > @@ -0,0 +1,36 @@
> > +import logging
> > +import time
> > +import distutils.spawn
> > +
> > +from avocado import skipUnless
> > +from avocado_qemu import Test
> > +from avocado.utils import process
> > +
> > +class AVR6Machine(Test):
> > + timeout = 5
> > +
> > + def test_freertos(self):
> > + """
> > + :avocado: tags=arch:avr
> > + :avocado: tags=machine:sample
> > + """
> > + """
> > +
> https://github.com/seharris/qemu-avr-tests/raw/master/free-rtos/Demo/AVR_ATMega2560_GCC/demo.elf
> > + constantly prints out
> 'ABCDEFGHIJKLMNOPQRSTUVWXABCDEFGHIJKLMNOPQRSTUVWX'
> > + """
> > + rom_url = 'https://github.com/seharris/qemu-avr-tests'
> > + rom_url +=
> '/raw/master/free-rtos/Demo/AVR_ATMega2560_GCC/demo.elf'
> > + rom_hash = '7eb521f511ca8f2622e0a3c5e8dd686efbb911d4'
> > + rom_path = self.fetch_asset(rom_url, asset_hash=rom_hash)
> > +
> > + self.vm.set_machine('sample')
> > + self.vm.add_args('-bios', rom_path)
> > + self.vm.add_args('-nographic')
> > + self.vm.launch()
> > +
> > + time.sleep(2)
> > + self.vm.shutdown()
> > +
> > + match = 'ABCDEFGHIJKLMNOPQRSTUVWXABCDEFGHIJKLMNOPQRSTUVWX'
> > +
> > + self.assertIn(match, self.vm.get_log())
> > diff --git a/tests/boot-serial-test.c b/tests/boot-serial-test.c
> > index 24852d4c7d..73d273b73f 100644
> > --- a/tests/boot-serial-test.c
> > +++ b/tests/boot-serial-test.c
> > @@ -16,6 +16,17 @@
> > #include "qemu/osdep.h"
> > #include "libqtest.h"
> >
> > +static const uint8_t bios_avr[] = {
> > + 0x89, 0xe1, /* ldi r24, 0x19 */
> > + 0x80, 0x93, 0xc5, 0x00, /* sts 0x00C5, r24 ; set baud rate to 38400
> */
>
> FWIW we can remove the previous two lines, we don't care about the
> baudrate in this test.
>
> > + 0x88, 0xe0, /* ldi r24, 0x08 */
> > + 0x80, 0x93, 0xc1, 0x00, /* sts 0x00C1, r24 ; Enable tx */
> > + 0x86, 0xe0, /* ldi r24, 0x06 */
> > + 0x80, 0x93, 0xc2, 0x00, /* sts 0x00C2, r24 ; Set the data bits to 8
> */
> > + 0x84, 0xe5, /* ldi r24, 0x54 */
> > + 0x80, 0x93, 0xc6, 0x00, /* sts 0x00C6, r24 ; Output 'T' */
> > +};
> > +
> > static const uint8_t kernel_mcf5208[] = {
> > 0x41, 0xf9, 0xfc, 0x06, 0x00, 0x00, /* lea 0xfc060000,%a0 */
> > 0x10, 0x3c, 0x00, 0x54, /* move.b #'T',%d0 */
> > @@ -92,6 +103,7 @@ typedef struct testdef {
> >
> > static testdef_t tests[] = {
> > { "alpha", "clipper", "", "PCI:" },
> > + { "avr", "sample", "", "T", sizeof(bios_avr), NULL, bios_avr },
> > { "ppc", "ppce500", "", "U-Boot" },
> > { "ppc", "40p", "-vga none -boot d", "Trying cd:," },
> > { "ppc", "g3beige", "", "PowerPC,750" },
> >
>
> Testing shows:
>
> TEST check-qtest-avr: tests/boot-serial-test
> qemu-system-avr: Unable to load /tmp/qtest-boot-serial-cOndewD as ELF,
> trying again as raw binary
>
> I wonder if this might fail Peter's testing, so Cc'ing Thomas.
>
> Reviewed-by: Philippe Mathieu-Daudé <address@hidden>
> Tested-by: Philippe Mathieu-Daudé <address@hidden>
>
> Regards,
>
> Phil.
>
--
Best Regards,
Michael Rolnik
- [Qemu-devel] [PATCH v27 6/8] target/avr: Add example board configuration, (continued)
[Qemu-devel] [PATCH v27 7/8] target/avr: Register AVR support with the rest of QEMU, the build system, and the MAINTAINERS file, Michael Rolnik, 2019/07/19
[Qemu-devel] [PATCH v27 8/8] target/avr: Add tests, Michael Rolnik, 2019/07/19
- Re: [Qemu-devel] [PATCH v27 8/8] target/avr: Add tests, Philippe Mathieu-Daudé, 2019/07/19
- Re: [Qemu-devel] [PATCH v27 8/8] target/avr: Add tests,
Michael Rolnik <=
- Re: [Qemu-devel] [PATCH v27 8/8] target/avr: Add tests, Thomas Huth, 2019/07/21
- Re: [Qemu-devel] [PATCH v27 8/8] target/avr: Add tests, Michael Rolnik, 2019/07/22
- Re: [Qemu-devel] [PATCH v27 8/8] target/avr: Add tests, Thomas Huth, 2019/07/22
- Re: [Qemu-devel] [PATCH v27 8/8] target/avr: Add tests, Michael Rolnik, 2019/07/22
- Re: [Qemu-devel] [PATCH v27 8/8] target/avr: Add tests, Peter Maydell, 2019/07/22
- Re: [Qemu-devel] [PATCH v27 8/8] target/avr: Add tests, Michael Rolnik, 2019/07/22
[Qemu-devel] [PATCH v27 5/8] target/avr: Add limited support for USART and 16 bit timer peripherals, Michael Rolnik, 2019/07/19