From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:34569) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TstTa-0002zR-Ib for qemu-devel@nongnu.org; Wed, 09 Jan 2013 06:05:46 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1TstTX-0006C2-JR for qemu-devel@nongnu.org; Wed, 09 Jan 2013 06:05:42 -0500 Received: from mx1.redhat.com ([209.132.183.28]:48952) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TstTX-0006Bt-Az for qemu-devel@nongnu.org; Wed, 09 Jan 2013 06:05:39 -0500 Date: Wed, 9 Jan 2013 13:09:27 +0200 From: "Michael S. Tsirkin" Message-ID: <20130109110926.GA17317@redhat.com> References: <1355761490-10073-1-git-send-email-pbonzini@redhat.com> <877gnovmgj.fsf@codemonkey.ws> <50ED3971.5030600@redhat.com> <20130109102238.GC30914@redhat.com> <50ED4C0C.5070109@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <50ED4C0C.5070109@redhat.com> Subject: Re: [Qemu-devel] [PATCH 00/15] qdev: make reset semantics more clear and consistent, reset qbuses under virtio devices List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Paolo Bonzini Cc: Peter Maydell , Anthony Liguori , qemu-devel@nongnu.org, Andreas =?iso-8859-1?Q?F=E4rber?= On Wed, Jan 09, 2013 at 11:53:00AM +0100, Paolo Bonzini wrote: > Il 09/01/2013 11:22, Michael S. Tsirkin ha scritto: > > > It's possible. I'll move the SCSI bus away from qdev reset. > > > Anthony/Michael, can you help doing the same with PCIDevice? And > > > perhaps Peter and Andreas with sysbus? > > > > I'm not sure what would you like to change with PCIDevice. > > Replace the DeviceState reset method with one in PCIDevice, and call it > from the PCI bus reset. > > Paolo You are talking about the call to qdev_reset_all(&dev->qdev) in pci_device_reset, and you want to detect, there, that device is a bridge and replace qdev_reset_all(&dev->qdev) with cast and call to pci_bus_reset for the secondary bus? -- MST