From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:54176) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bz1YZ-0005kA-Fk for qemu-devel@nongnu.org; Tue, 25 Oct 2016 09:14:23 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1bz1YU-0003qJ-3k for qemu-devel@nongnu.org; Tue, 25 Oct 2016 09:14:19 -0400 Received: from 1.mo6.mail-out.ovh.net ([46.105.56.136]:33753) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1bz1YT-0003pq-RU for qemu-devel@nongnu.org; Tue, 25 Oct 2016 09:14:14 -0400 Received: from player786.ha.ovh.net (b7.ovh.net [213.186.33.57]) by mo6.mail-out.ovh.net (Postfix) with ESMTP id 90A4F55B85 for ; Tue, 25 Oct 2016 15:14:12 +0200 (CEST) Date: Tue, 25 Oct 2016 15:14:00 +0200 From: Greg Kurz Message-ID: <20161025151400.404e52aa@bahia> In-Reply-To: <20161025033552.GT11052@umbus.fritz.box> References: <1477285201-10244-1-git-send-email-david@gibson.dropbear.id.au> <20161025033552.GT11052@umbus.fritz.box> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; boundary="Sig_/MlwQiztZ4rFH_SJBJdi_NEY"; protocol="application/pgp-signature" Subject: Re: [Qemu-devel] [PATCHv5 00/12] Cleanups to qtest PCI handling List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: David Gibson Cc: pbonzini@redhat.com, qemu-devel@nongnu.org, qemu-ppc@nongnu.org, lvivier@redhat.com, agraf@suse.de, stefanha@redhat.com, mst@redhat.com, aik@ozlabs.ru, mdroth@linux.vnet.ibm.com, thuth@redhat.com --Sig_/MlwQiztZ4rFH_SJBJdi_NEY Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: quoted-printable On Tue, 25 Oct 2016 14:35:52 +1100 David Gibson wrote: > On Mon, Oct 24, 2016 at 03:59:49PM +1100, David Gibson wrote: > > This series contains a number of cleanups to the libqos code for > > accessing PCI devices, and to tests which use it. > >=20 > > The general aim is to improve the consistency of semantics across > > functions, and reduce the amount of intimate knowledge of the libqos > > PCI layer needed by tests. > >=20 > > This should make it easier to write PCI tests which will be portable > > to different guest machines with different PCI host bridge > > arrangements. > >=20 > > This series is on top of my ppc-for-2.8 branch, since it contains > > patches enabling the virtio tests on ppc, which would otherwise > > conflict with the changes here. =20 >=20 > Greg, Alexey, Michael, >=20 > Some reviews from outside RH would be really welcome. >=20 Done. I also took time to run 'make check' with all targets on ppc64le, ppc64be, i686 and ppc32 hosts. Everything passes for ppc64le, ppc64be and i686. It fails on ppc32 but this seems to be a TCG issue (QEMU fails with SIGILL in code_gen_buffer()), not related to this patchset. Tested-by: Greg Kurz > >=20 > > Changes since v4: > > * Fixed some remaining abstraction breaks in ahci-test > > * Removed QPCI_BAR_INVALID, turned out not to really be useful > >=20 > > Changes since v3: > > * Fixed another endian bug introduced in ide-test > >=20 > > Changes since v2: > > * Fixed build bugs in virtio-9p-test, which I didn't find earlier > > due to not having the right libraries installed > > * Fixed an endian bug I accidentally introduced in ide-test > > * Better handling of invalid BAR tokens > >=20 > > Changes since v1: > > * Split out updates to tco-test into separate patch > > * Split out updates to ide-test into separate patch > > * Neater and more general handling of legacy PIO addresses > > * Removed now-redundant fields from platform specific bus structures > > * Introduced CONFIG_BASE() macro to virtio-pci to remove many > > similar assignments > > * Fixed handling of two guest testcasesin ivshmem > > * Added 64-bit accessors > > * Rebase on ppc-for-2.8 to avoid conflict with Laurent's series in > > the same area > >=20 > > David Gibson (12): > > libqos: Give qvirtio_config_read*() consistent semantics > > libqos: Handle PCI IO de-multiplexing in common code > > libqos: Move BAR assignment to common code > > libqos: Better handling of PCI legacy IO > > tests: Adjust tco-test to use qpci_legacy_iomap() > > libqos: Add streaming accessors for PCI MMIO > > libqos: Implement mmio accessors in terms of mem{read,write} > > tests: Clean up IO handling in ide-test > > libqos: Add 64-bit PCI IO accessors > > tests: Use qpci_mem{read,write} in ivshmem-test > > tests: Don't assume structure of PCI IO base in ahci-test > > libqos: Change PCI accessors to take opaque BAR handle > >=20 > > tests/ahci-test.c | 13 +-- > > tests/e1000e-test.c | 7 +- > > tests/ide-test.c | 177 +++++++++++++++++++++++++++----------= ---- > > tests/ivshmem-test.c | 47 +++++++---- > > tests/libqos/ahci.c | 4 +- > > tests/libqos/ahci.h | 7 +- > > tests/libqos/pci-pc.c | 187 ++++++++++---------------------------= ------ > > tests/libqos/pci-spapr.c | 194 ++++++++++++-------------------------= -------- > > tests/libqos/pci.c | 194 +++++++++++++++++++++++++++++++++++++= -------- > > tests/libqos/pci.h | 66 ++++++++++----- > > tests/libqos/usb.c | 6 +- > > tests/libqos/usb.h | 2 +- > > tests/libqos/virtio-mmio.c | 16 ++-- > > tests/libqos/virtio-pci.c | 122 ++++++++++++++-------------- > > tests/libqos/virtio-pci.h | 2 +- > > tests/rtl8139-test.c | 10 +-- > > tests/tco-test.c | 80 +++++++++---------- > > tests/usb-hcd-ehci-test.c | 5 +- > > tests/virtio-9p-test.c | 8 +- > > tests/virtio-blk-test.c | 42 +++------- > > tests/virtio-scsi-test.c | 4 +- > > 21 files changed, 598 insertions(+), 595 deletions(-) > > =20 >=20 --Sig_/MlwQiztZ4rFH_SJBJdi_NEY Content-Type: application/pgp-signature Content-Description: OpenPGP digital signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iEYEARECAAYFAlgPWpgACgkQAvw66wEB28L53wCfTsnNqW8p1QFlu/Pv1VVT4Wk4 bKUAn1Y4zN9Pc9OmJMTL4CAfCpSx4AiR =QcjW -----END PGP SIGNATURE----- --Sig_/MlwQiztZ4rFH_SJBJdi_NEY--