libxenstore: Use PTHREAD_STACK_MIN
authorIan Campbell <ian.campbell@citrix.com>
Fri, 25 Oct 2013 07:47:35 +0000 (08:47 +0100)
committerIan Jackson <Ian.Jackson@eu.citrix.com>
Mon, 28 Oct 2013 15:33:04 +0000 (15:33 +0000)
The existing value of 16K is smaller than the arm64 minimum stack size, which
is 128K. PTHREAD_STACK_MIN appears to be standard
http://pubs.opengroup.org/onlinepubs/009695399/functions/pthread_attr_setstacksize.html

Consindered setting a lower bound but the stack requirements of the watcher
thread are pretty minimal (tens of bytes from the looks of it) and unlikely to
blow PTHREAD_STACK_MIN on any useful platform.

Signed-off-by: Ian Campbell <ian.campbell@citrix.com>
Acked-by: Ian Jackson <ian.jackson@eu.citrix.com>
tools/xenstore/xs.c

index b1e6820cf441c4133c8f20ab9460f84c3328d682..261b8413102578537fc5c9494ff818fdbb2e094c 100644 (file)
@@ -723,7 +723,7 @@ bool xs_watch(struct xs_handle *h, const char *path, const char *token)
        struct iovec iov[2];
 
 #ifdef USE_PTHREAD
-#define READ_THREAD_STACKSIZE (16 * 1024)
+#define READ_THREAD_STACKSIZE PTHREAD_STACK_MIN
 
        /* We dynamically create a reader thread on demand. */
        mutex_lock(&h->request_mutex);