From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:41297) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TCb9N-0000CW-7a for qemu-devel@nongnu.org; Fri, 14 Sep 2012 15:02:02 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1TCb9L-0000bi-Iv for qemu-devel@nongnu.org; Fri, 14 Sep 2012 15:02:01 -0400 Received: from mx1.redhat.com ([209.132.183.28]:47988) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TCb9L-0000be-9X for qemu-devel@nongnu.org; Fri, 14 Sep 2012 15:01:59 -0400 Date: Fri, 14 Sep 2012 15:01:55 -0400 From: Jason Baron Message-ID: <20120914190154.GM1821@redhat.com> References: <20120914151401.GL22827@valinux.co.jp> <20120914152329.GL1821@redhat.com> <20120914173418.GM22827@valinux.co.jp> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20120914173418.GM22827@valinux.co.jp> Subject: Re: [Qemu-devel] [PATCH 00/25] q35 series take #1 List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Isaku Yamahata Cc: aliguori@us.ibm.com, juzhang@redhat.com, mst@redhat.com, jan.kiszka@siemens.com, qemu-devel@nongnu.org, agraf@suse.de, alex.williamson@redhat.com, kevin@koconnor.net, avi@redhat.com, mkletzan@redhat.com, lcapitulino@redhat.com, afaerber@suse.de, armbru@redhat.com On Sat, Sep 15, 2012 at 02:34:18AM +0900, Isaku Yamahata wrote: > On Fri, Sep 14, 2012 at 11:23:29AM -0400, Jason Baron wrote: > > On Sat, Sep 15, 2012 at 12:14:01AM +0900, Isaku Yamahata wrote: > > > On Thu, Sep 13, 2012 at 04:12:34PM -0400, Jason Baron wrote: > > > > 2) hotplug > > > > I've added piix acpi style hotplug to ich9. > > > > > > What's the point of this? > > > Its design is ad-hoc and shpc/pcie hotplug are available. > > > > The point was to get to feature parity with piix. I'm not sure how hard > > pcie hotplug is to implement really. I was thinking that we could > > replace the piix hotplug style with pcie hotplug, once that was > > implemented... > > shpc/pcie hotplug emulators are already available in qemu. > hw/shpc.c, hw/pcie.c and hw/xio3130_downstream.c. > > thanks, > -- > yamahata > ok, yes pcie hotplug seems to work :) I guess I got confused b/c I thought devices would show up on the host bus. So the host bus in q35 simply doesn't support hotplug? Also, the way I'm doing hotplug is by specifying a 'bus=pcie.n' param. Without the 'bus=pcie.n' param, i get: Bus 'pcie.0' does not support hotplugging We probably then need to enhance 'device_add' to find a reasonable default bus for the device on hotplug? Thanks, -Jason