From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([140.186.70.92]:48475) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1QcFgh-0002P0-1W for qemu-devel@nongnu.org; Thu, 30 Jun 2011 07:45:40 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1QcFgf-00038K-IV for qemu-devel@nongnu.org; Thu, 30 Jun 2011 07:45:38 -0400 Received: from cantor2.suse.de ([195.135.220.15]:39361 helo=mx2.suse.de) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1QcFgf-000389-7M for qemu-devel@nongnu.org; Thu, 30 Jun 2011 07:45:37 -0400 Message-ID: <4E0C61DF.2050606@suse.de> Date: Thu, 30 Jun 2011 13:45:35 +0200 From: Alexander Graf MIME-Version: 1.0 References: <1309260558-3332-1-git-send-email-stefano.stabellini@eu.citrix.com> In-Reply-To: <1309260558-3332-1-git-send-email-stefano.stabellini@eu.citrix.com> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH v3] xen: implement unplug protocol in xen_platform List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: stefano.stabellini@eu.citrix.com Cc: kwolf@redhat.com, xen-devel@lists.xensource.com, mst@redhat.com, qemu-devel@nongnu.org, armbru@redhat.com, anthony.perard@citrix.com On 06/28/2011 01:29 PM, stefano.stabellini@eu.citrix.com wrote: > From: Stefano Stabellini > > The unplug protocol is necessary to support PV drivers in the guest: the > drivers expect to be able to "unplug" emulated disks and nics before > initializing the Xen PV interfaces. > It is responsibility of the guest to make sure that the unplug is done > before the emulated devices or the PV interface start to be used. > > We use pci_for_each_device to walk the PCI bus, identify the devices and > disks that we want to disable and dynamically unplug them. $ ~/git/qemu/scripts/checkpatch.pl * ERROR: space required after that ',' (ctx:VxV) #158: FILE: hw/ide/piix.c:240: + },{ ^ total: 1 errors, 0 warnings, 130 lines checked I definitely want to see an ack from Kevin here first though. Alex From mboxrd@z Thu Jan 1 00:00:00 1970 From: Alexander Graf Subject: Re: [PATCH v3] xen: implement unplug protocol in xen_platform Date: Thu, 30 Jun 2011 13:45:35 +0200 Message-ID: <4E0C61DF.2050606@suse.de> References: <1309260558-3332-1-git-send-email-stefano.stabellini@eu.citrix.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1309260558-3332-1-git-send-email-stefano.stabellini@eu.citrix.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xensource.com Errors-To: xen-devel-bounces@lists.xensource.com To: stefano.stabellini@eu.citrix.com Cc: kwolf@redhat.com, xen-devel@lists.xensource.com, mst@redhat.com, qemu-devel@nongnu.org, armbru@redhat.com, anthony@codemonkey.ws, anthony.perard@citrix.com List-Id: xen-devel@lists.xenproject.org On 06/28/2011 01:29 PM, stefano.stabellini@eu.citrix.com wrote: > From: Stefano Stabellini > > The unplug protocol is necessary to support PV drivers in the guest: the > drivers expect to be able to "unplug" emulated disks and nics before > initializing the Xen PV interfaces. > It is responsibility of the guest to make sure that the unplug is done > before the emulated devices or the PV interface start to be used. > > We use pci_for_each_device to walk the PCI bus, identify the devices and > disks that we want to disable and dynamically unplug them. $ ~/git/qemu/scripts/checkpatch.pl * ERROR: space required after that ',' (ctx:VxV) #158: FILE: hw/ide/piix.c:240: + },{ ^ total: 1 errors, 0 warnings, 130 lines checked I definitely want to see an ack from Kevin here first though. Alex