OSDN Git Service

* configure.in (all_headers, all_lib2funcs): Remove.
[pf3gnuchains/gcc-fork.git] / boehm-gc / alloc.c
index c9d145e..1664e4a 100644 (file)
@@ -1,6 +1,8 @@
 /*
  * Copyright 1988, 1989 Hans-J. Boehm, Alan J. Demers
- * Copyright (c) 1991-1994 by Xerox Corporation.  All rights reserved.
+ * Copyright (c) 1991-1996 by Xerox Corporation.  All rights reserved.
+ * Copyright (c) 1998 by Silicon Graphics.  All rights reserved.
+ * Copyright (c) 1999 by Hewlett-Packard Company. All rights reserved.
  *
  * THIS MATERIAL IS PROVIDED AS IS, WITH ABSOLUTELY NO WARRANTY EXPRESSED
  * OR IMPLIED.  ANY USE IS AT YOUR OWN RISK.
  * modified is included with the above copyright notice.
  *
  */
-/* Boehm, February 16, 1996 2:26 pm PST */
 
 
-# include "gc_priv.h"
+# include "private/gc_priv.h"
 
 # include <stdio.h>
-# ifndef MACOS
+# if !defined(MACOS) && !defined(MSWINCE)
 #   include <signal.h>
 #   include <sys/types.h>
 # endif
@@ -58,15 +59,26 @@ word GC_non_gc_bytes = 0;  /* Number of bytes not intended to be collected */
 
 word GC_gc_no = 0;
 
-int GC_incremental = 0;    /* By default, stop the world.      */
+#ifndef SMALL_CONFIG
+  int GC_incremental = 0;  /* By default, stop the world.      */
+#endif
+
+int GC_parallel = FALSE;   /* By default, parallel GC is off.  */
+
+int GC_full_freq = 19;    /* Every 20th collection is a full   */
+                          /* collection, whether we need it    */
+                          /* or not.                           */
 
-int GC_full_freq = 4;     /* Every 5th collection is a full    */
-                          /* collection.                       */
+GC_bool GC_need_full_gc = FALSE;
+                          /* Need full GC do to heap growth.   */
+
+word GC_used_heap_size_after_full = 0;
 
 char * GC_copyright[] =
 {"Copyright 1988,1989 Hans-J. Boehm and Alan J. Demers ",
 "Copyright (c) 1991-1995 by Xerox Corporation.  All rights reserved. ",
-"Copyright (c) 1996-1997 by Silicon Graphics.  All rights reserved. ",
+"Copyright (c) 1996-1998 by Silicon Graphics.  All rights reserved. ",
+"Copyright (c) 1999-2000 by Hewlett-Packard Company.  All rights reserved. ",
 "THIS MATERIAL IS PROVIDED AS IS, WITH ABSOLUTELY NO WARRANTY",
 " EXPRESSED OR IMPLIED.  ANY USE IS AT YOUR OWN RISK.",
 "See source code for details." };
@@ -80,16 +92,24 @@ extern signed_word GC_mem_found;  /* Number of reclaimed longwords  */
 
 GC_bool GC_dont_expand = 0;
 
-word GC_free_space_divisor = 4;
+word GC_free_space_divisor = 3;
 
 extern GC_bool GC_collection_in_progress();
+               /* Collection is in progress, or was abandoned. */
 
 int GC_never_stop_func GC_PROTO((void)) { return(0); }
 
-CLOCK_TYPE GC_start_time;
+CLOCK_TYPE GC_start_time;      /* Time at which we stopped world.      */
+                               /* used only in GC_timeout_stop_func.   */
 
-int GC_timeout_stop_func GC_PROTO((void))
-{
+int GC_n_attempts = 0;         /* Number of attempts at finishing      */
+                               /* collection within TIME_LIMIT         */
+
+#if defined(SMALL_CONFIG) || defined(NO_CLOCK)
+#   define GC_timeout_stop_func GC_never_stop_func
+#else
+  int GC_timeout_stop_func GC_PROTO((void))
+  {
     CLOCK_TYPE current_time;
     static unsigned count = 0;
     unsigned long time_diff;
@@ -99,15 +119,19 @@ int GC_timeout_stop_func GC_PROTO((void))
     GET_TIME(current_time);
     time_diff = MS_TIME_DIFF(current_time,GC_start_time);
     if (time_diff >= TIME_LIMIT) {
-#      ifdef PRINTSTATS
+#      ifdef CONDPRINT
+         if (GC_print_stats) {
            GC_printf0("Abandoning stopped marking after ");
-           GC_printf1("%lu msecs\n", (unsigned long)time_diff);
+           GC_printf1("%lu msecs", (unsigned long)time_diff);
+           GC_printf1("(attempt %d)\n", (unsigned long) GC_n_attempts);
+         }
 #      endif
        return(1);
     }
 #endif
     return(0);
-}
+  }
+#endif /* !SMALL_CONFIG */
 
 /* Return the minimum number of words that must be allocated between   */
 /* collections to amortize the collection cost.                                */
@@ -120,18 +144,22 @@ static word min_words_allocd()
         int dummy;
         register signed_word stack_size = (ptr_t)(&dummy) - GC_stackbottom;
 #   endif
-    register word total_root_size;  /* includes double stack size,     */
+    word total_root_size;          /* includes double stack size,      */
                                    /* since the stack is expensive     */
                                    /* to scan.                         */
+    word scan_size;            /* Estimate of memory to be scanned     */
+                               /* during normal GC.                    */
     
     if (stack_size < 0) stack_size = -stack_size;
     total_root_size = 2 * stack_size + GC_root_size;
+    scan_size = BYTES_TO_WORDS(GC_heapsize - GC_large_free_bytes
+                              + (GC_large_free_bytes >> 2)
+                                  /* use a bit more of large empty heap */
+                              + total_root_size);
     if (GC_incremental) {
-        return(BYTES_TO_WORDS(GC_heapsize + total_root_size)
-               / (2 * GC_free_space_divisor));
+        return scan_size / (2 * GC_free_space_divisor);
     } else {
-        return(BYTES_TO_WORDS(GC_heapsize + total_root_size)
-               / GC_free_space_divisor);
+        return scan_size / GC_free_space_divisor;
     }
 }
 
@@ -196,13 +224,16 @@ GC_bool GC_should_collect()
     return(GC_adj_words_allocd() >= min_words_allocd());
 }
 
+
 void GC_notify_full_gc()
 {
-    if (GC_start_call_back != (void (*)())0) {
+    if (GC_start_call_back != (void (*) GC_PROTO((void)))0) {
        (*GC_start_call_back)();
     }
 }
 
+GC_bool GC_is_full_gc = FALSE;
+
 /* 
  * Initiate a garbage collection if appropriate.
  * Choose judiciously
@@ -212,39 +243,53 @@ void GC_notify_full_gc()
 void GC_maybe_gc()
 {
     static int n_partial_gcs = 0;
+
     if (GC_should_collect()) {
         if (!GC_incremental) {
            GC_notify_full_gc();
             GC_gcollect_inner();
             n_partial_gcs = 0;
             return;
-        } else if (n_partial_gcs >= GC_full_freq) {
-#          ifdef PRINTSTATS
-             GC_printf2(
-               "***>Full mark for collection %lu after %ld allocd bytes\n",
-               (unsigned long) GC_gc_no+1,
-               (long)WORDS_TO_BYTES(GC_words_allocd));
+        } else {
+#        ifdef PARALLEL_MARK
+           GC_wait_for_reclaim();
+#        endif
+         if (GC_need_full_gc || n_partial_gcs >= GC_full_freq) {
+#          ifdef CONDPRINT
+             if (GC_print_stats) {
+               GC_printf2(
+                 "***>Full mark for collection %lu after %ld allocd bytes\n",
+                 (unsigned long) GC_gc_no+1,
+                 (long)WORDS_TO_BYTES(GC_words_allocd));
+             }
 #           endif
            GC_promote_black_lists();
            (void)GC_reclaim_all((GC_stop_func)0, TRUE);
            GC_clear_marks();
             n_partial_gcs = 0;
            GC_notify_full_gc();
-        } else {
+           GC_is_full_gc = TRUE;
+          } else {
             n_partial_gcs++;
-        }
+          }
+       }
         /* We try to mark with the world stopped.      */
         /* If we run out of time, this turns into      */
         /* incremental marking.                        */
-#ifndef NO_CLOCK
-        GET_TIME(GC_start_time);
-#endif
+#      ifndef NO_CLOCK
+          GET_TIME(GC_start_time);
+#      endif
         if (GC_stopped_mark(GC_timeout_stop_func)) {
 #           ifdef SAVE_CALL_CHAIN
                 GC_save_callers(GC_last_stack);
 #           endif
             GC_finish_collection();
-        }
+        } else {
+           if (!GC_is_full_gc) {
+               /* Count this as the first attempt */
+               GC_n_attempts++;
+           }
+       }
     }
 }
 
@@ -256,28 +301,37 @@ void GC_maybe_gc()
 GC_bool GC_try_to_collect_inner(stop_func)
 GC_stop_func stop_func;
 {
-    if (GC_collection_in_progress()) {
-#   ifdef PRINTSTATS
+    if (GC_incremental && GC_collection_in_progress()) {
+#   ifdef CONDPRINT
+      if (GC_print_stats) {
        GC_printf0(
            "GC_try_to_collect_inner: finishing collection in progress\n");
-#    endif /* PRINTSTATS */
+      }
+#   endif /* CONDPRINT */
       /* Just finish collection already in progress.   */
        while(GC_collection_in_progress()) {
            if (stop_func()) return(FALSE);
            GC_collect_a_little_inner(1);
        }
     }
-#   ifdef PRINTSTATS
+#   ifdef CONDPRINT
+      if (GC_print_stats) {
        GC_printf2(
           "Initiating full world-stop collection %lu after %ld allocd bytes\n",
           (unsigned long) GC_gc_no+1,
           (long)WORDS_TO_BYTES(GC_words_allocd));
+      }
 #   endif
     GC_promote_black_lists();
     /* Make sure all blocks have been reclaimed, so sweep routines     */
     /* don't see cleared mark bits.                                    */
     /* If we're guaranteed to finish, then this is unnecessary.                */
-       if (stop_func != GC_never_stop_func
+    /* In the find_leak case, we have to finish to guarantee that      */
+    /* previously unmarked objects are not reported as leaks.          */
+#       ifdef PARALLEL_MARK
+           GC_wait_for_reclaim();
+#       endif
+       if ((GC_find_leak || stop_func != GC_never_stop_func)
            && !GC_reclaim_all(stop_func, FALSE)) {
            /* Aborted.  So far everything is still consistent. */
            return(FALSE);
@@ -287,6 +341,7 @@ GC_stop_func stop_func;
 #   ifdef SAVE_CALL_CHAIN
         GC_save_callers(GC_last_stack);
 #   endif
+    GC_is_full_gc = TRUE;
     if (!GC_stopped_mark(stop_func)) {
       if (!GC_incremental) {
        /* We're partially done and have no way to complete or use      */
@@ -306,32 +361,51 @@ GC_stop_func stop_func;
 
 /*
  * Perform n units of garbage collection work.  A unit is intended to touch
- * roughly a GC_RATE pages.  Every once in a while, we do more than that.
+ * roughly GC_RATE pages.  Every once in a while, we do more than that.
+ * This needa to be a fairly large number with our current incremental
+ * GC strategy, since otherwise we allocate too much during GC, and the
+ * cleanup gets expensive.
  */
-# define GC_RATE 8
+# define GC_RATE 10 
+# define MAX_PRIOR_ATTEMPTS 1
+       /* Maximum number of prior attempts at world stop marking       */
+       /* A value of 1 means that we finish the second time, no matter */
+       /* how long it takes.  Doesn't count the initial root scan      */
+       /* for a full GC.                                               */
 
 int GC_deficit = 0;    /* The number of extra calls to GC_mark_some    */
                        /* that we have made.                           */
-                       /* Negative values are equivalent to 0.         */
 
 void GC_collect_a_little_inner(n)
 int n;
 {
     register int i;
     
-    if (GC_collection_in_progress()) {
+    if (GC_incremental && GC_collection_in_progress()) {
        for (i = GC_deficit; i < GC_RATE*n; i++) {
-           if (GC_mark_some()) {
+           if (GC_mark_some((ptr_t)0)) {
                /* Need to finish a collection */
 #              ifdef SAVE_CALL_CHAIN
                    GC_save_callers(GC_last_stack);
 #              endif
-               (void) GC_stopped_mark(GC_never_stop_func);
+#              ifdef PARALLEL_MARK
+                   GC_wait_for_reclaim();
+#              endif
+               if (GC_n_attempts < MAX_PRIOR_ATTEMPTS) {
+                 GET_TIME(GC_start_time);
+                 if (!GC_stopped_mark(GC_timeout_stop_func)) {
+                   GC_n_attempts++;
+                   break;
+                 }
+               } else {
+                 (void)GC_stopped_mark(GC_never_stop_func);
+               }
                GC_finish_collection();
                break;
            }
        }
        if (GC_deficit > 0) GC_deficit -= GC_RATE*n;
+       if (GC_deficit < 0) GC_deficit = 0;
     } else {
         GC_maybe_gc();
     }
@@ -354,27 +428,30 @@ int GC_collect_a_little GC_PROTO(())
 /*
  * Assumes lock is held, signals are disabled.
  * We stop the world.
- * If final is TRUE, then we finish the collection, no matter how long
- * it takes.
- * Otherwise we may fail and return FALSE if this takes too long.
+ * If stop_func() ever returns TRUE, we may fail and return FALSE.
  * Increment GC_gc_no if we succeed.
  */
 GC_bool GC_stopped_mark(stop_func)
 GC_stop_func stop_func;
 {
     register int i;
-#   ifdef PRINTSTATS
+    int dummy;
+#   ifdef PRINTTIMES
        CLOCK_TYPE start_time, current_time;
 #   endif
        
     STOP_WORLD();
-#   ifdef PRINTSTATS
+#   ifdef PRINTTIMES
        GET_TIME(start_time);
+#   endif
+#   ifdef CONDPRINT
+      if (GC_print_stats) {
        GC_printf1("--> Marking for collection %lu ",
                   (unsigned long) GC_gc_no + 1);
        GC_printf2("after %lu allocd bytes + %lu wasted bytes\n",
                   (unsigned long) WORDS_TO_BYTES(GC_words_allocd),
                   (unsigned long) WORDS_TO_BYTES(GC_words_wasted));
+      }
 #   endif
 
     /* Mark from all roots.  */
@@ -384,16 +461,18 @@ GC_stop_func stop_func;
        GC_initiate_gc();
        for(i = 0;;i++) {
            if ((*stop_func)()) {
-#                  ifdef PRINTSTATS
+#                  ifdef CONDPRINT
+                     if (GC_print_stats) {
                        GC_printf0("Abandoned stopped marking after ");
                        GC_printf1("%lu iterations\n",
                                   (unsigned long)i);
+                     }
 #                  endif
                    GC_deficit = i; /* Give the mutator a chance. */
                    START_WORLD();
                    return(FALSE);
            }
-           if (GC_mark_some()) break;
+           if (GC_mark_some((ptr_t)(&dummy))) break;
        }
        
     GC_gc_no++;
@@ -401,12 +480,22 @@ GC_stop_func stop_func;
       GC_printf2("Collection %lu reclaimed %ld bytes",
                  (unsigned long) GC_gc_no - 1,
                  (long)WORDS_TO_BYTES(GC_mem_found));
-      GC_printf1(" ---> heapsize = %lu bytes\n",
-               (unsigned long) GC_heapsize);
-      /* Printf arguments may be pushed in funny places.  Clear the    */
-      /* space.                                                                */
-      GC_printf0("");
-#   endif      
+#   else
+#     ifdef CONDPRINT
+        if (GC_print_stats) {
+         GC_printf1("Collection %lu finished", (unsigned long) GC_gc_no - 1);
+       }
+#     endif
+#   endif /* !PRINTSTATS */
+#   ifdef CONDPRINT
+      if (GC_print_stats) {
+        GC_printf1(" ---> heapsize = %lu bytes\n",
+                  (unsigned long) GC_heapsize);
+        /* Printf arguments may be pushed in funny places.  Clear the  */
+        /* space.                                                      */
+        GC_printf0("");
+      }
+#   endif  /* CONDPRINT  */
 
     /* Check all debugged objects for consistency */
         if (GC_debugging_started) {
@@ -422,6 +511,57 @@ GC_stop_func stop_func;
     return(TRUE);
 }
 
+/* Set all mark bits for the free list whose first entry is q  */
+#ifdef __STDC__
+  void GC_set_fl_marks(ptr_t q)
+#else
+  void GC_set_fl_marks(q)
+  ptr_t q;
+#endif
+{
+   ptr_t p;
+   struct hblk * h, * last_h = 0;
+   hdr *hhdr;
+   int word_no;
+
+   for (p = q; p != 0; p = obj_link(p)){
+       h = HBLKPTR(p);
+       if (h != last_h) {
+         last_h = h; 
+         hhdr = HDR(h);
+       }
+       word_no = (((word *)p) - ((word *)h));
+       set_mark_bit_from_hdr(hhdr, word_no);
+   }
+}
+
+/* Clear all mark bits for the free list whose first entry is q        */
+/* Decrement GC_mem_found by number of words on free list.     */
+#ifdef __STDC__
+  void GC_clear_fl_marks(ptr_t q)
+#else
+  void GC_clear_fl_marks(q)
+  ptr_t q;
+#endif
+{
+   ptr_t p;
+   struct hblk * h, * last_h = 0;
+   hdr *hhdr;
+   int word_no;
+
+   for (p = q; p != 0; p = obj_link(p)){
+       h = HBLKPTR(p);
+       if (h != last_h) {
+         last_h = h; 
+         hhdr = HDR(h);
+       }
+       word_no = (((word *)p) - ((word *)h));
+       clear_mark_bit_from_hdr(hhdr, word_no);
+#      ifdef GATHERSTATS
+           GC_mem_found -= hhdr -> hb_sz;
+#      endif
+   }
+}
 
 /* Finish up a collection.  Assumes lock is held, signals are disabled,        */
 /* but the world is otherwise running.                                 */
@@ -439,93 +579,90 @@ void GC_finish_collection()
 #   ifdef GATHERSTATS
         GC_mem_found = 0;
 #   endif
-#   ifdef FIND_LEAK
+#   if defined(LINUX) && defined(__ELF__) && !defined(SMALL_CONFIG)
+       if (getenv("GC_PRINT_ADDRESS_MAP") != 0) {
+         GC_print_address_map();
+       }
+#   endif
+    if (GC_find_leak) {
       /* Mark all objects on the free list.  All objects should be */
       /* marked when we're done.                                  */
        {
          register word size;           /* current object size          */
-         register ptr_t p;     /* pointer to current object    */
-         register struct hblk * h;     /* pointer to block containing *p */
-         register hdr * hhdr;
-         register int word_no;           /* "index" of *p in *q          */
          int kind;
+         ptr_t q;
 
          for (kind = 0; kind < GC_n_kinds; kind++) {
            for (size = 1; size <= MAXOBJSZ; size++) {
-             for (p= GC_obj_kinds[kind].ok_freelist[size];
-                  p != 0; p=obj_link(p)){
-               h = HBLKPTR(p);
-               hhdr = HDR(h);
-               word_no = (((word *)p) - ((word *)h));
-               set_mark_bit_from_hdr(hhdr, word_no);
-             }
+             q = GC_obj_kinds[kind].ok_freelist[size];
+             if (q != 0) GC_set_fl_marks(q);
            }
          }
        }
-      /* Check that everything is marked */
        GC_start_reclaim(TRUE);
-#   else
+         /* The above just checks; it doesn't really reclaim anything. */
+    }
 
-      GC_finalize();
-#     ifdef STUBBORN_ALLOC
-        GC_clean_changing_list();
-#     endif
+    GC_finalize();
+#   ifdef STUBBORN_ALLOC
+      GC_clean_changing_list();
+#   endif
 
-#     ifdef PRINTTIMES
-       GET_TIME(finalize_time);
-#     endif
+#   ifdef PRINTTIMES
+      GET_TIME(finalize_time);
+#   endif
 
-      /* Clear free list mark bits, in case they got accidentally marked   */
-      /* Note: HBLKPTR(p) == pointer to head of block containing *p        */
-      /* Also subtract memory remaining from GC_mem_found count.           */
-      /* Note that composite objects on free list are cleared.             */
-      /* Thus accidentally marking a free list is not a problem;  only     */
-      /* objects on the list itself will be marked, and that's fixed here. */
+    /* Clear free list mark bits, in case they got accidentally marked   */
+    /* (or GC_find_leak is set and they were intentionally marked).     */
+    /* Also subtract memory remaining from GC_mem_found count.           */
+    /* Note that composite objects on free list are cleared.             */
+    /* Thus accidentally marking a free list is not a problem;  only     */
+    /* objects on the list itself will be marked, and that's fixed here. */
       {
        register word size;             /* current object size          */
-       register ptr_t p;       /* pointer to current object    */
-       register struct hblk * h;       /* pointer to block containing *p */
-       register hdr * hhdr;
-       register int word_no;           /* "index" of *p in *q          */
+       register ptr_t q;       /* pointer to current object    */
        int kind;
 
        for (kind = 0; kind < GC_n_kinds; kind++) {
          for (size = 1; size <= MAXOBJSZ; size++) {
-           for (p= GC_obj_kinds[kind].ok_freelist[size];
-                p != 0; p=obj_link(p)){
-               h = HBLKPTR(p);
-               hhdr = HDR(h);
-               word_no = (((word *)p) - ((word *)h));
-               clear_mark_bit_from_hdr(hhdr, word_no);
-#              ifdef GATHERSTATS
-                   GC_mem_found -= size;
-#              endif
-           }
+           q = GC_obj_kinds[kind].ok_freelist[size];
+           if (q != 0) GC_clear_fl_marks(q);
          }
        }
       }
 
 
-#     ifdef PRINTSTATS
+#   ifdef PRINTSTATS
        GC_printf1("Bytes recovered before sweep - f.l. count = %ld\n",
                  (long)WORDS_TO_BYTES(GC_mem_found));
-#     endif
-
+#   endif
     /* Reconstruct free lists to contain everything not marked */
-      GC_start_reclaim(FALSE);
-    
-#   endif /* !FIND_LEAK */
+        GC_start_reclaim(FALSE);
+        if (GC_is_full_gc)  {
+           GC_used_heap_size_after_full = USED_HEAP_SIZE;
+           GC_need_full_gc = FALSE;
+       } else {
+           GC_need_full_gc =
+                BYTES_TO_WORDS(USED_HEAP_SIZE - GC_used_heap_size_after_full)
+                > min_words_allocd();
+       }
 
 #   ifdef PRINTSTATS
        GC_printf2(
-                 "Immediately reclaimed %ld bytes in heap of size %lu bytes\n",
+                 "Immediately reclaimed %ld bytes in heap of size %lu bytes",
                  (long)WORDS_TO_BYTES(GC_mem_found),
                  (unsigned long)GC_heapsize);
-       GC_printf2("%lu (atomic) + %lu (composite) collectable bytes in use\n",
-                  (unsigned long)WORDS_TO_BYTES(GC_atomic_in_use),
-                  (unsigned long)WORDS_TO_BYTES(GC_composite_in_use));
+#      ifdef USE_MUNMAP
+         GC_printf1("(%lu unmapped)", GC_unmapped_bytes);
+#      endif
+       GC_printf2(
+               "\n%lu (atomic) + %lu (composite) collectable bytes in use\n",
+               (unsigned long)WORDS_TO_BYTES(GC_atomic_in_use),
+               (unsigned long)WORDS_TO_BYTES(GC_composite_in_use));
 #   endif
 
+      GC_n_attempts = 0;
+      GC_is_full_gc = FALSE;
     /* Reset or increment counters for next cycle */
       GC_words_allocd_before_gc += GC_words_allocd;
       GC_non_gc_bytes_at_gc = GC_non_gc_bytes;
@@ -533,6 +670,9 @@ void GC_finish_collection()
       GC_words_wasted = 0;
       GC_mem_freed = 0;
       
+#   ifdef USE_MUNMAP
+      GC_unmap_old();
+#   endif
 #   ifdef PRINTTIMES
        GET_TIME(done_time);
        GC_printf2("Finalize + initiate sweep took %lu + %lu msecs\n",
@@ -576,7 +716,7 @@ void GC_gcollect GC_PROTO(())
 word GC_n_heap_sects = 0;      /* Number of sections currently in heap. */
 
 /*
- * Use the chunk of memory starting at p of syze bytes as part of the heap.
+ * Use the chunk of memory starting at p of size bytes as part of the heap.
  * Assumes p is HBLKSIZE aligned, and bytes is a multiple of HBLKSIZE.
  */
 void GC_add_to_heap(p, bytes)
@@ -584,11 +724,13 @@ struct hblk *p;
 word bytes;
 {
     word words;
+    hdr * phdr;
     
     if (GC_n_heap_sects >= MAX_HEAP_SECTS) {
        ABORT("Too many heap sections: Increase MAXHINCR or MAX_HEAP_SECTS");
     }
-    if (!GC_install_header(p)) {
+    phdr = GC_install_header(p);
+    if (0 == phdr) {
        /* This is extremely unlikely. Can't add it.  This will         */
        /* almost certainly result in a 0 return from the allocator,    */
        /* which is entirely appropriate.                               */
@@ -597,38 +739,25 @@ word bytes;
     GC_heap_sects[GC_n_heap_sects].hs_start = (ptr_t)p;
     GC_heap_sects[GC_n_heap_sects].hs_bytes = bytes;
     GC_n_heap_sects++;
-    words = BYTES_TO_WORDS(bytes - HDR_BYTES);
-    HDR(p) -> hb_sz = words;
+    words = BYTES_TO_WORDS(bytes);
+    phdr -> hb_sz = words;
+    phdr -> hb_map = (unsigned char *)1;   /* A value != GC_invalid_map        */
+    phdr -> hb_flags = 0;
     GC_freehblk(p);
     GC_heapsize += bytes;
-    if ((ptr_t)p <= GC_least_plausible_heap_addr
+    if ((ptr_t)p <= (ptr_t)GC_least_plausible_heap_addr
         || GC_least_plausible_heap_addr == 0) {
-        GC_least_plausible_heap_addr = (ptr_t)p - sizeof(word);
+        GC_least_plausible_heap_addr = (GC_PTR)((ptr_t)p - sizeof(word));
                /* Making it a little smaller than necessary prevents   */
                /* us from getting a false hit from the variable        */
                /* itself.  There's some unintentional reflection       */
                /* here.                                                */
     }
-    if ((ptr_t)p + bytes >= GC_greatest_plausible_heap_addr) {
-        GC_greatest_plausible_heap_addr = (ptr_t)p + bytes;
+    if ((ptr_t)p + bytes >= (ptr_t)GC_greatest_plausible_heap_addr) {
+        GC_greatest_plausible_heap_addr = (GC_PTR)((ptr_t)p + bytes);
     }
 }
 
-#ifdef PRESERVE_LAST
-GC_bool GC_in_last_heap_sect(p)
-ptr_t p;
-{
-    struct HeapSect * last_heap_sect = &(GC_heap_sects[GC_n_heap_sects-1]);
-    ptr_t start = last_heap_sect -> hs_start;
-    ptr_t end;
-
-    if (p < start) return FALSE;
-    end = start + last_heap_sect -> hs_bytes;
-    if (p >= end) return FALSE;
-    return TRUE;
-}
-#endif
-
 # if !defined(NO_DEBUGGING)
 void GC_print_heap_sects()
 {
@@ -652,8 +781,8 @@ void GC_print_heap_sects()
 }
 # endif
 
-ptr_t GC_least_plausible_heap_addr = (ptr_t)ONES;
-ptr_t GC_greatest_plausible_heap_addr = 0;
+GC_PTR GC_least_plausible_heap_addr = (GC_PTR)ONES;
+GC_PTR GC_greatest_plausible_heap_addr = 0;
 
 ptr_t GC_max(x,y)
 ptr_t x, y;
@@ -709,9 +838,16 @@ word n;
     }
     space = GET_MEM(bytes);
     if( space == 0 ) {
+#      ifdef CONDPRINT
+         if (GC_print_stats) {
+           GC_printf1("Failed to expand heap by %ld bytes\n",
+                      (unsigned long)bytes);
+         }
+#       endif
        return(FALSE);
     }
-#   ifdef PRINTSTATS
+#   ifdef CONDPRINT
+      if (GC_print_stats) {
        GC_printf2("Increasing heap size by %lu after %lu allocated bytes\n",
                   (unsigned long)bytes,
                   (unsigned long)WORDS_TO_BYTES(GC_words_allocd));
@@ -720,6 +856,7 @@ word n;
          GC_print_block_list(); GC_print_hblkfreelist();
          GC_printf0("\n");
 #      endif
+      }
 #   endif
     expansion_slop = 8 * WORDS_TO_BYTES(min_words_allocd());
     if (5 * HBLKSIZE * MAXHINCR > expansion_slop) {
@@ -759,6 +896,7 @@ word n;
     LOCK();
     if (!GC_is_initialized) GC_init_inner();
     result = (int)GC_expand_hp_inner(divHBLKSZ((word)bytes));
+    if (result) GC_requested_heapsize += bytes;
     UNLOCK();
     ENABLE_SIGNALS();
     return(result);
@@ -772,8 +910,8 @@ GC_bool GC_collect_or_expand(needed_blocks, ignore_off_page)
 word needed_blocks;
 GC_bool ignore_off_page;
 {
-    
-    if (!GC_incremental && !GC_dont_gc && GC_should_collect()) {
+    if (!GC_incremental && !GC_dont_gc &&
+       (GC_dont_expand && GC_words_allocd > 0 || GC_should_collect())) {
       GC_notify_full_gc();
       GC_gcollect_inner();
     } else {
@@ -802,12 +940,16 @@ GC_bool ignore_off_page;
            GC_notify_full_gc();
            GC_gcollect_inner();
        } else {
-           WARN("Out of Memory!  Returning NIL!\n", 0);
+#          if !defined(AMIGA) || !defined(GC_AMIGA_FASTALLOC)
+             WARN("Out of Memory!  Returning NIL!\n", 0);
+#          endif
            return(FALSE);
        }
-      } else if (GC_fail_count) {
-#        ifdef PRINTSTATS
-           GC_printf0("Memory available again ...\n");
+      } else {
+#        ifdef CONDPRINT
+            if (GC_fail_count && GC_print_stats) {
+             GC_printf0("Memory available again ...\n");
+           }
 #        endif
       }
     }