[Xen-devel] [PATCH] tools/xenstore: try to get minimum thread stack size for watch thread

[Juergen Gross <jgross@xxxxxxxx>]

When creating a pthread in xs_watch() try to get the minimal needed
size of the thread from glibc instead of using a constant. This avoids
problems when the library is used in programs with large per-thread
memory.

Use dlsym() to get the pointer to __pthread_get_minstack() in order to
avoid linkage problems and fall back to the current constant size if
not found.

Signed-off-by: Juergen Gross <jgross@xxxxxxxx>
---
Only compile tested. Jim, can you please verify this patch is solving
your original problem?
---
 tools/xenstore/Makefile |  4 ++++
 tools/xenstore/xs.c     | 19 ++++++++++++++++++-
 2 files changed, 22 insertions(+), 1 deletion(-)

diff --git a/tools/xenstore/Makefile b/tools/xenstore/Makefile
index 2b99d2bc1b..fb6c73e297 100644
--- a/tools/xenstore/Makefile
+++ b/tools/xenstore/Makefile
@@ -100,6 +100,10 @@ libxenstore.so.$(MAJOR): libxenstore.so.$(MAJOR).$(MINOR)
        ln -sf $< $@
 
 xs.opic: CFLAGS += -DUSE_PTHREAD
+ifeq ($(CONFIG_Linux),y)
+xs.opic: CFLAGS += -DUSE_DLSYM
+xs.opic: LDFLAGS += -ldl
+endif
 
 libxenstore.so.$(MAJOR).$(MINOR): xs.opic xs_lib.opic
        $(CC) $(LDFLAGS) $(PTHREAD_LDFLAGS) -Wl,$(SONAME_LDFLAG) 
-Wl,libxenstore.so.$(MAJOR) $(SHLIB_LDFLAGS) -o $@ $^ $(LDLIBS_libxentoolcore) 
$(SOCKET_LIBS) $(PTHREAD_LIBS) $(APPEND_LDFLAGS)
diff --git a/tools/xenstore/xs.c b/tools/xenstore/xs.c
index abffd9cd80..8372f5b1a4 100644
--- a/tools/xenstore/xs.c
+++ b/tools/xenstore/xs.c
@@ -47,6 +47,11 @@ struct xs_stored_msg {
 
 #include <pthread.h>
 
+#ifdef USE_DLSYM
+#define __USE_GNU
+#include <dlfcn.h>
+#endif
+
 struct xs_handle {
        /* Communications channel to xenstore daemon. */
        int fd;
@@ -810,12 +815,24 @@ bool xs_watch(struct xs_handle *h, const char *path, 
const char *token)
        if (!h->read_thr_exists) {
                sigset_t set, old_set;
                pthread_attr_t attr;
+               static size_t stack_size;
+#ifdef USE_DLSYM
+               size_t (*getsz)(void);
+#endif
 
+               if (!stack_size) {
+#ifdef USE_DLSYM
+                       getsz = dlsym(RTLD_DEFAULT, "__pthread_get_minstack");
+                       stack_size = getsz ? getsz() : READ_THREAD_STACKSIZE;
+#else
+                       stack_size = READ_THREAD_STACKSIZE;
+#endif
+               }
                if (pthread_attr_init(&attr) != 0) {
                        mutex_unlock(&h->request_mutex);
                        return false;
                }
-               if (pthread_attr_setstacksize(&attr, READ_THREAD_STACKSIZE) != 
0) {
+               if (pthread_attr_setstacksize(&attr, stack_size) != 0) {
                        pthread_attr_destroy(&attr);
                        mutex_unlock(&h->request_mutex);
                        return false;
-- 
2.13.6


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel