[ruby-dev:39751] build error if PTHREAD_STACK_MIN is a function call

遠藤です。

OpenSolaris で ruby をビルドしてみたらエラーが出ました。


In file included from thread.c:213:
thread_pthread.c:470: error: enumerator value for `RUBY_STACK_MIN' not
integer constant
thread.c: In function `rb_fd_select':
thread.c:2307: warning: implicit declaration of function `select_large_fdset'
make: *** [thread.o] Error 1


PTHREAD_STACK_MIN の定義が

#define	PTHREAD_STACK_MIN	((size_t)_sysconf(_SC_THREAD_STACK_MIN))

という関数呼び出しになっていて、これを enum の値にしようとしていることが
原因です。#if (RUBY_STACK_MIN_LIMIT < PTHREAD_STACK_MIN) などもできません。

enum でなくグローバル変数にすれば動きますが、そうしてもいいでしょうか。

OpenSolaris は supported でも best effort でもないので構わないといえば
構わないのですが、他のプラットフォームでも起きるかもしれない話ですし、
ここを直せば make && make test が通るくらいには動いたので、できれば
直したいと思います。


diff --git a/thread_pthread.c b/thread_pthread.c
index c8ee4e1..43adc39 100644
--- a/thread_pthread.c
+++ b/thread_pthread.c
@@ -264,6 +264,8 @@ static struct {
 extern void *STACK_END_ADDRESS;
 #endif

+static int RUBY_STACK_MIN_LIMIT, RUBY_STACK_MIN,
RUBY_STACK_SPACE_LIMIT, RUBY_STACK_SPACE;
+
 #undef ruby_init_stack
 void
 ruby_init_stack(volatile VALUE *addr
@@ -301,6 +303,20 @@ ruby_init_stack(volatile VALUE *addr
        }
     }
 #endif
+
+#ifdef __SYMBIAN32__
+    RUBY_STACK_MIN_LIMIT = 64 * 1024;  /* 64KB: Let's be slightly
more frugal on mobile platform */
+#else
+    RUBY_STACK_MIN_LIMIT = 512 * 1024; /* 512KB */
+#endif
+#ifdef PTHREAD_STACK_MIN
+    RUBY_STACK_MIN = (RUBY_STACK_MIN_LIMIT < PTHREAD_STACK_MIN) ?
PTHREAD_STACK_MIN * 2 : RUBY_STACK_MIN_LIMIT;
+#else
+    RUBY_STACK_MIN = RUBY_STACK_MIN_LIMIT;
+#endif
+    RUBY_STACK_SPACE_LIMIT = 1024 * 1024;
+    RUBY_STACK_SPACE = (RUBY_STACK_MIN/5 > RUBY_STACK_SPACE_LIMIT ?
+                       RUBY_STACK_SPACE_LIMIT : RUBY_STACK_MIN/5);
 }

 #define CHECK_ERR(expr) \
@@ -457,22 +473,6 @@ use_cached_thread(rb_thread_t *th)
     return result;
 }

-enum {
-#ifdef __SYMBIAN32__
-    RUBY_STACK_MIN_LIMIT = 64 * 1024,  /* 64KB: Let's be slightly
more frugal on mobile platform */
-#else
-    RUBY_STACK_MIN_LIMIT = 512 * 1024, /* 512KB */
-#endif
-    RUBY_STACK_MIN = (
-#ifdef PTHREAD_STACK_MIN
-       (RUBY_STACK_MIN_LIMIT < PTHREAD_STACK_MIN) ? PTHREAD_STACK_MIN * 2 :
-#endif
-       RUBY_STACK_MIN_LIMIT),
-    RUBY_STACK_SPACE_LIMIT = 1024 * 1024,
-    RUBY_STACK_SPACE = (RUBY_STACK_MIN/5 > RUBY_STACK_SPACE_LIMIT ?
-                       RUBY_STACK_SPACE_LIMIT : RUBY_STACK_MIN/5)
-};
-
 static int
 native_thread_create(rb_thread_t *th)
 {

-- 
Yusuke ENDOH <[e-mail]>