From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:57634) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WxblD-0002P4-QK for qemu-devel@nongnu.org; Thu, 19 Jun 2014 08:48:17 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Wxbl7-0003Cl-MS for qemu-devel@nongnu.org; Thu, 19 Jun 2014 08:48:11 -0400 Received: from mx1.redhat.com ([209.132.183.28]:56470) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Wxbl7-0003CT-DT for qemu-devel@nongnu.org; Thu, 19 Jun 2014 08:48:05 -0400 Message-ID: <1403182025.22530.13.camel@nilsson.home.kraxel.org> From: Gerd Hoffmann Date: Thu, 19 Jun 2014 14:47:05 +0200 In-Reply-To: <53A2D682.1010903@suse.de> References: <1402992243-10224-1-git-send-email-arei.gonglei@huawei.com> <1403172316.22530.8.camel@nilsson.home.kraxel.org> <53A2D682.1010903@suse.de> Content-Type: text/plain; charset="UTF-8" Mime-Version: 1.0 Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [PATCH 0/5] tests: add usb host adapter qtest cases List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Andreas =?ISO-8859-1?Q?F=E4rber?= Cc: weidong.huang@huawei.com, luonengjun@huawei.com, qemu-devel@nongnu.org, arei.gonglei@huawei.com, stefanha@redhat.com, akong@redhat.com On Do, 2014-06-19 at 14:24 +0200, Andreas F=C3=A4rber wrote: > Hi Gerd, >=20 > Am 19.06.2014 12:05, schrieb Gerd Hoffmann: > >> Gonglei (5): > >> tests: add OHCI qtest > >> tests: add UHCI qtest > >> tests: add xHCI qtest > >> usb-hcd-ehci-test: change ehci qtest more expansively > >> usb-hcd-ehci-test: add ehci hotplug/unplug qtest > >=20 > > I'd prefer to leave the usb-hcd-ehci-test alone. >=20 > Why? My idea was to put all EHCI testing code into that ehci file. This > is so that any defines or helper functions can be shared within the > file. Functional testing of the companion core should go into the > respective *hci file, unless there is some interaction to test. The hotplugging and the tests currently in usb-hcd-ehci-test are quite different. Hotplugging creates a empty machine, then plugs + unplugs stuff. usb-hcd-ehci-test creates a machine with ehci+ohci companions, then goes run a bunch of tests for it. Maybe it makes sense to have a usb-hcd-hotplug-test.c file and collect hotplug testing for all usb hcds we have there. For code sharing that'll be better for sure as the basic code flow is the same for all hcd types. cheers, Gerd