From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:58642) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bojTl-0006Kn-40 for qemu-devel@nongnu.org; Mon, 26 Sep 2016 23:54:50 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1bojTj-0002tv-Nm for qemu-devel@nongnu.org; Mon, 26 Sep 2016 23:54:49 -0400 Date: Tue, 27 Sep 2016 13:49:55 +1000 From: David Gibson Message-ID: <20160927034955.GB30322@umbus.fritz.box> References: <1474899049-12506-1-git-send-email-lvivier@redhat.com> <1474899049-12506-4-git-send-email-lvivier@redhat.com> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="ADZbWkCsHQ7r3kzd" Content-Disposition: inline In-Reply-To: <1474899049-12506-4-git-send-email-lvivier@redhat.com> Subject: Re: [Qemu-devel] [Qemu-ppc] [PATCH 3/4] libqos: use generic qtest_shutdown() 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 --ADZbWkCsHQ7r3kzd Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Mon, Sep 26, 2016 at 04:10:48PM +0200, Laurent Vivier wrote: > Machine specific shutdown function can be registered by > the machine specific qtest_XXX_boot() if needed. >=20 > So we will not have to test twice the archicture (on boot and on > shutdown) if the test can be run on several architectures. >=20 > Signed-off-by: Laurent Vivier Reviewed-by: David Gibson > --- > tests/libqos/libqos-pc.c | 3 ++- > tests/libqos/libqos-spapr.c | 5 +++-- > tests/libqos/libqos.c | 11 ++++++++++- > tests/libqos/libqos.h | 8 ++++++-- > tests/rtas-test.c | 2 +- > 5 files changed, 22 insertions(+), 7 deletions(-) >=20 > diff --git a/tests/libqos/libqos-pc.c b/tests/libqos/libqos-pc.c > index aa17c98..219824d 100644 > --- a/tests/libqos/libqos-pc.c > +++ b/tests/libqos/libqos-pc.c > @@ -8,6 +8,7 @@ static QOSOps qos_ops =3D { > .uninit_allocator =3D pc_alloc_uninit, > .qpci_init =3D qpci_init_pc, > .qpci_free =3D qpci_free_pc, > + .shutdown =3D qtest_pc_shutdown > }; > =20 > QOSState *qtest_pc_vboot(const char *cmdline_fmt, va_list ap) > @@ -31,5 +32,5 @@ QOSState *qtest_pc_boot(const char *cmdline_fmt, ...) > =20 > void qtest_pc_shutdown(QOSState *qs) > { > - return qtest_shutdown(qs); > + return qtest_common_shutdown(qs); > } > diff --git a/tests/libqos/libqos-spapr.c b/tests/libqos/libqos-spapr.c > index 125c6b3..8f7fc14 100644 > --- a/tests/libqos/libqos-spapr.c > +++ b/tests/libqos/libqos-spapr.c > @@ -7,7 +7,8 @@ static QOSOps qos_ops =3D { > .init_allocator =3D spapr_alloc_init_flags, > .uninit_allocator =3D spapr_alloc_uninit, > .qpci_init =3D qpci_init_spapr, > - .qpci_free =3D qpci_free_spapr > + .qpci_free =3D qpci_free_spapr, > + .shutdown =3D qtest_spapr_shutdown > }; > =20 > QOSState *qtest_spapr_vboot(const char *cmdline_fmt, va_list ap) > @@ -29,5 +30,5 @@ QOSState *qtest_spapr_boot(const char *cmdline_fmt, ...) > =20 > void qtest_spapr_shutdown(QOSState *qs) > { > - return qtest_shutdown(qs); > + return qtest_common_shutdown(qs); > } > diff --git a/tests/libqos/libqos.c b/tests/libqos/libqos.c > index 332d60e..2f8a1ad 100644 > --- a/tests/libqos/libqos.c > +++ b/tests/libqos/libqos.c > @@ -52,7 +52,7 @@ QOSState *qtest_boot(QOSOps *ops, const char *cmdline_f= mt, ...) > /** > * Tear down the QEMU instance. > */ > -void qtest_shutdown(QOSState *qs) > +void qtest_common_shutdown(QOSState *qs) > { > if (qs->ops) { > if (qs->alloc && qs->ops->uninit_allocator) { > @@ -68,6 +68,15 @@ void qtest_shutdown(QOSState *qs) > g_free(qs); > } > =20 > +void qtest_shutdown(QOSState *qs) > +{ > + if (qs->ops && qs->ops->shutdown) { > + qs->ops->shutdown(qs); > + } else { > + qtest_common_shutdown(qs); > + } > +} > + > void set_context(QOSState *s) > { > global_qtest =3D s->qts; > diff --git a/tests/libqos/libqos.h b/tests/libqos/libqos.h > index a9f6990..2319697 100644 > --- a/tests/libqos/libqos.h > +++ b/tests/libqos/libqos.h > @@ -5,22 +5,26 @@ > #include "libqos/pci.h" > #include "libqos/malloc-pc.h" > =20 > +typedef struct QOSState QOSState; > + > typedef struct QOSOps { > QGuestAllocator *(*init_allocator)(QAllocOpts); > void (*uninit_allocator)(QGuestAllocator *); > QPCIBus *(*qpci_init)(QGuestAllocator *alloc); > void (*qpci_free)(QPCIBus *bus); > + void (*shutdown)(QOSState *); > } QOSOps; > =20 > -typedef struct QOSState { > +struct QOSState { > QTestState *qts; > QGuestAllocator *alloc; > QPCIBus *pcibus; > QOSOps *ops; > -} QOSState; > +}; > =20 > QOSState *qtest_vboot(QOSOps *ops, const char *cmdline_fmt, va_list ap); > QOSState *qtest_boot(QOSOps *ops, const char *cmdline_fmt, ...); > +void qtest_common_shutdown(QOSState *qs); > void qtest_shutdown(QOSState *qs); > bool have_qemu_img(void); > void mkimg(const char *file, const char *fmt, unsigned size_mb); > diff --git a/tests/rtas-test.c b/tests/rtas-test.c > index 73c7803..ba0867a 100644 > --- a/tests/rtas-test.c > +++ b/tests/rtas-test.c > @@ -22,7 +22,7 @@ static void test_rtas_get_time_of_day(void) > t2 =3D mktimegm(&tm); > g_assert(t2 - t1 < 5); /* 5 sec max to run the test */ > =20 > - qtest_spapr_shutdown(qs); > + qtest_shutdown(qs); > } > =20 > int main(int argc, char *argv[]) --=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 --ADZbWkCsHQ7r3kzd Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iQIcBAEBCAAGBQJX6exjAAoJEGw4ysog2bOS7FUP/3IVR3sLCpyOhf4lCkyxTyIX mScl39GB0ai3ev5iU9DH+uw/NucadBnkRAuo7sEbcMhNe0rEtA4zzH4rVeznbVuf qPzvE9Ec/+ldyVUVDCrO0ejGeTIYFKde67u476Q6qyoJFI+wwyC0O+cuqfnM6gF/ QiQrXPuXaUNxvLv5pvycO6CeAaCv2iMYgVKBj/vu3A9WYlyW8Mc9RNAGTpwascg1 P115BmgsRJCoCCzEOArGxbpVfRFGVU/okl3v+fyZRXGJLL6u8ZCQIJ5y4vFgJ3r4 K2EIXgcSEvfIQfRrfWTQMaEKhZsra0sTNBCljZdzXAjitXYQzz/HSbfyRi6M+Uz4 wbR8A6xq4MCBjqCDj+q05AwpOJCoxQsZYuBSxVQ3x4iMPMFRyQrRzq6Eq6yKEIoc xYfwEyj3QbvuK7x7TFjB+9gEkY+Jxx1mdCNlD3JXR53SYfN7elv7cEGhkfn0OsNe lQ/c7w/YT8ERcN2JTo3I/J0gC3yROs7APyVDSzAlX89ply6mleFrC3Jrk/6L21jq oB6p4sml5gaimpmhq3aW2+l8Vq+lUbb3VEyHLUdybuLheEYMN6nti10lsa6jJsqn uE9xy/NQH3fRjqCI8yBI4acsad61hAlr9DDk6nMabtUhQx0QYNnWnGiyGhRbQ/6q fyIfq3C2B6Ufa0kVQAHU =tFG4 -----END PGP SIGNATURE----- --ADZbWkCsHQ7r3kzd--