From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:45308) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YDK30-0003uW-Kf for qemu-devel@nongnu.org; Mon, 19 Jan 2015 16:39:47 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1YDK2w-0001rd-G5 for qemu-devel@nongnu.org; Mon, 19 Jan 2015 16:39:46 -0500 Received: from mx1.redhat.com ([209.132.183.28]:47980) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YDK2w-0001rZ-8s for qemu-devel@nongnu.org; Mon, 19 Jan 2015 16:39:42 -0500 Date: Mon, 19 Jan 2015 23:39:34 +0200 From: "Michael S. Tsirkin" Message-ID: <20150119213934.GA10427@redhat.com> References: <1421679157-3510-1-git-send-email-armbru@redhat.com> <54BD1E1A.2020706@suse.de> <87iog2hiah.fsf@blackfin.pond.sub.org> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline In-Reply-To: <87iog2hiah.fsf@blackfin.pond.sub.org> Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [PATCH 00/10] pci: Partial conversion to realize List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Markus Armbruster Cc: pbonzini@redhat.com, arei.gonglei@huawei.com, kraxel@redhat.com, Andreas =?iso-8859-1?Q?F=E4rber?= , qemu-devel@nongnu.org On Mon, Jan 19, 2015 at 05:00:38PM +0100, Markus Armbruster wrote: > Andreas F=E4rber writes: >=20 > > Hi Markus, > > > > Am 19.01.2015 um 15:52 schrieb Markus Armbruster: > >> I posted this series as RFC back in October, but it depended on > >> patches then still under review, so I put it aside, and promptly > >> forgot. Fortunately, rebasing and updating it wasn't much trouble. > > > > A question I had brought up for the RFC was that of testing: Did you > > check which of your conversions are covered by QTest or where we may > > first need additional test cases to verify this doesn't regress? > > > > As a reminder, when converting ISA devices, I accepted patches that w= ere > > only tested with one of two serial devices contained in a file, promp= tly > > breaking the other in some target. I'd like to avoid that for PCI. ;) >=20 > I got this after I sent my reply to your original question. >=20 > Perhaps we could create a qtest that tries to -device every known PCI > device: grab them from >=20 > { "execute": "qom-list-types", "arguments": { "implements": "pci-devi= ce" } } >=20 > Some require properties, typically backends. We could introspect with > device-list-properties and try to supply common ones, like "drive", > "chardev", "netdev". >=20 > Doing it as a big device_add / device_del orgy would be cute, except > device_del requires guest cooperation, complicating the test. We could > special-case device_del before guest starts if we wanted. You can drive the hot-unplug register from qemu monitor without running guests. Target specific. Alternatively, e.g. PC completes unplug on reset. Maybe we should teach all targets to do that, then you can just reset to complete unplug. --=20 MST