diff mbox

version: Document guest_handle

Message ID 20160223190401.GA15838@char.us.oracle.com (mailing list archive)
State New, archived
Headers show

Commit Message

Konrad Rzeszutek Wilk Feb. 23, 2016, 7:04 p.m. UTC 
On Mon, Feb 22, 2016 at 05:32:53AM -0700, Jan Beulich wrote:
> >>> On 19.02.16 at 15:32, <konrad@kernel.org> wrote:
> > --- a/xen/include/public/version.h
> > +++ b/xen/include/public/version.h
> > @@ -79,6 +79,10 @@ typedef struct xen_feature_info xen_feature_info_t;
> >  #define XENVER_pagesize 7
> >  
> >  /* arg == xen_domain_handle_t. */
> > +/*
> > + * The toolstack fills it out for the guest consumption. Usually it is
> > + * the UUID of the guest but can be anything.
> > + */
> >  #define XENVER_guest_handle 8
> 
> Generally fine, but I'm not overly happy with the wording. I'd
> prefer if it was made slightly less vague this is meant to be the
> UUID. Maybe "The toolstack fills it out for guest consumption. It
> is intended to hold the UUID of the guest"?
> 
> Jan
> 

Attached as well:

From 7da6d169835acb4aa93bfa48b1ae6b8e198f8e5f Mon Sep 17 00:00:00 2001
From: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
Date: Fri, 19 Feb 2016 09:26:02 -0500
Subject: [PATCH] version: Document guest_handle

And what it is usually used for.

Signed-off-by: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
Reviewed-by: Razvan Cojocaru <rcojocaru@bitdefender.com>

---
v2: Reword per Jan's suggestion.
---
 xen/include/public/version.h | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

Comments

Jan Beulich Feb. 24, 2016, 7:46 a.m. UTC | #1 
>>> On 23.02.16 at 20:04, <konrad.wilk@oracle.com> wrote:
> From 7da6d169835acb4aa93bfa48b1ae6b8e198f8e5f Mon Sep 17 00:00:00 2001
> From: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
> Date: Fri, 19 Feb 2016 09:26:02 -0500
> Subject: [PATCH] version: Document guest_handle
> 
> And what it is usually used for.
> 
> Signed-off-by: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
> Reviewed-by: Razvan Cojocaru <rcojocaru@bitdefender.com>
> 
> ---
> v2: Reword per Jan's suggestion.

Acked-by: Jan Beulich <jbeulich@suse.com>
diff mbox

Patch

diff --git a/xen/include/public/version.h b/xen/include/public/version.h
index adca602..5e6a7ea 100644
--- a/xen/include/public/version.h
+++ b/xen/include/public/version.h
@@ -78,7 +78,11 @@  typedef struct xen_feature_info xen_feature_info_t;
 /* arg == NULL; returns host memory page size. */
 #define XENVER_pagesize 7
 
-/* arg == xen_domain_handle_t. */
+/* arg == xen_domain_handle_t.
+ *
+ * The toolstack fills it out for guest consumption. It is intended to hold
+ * the UUID of the guest.
+ */
 #define XENVER_guest_handle 8
 
 #define XENVER_commandline 9