diff --git a/boehm-gc/ChangeLog b/boehm-gc/ChangeLog index ff94d2b7db116e1efb45c53d861b883b60b44337..58c5f39071ec7615706c928874efe3100357dc5b 100644 --- a/boehm-gc/ChangeLog +++ b/boehm-gc/ChangeLog @@ -1,3 +1,15 @@ +2001-10-16 Loren J. Rittle <ljrittle@acm.org> + + * include/gc_pthread_redirects.h: Generalize test to use GC_PTHREADS. + * linux_threads.c (WRAP_FUNC(pthread_join)): Conditionalized on + GC_FREEBSD_THREADS, handle strange interaction between system + pthread implementation and boehm-gc signal-handler architecture. + * tests/test.c (main): Conditionalized on GC_FREEBSD_THREADS, + set stack. + * include/private/gcconfig.h (configuration keyed off FREEBSD): + Define SIG_SUSPEND, SIG_THR_RESTART. Do not define + MPROTECT_VDB when GC_FREEBSD_THREADS is defined. + 2001-10-16 Loren J. Rittle <ljrittle@acm.org> * Makefile.am (test.o): Find tests/test.c in $(srcdir). diff --git a/boehm-gc/include/gc_pthread_redirects.h b/boehm-gc/include/gc_pthread_redirects.h index ac254a86726ea067736aaec03410c8bf165c710c..47284fbc97a65673c5818e652b6fab81bad65f23 100644 --- a/boehm-gc/include/gc_pthread_redirects.h +++ b/boehm-gc/include/gc_pthread_redirects.h @@ -44,9 +44,7 @@ #endif /* SOLARIS_THREADS || SOLARIS_PTHREADS */ -#if !defined(GC_USE_LD_WRAP) && \ - (defined(GC_IRIX_THREADS) || defined(GC_LINUX_THREADS) \ - || defined(GC_HPUX_THREADS) || defined(GC_OSF1_THREADS)) +#if !defined(GC_USE_LD_WRAP) && defined(GC_PTHREADS) && !defined(GC_SOLARIS_PTHREADS) /* We treat these similarly. */ # include <pthread.h> # include <signal.h> diff --git a/boehm-gc/include/private/gcconfig.h b/boehm-gc/include/private/gcconfig.h index e92f2a4eb0cb3f0f8cfac323e6b83afc24d612df..4334dbd29e466a03b91a9f350c9feba9f2c84047 100644 --- a/boehm-gc/include/private/gcconfig.h +++ b/boehm-gc/include/private/gcconfig.h @@ -1017,7 +1017,11 @@ # endif # ifdef FREEBSD # define OS_TYPE "FREEBSD" -# define MPROTECT_VDB +# ifndef GC_FREEBSD_THREADS +# define MPROTECT_VDB +# endif +# define SIG_SUSPEND SIGUSR1 +# define SIG_THR_RESTART SIGUSR2 # define FREEBSD_STACKBOTTOM # ifdef __ELF__ # define DYNAMIC_LOADING diff --git a/boehm-gc/linux_threads.c b/boehm-gc/linux_threads.c index f52eab8c1247ae37313771242a777126a710a14c..ccd7dc65a743fa8919672cf3e9d54b9a50fd728e 100644 --- a/boehm-gc/linux_threads.c +++ b/boehm-gc/linux_threads.c @@ -1272,6 +1272,17 @@ int WRAP_FUNC(pthread_join)(pthread_t thread, void **retval) /* cant have been recycled by pthreads. */ UNLOCK(); result = REAL_FUNC(pthread_join)(thread, retval); +# if defined (GC_FREEBSD_THREADS) + /* On FreeBSD, the wrapped pthread_join() sometimes returns (what + appears to be) a spurious EINTR which caused the test and real code + to gratuitously fail. Having looked at system pthread library source + code, I see how this return code may be generated. In one path of + code, pthread_join() just returns the errno setting of the thread + being joined. This does not match the POSIX specification or the + local man pages thus I have taken the liberty to catch this one + spurious return value properly conditionalized on GC_FREEBSD_THREADS. */ + if (result == EINTR) result = 0; +# endif if (result == 0) { LOCK(); /* Here the pthread thread id may have been recycled. */ diff --git a/boehm-gc/tests/test.c b/boehm-gc/tests/test.c index 35cb1357883a6dc8f5959b9e269571dce8662ad8..e8cc9763f44231db1d5238e922636d67958d5a7e 100644 --- a/boehm-gc/tests/test.c +++ b/boehm-gc/tests/test.c @@ -1624,7 +1624,7 @@ main() } # endif /* GC_HPUX_THREADS */ pthread_attr_init(&attr); -# if defined(GC_IRIX_THREADS) +# if defined(GC_IRIX_THREADS) || defined(GC_FREEBSD_THREADS) pthread_attr_setstacksize(&attr, 1000000); # endif n_tests = 0;