Hello, I wanted to map several grants at a time, and I found that xc_gnttab_map_grant_refs permits that, but it requires you to fill a domid array with the domids of each page. Isn''t just usually the same? Or put another way, in which case would you want to contiguously map pages from different domains? It looks to me like the interface puts burden on 99% usage case for little reason... Samuel _______________________________________________ Xen-devel mailing list Xen-devel@lists.xensource.com http://lists.xensource.com/xen-devel
Mea culpa for that bit of API. I think this is a case where the underlying implementation makes it possible, so I decided to make it available through the user API. No particular reason for it, so I have no objection to it being simplified (or wrapped by an additional API function). Regards, Derek Murray. On Wed, Jul 16, 2008 at 3:40 PM, Samuel Thibault <samuel.thibault@eu.citrix.com> wrote:> Hello, > > I wanted to map several grants at a time, and I found that > xc_gnttab_map_grant_refs permits that, but it requires you to fill a > domid array with the domids of each page. Isn''t just usually the same? > Or put another way, in which case would you want to contiguously map > pages from different domains? It looks to me like the interface puts > burden on 99% usage case for little reason... > > Samuel > > _______________________________________________ > Xen-devel mailing list > Xen-devel@lists.xensource.com > http://lists.xensource.com/xen-devel >_______________________________________________ Xen-devel mailing list Xen-devel@lists.xensource.com http://lists.xensource.com/xen-devel
Derek Murray, le Wed 16 Jul 2008 15:51:18 +0100, a écrit :> Mea culpa for that bit of API. I think this is a case where the > underlying implementation makes it possible, so I decided to make it > available through the user API. No particular reason for it, so I have > no objection to it being simplified (or wrapped by an additional API > function).That''s why I asked whether somebody actually use that function :) Samuel _______________________________________________ Xen-devel mailing list Xen-devel@lists.xensource.com http://lists.xensource.com/xen-devel
Samuel Thibault writes ("Re: [Xen-devel] Who uses
xc_gnttab_map_grant_refs?"):> Derek Murray, le Wed 16 Jul 2008 15:51:18 +0100, a écrit :
> > Mea culpa for that bit of API. I think this is a case where the
> > underlying implementation makes it possible, so I decided to make it
> > available through the user API. No particular reason for it, so I have
> > no objection to it being simplified (or wrapped by an additional API
> > function).
>
> That''s why I asked whether somebody actually use that function :)
Of course other users of that function may not be reading this list.
If you''re suggesting to change the ABI or API, why not give the new
interface a new name ?
Ian.
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xensource.com
http://lists.xensource.com/xen-devel
Ian Jackson, le Fri 18 Jul 2008 15:21:53 +0100, a écrit :> Samuel Thibault writes ("Re: [Xen-devel] Who uses xc_gnttab_map_grant_refs?"): > > Derek Murray, le Wed 16 Jul 2008 15:51:18 +0100, a écrit : > > > Mea culpa for that bit of API. I think this is a case where the > > > underlying implementation makes it possible, so I decided to make it > > > available through the user API. No particular reason for it, so I have > > > no objection to it being simplified (or wrapped by an additional API > > > function). > > > > That''s why I asked whether somebody actually use that function :) > > Of course other users of that function may not be reading this list.Sure> If you''re suggesting to change the ABI or API, why not give the new > interface a new name ?But still kill the old function at least? I''m asking because not killing the old function would mean either duplicating the code, or writing a wrapper that merely allocate an array full of the same value... Samuel _______________________________________________ Xen-devel mailing list Xen-devel@lists.xensource.com http://lists.xensource.com/xen-devel
Samuel Thibault writes ("Re: [Xen-devel] Who uses
xc_gnttab_map_grant_refs?"):> But still kill the old function at least?
Is that a change we want to make at this stage of the release ? I
don''t necessarily have an opinion but it''s a question we
should be
asking.
Ian.
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xensource.com
http://lists.xensource.com/xen-devel
Samuel Thibault
2008-Jul-21 10:48 UTC
[Xen-devel] [PATCH] Re: Who uses xc_gnttab_map_grant_refs?
Ian Jackson, le Fri 18 Jul 2008 15:55:58 +0100, a écrit :> Samuel Thibault writes ("Re: [Xen-devel] Who uses xc_gnttab_map_grant_refs?"): > > But still kill the old function at least? > > Is that a change we want to make at this stage of the release ? I > don''t necessarily have an opinion but it''s a question we should be > asking.Ok, well, here is a patch that keeps the old function, just making its implementation an internal with an additional domids_stride parameter in order to also both efficiency. Samuel libxc: add xc_gnttab_map_domain_grant_refs. xc_gnttab_map_domain_grant_refs permits to simply map several pages from the same domain. Signed-off-by: Samuel Thibault <samuel.thibault@eu.citrix.com> diff -r 3139383b36a0 tools/libxc/xc_linux.c --- a/tools/libxc/xc_linux.c Fri Jul 18 18:13:53 2008 +0100 +++ b/tools/libxc/xc_linux.c Mon Jul 21 11:44:43 2008 +0100 @@ -418,9 +418,10 @@ return addr; } -void *xc_gnttab_map_grant_refs(int xcg_handle, +static void *do_gnttab_map_grant_refs(int xcg_handle, uint32_t count, uint32_t *domids, + int domids_stride, uint32_t *refs, int prot) { @@ -435,7 +436,7 @@ for ( i = 0; i < count; i++ ) { - map->refs[i].domid = domids[i]; + map->refs[i].domid = domids[i * domids_stride]; map->refs[i].ref = refs[i]; } @@ -462,6 +463,24 @@ out: free(map); return addr; +} + +void *xc_gnttab_map_grant_refs(int xcg_handle, + uint32_t count, + uint32_t *domids, + uint32_t *refs, + int prot) +{ + return do_gnttab_map_grant_refs(xcg_handle, count, domids, 1, refs, prot); +} + +void *xc_gnttab_map_domain_grant_refs(int xcg_handle, + uint32_t count, + uint32_t domid, + uint32_t *refs, + int prot) +{ + return do_gnttab_map_grant_refs(xcg_handle, count, &domid, 0, refs, prot); } int xc_gnttab_munmap(int xcg_handle, diff -r 3139383b36a0 tools/libxc/xenctrl.h --- a/tools/libxc/xenctrl.h Fri Jul 18 18:13:53 2008 +0100 +++ b/tools/libxc/xenctrl.h Mon Jul 21 11:44:43 2008 +0100 @@ -865,6 +865,23 @@ uint32_t *refs, int prot); +/** + * Memory maps one or more grant references from one domain to a + * contiguous local address range. Mappings should be unmapped with + * xc_gnttab_munmap. Returns NULL on failure. + * + * @parm xcg_handle a handle on an open grant table interface + * @parm count the number of grant references to be mapped + * @parm domid the domain to map memory from + * @parm refs an array of @count grant references to be mapped + * @parm prot same flag as in mmap() + */ +void *xc_gnttab_map_domain_grant_refs(int xcg_handle, + uint32_t count, + uint32_t domid, + uint32_t *refs, + int prot); + /* * Unmaps the @count pages starting at @start_address, which were mapped by a * call to xc_gnttab_map_grant_ref or xc_gnttab_map_grant_refs. Returns zero _______________________________________________ Xen-devel mailing list Xen-devel@lists.xensource.com http://lists.xensource.com/xen-devel