qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH qom-next 0/5] qtest: Prepare PCI NIC tests


From: Stefan Hajnoczi
Subject: Re: [Qemu-devel] [PATCH qom-next 0/5] qtest: Prepare PCI NIC tests
Date: Fri, 7 Feb 2014 16:50:31 +0100
User-agent: Mutt/1.5.21 (2010-09-15)

On Wed, Feb 05, 2014 at 06:01:12PM +0100, Andreas Färber wrote:
> Hello Stefan,
> 
> As discussed late last year, this series starts preparing tests to cover
> optional NICs. They don't do any functional tests but assure that QOM/PCI
> refactorings don't break the basic usage of these devices.
> 
> pcnet-test.c is intended to handle sparc's lance as well.
> 
> virtio-net is missing among others (is there any ongoing work on virtio
> qtests?!), but I thought I'd get this out as a start. My goal will be to
> have every optional device in 2.0 covered by such basic tests.
> Hopefully it will also encourage people to write functional tests later on
> and to copy these stubs for new devices!
> 
> Regards,
> Andreas
> 
> Cc: Stefan Hajnoczi <address@hidden>
> Cc: Anthony Liguori <address@hidden>
> Cc: Paolo Bonzini <address@hidden>
> Cc: Peter Maydell <address@hidden>
> 
> Andreas Färber (5):
>   tests: Add e1000 qtest
>   tests: Add vmxnet3 qtest
>   tests: Add rtl8139 qtest
>   tests: Add PC-Net qtest
>   tests: Add eepro100 qtest
> 
>  tests/Makefile        | 22 +++++++++++++++++++
>  tests/e1000-test.c    | 35 +++++++++++++++++++++++++++++
>  tests/eepro100-test.c | 61 
> +++++++++++++++++++++++++++++++++++++++++++++++++++
>  tests/pcnet-test.c    | 35 +++++++++++++++++++++++++++++
>  tests/rtl8139-test.c  | 35 +++++++++++++++++++++++++++++
>  tests/vmxnet3-test.c  | 35 +++++++++++++++++++++++++++++
>  6 files changed, 223 insertions(+)
>  create mode 100644 tests/e1000-test.c
>  create mode 100644 tests/eepro100-test.c
>  create mode 100644 tests/pcnet-test.c
>  create mode 100644 tests/rtl8139-test.c
>  create mode 100644 tests/vmxnet3-test.c

Not much going on here, should be easy to merge.  I did leave comments
on the first patch, they apply to all other patches in the series as
well.



reply via email to

[Prev in Thread] Current Thread [Next in Thread]