[patch] MPROTECT_VDB and gcconfig.h

[Bryce McKinlay]

MPROTECT_VDB needs to be commented out for i386/linux or multithreaded read()
calls don't work right. I've tested it and this is still the case with the new
GC version. The following patch has been committed. Also, include gcconfig.h in
posix-threads.cc (I don't know if this is really neccessary, but better to be
safe...)

regards

  [ bryce ]

1999-11-02  Bryce McKinlay  <bryce@albatross.co.nz>

        * gcconfig.h: Don't define MPROTECT_VDB on Linux/i386.

Index: gcconfig.h
===================================================================
RCS file: /cvs/java/libgcj/boehm-gc/gcconfig.h,v
retrieving revision 1.2
diff -u -r1.2 gcconfig.h
--- gcconfig.h  1999/11/03 01:11:03     1.2
+++ gcconfig.h  1999/11/03 03:03:18
@@ -713,7 +713,9 @@
        /* with 2GB physical memory will usually move the user          */
        /* address space limit, and hence initial SP to 0x80000000.     */
 #       if !defined(LINUX_THREADS) || !defined(REDIRECT_MALLOC)
-#          define MPROTECT_VDB
+       /* libgcj: Linux threads don't interact well with the read() wrapper.
+          Not defining MPROTECT_VDB fixes this.  */
+/* #       define MPROTECT_VDB */
 #      else
            /* We seem to get random errors in incremental mode,        */
            /* possibly because Linux threads is itself a malloc client */


1999-11-02  Bryce McKinlay  <bryce@albatross.co.nz>

        * posix-threads.cc: Don't include boehm-config.h. Include gcconfig.h
        instead.

Index: posix-threads.cc
===================================================================
RCS file: /cvs/java/libgcj/libjava/posix-threads.cc,v
retrieving revision 1.11
diff -u -r1.11 posix-threads.cc
--- posix-threads.cc    1999/11/03 02:13:40     1.11
+++ posix-threads.cc    1999/11/03 03:05:39
@@ -18,6 +18,7 @@
 #ifdef HAVE_BOEHM_GC
 extern "C"
 {
+#include <gcconfig.h>
 #include <gc.h>
 };
 #endif /* HAVE_BOEHM_GC */