From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:47210) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bpU0n-0005h7-7T for qemu-devel@nongnu.org; Thu, 29 Sep 2016 01:36:02 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1bpU0l-0007t1-I6 for qemu-devel@nongnu.org; Thu, 29 Sep 2016 01:36:00 -0400 Date: Thu, 29 Sep 2016 15:35:02 +1000 From: David Gibson Message-ID: <20160929053502.GV8390@umbus.fritz.box> References: <1475088693-29091-1-git-send-email-lvivier@redhat.com> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="BlkQeOBdElZ1aiuH" Content-Disposition: inline In-Reply-To: <1475088693-29091-1-git-send-email-lvivier@redhat.com> Subject: Re: [Qemu-devel] [Qemu-ppc] [PATCH v3 0/6] tests: enable ohci/uhci/xhci tests on PPC64 List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Laurent Vivier Cc: qemu-devel@nongnu.org, thuth@redhat.com, Greg Kurz , qemu-ppc@nongnu.org, Gerd Hoffmann , dgibson@redhat.com --BlkQeOBdElZ1aiuH Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Wed, Sep 28, 2016 at 08:51:27PM +0200, Laurent Vivier wrote: > This series enables USB tests on PPC64, and for > that implements libqos SPAPR PCI support. These look good, except I still think the endianness is a bit confused. A more detailed analysis in response to patch 1. Also, I'm afraid this is going to need another rebase, due to things I've merged in ppc-for-2.8 today. >=20 > v3: > - rebase > - minor fixes (commas, comments) > - add Greg's R-b >=20 > v2: > - rebase > - revert order in qtest_common_shutdown() to release > PCI and then memory allocator > - remove useless byte-swapping in qpci-spapr.c > - compute machine endianness on qtest_init() > - introduce target_cpu_to_leXX()/target_cpu_to_beXX() > target_leXX_to_cpu()/target_beXX_to_cpu() >=20 > Laurent Vivier (6): > libqos: add PPC64 PCI support > libqos: add PCI management in qtest_vboot()/qtest_shutdown() > libqos: use generic qtest_shutdown() > qtest: evaluate endianness of the target in qtest_init() > qtest: define target cpu endianness conversion functions > tests: enable ohci/uhci/xhci tests on PPC64 >=20 > tests/Makefile.include | 9 +- > tests/e1000e-test.c | 2 +- > tests/i440fx-test.c | 2 +- > tests/ide-test.c | 2 +- > tests/ivshmem-test.c | 2 +- > tests/libqos/ahci.c | 2 +- > tests/libqos/libqos-pc.c | 8 +- > tests/libqos/libqos-spapr.c | 8 +- > tests/libqos/libqos.c | 32 ++++- > tests/libqos/libqos.h | 11 +- > tests/libqos/pci-pc.c | 24 +--- > tests/libqos/pci-pc.h | 3 +- > tests/libqos/pci-spapr.c | 280 ++++++++++++++++++++++++++++++++++++++= ++++++ > tests/libqos/pci-spapr.h | 17 +++ > tests/libqos/pci.c | 22 +++- > tests/libqos/rtas.c | 45 +++++++ > tests/libqos/rtas.h | 4 + > tests/libqos/usb.c | 2 +- > tests/libqos/virtio-pci.c | 2 +- > tests/libqtest.c | 96 ++++++++------- > tests/libqtest.h | 71 ++++++++++- > tests/q35-test.c | 2 +- > tests/rtas-test.c | 2 +- > tests/rtl8139-test.c | 2 +- > tests/tco-test.c | 2 +- > tests/usb-hcd-ehci-test.c | 2 +- > tests/usb-hcd-uhci-test.c | 24 ++-- > tests/vhost-user-test.c | 4 +- > tests/virtio-9p-test.c | 2 +- > tests/virtio-blk-test.c | 4 +- > tests/virtio-net-test.c | 2 +- > tests/virtio-scsi-test.c | 2 +- > 32 files changed, 580 insertions(+), 112 deletions(-) > create mode 100644 tests/libqos/pci-spapr.c > create mode 100644 tests/libqos/pci-spapr.h >=20 --=20 David Gibson | I'll have my music baroque, and my code david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_ | _way_ _around_! http://www.ozlabs.org/~dgibson --BlkQeOBdElZ1aiuH Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iQIcBAEBCAAGBQJX7KgGAAoJEGw4ysog2bOSjjwP/1935UvcK5UDmQAx7ulBuedS D5VT4k423p/aqyioULk9CMHJtQQ2EkaDI4Q3w0QwOpEQZJGnjeQbR/k5xvZNElUO GNhMZMCQBcKiEFFHOljhRBBG57AZv1mTWqjwdwjFCGBVVONpr0ImZ3i1DJ0IPcSN k1FmGct8W1L/ykRpWgG4kyHPy46eTUwpctYQNb+y92ac7Z8a/gAJ/lr9fqhooZhG h5YOjAT1i0Gf8AhvmxTDLwNvIcakOzrp5oi/n5ElH0XMAH5gFqOcPBH6KngEqXGX H5xKmbvb5h47M19R3uzu/WSallGzkgJTfVN6khmHYNBVqoel+ymVXF0NrBa2yjEF KDZ5vFrA9mNurg9dg2ZPjvQ4hIv2e/c22mPfeXvuTmPfuuWDwxFAijjFCrzizC3n f4s9y9BHtlMugf9w0HYMG5UMkhtYYLtP5vPGBJX7E4nkuZm7TqPmHP1T/LVOr1Ok imGfHtVGHH1uhxXwZNn/DFh8WtQGdlMMWUqTrY4yq4qlISw1or8wfIbZ9SVbUXQ/ JUHZQ/gvPiTfYYJhy9VNRzT1J3I6fNNeYeE1U/xm28cIBTTQTK2PewRakC7Qkqcb erqPMLF11lTHCCN2jzVcWedjOsRI65tjjdX8Xgslrsryn6tcy28zN5VtPcXEdiAb wTbM2+KWXIrdXNgcg2AE =FQom -----END PGP SIGNATURE----- --BlkQeOBdElZ1aiuH--