lkml.org 
[lkml]   [2011]   [Dec]   [9]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
SubjectRe: [PATCH V3 1/2] xen/granttable: Support sub-page grants
From
Date
On Fri, 2011-12-09 at 11:32 +0000, annie.li@oracle.com wrote:
> -- They can't be used to map the page (so can only be used in a GNTTABOP_copy
> hypercall).
> -- It's possible to grant access with a finer granularity than whole pages.
> -- Xen guarantees that they can be revoked quickly (a normal map grant can
> only be revoked with the cooperation of the domain which has been granted
> access).
>
> Signed-off-by: Annie Li <annie.li@oracle.com>
> ---
> drivers/xen/grant-table.c | 71 +++++++++++++++++++++++++++++++++++++++++++++
> include/xen/grant_table.h | 13 ++++++++
> 2 files changed, 84 insertions(+), 0 deletions(-)
>
> diff --git a/drivers/xen/grant-table.c b/drivers/xen/grant-table.c
> index bd325fd..0ac16fa 100644
> --- a/drivers/xen/grant-table.c
> +++ b/drivers/xen/grant-table.c
> @@ -120,6 +120,16 @@ struct gnttab_ops {
> * by bit operations.
> */
> int (*query_foreign_access)(grant_ref_t);
> + /*
> + * Grant a domain to access a range of bytes within the page referred by
> + * an available grant entry. First parameter is grant entry reference
> + * number, second one is id of grantee domain, third one is frame
> + * address of subpage grant, forth one is grant type and flag
> + * information, fifth one is offset of the range of bytes, and last one
> + * is length of bytes to be accessed.
> + */
> + void (*update_subpage_entry)(grant_ref_t, domid_t, unsigned long, int,
> + unsigned, unsigned);

Please can you name the arguments here and then refer to them by name in
the comments instead of all this "First parameter", "second one" stuff.

Similarly for the existing comments sorry I didn't notice this in
previous review.

Ian.



\
 
 \ /
  Last update: 2011-12-09 18:41    [W:0.047 / U:1.524 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site