diff mbox

[v3,15/15] docs: document xenstore domain xenstore paths

Message ID 1452258526-4797-16-git-send-email-jgross@suse.com (mailing list archive)
State New, archived
Headers show

Commit Message

Jürgen Groß Jan. 8, 2016, 1:08 p.m. UTC
Add the xenstore domain specific xenstore paths to
docs/misc/xenstore-paths.markdown

Signed-off-by: Juergen Gross <jgross@suse.com>
---
 docs/misc/xenstore-paths.markdown | 10 ++++++++++
 1 file changed, 10 insertions(+)

Comments

Ian Campbell Jan. 15, 2016, 4:14 p.m. UTC | #1
On Fri, 2016-01-08 at 14:08 +0100, Juergen Gross wrote:
> Add the xenstore domain specific xenstore paths to
> docs/misc/xenstore-paths.markdown
> 
> Signed-off-by: Juergen Gross <jgross@suse.com>

One typo below, otherwise:

Acked-by: Ian Campbell <ian.campbell@citrix.com>

You could fold this into some earlier patch which first uses these paths,
if such a patch exists (and you can be bothered).

> ---
>  docs/misc/xenstore-paths.markdown | 10 ++++++++++
>  1 file changed, 10 insertions(+)
> 
> diff --git a/docs/misc/xenstore-paths.markdown b/docs/misc/xenstore-
> paths.markdown
> index 197bb0f..4712e3a 100644
> --- a/docs/misc/xenstore-paths.markdown
> +++ b/docs/misc/xenstore-paths.markdown
> @@ -509,6 +509,16 @@ The device model version for a domain.
>  
>  ifb device used by Remus to buffer network output from the associated
> vif.
>  
> +### xenstored specific paths
> +
> +The /tool/xenstored namespace is created by the xenstore daemon or
> domain
> +for the toolstack to obtain e.g. the domain id of a xenstore domain.
> +
> +#### /tool/xenstored/domid = INTEGER [n,INTERNAL]
> +
> +Domain Id of the xenstroe domain in case xenstore is provided via a

"xenstore"
Jürgen Groß Jan. 15, 2016, 4:22 p.m. UTC | #2
On 15/01/16 17:14, Ian Campbell wrote:
> On Fri, 2016-01-08 at 14:08 +0100, Juergen Gross wrote:
>> Add the xenstore domain specific xenstore paths to
>> docs/misc/xenstore-paths.markdown
>>
>> Signed-off-by: Juergen Gross <jgross@suse.com>
> 
> One typo below, otherwise:
> 
> Acked-by: Ian Campbell <ian.campbell@citrix.com>
> 
> You could fold this into some earlier patch which first uses these paths,
> if such a patch exists (and you can be bothered).

As the /tool/xenstored/ path isn't introduced by any of my patches in
this series I don't think this would be appropriate. And just due to
consuming it in patch 14 I don't think these two patches should be
merged.

> 
>> ---
>>  docs/misc/xenstore-paths.markdown | 10 ++++++++++
>>  1 file changed, 10 insertions(+)
>>
>> diff --git a/docs/misc/xenstore-paths.markdown b/docs/misc/xenstore-
>> paths.markdown
>> index 197bb0f..4712e3a 100644
>> --- a/docs/misc/xenstore-paths.markdown
>> +++ b/docs/misc/xenstore-paths.markdown
>> @@ -509,6 +509,16 @@ The device model version for a domain.
>>  
>>  ifb device used by Remus to buffer network output from the associated
>> vif.
>>  
>> +### xenstored specific paths
>> +
>> +The /tool/xenstored namespace is created by the xenstore daemon or
>> domain
>> +for the toolstack to obtain e.g. the domain id of a xenstore domain.
>> +
>> +#### /tool/xenstored/domid = INTEGER [n,INTERNAL]
>> +
>> +Domain Id of the xenstroe domain in case xenstore is provided via a
> 
> "xenstore"

Uuh, yes. I'll correct it.


Thanks,

Juergen
Ian Campbell Jan. 15, 2016, 4:27 p.m. UTC | #3
On Fri, 2016-01-15 at 17:22 +0100, Juergen Gross wrote:
> On 15/01/16 17:14, Ian Campbell wrote:
> > On Fri, 2016-01-08 at 14:08 +0100, Juergen Gross wrote:
> > > Add the xenstore domain specific xenstore paths to
> > > docs/misc/xenstore-paths.markdown
> > > 
> > > Signed-off-by: Juergen Gross <jgross@suse.com>
> > 
> > One typo below, otherwise:
> > 
> > Acked-by: Ian Campbell <ian.campbell@citrix.com>
> > 
> > You could fold this into some earlier patch which first uses these
> > paths,
> > if such a patch exists (and you can be bothered).
> 
> As the /tool/xenstored/ path isn't introduced by any of my patches in
> this series I don't think this would be appropriate. And just due to
> consuming it in patch 14 I don't think these two patches should be
> merged.

Ah, fair enough.
diff mbox

Patch

diff --git a/docs/misc/xenstore-paths.markdown b/docs/misc/xenstore-paths.markdown
index 197bb0f..4712e3a 100644
--- a/docs/misc/xenstore-paths.markdown
+++ b/docs/misc/xenstore-paths.markdown
@@ -509,6 +509,16 @@  The device model version for a domain.
 
 ifb device used by Remus to buffer network output from the associated vif.
 
+### xenstored specific paths
+
+The /tool/xenstored namespace is created by the xenstore daemon or domain
+for the toolstack to obtain e.g. the domain id of a xenstore domain.
+
+#### /tool/xenstored/domid = INTEGER [n,INTERNAL]
+
+Domain Id of the xenstroe domain in case xenstore is provided via a
+domain instead of a daemon in dom0.
+
 [BLKIF]: http://xenbits.xen.org/docs/unstable/hypercall/include,public,io,blkif.h.html
 [FBIF]: http://xenbits.xen.org/docs/unstable/hypercall/include,public,io,fbif.h.html
 [HVMPARAMS]: http://xenbits.xen.org/docs/unstable/hypercall/include,public,hvm,params.h.html