From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:42761) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WChpe-0001dR-84 for qemu-devel@nongnu.org; Sun, 09 Feb 2014 22:46:58 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1WChpZ-0002Ka-Mt for qemu-devel@nongnu.org; Sun, 09 Feb 2014 22:46:54 -0500 Received: from mail-pd0-f174.google.com ([209.85.192.174]:39861) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WChpZ-0002KN-E4 for qemu-devel@nongnu.org; Sun, 09 Feb 2014 22:46:49 -0500 Received: by mail-pd0-f174.google.com with SMTP id z10so5498655pdj.5 for ; Sun, 09 Feb 2014 19:46:47 -0800 (PST) Message-ID: <52F84BA0.4030509@ozlabs.ru> Date: Mon, 10 Feb 2014 14:46:40 +1100 From: Alexey Kardashevskiy MIME-Version: 1.0 References: <1391947053-26753-1-git-send-email-afaerber@suse.de> In-Reply-To: <1391947053-26753-1-git-send-email-afaerber@suse.de> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Subject: Re: [Qemu-devel] [PATCH qom-next v2 00/12] qtest: Prepare PCI NIC, virtio, IndustryPack tests List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: =?UTF-8?B?QW5kcmVhcyBGw6RyYmVy?= , qemu-devel@nongnu.org Cc: Kevin Wolf , Peter Maydell , Stefan Hajnoczi , "Michael S. Tsirkin" , Alberto Garcia , Anthony Liguori , Paolo Bonzini On 02/09/2014 10:57 PM, Andreas Färber wrote: > Hello Stefan and Michael, > > This series starts preparing tests to cover optional PCI devices. They don't do > any functional tests but assure that QOM/PCI/VMState refactorings don't break > the basic usage of devices not covered by qom-test. > > pcnet-test.c is intended to handle sparc's lance as well. > > v2 starts to cover virtio devices and covers ipack. > > NB that the Makefile is being prepared for testing ... > * PCI devices on different architectures and > * VirtIO devices on different transports and > * IndustryPack devices on different bridges, > but the tests themselves will need some follow-up for choosing the right -machine > or adding the required bus -device. My focus for now is on testing devices at all. > > I'd like to include the IndustryPack tests in my upcoming pull (Monday) and am > hoping I can do so on top of the original set of NIC tests for convenience. How to apply these patches? They are quoted printable and "git am" does not parse them. Besides that, I have a spapr-phb test which I posted here months ago but nobody seems to notice it :) I'll post it in this thread, it could go together with the other tests in this set. > > Regards, > Andreas > > v1 -> v2: > * Reordered includes (stefanha) > * Used qtest_end() where applicable > * Added qtest for tpci200 and ipoctal232 (for my IndustryPack refactoring series) > * Added qtests for ne2000, virtio-net, virtio-blk, virtio-balloon, virtio-rng > > Cc: Stefan Hajnoczi > Cc: Michael S. Tsirkin > Cc: Kevin Wolf > Cc: Alberto Garcia > Cc: Anthony Liguori > Cc: Paolo Bonzini > Cc: Peter Maydell > Cc: Alexey Kardashevskiy > > Andreas Färber (12): > tests: Add e1000 qtest > tests: Add vmxnet3 qtest > tests: Add rtl8139 qtest > tests: Add pcnet qtest > tests: Add eepro100 qtest > tests: Add tpci200 qtest > tests: Add ne2000 qtest > tests: Add virtio-net qtest > tests: Add virtio-blk qtest > tests: Add virtio-balloon qtest > tests: Add virtio-rng qtest > tests: Add ipoctal232 qtest > > tests/Makefile | 49 +++++++++++++++++++++++++++++++++++ > tests/e1000-test.c | 33 ++++++++++++++++++++++++ > tests/eepro100-test.c | 63 +++++++++++++++++++++++++++++++++++++++++++++ > tests/ipoctal232-test.c | 33 ++++++++++++++++++++++++ > tests/ne2000-test.c | 33 ++++++++++++++++++++++++ > tests/pcnet-test.c | 33 ++++++++++++++++++++++++ > tests/rtl8139-test.c | 33 ++++++++++++++++++++++++ > tests/tpci200-test.c | 33 ++++++++++++++++++++++++ > tests/virtio-balloon-test.c | 33 ++++++++++++++++++++++++ > tests/virtio-blk-test.c | 34 ++++++++++++++++++++++++ > tests/virtio-net-test.c | 33 ++++++++++++++++++++++++ > tests/virtio-rng-test.c | 33 ++++++++++++++++++++++++ > tests/vmxnet3-test.c | 33 ++++++++++++++++++++++++ > 13 files changed, 476 insertions(+) > create mode 100644 tests/e1000-test.c > create mode 100644 tests/eepro100-test.c > create mode 100644 tests/ipoctal232-test.c > create mode 100644 tests/ne2000-test.c > create mode 100644 tests/pcnet-test.c > create mode 100644 tests/rtl8139-test.c > create mode 100644 tests/tpci200-test.c > create mode 100644 tests/virtio-balloon-test.c > create mode 100644 tests/virtio-blk-test.c > create mode 100644 tests/virtio-net-test.c > create mode 100644 tests/virtio-rng-test.c > create mode 100644 tests/vmxnet3-test.c > -- Alexey