From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Vrabel Date: Wed, 04 Dec 2013 13:00:08 +0000 Subject: Re: [PATCH] xen/pvhvm: If xen_platform_pci=0 is set don't blow up. Message-Id: <529F2758.9070304@citrix.com> List-Id: References: <1386105246-14337-1-git-send-email-konrad.wilk@oracle.com> In-Reply-To: <1386105246-14337-1-git-send-email-konrad.wilk@oracle.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Konrad Rzeszutek Wilk Cc: stefano.stabellini@eu.citrix.com, ian.campbell@citrix.com, xen-devel@lists.xenproject.org, linux-kernel@vger.kernel.org, boris.ostrovsky@oracle.com, leosilva@linux.vnet.ibm.com, ashley@ashleylai.com, peterhuewe@gmx.de, mail@srajiv.net, tpmdd@selhorst.net, tpmdd@sirrix.com, dmitry.torokhov@gmail.com, bhelgaas@google.com, plagnioj@jcrosoft.com, tomi.valkeinen@ti.com, tpmdd-devel@lists.sourceforge.net, linux-input@vger.kernel.org, netdev@vger.kernel.org, linux-pci@vger.kernel.org, linux-fbdev@vger.kernel.org On 03/12/13 21:14, Konrad Rzeszutek Wilk wrote: > > Ian Campbell suggested getting rid of 'xen_platform_pci_unplug' > but unfortunatly the xen-blkfront driver is using it, so we > cannot do it. I had a look at what blkfront was using this for and it seems dumb. How did we end up with the frontend driver working around toolstack bugs? If HVM Linux guest didn't want (e.g.,) PV CDROM, the toolstack shouldn't have created one. And perhaps more importantly, if you actually want a PV CDROM in a HVM guest, it's not possible. David From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Vrabel Subject: Re: [PATCH] xen/pvhvm: If xen_platform_pci=0 is set don't blow up. Date: Wed, 4 Dec 2013 13:00:08 +0000 Message-ID: <529F2758.9070304@citrix.com> References: <1386105246-14337-1-git-send-email-konrad.wilk@oracle.com> Mime-Version: 1.0 Content-Type: text/plain; charset="ISO-8859-1" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1386105246-14337-1-git-send-email-konrad.wilk@oracle.com> Sender: linux-pci-owner@vger.kernel.org To: Konrad Rzeszutek Wilk Cc: stefano.stabellini@eu.citrix.com, ian.campbell@citrix.com, xen-devel@lists.xenproject.org, linux-kernel@vger.kernel.org, boris.ostrovsky@oracle.com, leosilva@linux.vnet.ibm.com, ashley@ashleylai.com, peterhuewe@gmx.de, mail@srajiv.net, tpmdd@selhorst.net, tpmdd@sirrix.com, dmitry.torokhov@gmail.com, bhelgaas@google.com, plagnioj@jcrosoft.com, tomi.valkeinen@ti.com, tpmdd-devel@lists.sourceforge.net, linux-input@vger.kernel.org, netdev@vger.kernel.org, linux-pci@vger.kernel.org, linux-fbdev@vger.kernel.org List-Id: linux-input@vger.kernel.org On 03/12/13 21:14, Konrad Rzeszutek Wilk wrote: > > Ian Campbell suggested getting rid of 'xen_platform_pci_unplug' > but unfortunatly the xen-blkfront driver is using it, so we > cannot do it. I had a look at what blkfront was using this for and it seems dumb. How did we end up with the frontend driver working around toolstack bugs? If HVM Linux guest didn't want (e.g.,) PV CDROM, the toolstack shouldn't have created one. And perhaps more importantly, if you actually want a PV CDROM in a HVM guest, it's not possible. David From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from smtp.citrix.com ([66.165.176.89]:47592 "EHLO SMTP.CITRIX.COM" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932201Ab3LDNAN (ORCPT ); Wed, 4 Dec 2013 08:00:13 -0500 Message-ID: <529F2758.9070304@citrix.com> Date: Wed, 4 Dec 2013 13:00:08 +0000 From: David Vrabel MIME-Version: 1.0 To: Konrad Rzeszutek Wilk CC: , , , , , , , , , , , , , , , , , , , Subject: Re: [PATCH] xen/pvhvm: If xen_platform_pci=0 is set don't blow up. References: <1386105246-14337-1-git-send-email-konrad.wilk@oracle.com> In-Reply-To: <1386105246-14337-1-git-send-email-konrad.wilk@oracle.com> Content-Type: text/plain; charset="ISO-8859-1" Sender: linux-pci-owner@vger.kernel.org List-ID: On 03/12/13 21:14, Konrad Rzeszutek Wilk wrote: > > Ian Campbell suggested getting rid of 'xen_platform_pci_unplug' > but unfortunatly the xen-blkfront driver is using it, so we > cannot do it. I had a look at what blkfront was using this for and it seems dumb. How did we end up with the frontend driver working around toolstack bugs? If HVM Linux guest didn't want (e.g.,) PV CDROM, the toolstack shouldn't have created one. And perhaps more importantly, if you actually want a PV CDROM in a HVM guest, it's not possible. David