Re: [Qemu-devel] [PATCH v4 1/8] xen: import ring.h from xen

2017-03-29 Thread Stefano Stabellini
On Wed, 29 Mar 2017, Paolo Bonzini wrote: > On 29/03/2017 01:54, Stefano Stabellini wrote: > >>> I understand your point of view, and honestly it wouldn't be a problem > >>> doing it the way you suggested either. However, I think that going > >>> forward it will be less of a maintenance pain to kee

Re: [Qemu-devel] [PATCH v4 1/8] xen: import ring.h from xen

2017-03-29 Thread Paolo Bonzini
On 29/03/2017 01:54, Stefano Stabellini wrote: >>> I understand your point of view, and honestly it wouldn't be a problem >>> doing it the way you suggested either. However, I think that going >>> forward it will be less of a maintenance pain to keep ring.h in sync, >>> compared to maintaining a

Re: [Qemu-devel] [PATCH v4 1/8] xen: import ring.h from xen

2017-03-28 Thread Juergen Gross
On 29/03/17 01:54, Stefano Stabellini wrote: > On Tue, 28 Mar 2017, Juergen Gross wrote: >> On 28/03/17 00:48, Stefano Stabellini wrote: >>> On Mon, 27 Mar 2017, Juergen Gross wrote: On 24/03/17 18:37, Stefano Stabellini wrote: > On Fri, 24 Mar 2017, Juergen Gross wrote: >> On 23/03/17

Re: [Qemu-devel] [PATCH v4 1/8] xen: import ring.h from xen

2017-03-28 Thread Stefano Stabellini
On Tue, 28 Mar 2017, Juergen Gross wrote: > On 28/03/17 00:48, Stefano Stabellini wrote: > > On Mon, 27 Mar 2017, Juergen Gross wrote: > >> On 24/03/17 18:37, Stefano Stabellini wrote: > >>> On Fri, 24 Mar 2017, Juergen Gross wrote: > On 23/03/17 19:22, Stefano Stabellini wrote: > > On Thu

Re: [Qemu-devel] [PATCH v4 1/8] xen: import ring.h from xen

2017-03-27 Thread Juergen Gross
On 28/03/17 00:48, Stefano Stabellini wrote: > On Mon, 27 Mar 2017, Juergen Gross wrote: >> On 24/03/17 18:37, Stefano Stabellini wrote: >>> On Fri, 24 Mar 2017, Juergen Gross wrote: On 23/03/17 19:22, Stefano Stabellini wrote: > On Thu, 23 Mar 2017, Paolo Bonzini wrote: >> On 23/03/20

Re: [Qemu-devel] [PATCH v4 1/8] xen: import ring.h from xen

2017-03-27 Thread Stefano Stabellini
On Mon, 27 Mar 2017, Juergen Gross wrote: > On 24/03/17 18:37, Stefano Stabellini wrote: > > On Fri, 24 Mar 2017, Juergen Gross wrote: > >> On 23/03/17 19:22, Stefano Stabellini wrote: > >>> On Thu, 23 Mar 2017, Paolo Bonzini wrote: > On 23/03/2017 14:55, Juergen Gross wrote: > > On 23/03/

Re: [Qemu-devel] [PATCH v4 1/8] xen: import ring.h from xen

2017-03-27 Thread Juergen Gross
On 24/03/17 18:37, Stefano Stabellini wrote: > On Fri, 24 Mar 2017, Juergen Gross wrote: >> On 23/03/17 19:22, Stefano Stabellini wrote: >>> On Thu, 23 Mar 2017, Paolo Bonzini wrote: On 23/03/2017 14:55, Juergen Gross wrote: > On 23/03/17 14:00, Greg Kurz wrote: >> On Mon, 20 Mar 2017

Re: [Qemu-devel] [PATCH v4 1/8] xen: import ring.h from xen

2017-03-24 Thread Stefano Stabellini
On Fri, 24 Mar 2017, Juergen Gross wrote: > On 23/03/17 19:22, Stefano Stabellini wrote: > > On Thu, 23 Mar 2017, Paolo Bonzini wrote: > >> On 23/03/2017 14:55, Juergen Gross wrote: > >>> On 23/03/17 14:00, Greg Kurz wrote: > On Mon, 20 Mar 2017 11:19:05 -0700 > Stefano Stabellini wrote:

Re: [Qemu-devel] [PATCH v4 1/8] xen: import ring.h from xen

2017-03-23 Thread Juergen Gross
On 23/03/17 19:22, Stefano Stabellini wrote: > On Thu, 23 Mar 2017, Paolo Bonzini wrote: >> On 23/03/2017 14:55, Juergen Gross wrote: >>> On 23/03/17 14:00, Greg Kurz wrote: On Mon, 20 Mar 2017 11:19:05 -0700 Stefano Stabellini wrote: > Do not use the ring.h header installed on

Re: [Qemu-devel] [PATCH v4 1/8] xen: import ring.h from xen

2017-03-23 Thread Stefano Stabellini
On Thu, 23 Mar 2017, Paolo Bonzini wrote: > On 23/03/2017 14:55, Juergen Gross wrote: > > On 23/03/17 14:00, Greg Kurz wrote: > >> On Mon, 20 Mar 2017 11:19:05 -0700 > >> Stefano Stabellini wrote: > >> > >>> Do not use the ring.h header installed on the system. Instead, import > >>> the header int

Re: [Qemu-devel] [PATCH v4 1/8] xen: import ring.h from xen

2017-03-23 Thread Greg Kurz
On Thu, 23 Mar 2017 15:19:28 +0100 Paolo Bonzini wrote: > On 23/03/2017 14:55, Juergen Gross wrote: > > On 23/03/17 14:00, Greg Kurz wrote: > >> On Mon, 20 Mar 2017 11:19:05 -0700 > >> Stefano Stabellini wrote: > >> > >>> Do not use the ring.h header installed on the system. Instead, import

Re: [Qemu-devel] [PATCH v4 1/8] xen: import ring.h from xen

2017-03-23 Thread Greg Kurz
On Thu, 23 Mar 2017 14:55:26 +0100 Juergen Gross wrote: > On 23/03/17 14:00, Greg Kurz wrote: > > On Mon, 20 Mar 2017 11:19:05 -0700 > > Stefano Stabellini wrote: > > > >> Do not use the ring.h header installed on the system. Instead, import > >> the header into the QEMU codebase. This avoids

Re: [Qemu-devel] [PATCH v4 1/8] xen: import ring.h from xen

2017-03-23 Thread Paolo Bonzini
On 23/03/2017 14:55, Juergen Gross wrote: > On 23/03/17 14:00, Greg Kurz wrote: >> On Mon, 20 Mar 2017 11:19:05 -0700 >> Stefano Stabellini wrote: >> >>> Do not use the ring.h header installed on the system. Instead, import >>> the header into the QEMU codebase. This avoids problems when QEMU is

Re: [Qemu-devel] [PATCH v4 1/8] xen: import ring.h from xen

2017-03-23 Thread Juergen Gross
On 23/03/17 14:00, Greg Kurz wrote: > On Mon, 20 Mar 2017 11:19:05 -0700 > Stefano Stabellini wrote: > >> Do not use the ring.h header installed on the system. Instead, import >> the header into the QEMU codebase. This avoids problems when QEMU is >> built against a Xen version too old to provide

Re: [Qemu-devel] [PATCH v4 1/8] xen: import ring.h from xen

2017-03-23 Thread Greg Kurz
On Mon, 20 Mar 2017 11:19:05 -0700 Stefano Stabellini wrote: > Do not use the ring.h header installed on the system. Instead, import > the header into the QEMU codebase. This avoids problems when QEMU is > built against a Xen version too old to provide all the ring macros. > > Signed-off-by: Ste

[Qemu-devel] [PATCH v4 1/8] xen: import ring.h from xen

2017-03-20 Thread Stefano Stabellini
Do not use the ring.h header installed on the system. Instead, import the header into the QEMU codebase. This avoids problems when QEMU is built against a Xen version too old to provide all the ring macros. Signed-off-by: Stefano Stabellini Reviewed-by: Greg Kurz CC: anthony.per...@citrix.com CC