Re: [Qemu-devel] [Xen-devel] [PATCH] Add Xen platform PCI device version 2.

2013-06-27 Thread Paul Durrant
Campbell; Matt Wilson; xen- > de...@lists.xen.org > Subject: Re: [Qemu-devel] [Xen-devel] [PATCH] Add Xen platform PCI device > version 2. > > > -Original Message- > > From: Alex Bligh [mailto:a...@alex.org.uk] > > Sent: 26 June 2013 21:00 > > To: Paul

Re: [Qemu-devel] [Xen-devel] [PATCH] Add Xen platform PCI device version 2.

2013-06-27 Thread Paul Durrant
> -Original Message- > From: Alex Bligh [mailto:a...@alex.org.uk] > Sent: 26 June 2013 21:00 > To: Paul Durrant; Tim (Xen.org) > Cc: Ian Campbell; Matt Wilson; xen-de...@lists.xen.org; qemu- > de...@nongnu.org; Alex Bligh > Subject: RE: [Xen-devel] [Qemu-devel] [PATCH] Add Xen platform PCI

Re: [Qemu-devel] [Xen-devel] [PATCH] Add Xen platform PCI device version 2.

2013-06-26 Thread Alex Bligh
--On 26 June 2013 12:06:31 + Paul Durrant wrote: The issue is the old s/w not the new s/w. The old drivers make assumptions about each other's presence as we can't change that because they are already out there. Then (without knowing the details) what's to prevent the new drivers not

Re: [Qemu-devel] [Xen-devel] [PATCH] Add Xen platform PCI device version 2.

2013-06-26 Thread Paul Durrant
> -Original Message- > From: Tim Deegan [mailto:t...@xen.org] > Sent: 26 June 2013 13:36 > To: Paul Durrant > Cc: Ian Campbell; Matt Wilson; Alex Bligh; xen-de...@lists.xen.org; qemu- > de...@nongnu.org > Subject: Re: [Xen-devel] [Qemu-devel] [PATCH] Add Xen platform PCI device > version 2.

Re: [Qemu-devel] [Xen-devel] [PATCH] Add Xen platform PCI device version 2.

2013-06-26 Thread Tim Deegan
At 12:06 + on 26 Jun (1372248391), Paul Durrant wrote: > > -Original Message- > > From: Tim Deegan [mailto:t...@xen.org] > > Sent: 26 June 2013 12:58 > > To: Paul Durrant > > Cc: Ian Campbell; Matt Wilson; Alex Bligh; xen-de...@lists.xen.org; qemu- > > de...@nongnu.org > > Subject: Re:

Re: [Qemu-devel] [Xen-devel] [PATCH] Add Xen platform PCI device version 2.

2013-06-26 Thread Paul Durrant
> -Original Message- > From: Tim Deegan [mailto:t...@xen.org] > Sent: 26 June 2013 12:58 > To: Paul Durrant > Cc: Ian Campbell; Matt Wilson; Alex Bligh; xen-de...@lists.xen.org; qemu- > de...@nongnu.org > Subject: Re: [Xen-devel] [Qemu-devel] [PATCH] Add Xen platform PCI device > version 2.

Re: [Qemu-devel] [Xen-devel] [PATCH] Add Xen platform PCI device version 2.

2013-06-26 Thread Tim Deegan
At 11:23 + on 26 Jun (1372245783), Paul Durrant wrote: > We could blacklist all existing Citrix PV drivers in upstream QEMU, > to avoid the clash, but that seems like a very unfriendly > approach. Also, it's not going to stop someone with an existing VM, > who happens to be using legacy Citrix

Re: [Qemu-devel] [Xen-devel] [PATCH] Add Xen platform PCI device version 2.

2013-06-26 Thread Ian Campbell
On Wed, 2013-06-26 at 12:23 +0100, Paul Durrant wrote: > > -Original Message- > > From: Ian Campbell > > Sent: 26 June 2013 11:40 > > To: Tim (Xen.org) > > Cc: Paul Durrant; Matt Wilson; Alex Bligh; xen-de...@lists.xen.org; qemu- > > de...@nongnu.org > > Subject: Re: [Xen-devel] [Qemu-devel

Re: [Qemu-devel] [Xen-devel] [PATCH] Add Xen platform PCI device version 2.

2013-06-26 Thread Paul Durrant
> -Original Message- > From: Ian Campbell > Sent: 26 June 2013 11:40 > To: Tim (Xen.org) > Cc: Paul Durrant; Matt Wilson; Alex Bligh; xen-de...@lists.xen.org; qemu- > de...@nongnu.org > Subject: Re: [Xen-devel] [Qemu-devel] [PATCH] Add Xen platform PCI device > version 2. > > On Thu, 2013-

Re: [Qemu-devel] [Xen-devel] [PATCH] Add Xen platform PCI device version 2.

2013-06-26 Thread Ian Campbell
On Thu, 2013-06-20 at 09:56 +0100, Tim Deegan wrote: > At 07:47 + on 20 Jun (1371714432), Paul Durrant wrote: > > > > I agree. If this is really the only solution, we would need to have > > > > both versions presented to the guest so that old drivers continue to > > > > work without any interve

Re: [Qemu-devel] [Xen-devel] [PATCH] Add Xen platform PCI device version 2.

2013-06-26 Thread Ian Campbell
On Wed, 2013-06-19 at 12:31 +0100, Paul Durrant wrote: > That's correct. If a vendor wishes to ship a new driver for an > existing piece of h/w they just post it. However, at some point the > vendor will sell a new piece of h/w which requires a driver that will > not work with older h/w. So, they

Re: [Qemu-devel] [Xen-devel] [PATCH] Add Xen platform PCI device version 2.

2013-06-20 Thread Paul Durrant
> -Original Message- > From: Tim Deegan [mailto:t...@xen.org] > Sent: 20 June 2013 09:56 > To: Paul Durrant > Cc: Matt Wilson; Alex Bligh; xen-de...@lists.xen.org; Ian Campbell; qemu- > de...@nongnu.org > Subject: Re: [Xen-devel] [Qemu-devel] [PATCH] Add Xen platform PCI device > version 2.

Re: [Qemu-devel] [Xen-devel] [PATCH] Add Xen platform PCI device version 2.

2013-06-20 Thread Tim Deegan
At 07:47 + on 20 Jun (1371714432), Paul Durrant wrote: > > > I agree. If this is really the only solution, we would need to have > > > both versions presented to the guest so that old drivers continue to > > > work without any intervention. > > > > I suspect that if we expose both, both sets o

Re: [Qemu-devel] [Xen-devel] [PATCH] Add Xen platform PCI device version 2.

2013-06-20 Thread Paul Durrant
> -Original Message- > From: Alex Bligh [mailto:a...@alex.org.uk] > Sent: 20 June 2013 09:09 > To: Paul Durrant; Tim (Xen.org); Matt Wilson > Cc: xen-de...@lists.xen.org; Ian Campbell; qemu-devel@nongnu.org; Alex > Bligh > Subject: RE: [Xen-devel] [Qemu-devel] [PATCH] Add Xen platform PCI d

Re: [Qemu-devel] [Xen-devel] [PATCH] Add Xen platform PCI device version 2.

2013-06-20 Thread Alex Bligh
--On 20 June 2013 07:47:12 + Paul Durrant 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-

Re: [Qemu-devel] [Xen-devel] [PATCH] Add Xen platform PCI device version 2.

2013-06-20 Thread Paul Durrant
> -Original Message- > From: Tim Deegan [mailto:t...@xen.org] > Sent: 19 June 2013 21:15 > To: Matt Wilson > Cc: Alex Bligh; Paul Durrant; xen-de...@lists.xen.org; Ian Campbell; qemu- > de...@nongnu.org > Subject: Re: [Xen-devel] [Qemu-devel] [PATCH] Add Xen platform PCI device > version 2.

Re: [Qemu-devel] [Xen-devel] [PATCH] Add Xen platform PCI device version 2.

2013-06-19 Thread Tim Deegan
At 11:21 -0700 on 19 Jun (1371640904), Matt Wilson wrote: > On Wed, Jun 19, 2013 at 11:42:06AM +0100, Alex Bligh wrote: > > > > --On 19 June 2013 10:13:17 + Paul Durrant > > wrote: > > > > >>We obviously can't say to users "Are you running Windows and are you > > >>running PV drivers >= X.Y,

Re: [Qemu-devel] [Xen-devel] [PATCH] Add Xen platform PCI device version 2.

2013-06-19 Thread Matt Wilson
On Wed, Jun 19, 2013 at 11:42:06AM +0100, Alex Bligh wrote: > > --On 19 June 2013 10:13:17 + Paul Durrant > wrote: > > >>We obviously can't say to users "Are you running Windows and are you > >>running PV drivers >= X.Y, if so set lever A to position B, otherwise if > >>you are running some o

Re: [Qemu-devel] [Xen-devel] [PATCH] Add Xen platform PCI device version 2.

2013-06-19 Thread James Harper
> > > -Original Message- > > From: Ian Campbell > > Sent: 19 June 2013 10:42 > > To: Paul Durrant > > Cc: qemu-devel@nongnu.org; xen-de...@lists.xen.org > > Subject: Re: [Xen-devel] [PATCH] Add Xen platform PCI device version 2. > > > > On Wed, 2013-06-19 at 10:32 +0100, Paul Durrant wrote

Re: [Qemu-devel] [Xen-devel] [PATCH] Add Xen platform PCI device version 2.

2013-06-19 Thread Paolo Bonzini
Il 19/06/2013 13:31, Paul Durrant ha scritto: > This is model we have followed in XenServer: the platform device > represents 'the set of PV drivers' and therefore to ship a new and > non-backwards-compatible set of PV drivers we incremented the > platform device id. But what is exactly the incomp

Re: [Qemu-devel] [Xen-devel] [PATCH] Add Xen platform PCI device version 2.

2013-06-19 Thread Paul Durrant
> -Original Message- > From: Tim Deegan [mailto:t...@xen.org] > Sent: 19 June 2013 16:46 > To: Paul Durrant > Cc: James Harper; Ian Campbell; qemu-devel@nongnu.org; xen- > de...@lists.xen.org > Subject: Re: [Xen-devel] [PATCH] Add Xen platform PCI device version 2. > > At 11:23 + on 19

Re: [Qemu-devel] [Xen-devel] [PATCH] Add Xen platform PCI device version 2.

2013-06-19 Thread Tim Deegan
At 11:23 + on 19 Jun (1371641017), Paul Durrant wrote: > The problem is that the old Citrix PV drivers bind their version of > xenvbd directly to the platform device (id=1). The new PV drivers bind > their xenbus driver to their platform device, because to go onto > Windows Update you cannot ha

Re: [Qemu-devel] [Xen-devel] [PATCH] Add Xen platform PCI device version 2.

2013-06-19 Thread Paul Durrant
> -Original Message- > From: James Harper [mailto:james.har...@bendigoit.com.au] > Sent: 19 June 2013 11:55 > To: Paul Durrant; Ian Campbell > Cc: qemu-devel@nongnu.org; xen-de...@lists.xen.org > Subject: RE: [Xen-devel] [PATCH] Add Xen platform PCI device version 2. > > > > > > -Origi

Re: [Qemu-devel] [Xen-devel] [PATCH] Add Xen platform PCI device version 2.

2013-06-19 Thread Paul Durrant
> -Original Message- > From: Alex Bligh [mailto:a...@alex.org.uk] > Sent: 19 June 2013 11:42 > To: Paul Durrant; Ian Campbell > Cc: qemu-devel@nongnu.org; xen-de...@lists.xen.org; Alex Bligh > Subject: Re: [Qemu-devel] [Xen-devel] [PATCH] Add Xen platform PCI

Re: [Qemu-devel] [Xen-devel] [PATCH] Add Xen platform PCI device version 2.

2013-06-19 Thread Paul Durrant
n-de...@lists.xen.org > Subject: Re: [Qemu-devel] [Xen-devel] [PATCH] Add Xen platform PCI device > version 2. > > At 11:07 +0100 on 19 Jun (1371640052), Ian Campbell wrote: > > On Wed, 2013-06-19 at 10:43 +0100, Paul Durrant wrote: > > > > -Original Message- >

Re: [Qemu-devel] [Xen-devel] [PATCH] Add Xen platform PCI device version 2.

2013-06-19 Thread Paul Durrant
> -Original Message- > From: James Harper [mailto:james.har...@bendigoit.com.au] > Sent: 19 June 2013 11:55 > To: Paul Durrant; Ian Campbell > Cc: qemu-devel@nongnu.org; xen-de...@lists.xen.org > Subject: RE: [Xen-devel] [PATCH] Add Xen platform PCI device version 2. > > > > > > -Origi

Re: [Qemu-devel] [Xen-devel] [PATCH] Add Xen platform PCI device version 2.

2013-06-19 Thread Ian Campbell
On Wed, 2013-06-19 at 11:13 +0100, Paul Durrant wrote: > > -Original Message- > > From: Ian Campbell > > Sent: 19 June 2013 11:08 > > To: Paul Durrant > > Cc: qemu-devel@nongnu.org; xen-de...@lists.xen.org > > Subject: Re: [Xen-devel] [PATCH] Add Xen platform PCI device version 2. > > > >

Re: [Qemu-devel] [Xen-devel] [PATCH] Add Xen platform PCI device version 2.

2013-06-19 Thread Alex Bligh
--On 19 June 2013 10:13:17 + Paul Durrant wrote: We obviously can't say to users "Are you running Windows and are you running PV drivers >= X.Y, if so set lever A to position B, otherwise if you are running some other OS or an earlier version of the Windows PV driver set it to position

Re: [Qemu-devel] [Xen-devel] [PATCH] Add Xen platform PCI device version 2.

2013-06-19 Thread Tim Deegan
At 11:07 +0100 on 19 Jun (1371640052), Ian Campbell wrote: > On Wed, 2013-06-19 at 10:43 +0100, Paul Durrant wrote: > > > -Original Message- > > > From: Ian Campbell > > > Sent: 19 June 2013 10:42 > > > To: Paul Durrant > > > Cc: qemu-devel@nongnu.org; xen-de...@lists.xen.org > > > Subject:

Re: [Qemu-devel] [Xen-devel] [PATCH] Add Xen platform PCI device version 2.

2013-06-19 Thread Paul Durrant
> -Original Message- > From: Ian Campbell > Sent: 19 June 2013 11:08 > To: Paul Durrant > Cc: qemu-devel@nongnu.org; xen-de...@lists.xen.org > Subject: Re: [Xen-devel] [PATCH] Add Xen platform PCI device version 2. > > On Wed, 2013-06-19 at 10:43 +0100, Paul Durrant wrote: > > > -Origi

Re: [Qemu-devel] [Xen-devel] [PATCH] Add Xen platform PCI device version 2.

2013-06-19 Thread Ian Campbell
On Wed, 2013-06-19 at 10:43 +0100, Paul Durrant wrote: > > -Original Message- > > From: Ian Campbell > > Sent: 19 June 2013 10:42 > > To: Paul Durrant > > Cc: qemu-devel@nongnu.org; xen-de...@lists.xen.org > > Subject: Re: [Xen-devel] [PATCH] Add Xen platform PCI device version 2. > > > >

Re: [Qemu-devel] [Xen-devel] [PATCH] Add Xen platform PCI device version 2.

2013-06-19 Thread Ian Campbell
On Wed, 2013-06-19 at 10:32 +0100, Paul Durrant wrote: > The XenServer 6.1+ Citrix Windows PV bus driver binds to a new version > of the Xen platform device (since the newer driver set cannot co-exist > with previous drivers which bind to the existing "xen-platform" type of > device). This patch in

Re: [Qemu-devel] [Xen-devel] [PATCH] Add Xen platform PCI device version 2.

2013-06-19 Thread Paul Durrant
> -Original Message- > From: Ian Campbell > Sent: 19 June 2013 10:42 > To: Paul Durrant > Cc: qemu-devel@nongnu.org; xen-de...@lists.xen.org > Subject: Re: [Xen-devel] [PATCH] Add Xen platform PCI device version 2. > > On Wed, 2013-06-19 at 10:32 +0100, Paul Durrant wrote: > > The XenServe