Re: [PATCH v3 15/15] docs: document xenstore domain xenstore paths

[Juergen Gross <jgross@suse.com>]

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