[PATCH] 1/1 Better expose xenstore limits

[Tim Post <echo@echoreply.us>]

[-- Attachment #1: Type: text/plain, Size: 698 bytes --]

Hi,

The attached patch announces the limits regarding:

XENSTORE_PAYLOAD_MAX, XENSTORE_ABS_PATH_MAX and XENSTORE_REL_PATH_MAX in
the public xs.h.

These are already defined in xs_wire, however, not many people dig that
far when using xs.h.

Since xs.h already includes xs_wire, this patch duplicates those limits
as XS_PAYLOAD_MAX, XS_ABS_PATH_MAX and XS_REL_PATH_MAX. Moving these
would be a pain, but they really need to be advertised.

Such limits should be obvious in the headers that people actually use
outside of the kernel, saving them from wondering what went wrong.

Added comments direct people to xs_wire.h , in case those limits change.

Signed-off-by: Tim Post <echo@echoreply.us>



[-- Attachment #2: xenstore-expose-limits.patch --]
[-- Type: text/x-patch, Size: 486 bytes --]

diff -r dc5bd14a4675 tools/xenstore/xs.h
--- a/tools/xenstore/xs.h	Fri Apr 24 13:46:17 2009 +0100
+++ b/tools/xenstore/xs.h	Thu Apr 30 00:39:14 2009 +0800
@@ -23,6 +23,11 @@
 #include <xs_lib.h>
 
 #define XBT_NULL 0
+
+/* Please see xen/io/xs_wire.h for more explantion on these limits */
+#define XS_PAYLOAD_MAX XENSTORE_PAYLOAD_MAX
+#define XS_ABS_PATH_MAX XENSTORE_ABS_PATH_MAX
+#define XS_REL_PATH_MAX XENSTORE_REL_PATH_MAX
 
 struct xs_handle;
 typedef uint32_t xs_transaction_t;

[-- Attachment #3: Type: text/plain, Size: 138 bytes --]

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xensource.com
http://lists.xensource.com/xen-devel