From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:43000) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Vq1cB-0004HI-0j for qemu-devel@nongnu.org; Mon, 09 Dec 2013 09:15:21 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Vq1c5-0004i2-25 for qemu-devel@nongnu.org; Mon, 09 Dec 2013 09:15:14 -0500 Received: from mx1.redhat.com ([209.132.183.28]:25894) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Vq1c4-0004hU-RX for qemu-devel@nongnu.org; Mon, 09 Dec 2013 09:15:09 -0500 Date: Mon, 9 Dec 2013 15:14:31 +0100 From: Igor Mammedov Message-ID: <20131209151431.57d49bd4@nial.usersys.redhat.com> In-Reply-To: <52A5C9D4.7080404@redhat.com> References: <1386349395-5710-1-git-send-email-imammedo@redhat.com> <1386349395-5710-8-git-send-email-imammedo@redhat.com> <52A20996.8020308@redhat.com> <20131209144139.7f9ac562@nial.usersys.redhat.com> <52A5C9D4.7080404@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH 7/7] hw/pci: convert PCI bus to use "hotplug-device" interface. List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Paolo Bonzini Cc: peter.maydell@linaro.org, ehabkost@redhat.com, marcel.a@redhat.com, mst@redhat.com, qemu-devel@nongnu.org, blauwirbel@gmail.com, alex.williamson@redhat.com, anthony@codemonkey.ws, afaerber@suse.de On Mon, 09 Dec 2013 14:47:00 +0100 Paolo Bonzini wrote: > Il 09/12/2013 14:41, Igor Mammedov ha scritto: > >> > Please move the parts under the "if" to hotplug.c (something like > >> > hotplug_handler_plug and hotplug_handler_unplug). Also, should this be > > sure. > > > >> > moved up to generic code (e.g. bus_add_child/bus_remove_child)? > > Current usage hints that it's more realize() related part. If handler > > fails device might want to do same device specific failure handling > > before returning from realize() with failure. > > It will work in bus_add_child/bus_remove_child as well but it would > > basically put handlers to nofail category. > > Makes sense. realize() for the "plug" handler, and qdev_unplug for the > unplug handler, I guess. Just to be sure, I've meant not DEVICE.realize() but each device specific one. qdev_unplug() might work for now, but I haven't checked all devices that might use interface and if it would break anything. Ideally it should be in device's unrealize() complementing realize() part. I'd wait till all buses converted to new interface before attempting to generalize current plug/unplug call pathes though. > > Paolo >