From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:58688) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bojTm-0006L5-ME for qemu-devel@nongnu.org; Mon, 26 Sep 2016 23:54:52 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1bojTl-0002uq-07 for qemu-devel@nongnu.org; Mon, 26 Sep 2016 23:54:50 -0400 Date: Tue, 27 Sep 2016 13:53:13 +1000 From: David Gibson Message-ID: <20160927035313.GC30322@umbus.fritz.box> References: <1474899049-12506-1-git-send-email-lvivier@redhat.com> <1474899049-12506-5-git-send-email-lvivier@redhat.com> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="YD3LsXFS42OYHhNZ" Content-Disposition: inline In-Reply-To: <1474899049-12506-5-git-send-email-lvivier@redhat.com> Subject: Re: [Qemu-devel] [Qemu-ppc] [PATCH 4/4] 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 --YD3LsXFS42OYHhNZ Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Mon, Sep 26, 2016 at 04:10:49PM +0200, Laurent Vivier wrote: > Signed-off-by: Laurent Vivier > --- > tests/Makefile.include | 8 +++++++- > tests/libqos/usb.c | 14 ++++++++++++++ > tests/usb-hcd-uhci-test.c | 24 ++++++++++++++++-------- > 3 files changed, 37 insertions(+), 9 deletions(-) >=20 > diff --git a/tests/Makefile.include b/tests/Makefile.include > index 967426a..aaf264f 100644 > --- a/tests/Makefile.include > +++ b/tests/Makefile.include > @@ -270,6 +270,12 @@ check-qtest-ppc64-y +=3D tests/drive_del-test$(EXESU= F) > check-qtest-ppc64-y +=3D tests/postcopy-test$(EXESUF) > check-qtest-ppc64-y +=3D tests/boot-serial-test$(EXESUF) > check-qtest-ppc64-y +=3D tests/rtas-test$(EXESUF) > +check-qtest-ppc64-y +=3D tests/usb-hcd-ohci-test$(EXESUF) > +gcov-files-ppc64-y +=3D hw/usb/hcd-ohci.c > +check-qtest-ppc64-y +=3D tests/usb-hcd-uhci-test$(EXESUF) > +gcov-files-ppc64-y +=3D hw/usb/hcd-uhci.c > +check-qtest-ppc64-y +=3D tests/usb-hcd-xhci-test$(EXESUF) > +gcov-files-ppc64-y +=3D hw/usb/hcd-xhci.c > =20 > check-qtest-sh4-y =3D tests/endianness-test$(EXESUF) > =20 > @@ -595,7 +601,7 @@ libqos-pc-obj-y +=3D tests/libqos/malloc-pc.o tests/l= ibqos/libqos-pc.o > libqos-pc-obj-y +=3D tests/libqos/ahci.o > libqos-omap-obj-y =3D $(libqos-obj-y) tests/libqos/i2c-omap.o > libqos-imx-obj-y =3D $(libqos-obj-y) tests/libqos/i2c-imx.o > -libqos-usb-obj-y =3D $(libqos-pc-obj-y) tests/libqos/usb.o > +libqos-usb-obj-y =3D $(libqos-spapr-obj-y) $(libqos-pc-obj-y) tests/libq= os/usb.o > libqos-virtio-obj-y =3D $(libqos-pc-obj-y) tests/libqos/virtio.o tests/l= ibqos/virtio-pci.o tests/libqos/virtio-mmio.o tests/libqos/malloc-generic.o > =20 > tests/device-introspect-test$(EXESUF): tests/device-introspect-test.o > diff --git a/tests/libqos/usb.c b/tests/libqos/usb.c > index f794d92..6168a2f 100644 > --- a/tests/libqos/usb.c > +++ b/tests/libqos/usb.c > @@ -12,10 +12,17 @@ > * See the COPYING file in the top-level directory. > */ > #include "qemu/osdep.h" > +#include "qemu/bswap.h" > #include "libqtest.h" > #include "hw/usb/uhci-regs.h" > #include "libqos/usb.h" > =20 > +#ifdef HOST_WORDS_BIGENDIAN > +static const bool host_big_endian =3D true; > +#else > +static const bool host_big_endian =3D false; > +#endif > + > void qusb_pci_init_one(QPCIBus *pcibus, struct qhc *hc, uint32_t devfn, = int bar) > { > hc->dev =3D qpci_device_find(pcibus, devfn); > @@ -31,6 +38,13 @@ void uhci_port_test(struct qhc *hc, int port, uint16_t= expect) > uint16_t value =3D qpci_io_readw(hc->dev, addr); > uint16_t mask =3D ~(UHCI_PORT_WRITE_CLEAR | UHCI_PORT_RSVD1); > =20 > + if (qtest_big_endian() && host_big_endian) { > + /* little endian device on big endian guest > + * must be swapped on big endian host > + */ > + value =3D bswap16(value); > + } > + Hm.. should the qpci_io_*() helpers handle the endian conversion? > g_assert((value & mask) =3D=3D (expect & mask)); > } > =20 > diff --git a/tests/usb-hcd-uhci-test.c b/tests/usb-hcd-uhci-test.c > index c24063e..4b951ce 100644 > --- a/tests/usb-hcd-uhci-test.c > +++ b/tests/usb-hcd-uhci-test.c > @@ -9,9 +9,13 @@ > =20 > #include "qemu/osdep.h" > #include "libqtest.h" > +#include "libqos/libqos.h" > #include "libqos/usb.h" > +#include "libqos/libqos-pc.h" > +#include "libqos/libqos-spapr.h" > #include "hw/usb/uhci-regs.h" > =20 > +static QOSState *qs; > =20 > static void test_uhci_init(void) > { > @@ -19,13 +23,10 @@ static void test_uhci_init(void) > =20 > static void test_port(int port) > { > - QPCIBus *pcibus; > struct qhc uhci; > =20 > g_assert(port > 0); > - pcibus =3D qpci_init_pc(NULL); > - g_assert(pcibus !=3D NULL); > - qusb_pci_init_one(pcibus, &uhci, QPCI_DEVFN(0x1d, 0), 4); > + qusb_pci_init_one(qs->pcibus, &uhci, QPCI_DEVFN(0x1d, 0), 4); > uhci_port_test(&uhci, port - 1, UHCI_PORT_CCS); > } > =20 > @@ -75,6 +76,7 @@ static void test_usb_storage_hotplug(void) > =20 > int main(int argc, char **argv) > { > + const char *arch =3D qtest_get_arch(); > int ret; > =20 > g_test_init(&argc, &argv, NULL); > @@ -84,11 +86,17 @@ int main(int argc, char **argv) > qtest_add_func("/uhci/pci/hotplug", test_uhci_hotplug); > qtest_add_func("/uhci/pci/hotplug/usb-storage", test_usb_storage_hot= plug); > =20 > - qtest_start("-device piix3-usb-uhci,id=3Duhci,addr=3D1d.0" > - " -drive id=3Ddrive0,if=3Dnone,file=3D/dev/null,format= =3Draw" > - " -device usb-tablet,bus=3Duhci.0,port=3D1"); > + if (strcmp(arch, "i386") =3D=3D 0 || strcmp(arch, "x86_64") =3D=3D 0= ) { > + qs =3D qtest_pc_boot("-device piix3-usb-uhci,id=3Duhci,addr=3D1d= =2E0" > + " -drive id=3Ddrive0,if=3Dnone,file=3D/dev/nu= ll,format=3Draw" > + " -device usb-tablet,bus=3Duhci.0,port=3D1"); > + } else if (strcmp(arch, "ppc64") =3D=3D 0) { > + qs =3D qtest_spapr_boot("-device piix3-usb-uhci,id=3Duhci,addr= =3D1d.0" > + " -drive id=3Ddrive0,if=3Dnone,file=3D/dev/nu= ll,format=3Draw" > + " -device usb-tablet,bus=3Duhci.0,port=3D1"); > + } > ret =3D g_test_run(); > - qtest_end(); > + qtest_shutdown(qs); > =20 > return ret; > } --=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 --YD3LsXFS42OYHhNZ Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iQIcBAEBCAAGBQJX6e0pAAoJEGw4ysog2bOSetcP/A36vKHU8Nb8OZIuONQL4KBS 2yweX3i4GM62mRyPrWtESiWsyPskeR5um5P8KjJe2yA6atUVj+GI6ZTXZqFWuCS1 M6ZLBLkq9io/Qyn7cqi8DWgKs/FV00hH87zrYFRFtu6p/Ug/Dketr6a5YyTly98v JR1YGQD+RxXw1V+b04GiQpy2WJjKvMl8oO7bnqkb0hgzA57K+5kWIMRZg1KG+KBd e11mSzprfIy8mRonz8oUhAv7SEyGXebOzgaKplVc1GHefDPTanseTwJH+e1CiSIK k92LmbWvMBu1TpWUbss0jvV+LVIAIaCsbCWsz4ljNvjZzyuTynAr4oT+2OWR9Dtv seL8Ybripw3cYM/StF60yfMHDnPM5mVddv7NmVN2uhlPCVXw4nVznC+y2T5wR8AF uZEaG8GDB46ez6f760GzTPSu7b64O7H4RYP/aaAbN5ikG5teVCgix7RqM05jHJjQ UXvIzVSVQ+jCKMVKekEJp8RYFLCTzMPQY4E40zJNbkJ084BHirTHhSapQudMGI9C DBhZAx7WeHrPYEd4KAnn6jI1Vkn3p3YH5TGdzTMqtx7HKhU4gETiIjhse7+fBcGa eQ2qGMZCV7QqXxhpY9fGNj38j2+6pk89a0VUlLRvsXTtRkIFnMkdsNIAxP3guZvB bGFkLEYGb1xEnQ2EPrhz =0BiW -----END PGP SIGNATURE----- --YD3LsXFS42OYHhNZ--