From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stefano Stabellini Subject: Re: [Qemu-devel] [PATCH] Remove hardcoded xen-platform device initialization Date: Thu, 13 Jun 2013 18:33:06 +0100 Message-ID: References: <1371117054-5694-1-git-send-email-paul.durrant@citrix.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1371117054-5694-1-git-send-email-paul.durrant@citrix.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Paul Durrant Cc: qemu-devel@nongnu.org, xen-devel@lists.xen.org List-Id: xen-devel@lists.xenproject.org On Thu, 13 Jun 2013, Paul Durrant wrote: > The xen-platform device should be initialized by the Xen toolstack by > passing the appropriate -device argument on the command line. > > Signed-off-by: Paul Durrant This patch is problematic because we can't know for sure the version of upstream QEMU that is going to be used with Xen. If we apply this patch and QEMU 1.5 is going to be used with Xen 4.2, guests won't be able to use PV drivers. > hw/i386/pc_piix.c | 3 --- > 1 file changed, 3 deletions(-) > > diff --git a/hw/i386/pc_piix.c b/hw/i386/pc_piix.c > index d618570..e25012d 100644 > --- a/hw/i386/pc_piix.c > +++ b/hw/i386/pc_piix.c > @@ -174,9 +174,6 @@ static void pc_init1(MemoryRegion *system_memory, > pc_register_ferr_irq(gsi[13]); > > pc_vga_init(isa_bus, pci_enabled ? pci_bus : NULL); > - if (xen_enabled()) { > - pci_create_simple(pci_bus, -1, "xen-platform"); > - } > > /* init basic PC hardware */ > pc_basic_device_init(isa_bus, gsi, &rtc_state, &floppy, xen_enabled());