[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [Xen-devel] [PATCH] Add Xen platform PCI device version
From: |
Paul Durrant |
Subject: |
Re: [Qemu-devel] [Xen-devel] [PATCH] Add Xen platform PCI device version 2. |
Date: |
Thu, 20 Jun 2013 08:19:53 +0000 |
> -----Original Message-----
> From: Alex Bligh [mailto:address@hidden
> Sent: 20 June 2013 09:09
> To: Paul Durrant; Tim (Xen.org); Matt Wilson
> Cc: address@hidden; Ian Campbell; address@hidden; Alex
> Bligh
> Subject: RE: [Xen-devel] [Qemu-devel] [PATCH] Add Xen platform PCI device
> version 2.
>
>
>
> --On 20 June 2013 07:47:12 +0000 Paul Durrant <address@hidden>
> wrote:
>
> > If someone has a VM that has the old Citrix drivers installed, or GPLPV,
> > I think I should be able to spot this and make sure that the new bus
> > driver quiesces itself to prevent strangeness ensuing. If and when said
> > previous drivers are un-installed then the new bus driver can wake up and
> > enumerate the device nodes for the other pv drivers and Windows Update
> > can carry on doing its stuff.
>
> I have no clue about Windows device drivers, so this may be a silly
> suggestion. If your suggestion above already requires a Xen code change,
> one possibility might be copy the idea behind the PCI unplug logic. Either
> if the new PCI device is used, it could unplug the old one, or vice versa.
> Drivers magically unplugging themselves may not be ideal, but it beats
> having 2 drivers fighting over the same device.
>
Unfortunately, whilst it sounds good on the face of it, it's not as
straightforward as that. The old Citrix PV drivers did not just bind to the Xen
platform device, and make that device go away automagically would actually
cause the system disk to disappear without any clean fallback to emulation.
As long as nothing actually breaks if and when Windows fetches the new PV bus
driver from Windows Update then we can document the need to manually uninstall
any other PV drivers.
Paul
- Re: [Qemu-devel] [Xen-devel] [PATCH] Add Xen platform PCI device version 2., (continued)
- Re: [Qemu-devel] [Xen-devel] [PATCH] Add Xen platform PCI device version 2., Ian Campbell, 2013/06/19
- Re: [Qemu-devel] [Xen-devel] [PATCH] Add Xen platform PCI device version 2., Paul Durrant, 2013/06/19
- Re: [Qemu-devel] [Xen-devel] [PATCH] Add Xen platform PCI device version 2., Ian Campbell, 2013/06/19
- Re: [Qemu-devel] [Xen-devel] [PATCH] Add Xen platform PCI device version 2., Paul Durrant, 2013/06/19
- Re: [Qemu-devel] [Xen-devel] [PATCH] Add Xen platform PCI device version 2., Alex Bligh, 2013/06/19
- Re: [Qemu-devel] [Xen-devel] [PATCH] Add Xen platform PCI device version 2., Paul Durrant, 2013/06/19
- Re: [Qemu-devel] [Xen-devel] [PATCH] Add Xen platform PCI device version 2., Matt Wilson, 2013/06/19
- Re: [Qemu-devel] [Xen-devel] [PATCH] Add Xen platform PCI device version 2., Tim Deegan, 2013/06/19
- Re: [Qemu-devel] [Xen-devel] [PATCH] Add Xen platform PCI device version 2., Paul Durrant, 2013/06/20
- Re: [Qemu-devel] [Xen-devel] [PATCH] Add Xen platform PCI device version 2., Alex Bligh, 2013/06/20
- Re: [Qemu-devel] [Xen-devel] [PATCH] Add Xen platform PCI device version 2.,
Paul Durrant <=
- Re: [Qemu-devel] [Xen-devel] [PATCH] Add Xen platform PCI device version 2., Tim Deegan, 2013/06/20
- Re: [Qemu-devel] [Xen-devel] [PATCH] Add Xen platform PCI device version 2., Paul Durrant, 2013/06/20
- Re: [Qemu-devel] [Xen-devel] [PATCH] Add Xen platform PCI device version 2., Ian Campbell, 2013/06/26
- Re: [Qemu-devel] [Xen-devel] [PATCH] Add Xen platform PCI device version 2., Paul Durrant, 2013/06/26
- Re: [Qemu-devel] [Xen-devel] [PATCH] Add Xen platform PCI device version 2., Ian Campbell, 2013/06/26
- Re: [Qemu-devel] [Xen-devel] [PATCH] Add Xen platform PCI device version 2., Tim Deegan, 2013/06/26
- Re: [Qemu-devel] [Xen-devel] [PATCH] Add Xen platform PCI device version 2., Paul Durrant, 2013/06/26
- Re: [Qemu-devel] [Xen-devel] [PATCH] Add Xen platform PCI device version 2., Tim Deegan, 2013/06/26
- Re: [Qemu-devel] [Xen-devel] [PATCH] Add Xen platform PCI device version 2., Paul Durrant, 2013/06/26
- Re: [Qemu-devel] [Xen-devel] [PATCH] Add Xen platform PCI device version 2., Alex Bligh, 2013/06/26