Re: [Xen-devel] [PATCH] docs: retrospectively add XS_DIRECTORY_PART to the xenstore protocol...
> -Original Message- > From: Ian Jackson [mailto:ian.jack...@citrix.com] > Sent: 27 January 2020 15:49 > To: Jürgen Groß > Cc: Durrant, Paul ; xen-devel@lists.xenproject.org; > Andrew Cooper ; George Dunlap > ; Jan Beulich ; Julien Grall > ; Konrad Rzeszutek Wilk ; Stefano > Stabellini ; Wei Liu > Subject: Re: [PATCH] docs: retrospectively add XS_DIRECTORY_PART to the > xenstore protocol... > > Jürgen Groß writes ("Re: [PATCH] docs: retrospectively add > XS_DIRECTORY_PART to the xenstore protocol..."): > > On 27.01.20 16:33, Ian Jackson wrote: > > > Paul Durrant writes ("[PATCH] docs: retrospectively add > XS_DIRECTORY_PART to the xenstore protocol..."): > > >> ... specification. > > >> > > >> This was added by commit 0ca64ed8 "xenstore: add support for > > >> reading directory with many children" but not added to the > > >> specification at that point. A version of xenstored supporting the > > >> command was first released in Xen 4.9. > > > > > > Thanks for documenting this. A docs fix like this should be > > > backported if it applies, IMO. > > > > > > Acked-by: Ian Jackson > > > Backport: 4.9+ > > > > > > I will commit it to staging momentarily. > > > > > >> +DIRECTORY_PART | |* > > >> +Performs the same function as DIRECTORY, but returns a > > >> +sub-list of children starting at in the overall > > >> +child list and less than or equal to XENSTORE_PAYLOAD_MAX > > >> +octets in length. If is beyond the end of the > > >> +overall child list then the returned sub-list will be > > >> +empty. > > > > > > I wonder if it should be somehow made more explicit that `index' is > > > a count of directory entries, not bytes. Maybe this is obvious. > > > > But this is wrong. It is bytes, and the generation count returned is > > missing (see my original patch back in 2017). > > Sorry for being too quick. I have reverted my commit. > Since I got it wrong, I suggest just taking Juergen's original text (which I was unaware of before). It seems ok to me. Paul > Ian. ___ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel
Re: [Xen-devel] [PATCH] docs: retrospectively add XS_DIRECTORY_PART to the xenstore protocol...
Jürgen Groß writes ("Re: [PATCH] docs: retrospectively add XS_DIRECTORY_PART to the xenstore protocol..."): > On 27.01.20 16:33, Ian Jackson wrote: > > Paul Durrant writes ("[PATCH] docs: retrospectively add XS_DIRECTORY_PART > > to the xenstore protocol..."): > >> ... specification. > >> > >> This was added by commit 0ca64ed8 "xenstore: add support for reading > >> directory with many children" but not added to the specification at that > >> point. A version of xenstored supporting the command was first released > >> in Xen 4.9. > > > > Thanks for documenting this. A docs fix like this should be > > backported if it applies, IMO. > > > > Acked-by: Ian Jackson > > Backport: 4.9+ > > > > I will commit it to staging momentarily. > > > >> +DIRECTORY_PART| |* > >> + Performs the same function as DIRECTORY, but returns a > >> + sub-list of children starting at in the overall > >> + child list and less than or equal to XENSTORE_PAYLOAD_MAX > >> + octets in length. If is beyond the end of the > >> + overall child list then the returned sub-list will be > >> + empty. > > > > I wonder if it should be somehow made more explicit that `index' is a > > count of directory entries, not bytes. Maybe this is obvious. > > But this is wrong. It is bytes, and the generation count returned is > missing (see my original patch back in 2017). Sorry for being too quick. I have reverted my commit. Ian. ___ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel
Re: [Xen-devel] [PATCH] docs: retrospectively add XS_DIRECTORY_PART to the xenstore protocol...
On 27.01.20 16:33, Ian Jackson wrote: Paul Durrant writes ("[PATCH] docs: retrospectively add XS_DIRECTORY_PART to the xenstore protocol..."): ... specification. This was added by commit 0ca64ed8 "xenstore: add support for reading directory with many children" but not added to the specification at that point. A version of xenstored supporting the command was first released in Xen 4.9. Thanks for documenting this. A docs fix like this should be backported if it applies, IMO. Acked-by: Ian Jackson Backport: 4.9+ I will commit it to staging momentarily. +DIRECTORY_PART | |* + Performs the same function as DIRECTORY, but returns a + sub-list of children starting at in the overall + child list and less than or equal to XENSTORE_PAYLOAD_MAX + octets in length. If is beyond the end of the + overall child list then the returned sub-list will be + empty. I wonder if it should be somehow made more explicit that `index' is a count of directory entries, not bytes. Maybe this is obvious. But this is wrong. It is bytes, and the generation count returned is missing (see my original patch back in 2017). Juergen ___ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel
Re: [Xen-devel] [PATCH] docs: retrospectively add XS_DIRECTORY_PART to the xenstore protocol...
Paul Durrant writes ("[PATCH] docs: retrospectively add XS_DIRECTORY_PART to the xenstore protocol..."): > ... specification. > > This was added by commit 0ca64ed8 "xenstore: add support for reading > directory with many children" but not added to the specification at that > point. A version of xenstored supporting the command was first released > in Xen 4.9. Thanks for documenting this. A docs fix like this should be backported if it applies, IMO. Acked-by: Ian Jackson Backport: 4.9+ I will commit it to staging momentarily. > +DIRECTORY_PART | |* > + Performs the same function as DIRECTORY, but returns a > + sub-list of children starting at in the overall > + child list and less than or equal to XENSTORE_PAYLOAD_MAX > + octets in length. If is beyond the end of the > + overall child list then the returned sub-list will be > + empty. I wonder if it should be somehow made more explicit that `index' is a count of directory entries, not bytes. Maybe this is obvious. Ian. ___ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel
Re: [Xen-devel] [PATCH] docs: retrospectively add XS_DIRECTORY_PART to the xenstore protocol...
On 27.01.20 16:19, Paul Durrant wrote: ... specification. This was added by commit 0ca64ed8 "xenstore: add support for reading directory with many children" but not added to the specification at that point. A version of xenstored supporting the command was first released in Xen 4.9. Signed-off-by: Paul Durrant --- Cc: Juergen Gross Cc: Andrew Cooper Cc: George Dunlap Cc: Ian Jackson Cc: Jan Beulich Cc: Julien Grall Cc: Konrad Rzeszutek Wilk Cc: Stefano Stabellini Cc: Wei Liu --- docs/misc/xenstore.txt | 13 +++-- 1 file changed, 11 insertions(+), 2 deletions(-) diff --git a/docs/misc/xenstore.txt b/docs/misc/xenstore.txt index ae1b6a8c6e..bf42e9ec37 100644 --- a/docs/misc/xenstore.txt +++ b/docs/misc/xenstore.txt @@ -125,8 +125,9 @@ Values commonly included in payloads include: -The following are the actual type values, including the request and -reply payloads as applicable: +The following are the actual type values defined in io/xs_wire.h +(omitting the XS_ prefix), including the request and reply payloads +as applicable: -- Database read, write and permissions operations -- @@ -152,6 +153,14 @@ DIRECTORY | |* leafnames. The resulting children are each named /. +DIRECTORY_PART | |* + Performs the same function as DIRECTORY, but returns a + sub-list of children starting at in the overall + child list and less than or equal to XENSTORE_PAYLOAD_MAX + octets in length. If is beyond the end of the + overall child list then the returned sub-list will be + empty. + Hmm, not quite. I did send this some years ago: https://lists.xen.org/archives/html/xen-devel/2017-05/msg00650.html Ian wanted to suggest something better, but he never did. Juergen ___ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel
[Xen-devel] [PATCH] docs: retrospectively add XS_DIRECTORY_PART to the xenstore protocol...
... specification. This was added by commit 0ca64ed8 "xenstore: add support for reading directory with many children" but not added to the specification at that point. A version of xenstored supporting the command was first released in Xen 4.9. Signed-off-by: Paul Durrant --- Cc: Juergen Gross Cc: Andrew Cooper Cc: George Dunlap Cc: Ian Jackson Cc: Jan Beulich Cc: Julien Grall Cc: Konrad Rzeszutek Wilk Cc: Stefano Stabellini Cc: Wei Liu --- docs/misc/xenstore.txt | 13 +++-- 1 file changed, 11 insertions(+), 2 deletions(-) diff --git a/docs/misc/xenstore.txt b/docs/misc/xenstore.txt index ae1b6a8c6e..bf42e9ec37 100644 --- a/docs/misc/xenstore.txt +++ b/docs/misc/xenstore.txt @@ -125,8 +125,9 @@ Values commonly included in payloads include: -The following are the actual type values, including the request and -reply payloads as applicable: +The following are the actual type values defined in io/xs_wire.h +(omitting the XS_ prefix), including the request and reply payloads +as applicable: -- Database read, write and permissions operations -- @@ -152,6 +153,14 @@ DIRECTORY | |* leafnames. The resulting children are each named /. +DIRECTORY_PART | |* + Performs the same function as DIRECTORY, but returns a + sub-list of children starting at in the overall + child list and less than or equal to XENSTORE_PAYLOAD_MAX + octets in length. If is beyond the end of the + overall child list then the returned sub-list will be + empty. + GET_PERMS | |+ SET_PERMS ||+? is one of the following -- 2.20.1 ___ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel