Re: [Xen-devel] [PATCH XEN v5 07/23] tools: Refactor /dev/xen/gnt{dev, shr} wrappers into libxengnttab.

2015-11-25 Thread Ian Campbell
On Tue, 2015-11-24 at 09:34 +, Ian Campbell wrote: > Thinking about this some more overnight, it occurred to me that the > real issue is that gnttab and gntshr are actually two quite difference > APIs. gnttab is all about consuming grant references which are given to > you from elsewhere while

Re: [Xen-devel] [PATCH XEN v5 07/23] tools: Refactor /dev/xen/gnt{dev, shr} wrappers into libxengnttab.

2015-11-24 Thread Daniel De Graaf
On 16/11/15 07:30, Ian Campbell wrote: On Fri, 2015-11-13 at 15:38 -0500, Daniel De Graaf wrote: On 13/11/15 10:02, Ian Campbell wrote: On Wed, 2015-11-11 at 15:03 +, Ian Jackson wrote: Ian Campbell writes ("[PATCH XEN v5 07/23] tools: Refactor /dev/xen/gnt{dev,shr} wrappers into libxengnt

Re: [Xen-devel] [PATCH XEN v5 07/23] tools: Refactor /dev/xen/gnt{dev, shr} wrappers into libxengnttab.

2015-11-24 Thread Ian Campbell
On Mon, 2015-11-23 at 17:05 +, Ian Campbell wrote: Adding Daniel who I should have included the first time, plus a few more thoughts from me at the end. > On Wed, 2015-11-11 at 15:03 +, Ian Jackson wrote: > > > XXX consider combining into a single namespace (i.e. with > > > xengnttab_hand

Re: [Xen-devel] [PATCH XEN v5 07/23] tools: Refactor /dev/xen/gnt{dev, shr} wrappers into libxengnttab.

2015-11-23 Thread Ian Campbell
On Wed, 2015-11-11 at 15:03 +, Ian Jackson wrote: > > XXX consider combining into a single namespace (i.e. with > > xengnttab_handle having two open fd's in it on Linux) > [...] > I conclude it would be better to unify them. Starting to look into this I have some queries on the direction I ou

Re: [Xen-devel] [PATCH XEN v5 07/23] tools: Refactor /dev/xen/gnt{dev, shr} wrappers into libxengnttab.

2015-11-16 Thread Ian Campbell
On Fri, 2015-11-13 at 15:38 -0500, Daniel De Graaf wrote: > On 13/11/15 10:02, Ian Campbell wrote: > > On Wed, 2015-11-11 at 15:03 +, Ian Jackson wrote: > > > Ian Campbell writes ("[PATCH XEN v5 07/23] tools: Refactor > > > /dev/xen/gnt{dev,shr} wrappers into libxengnttab."): > > > > libxengntt

Re: [Xen-devel] [PATCH XEN v5 07/23] tools: Refactor /dev/xen/gnt{dev, shr} wrappers into libxengnttab.

2015-11-13 Thread Daniel De Graaf
On 13/11/15 10:02, Ian Campbell wrote: On Wed, 2015-11-11 at 15:03 +, Ian Jackson wrote: Ian Campbell writes ("[PATCH XEN v5 07/23] tools: Refactor /dev/xen/gnt{dev,shr} wrappers into libxengnttab."): libxengnttab will provide a stable API and ABI for accessing the grant table devices. The

Re: [Xen-devel] [PATCH XEN v5 07/23] tools: Refactor /dev/xen/gnt{dev, shr} wrappers into libxengnttab.

2015-11-13 Thread Ian Campbell
On Wed, 2015-11-11 at 15:03 +, Ian Jackson wrote: > Ian Campbell writes ("[PATCH XEN v5 07/23] tools: Refactor > /dev/xen/gnt{dev,shr} wrappers into libxengnttab."): > > libxengnttab will provide a stable API and ABI for accessing the > > grant table devices. > > > > The functions are moved in

Re: [Xen-devel] [PATCH XEN v5 07/23] tools: Refactor /dev/xen/gnt{dev, shr} wrappers into libxengnttab.

2015-11-11 Thread Ian Jackson
Ian Campbell writes ("[PATCH XEN v5 07/23] tools: Refactor /dev/xen/gnt{dev,shr} wrappers into libxengnttab."): > libxengnttab will provide a stable API and ABI for accessing the > grant table devices. > > The functions are moved into the xengnt{tab,shr} namespace to make a > clean break from lib

Re: [Xen-devel] [PATCH XEN v5 07/23] tools: Refactor /dev/xen/gnt{dev, shr} wrappers into libxengnttab.

2015-11-11 Thread Wei Liu
On Mon, Nov 09, 2015 at 12:00:42PM +, Ian Campbell wrote: > libxengnttab will provide a stable API and ABI for accessing the > grant table devices. > > The functions are moved into the xengnt{tab,shr} namespace to make a > clean break from libxc and avoid ambiguity regarding which interfaces >

[Xen-devel] [PATCH XEN v5 07/23] tools: Refactor /dev/xen/gnt{dev, shr} wrappers into libxengnttab.

2015-11-09 Thread Ian Campbell
libxengnttab will provide a stable API and ABI for accessing the grant table devices. The functions are moved into the xengnt{tab,shr} namespace to make a clean break from libxc and avoid ambiguity regarding which interfaces are stable. XXX consider combining into a single namespace (i.e. with xe