[project @ 2001-03-02 16:15:53 by simonmar]
[ghc-hetmet.git] / ghc / rts / GC.c
index 0600f97..5388590 100644 (file)
@@ -1,5 +1,5 @@
 /* -----------------------------------------------------------------------------
- * $Id: GC.c,v 1.84 2000/08/15 14:18:43 simonmar Exp $
+ * $Id: GC.c,v 1.98 2001/03/02 16:15:53 simonmar Exp $
  *
  * (c) The GHC Team 1998-1999
  *
@@ -37,6 +37,7 @@
 #include "Sanity.h"
 #include "GC.h"
 #include "BlockAlloc.h"
+#include "MBlock.h"
 #include "Main.h"
 #include "ProfHeap.h"
 #include "SchedAPI.h"
 #  include "ParallelDebug.h"
 # endif
 #endif
+#include "HsFFI.h"
+#include "Linker.h"
+#if defined(RTS_GTK_FRONTPANEL)
+#include "FrontPanel.h"
+#endif
 
 //@node STATIC OBJECT LIST, Static function declarations, Includes
 //@subsection STATIC OBJECT LIST
@@ -126,7 +132,6 @@ static rtsBool failed_to_evac;
  */
 bdescr *old_to_space;
 
-
 /* Data used for allocation area sizing.
  */
 lnat new_blocks;               /* blocks allocated during this GC */
@@ -147,8 +152,8 @@ static rtsBool      traverse_weak_ptr_list  ( void );
 static void         cleanup_weak_ptr_list   ( StgWeak **list );
 
 static void         scavenge_stack          ( StgPtr p, StgPtr stack_end );
-static void         scavenge_large          ( step *step );
-static void         scavenge                ( step *step );
+static void         scavenge_large          ( step * );
+static void         scavenge                ( step * );
 static void         scavenge_static         ( void );
 static void         scavenge_mutable_list   ( generation *g );
 static void         scavenge_mut_once_list  ( generation *g );
@@ -157,6 +162,9 @@ static void         scavenge_mut_once_list  ( generation *g );
 static void         gcCAFs                  ( void );
 #endif
 
+void revertCAFs   ( void );
+void scavengeCAFs ( void );
+
 //@node Garbage Collect, Weak Pointers, Static function declarations
 //@subsection Garbage Collect
 
@@ -187,7 +195,7 @@ static void         gcCAFs                  ( void );
 void GarbageCollect ( void (*get_roots)(void), rtsBool force_major_gc )
 {
   bdescr *bd;
-  step *step;
+  step *stp;
   lnat live, allocated, collected = 0, copied = 0;
   nat g, s;
 
@@ -209,7 +217,9 @@ void GarbageCollect ( void (*get_roots)(void), rtsBool force_major_gc )
   CCCS = CCS_GC;
 #endif
 
-  /* Approximate how much we allocated */
+  /* Approximate how much we allocated.  
+   * Todo: only when generating stats? 
+   */
   allocated = calcAllocated();
 
   /* Figure out which generation to collect
@@ -227,6 +237,12 @@ void GarbageCollect ( void (*get_roots)(void), rtsBool force_major_gc )
     major_gc = (N == RtsFlags.GcFlags.generations-1);
   }
 
+#ifdef RTS_GTK_FRONTPANEL
+  if (RtsFlags.GcFlags.frontpanel) {
+      updateFrontPanelBeforeGC(N);
+  }
+#endif
+
   /* check stack sanity *before* GC (ToDo: check all threads) */
 #if defined(GRAN)
   // ToDo!: check sanity  IF_DEBUG(sanity, checkTSOsSanity());
@@ -275,25 +291,25 @@ void GarbageCollect ( void (*get_roots)(void), rtsBool force_major_gc )
        * as necessary.
        */
       bd = allocBlock();
-      step = &generations[g].steps[s];
-      ASSERT(step->gen->no == g);
-      ASSERT(step->hp ? Bdescr(step->hp)->step == step : rtsTrue);
+      stp = &generations[g].steps[s];
+      ASSERT(stp->gen->no == g);
+      ASSERT(stp->hp ? Bdescr(stp->hp)->step == stp : rtsTrue);
       bd->gen  = &generations[g];
-      bd->step = step;
+      bd->step = stp;
       bd->link = NULL;
       bd->evacuated = 1;       /* it's a to-space block */
-      step->hp        = bd->start;
-      step->hpLim     = step->hp + BLOCK_SIZE_W;
-      step->hp_bd     = bd;
-      step->to_space  = bd;
-      step->to_blocks = 1;
-      step->scan      = bd->start;
-      step->scan_bd   = bd;
-      step->new_large_objects = NULL;
-      step->scavenged_large_objects = NULL;
+      stp->hp        = bd->start;
+      stp->hpLim     = stp->hp + BLOCK_SIZE_W;
+      stp->hp_bd     = bd;
+      stp->to_space  = bd;
+      stp->to_blocks = 1;
+      stp->scan      = bd->start;
+      stp->scan_bd   = bd;
+      stp->new_large_objects = NULL;
+      stp->scavenged_large_objects = NULL;
       new_blocks++;
       /* mark the large objects as not evacuated yet */
-      for (bd = step->large_objects; bd; bd = bd->link) {
+      for (bd = stp->large_objects; bd; bd = bd->link) {
        bd->evacuated = 0;
       }
     }
@@ -304,28 +320,28 @@ void GarbageCollect ( void (*get_roots)(void), rtsBool force_major_gc )
    */
   for (g = N+1; g < RtsFlags.GcFlags.generations; g++) {
     for (s = 0; s < generations[g].n_steps; s++) {
-      step = &generations[g].steps[s];
-      if (step->hp_bd == NULL) {
+      stp = &generations[g].steps[s];
+      if (stp->hp_bd == NULL) {
        bd = allocBlock();
        bd->gen = &generations[g];
-       bd->step = step;
+       bd->step = stp;
        bd->link = NULL;
        bd->evacuated = 0;      /* *not* a to-space block */
-       step->hp = bd->start;
-       step->hpLim = step->hp + BLOCK_SIZE_W;
-       step->hp_bd = bd;
-       step->blocks = bd;
-       step->n_blocks = 1;
+       stp->hp = bd->start;
+       stp->hpLim = stp->hp + BLOCK_SIZE_W;
+       stp->hp_bd = bd;
+       stp->blocks = bd;
+       stp->n_blocks = 1;
        new_blocks++;
       }
       /* Set the scan pointer for older generations: remember we
        * still have to scavenge objects that have been promoted. */
-      step->scan = step->hp;
-      step->scan_bd = step->hp_bd;
-      step->to_space = NULL;
-      step->to_blocks = 0;
-      step->new_large_objects = NULL;
-      step->scavenged_large_objects = NULL;
+      stp->scan = stp->hp;
+      stp->scan_bd = stp->hp_bd;
+      stp->to_space = NULL;
+      stp->to_blocks = 0;
+      stp->new_large_objects = NULL;
+      stp->scavenged_large_objects = NULL;
     }
   }
 
@@ -370,6 +386,8 @@ void GarbageCollect ( void (*get_roots)(void), rtsBool force_major_gc )
     }
   }
 
+  scavengeCAFs();
+
   /* follow all the roots that the application knows about.
    */
   evac_gen = 0;
@@ -451,15 +469,15 @@ void GarbageCollect ( void (*get_roots)(void), rtsBool force_major_gc )
          if (gen == 0 && st == 0 && RtsFlags.GcFlags.generations > 1) { 
            continue; 
          }
-         step = &generations[gen].steps[st];
+         stp = &generations[gen].steps[st];
          evac_gen = gen;
-         if (step->hp_bd != step->scan_bd || step->scan < step->hp) {
-           scavenge(step);
+         if (stp->hp_bd != stp->scan_bd || stp->scan < stp->hp) {
+           scavenge(stp);
            flag = rtsTrue;
            goto loop2;
          }
-         if (step->new_large_objects != NULL) {
-           scavenge_large(step);
+         if (stp->new_large_objects != NULL) {
+           scavenge_large(stp);
            flag = rtsTrue;
            goto loop2;
          }
@@ -526,35 +544,35 @@ void GarbageCollect ( void (*get_roots)(void), rtsBool force_major_gc )
 
     for (s = 0; s < generations[g].n_steps; s++) {
       bdescr *next;
-      step = &generations[g].steps[s];
+      stp = &generations[g].steps[s];
 
       if (!(g == 0 && s == 0 && RtsFlags.GcFlags.generations > 1)) {
        /* Tidy the end of the to-space chains */
-       step->hp_bd->free = step->hp;
-       step->hp_bd->link = NULL;
+       stp->hp_bd->free = stp->hp;
+       stp->hp_bd->link = NULL;
        /* stats information: how much we copied */
        if (g <= N) {
-         copied -= step->hp_bd->start + BLOCK_SIZE_W -
-           step->hp_bd->free;
+         copied -= stp->hp_bd->start + BLOCK_SIZE_W -
+           stp->hp_bd->free;
        }
       }
 
       /* for generations we collected... */
       if (g <= N) {
 
-       collected += step->n_blocks * BLOCK_SIZE_W; /* for stats */
+       collected += stp->n_blocks * BLOCK_SIZE_W; /* for stats */
 
        /* free old memory and shift to-space into from-space for all
         * the collected steps (except the allocation area).  These
         * freed blocks will probaby be quickly recycled.
         */
        if (!(g == 0 && s == 0)) {
-         freeChain(step->blocks);
-         step->blocks = step->to_space;
-         step->n_blocks = step->to_blocks;
-         step->to_space = NULL;
-         step->to_blocks = 0;
-         for (bd = step->blocks; bd != NULL; bd = bd->link) {
+         freeChain(stp->blocks);
+         stp->blocks = stp->to_space;
+         stp->n_blocks = stp->to_blocks;
+         stp->to_space = NULL;
+         stp->to_blocks = 0;
+         for (bd = stp->blocks; bd != NULL; bd = bd->link) {
            bd->evacuated = 0;  /* now from-space */
          }
        }
@@ -564,15 +582,15 @@ void GarbageCollect ( void (*get_roots)(void), rtsBool force_major_gc )
         * collection from large_objects.  Any objects left on
         * large_objects list are therefore dead, so we free them here.
         */
-       for (bd = step->large_objects; bd != NULL; bd = next) {
+       for (bd = stp->large_objects; bd != NULL; bd = next) {
          next = bd->link;
          freeGroup(bd);
          bd = next;
        }
-       for (bd = step->scavenged_large_objects; bd != NULL; bd = bd->link) {
+       for (bd = stp->scavenged_large_objects; bd != NULL; bd = bd->link) {
          bd->evacuated = 0;
        }
-       step->large_objects = step->scavenged_large_objects;
+       stp->large_objects = stp->scavenged_large_objects;
 
        /* Set the maximum blocks for this generation, interpolating
         * between the maximum size of the oldest and youngest
@@ -597,14 +615,14 @@ void GarbageCollect ( void (*get_roots)(void), rtsBool force_major_gc )
         * scavenged_large_object list (i.e. large objects that have been
         * promoted during this GC) to the large_object list for that step.
         */
-       for (bd = step->scavenged_large_objects; bd; bd = next) {
+       for (bd = stp->scavenged_large_objects; bd; bd = next) {
          next = bd->link;
          bd->evacuated = 0;
-         dbl_link_onto(bd, &step->large_objects);
+         dbl_link_onto(bd, &stp->large_objects);
        }
 
        /* add the new blocks we promoted during this GC */
-       step->n_blocks += step->to_blocks;
+       stp->n_blocks += stp->to_blocks;
       }
     }
   }
@@ -719,9 +737,11 @@ void GarbageCollect ( void (*get_roots)(void), rtsBool force_major_gc )
   }
 
  /* mark the garbage collected CAFs as dead */
-#ifdef DEBUG
+#if 0 /* doesn't work at the moment */
+#if defined(DEBUG)
   if (major_gc) { gcCAFs(); }
 #endif
+#endif
   
   /* zero the scavenged static object list */
   if (major_gc) {
@@ -758,6 +778,12 @@ void GarbageCollect ( void (*get_roots)(void), rtsBool force_major_gc )
   /* check for memory leaks if sanity checking is on */
   IF_DEBUG(sanity, memInventory());
 
+#ifdef RTS_GTK_FRONTPANEL
+  if (RtsFlags.GcFlags.frontpanel) {
+      updateFrontPanelAfterGC( N, live );
+  }
+#endif
+
   /* ok, GC over: tell the stats department what happened. */
   stat_endGC(allocated, collected, live, copied, N);
 }
@@ -814,7 +840,7 @@ traverse_weak_ptr_list(void)
     /* There might be a DEAD_WEAK on the list if finalizeWeak# was
      * called on a live weak pointer object.  Just remove it.
      */
-    if (w->header.info == &DEAD_WEAK_info) {
+    if (w->header.info == &stg_DEAD_WEAK_info) {
       next_w = ((StgDeadWeak *)w)->link;
       *last_w = next_w;
       continue;
@@ -869,7 +895,7 @@ traverse_weak_ptr_list(void)
          next = t->global_link;
          *prev = next;
          continue;
-      default:
+      default: ;
       }
 
       /* Threads which have already been determined to be alive are
@@ -994,10 +1020,6 @@ isAlive(StgClosure *p)
       /* alive! */
       return ((StgEvacuated *)p)->evacuee;
 
-    case BCO:
-      size = bco_sizeW((StgBCO*)p);
-      goto large;
-
     case ARR_WORDS:
       size = arr_words_sizeW((StgArrWords *)p);
       goto large;
@@ -1043,24 +1065,24 @@ MarkRoot(StgClosure *root)
 }
 
 //@cindex addBlock
-static void addBlock(step *step)
+static void addBlock(step *stp)
 {
   bdescr *bd = allocBlock();
-  bd->gen = step->gen;
-  bd->step = step;
+  bd->gen = stp->gen;
+  bd->step = stp;
 
-  if (step->gen->no <= N) {
+  if (stp->gen->no <= N) {
     bd->evacuated = 1;
   } else {
     bd->evacuated = 0;
   }
 
-  step->hp_bd->free = step->hp;
-  step->hp_bd->link = bd;
-  step->hp = bd->start;
-  step->hpLim = step->hp + BLOCK_SIZE_W;
-  step->hp_bd = bd;
-  step->to_blocks++;
+  stp->hp_bd->free = stp->hp;
+  stp->hp_bd->link = bd;
+  stp->hp = bd->start;
+  stp->hpLim = stp->hp + BLOCK_SIZE_W;
+  stp->hp_bd = bd;
+  stp->to_blocks++;
   new_blocks++;
 }
 
@@ -1069,14 +1091,14 @@ static void addBlock(step *step)
 static __inline__ void 
 upd_evacuee(StgClosure *p, StgClosure *dest)
 {
-  p->header.info = &EVACUATED_info;
+  p->header.info = &stg_EVACUATED_info;
   ((StgEvacuated *)p)->evacuee = dest;
 }
 
 //@cindex copy
 
 static __inline__ StgClosure *
-copy(StgClosure *src, nat size, step *step)
+copy(StgClosure *src, nat size, step *stp)
 {
   P_ to, from, dest;
 
@@ -1086,27 +1108,27 @@ copy(StgClosure *src, nat size, step *step)
    * evacuate to an older generation, adjust it here (see comment
    * by evacuate()).
    */
-  if (step->gen->no < evac_gen) {
+  if (stp->gen->no < evac_gen) {
 #ifdef NO_EAGER_PROMOTION    
     failed_to_evac = rtsTrue;
 #else
-    step = &generations[evac_gen].steps[0];
+    stp = &generations[evac_gen].steps[0];
 #endif
   }
 
   /* chain a new block onto the to-space for the destination step if
    * necessary.
    */
-  if (step->hp + size >= step->hpLim) {
-    addBlock(step);
+  if (stp->hp + size >= stp->hpLim) {
+    addBlock(stp);
   }
 
-  for(to = step->hp, from = (P_)src; size>0; --size) {
+  for(to = stp->hp, from = (P_)src; size>0; --size) {
     *to++ = *from++;
   }
 
-  dest = step->hp;
-  step->hp = to;
+  dest = stp->hp;
+  stp->hp = to;
   upd_evacuee(src,(StgClosure *)dest);
   return (StgClosure *)dest;
 }
@@ -1119,29 +1141,29 @@ copy(StgClosure *src, nat size, step *step)
 //@cindex copyPart
 
 static __inline__ StgClosure *
-copyPart(StgClosure *src, nat size_to_reserve, nat size_to_copy, step *step)
+copyPart(StgClosure *src, nat size_to_reserve, nat size_to_copy, step *stp)
 {
   P_ dest, to, from;
 
   TICK_GC_WORDS_COPIED(size_to_copy);
-  if (step->gen->no < evac_gen) {
+  if (stp->gen->no < evac_gen) {
 #ifdef NO_EAGER_PROMOTION    
     failed_to_evac = rtsTrue;
 #else
-    step = &generations[evac_gen].steps[0];
+    stp = &generations[evac_gen].steps[0];
 #endif
   }
 
-  if (step->hp + size_to_reserve >= step->hpLim) {
-    addBlock(step);
+  if (stp->hp + size_to_reserve >= stp->hpLim) {
+    addBlock(stp);
   }
 
-  for(to = step->hp, from = (P_)src; size_to_copy>0; --size_to_copy) {
+  for(to = stp->hp, from = (P_)src; size_to_copy>0; --size_to_copy) {
     *to++ = *from++;
   }
   
-  dest = step->hp;
-  step->hp += size_to_reserve;
+  dest = stp->hp;
+  stp->hp += size_to_reserve;
   upd_evacuee(src,(StgClosure *)dest);
   return (StgClosure *)dest;
 }
@@ -1166,7 +1188,7 @@ static inline void
 evacuate_large(StgPtr p, rtsBool mutable)
 {
   bdescr *bd = Bdescr(p);
-  step *step;
+  step *stp;
 
   /* should point to the beginning of the block */
   ASSERT(((W_)p & BLOCK_MASK) == 0);
@@ -1183,12 +1205,12 @@ evacuate_large(StgPtr p, rtsBool mutable)
     return;
   }
 
-  step = bd->step;
+  stp = bd->step;
   /* remove from large_object list */
   if (bd->back) {
     bd->back->link = bd->link;
   } else { /* first object in the list */
-    step->large_objects = bd->link;
+    stp->large_objects = bd->link;
   }
   if (bd->link) {
     bd->link->back = bd->back;
@@ -1196,19 +1218,19 @@ evacuate_large(StgPtr p, rtsBool mutable)
   
   /* link it on to the evacuated large object list of the destination step
    */
-  step = bd->step->to;
-  if (step->gen->no < evac_gen) {
+  stp = bd->step->to;
+  if (stp->gen->no < evac_gen) {
 #ifdef NO_EAGER_PROMOTION    
     failed_to_evac = rtsTrue;
 #else
-    step = &generations[evac_gen].steps[0];
+    stp = &generations[evac_gen].steps[0];
 #endif
   }
 
-  bd->step = step;
-  bd->gen = step->gen;
-  bd->link = step->new_large_objects;
-  step->new_large_objects = bd;
+  bd->step = stp;
+  bd->gen = stp->gen;
+  bd->link = stp->new_large_objects;
+  stp->new_large_objects = bd;
   bd->evacuated = 1;
 
   if (mutable) {
@@ -1230,21 +1252,21 @@ static StgClosure *
 mkMutCons(StgClosure *ptr, generation *gen)
 {
   StgMutVar *q;
-  step *step;
+  step *stp;
 
-  step = &gen->steps[0];
+  stp = &gen->steps[0];
 
   /* chain a new block onto the to-space for the destination step if
    * necessary.
    */
-  if (step->hp + sizeofW(StgIndOldGen) >= step->hpLim) {
-    addBlock(step);
+  if (stp->hp + sizeofW(StgIndOldGen) >= stp->hpLim) {
+    addBlock(stp);
   }
 
-  q = (StgMutVar *)step->hp;
-  step->hp += sizeofW(StgMutVar);
+  q = (StgMutVar *)stp->hp;
+  stp->hp += sizeofW(StgMutVar);
 
-  SET_HDR(q,&MUT_CONS_info,CCS_GC);
+  SET_HDR(q,&stg_MUT_CONS_info,CCS_GC);
   q->var = ptr;
   recordOldToNewPtrs((StgMutClosure *)q);
 
@@ -1282,7 +1304,7 @@ evacuate(StgClosure *q)
 {
   StgClosure *to;
   bdescr *bd = NULL;
-  step *step;
+  step *stp;
   const StgInfoTable *info;
 
 loop:
@@ -1300,10 +1322,10 @@ loop:
       }
       return q;
     }
-    step = bd->step->to;
+    stp = bd->step->to;
   }
 #ifdef DEBUG
-  else step = NULL; /* make sure copy() will crash if HEAP_ALLOCED is wrong */
+  else stp = NULL; /* make sure copy() will crash if HEAP_ALLOCED is wrong */
 #endif
 
   /* make sure the info pointer is into text space */
@@ -1321,32 +1343,32 @@ loop:
   
   switch (info -> type) {
 
-  case BCO:
-    {
-      nat size = bco_sizeW((StgBCO*)q);
-
-      if (size >= LARGE_OBJECT_THRESHOLD/sizeof(W_)) {
-       evacuate_large((P_)q, rtsFalse);
-       to = q;
-      } else {
-       /* just copy the block */
-       to = copy(q,size,step);
-      }
-      return to;
-    }
-
   case MUT_VAR:
-    ASSERT(q->header.info != &MUT_CONS_info);
+    ASSERT(q->header.info != &stg_MUT_CONS_info);
   case MVAR:
-    to = copy(q,sizeW_fromITBL(info),step);
+    to = copy(q,sizeW_fromITBL(info),stp);
     recordMutable((StgMutClosure *)to);
     return to;
 
+  case CONSTR_0_1:
+  { 
+      StgWord w = (StgWord)q->payload[0];
+      if (q->header.info == Czh_con_info &&
+         /* unsigned, so always true:  (StgChar)w >= MIN_CHARLIKE && */ 
+         (StgChar)w <= MAX_CHARLIKE) {
+         return (StgClosure *)CHARLIKE_CLOSURE((StgChar)w);
+      }
+      if (q->header.info == Izh_con_info &&
+         (StgInt)w >= MIN_INTLIKE && (StgInt)w <= MAX_INTLIKE) {
+         return (StgClosure *)INTLIKE_CLOSURE((StgInt)w);
+      }
+      /* else, fall through ... */
+  }
+
   case FUN_1_0:
   case FUN_0_1:
   case CONSTR_1_0:
-  case CONSTR_0_1:
-    return copy(q,sizeofW(StgHeader)+1,step);
+    return copy(q,sizeofW(StgHeader)+1,stp);
 
   case THUNK_1_0:              /* here because of MIN_UPD_SIZE */
   case THUNK_0_1:
@@ -1357,10 +1379,10 @@ loop:
     if (bd->gen->no == 0 && 
        bd->step->no != 0 &&
        bd->step->no == bd->gen->n_steps-1) {
-      step = bd->step;
+      stp = bd->step;
     }
 #endif
-    return copy(q,sizeofW(StgHeader)+2,step);
+    return copy(q,sizeofW(StgHeader)+2,stp);
 
   case FUN_1_1:
   case FUN_0_2:
@@ -1368,28 +1390,27 @@ loop:
   case CONSTR_1_1:
   case CONSTR_0_2:
   case CONSTR_2_0:
-    return copy(q,sizeofW(StgHeader)+2,step);
+    return copy(q,sizeofW(StgHeader)+2,stp);
 
   case FUN:
   case THUNK:
   case CONSTR:
   case IND_PERM:
   case IND_OLDGEN_PERM:
-  case CAF_UNENTERED:
-  case CAF_ENTERED:
   case WEAK:
   case FOREIGN:
   case STABLE_NAME:
-    return copy(q,sizeW_fromITBL(info),step);
+  case BCO:
+    return copy(q,sizeW_fromITBL(info),stp);
 
   case CAF_BLACKHOLE:
   case SE_CAF_BLACKHOLE:
   case SE_BLACKHOLE:
   case BLACKHOLE:
-    return copyPart(q,BLACKHOLE_sizeW(),sizeofW(StgHeader),step);
+    return copyPart(q,BLACKHOLE_sizeW(),sizeofW(StgHeader),stp);
 
   case BLACKHOLE_BQ:
-    to = copy(q,BLACKHOLE_sizeW(),step); 
+    to = copy(q,BLACKHOLE_sizeW(),stp); 
     recordMutable((StgMutClosure *)to);
     return to;
 
@@ -1448,10 +1469,6 @@ loop:
        selectee = ((StgInd *)selectee)->indirectee;
        goto selector_loop;
 
-      case CAF_ENTERED:
-       selectee = ((StgCAF *)selectee)->value;
-       goto selector_loop;
-
       case EVACUATED:
        selectee = ((StgEvacuated *)selectee)->evacuee;
        goto selector_loop;
@@ -1466,7 +1483,6 @@ loop:
       case THUNK_STATIC:
       case THUNK_SELECTOR:
        /* aargh - do recursively???? */
-      case CAF_UNENTERED:
       case CAF_BLACKHOLE:
       case SE_CAF_BLACKHOLE:
       case SE_BLACKHOLE:
@@ -1480,7 +1496,7 @@ loop:
             (int)(selectee_info->type));
       }
     }
-    return copy(q,THUNK_SELECTOR_sizeW(),step);
+    return copy(q,THUNK_SELECTOR_sizeW(),stp);
 
   case IND:
   case IND_OLDGEN:
@@ -1505,9 +1521,15 @@ loop:
     return q;
 
   case IND_STATIC:
+    /* a revertible CAF - it'll be on the CAF list, so don't do
+     * anything with it here (we'll scavenge it later).
+     */
+    if (((StgIndStatic *)q)->saved_info != NULL) {
+       return q;
+    }
     if (major_gc && IND_STATIC_LINK((StgClosure *)q) == NULL) {
-      IND_STATIC_LINK((StgClosure *)q) = static_objects;
-      static_objects = (StgClosure *)q;
+       IND_STATIC_LINK((StgClosure *)q) = static_objects;
+       static_objects = (StgClosure *)q;
     }
     return q;
 
@@ -1554,7 +1576,7 @@ loop:
        evacuate_large((P_)q, rtsFalse);
        return q;
       } else {
-       return copy(q,size,step);
+       return copy(q,size,stp);
       }
     }
 
@@ -1585,7 +1607,7 @@ loop:
        return q;
       } else {
        /* just copy the block */
-       return copy(q,size,step);
+       return copy(q,size,stp);
       }
     }
 
@@ -1599,7 +1621,7 @@ loop:
        to = q;
       } else {
        /* just copy the block */
-       to = copy(q,size,step);
+       to = copy(q,size,stp);
        if (info->type == MUT_ARR_PTRS) {
          recordMutable((StgMutClosure *)to);
        }
@@ -1630,7 +1652,7 @@ loop:
        * list it contains.  
        */
       } else {
-       StgTSO *new_tso = (StgTSO *)copy((StgClosure *)tso,tso_sizeW(tso),step);
+       StgTSO *new_tso = (StgTSO *)copy((StgClosure *)tso,tso_sizeW(tso),stp);
 
        diff = (StgPtr)new_tso - (StgPtr)tso; /* In *words* */
 
@@ -1649,9 +1671,9 @@ loop:
   case RBH: // cf. BLACKHOLE_BQ
     {
       //StgInfoTable *rip = get_closure_info(q, &size, &ptrs, &nonptrs, &vhs, str);
-      to = copy(q,BLACKHOLE_sizeW(),step); 
+      to = copy(q,BLACKHOLE_sizeW(),stp); 
       //ToDo: derive size etc from reverted IP
-      //to = copy(q,size,step);
+      //to = copy(q,size,stp);
       recordMutable((StgMutClosure *)to);
       IF_DEBUG(gc,
               belch("@@ evacuate: RBH %p (%s) to %p (%s)",
@@ -1661,7 +1683,7 @@ loop:
 
   case BLOCKED_FETCH:
     ASSERT(sizeofW(StgBlockedFetch) >= MIN_NONUPD_SIZE);
-    to = copy(q,sizeofW(StgBlockedFetch),step);
+    to = copy(q,sizeofW(StgBlockedFetch),stp);
     IF_DEBUG(gc,
             belch("@@ evacuate: %p (%s) to %p (%s)",
                   q, info_type(q), to, info_type(to)));
@@ -1669,7 +1691,7 @@ loop:
 
   case FETCH_ME:
     ASSERT(sizeofW(StgBlockedFetch) >= MIN_UPD_SIZE);
-    to = copy(q,sizeofW(StgFetchMe),step);
+    to = copy(q,sizeofW(StgFetchMe),stp);
     IF_DEBUG(gc,
             belch("@@ evacuate: %p (%s) to %p (%s)",
                   q, info_type(q), to, info_type(to)));
@@ -1677,7 +1699,7 @@ loop:
 
   case FETCH_ME_BQ:
     ASSERT(sizeofW(StgBlockedFetch) >= MIN_UPD_SIZE);
-    to = copy(q,sizeofW(StgFetchMeBlockingQueue),step);
+    to = copy(q,sizeofW(StgFetchMeBlockingQueue),stp);
     IF_DEBUG(gc,
             belch("@@ evacuate: %p (%s) to %p (%s)",
                   q, info_type(q), to, info_type(to)));
@@ -1824,15 +1846,15 @@ scavengeTSO (StgTSO *tso)
 //@cindex scavenge
 
 static void
-scavenge(step *step)
+scavenge(step *stp)
 {
   StgPtr p, q;
   const StgInfoTable *info;
   bdescr *bd;
   nat saved_evac_gen = evac_gen; /* used for temporarily changing evac_gen */
 
-  p = step->scan;
-  bd = step->scan_bd;
+  p = stp->scan;
+  bd = stp->scan_bd;
 
   failed_to_evac = rtsFalse;
 
@@ -1840,10 +1862,10 @@ scavenge(step *step)
    * evacuated objects 
    */
 
-  while (bd != step->hp_bd || p < step->hp) {
+  while (bd != stp->hp_bd || p < stp->hp) {
 
     /* If we're at the end of this block, move on to the next block */
-    if (bd != step->hp_bd && p == bd->free) {
+    if (bd != stp->hp_bd && p == bd->free) {
       bd = bd->link;
       p = bd->start;
       continue;
@@ -1862,17 +1884,6 @@ scavenge(step *step)
 
     switch (info -> type) {
 
-    case BCO:
-      {
-       StgBCO* bco = (StgBCO *)p;
-       nat i;
-       for (i = 0; i < bco->n_ptrs; i++) {
-         bcoConstCPtr(bco,i) = evacuate(bcoConstCPtr(bco,i));
-       }
-       p += bco_sizeW(bco);
-       break;
-      }
-
     case MVAR:
       /* treat MVars specially, because we don't want to evacuate the
        * mut_link field in the middle of the closure.
@@ -1945,6 +1956,7 @@ scavenge(step *step)
     case WEAK:
     case FOREIGN:
     case STABLE_NAME:
+    case BCO:
       {
        StgPtr end;
 
@@ -1957,8 +1969,8 @@ scavenge(step *step)
       }
 
     case IND_PERM:
-      if (step->gen->no != 0) {
-       SET_INFO(((StgClosure *)p), &IND_OLDGEN_PERM_info);
+      if (stp->gen->no != 0) {
+       SET_INFO(((StgClosure *)p), &stg_IND_OLDGEN_PERM_info);
       }
       /* fall through */
     case IND_OLDGEN_PERM:
@@ -1971,40 +1983,9 @@ scavenge(step *step)
       p += sizeofW(StgIndOldGen);
       break;
 
-    case CAF_UNENTERED:
-      {
-       StgCAF *caf = (StgCAF *)p;
-
-       caf->body = evacuate(caf->body);
-       if (failed_to_evac) {
-         failed_to_evac = rtsFalse;
-         recordOldToNewPtrs((StgMutClosure *)p);
-       } else {
-         caf->mut_link = NULL;
-       }
-        p += sizeofW(StgCAF);
-       break;
-      }
-
-    case CAF_ENTERED:
-      {
-       StgCAF *caf = (StgCAF *)p;
-
-       caf->body = evacuate(caf->body);
-       caf->value = evacuate(caf->value);
-       if (failed_to_evac) {
-         failed_to_evac = rtsFalse;
-         recordOldToNewPtrs((StgMutClosure *)p);
-       } else {
-         caf->mut_link = NULL;
-       }
-        p += sizeofW(StgCAF);
-       break;
-      }
-
     case MUT_VAR:
       /* ignore MUT_CONSs */
-      if (((StgMutVar *)p)->header.info != &MUT_CONS_info) {
+      if (((StgMutVar *)p)->header.info != &stg_MUT_CONS_info) {
        evac_gen = 0;
        ((StgMutVar *)p)->var = evacuate(((StgMutVar *)p)->var);
        evac_gen = saved_evac_gen;
@@ -2212,8 +2193,8 @@ scavenge(step *step)
     }
   }
 
-  step->scan_bd = bd;
-  step->scan = p;
+  stp->scan_bd = bd;
+  stp->scan = p;
 }    
 
 /* -----------------------------------------------------------------------------
@@ -2265,7 +2246,6 @@ scavenge_one(StgClosure *p)
   case FOREIGN:
   case IND_PERM:
   case IND_OLDGEN_PERM:
-  case CAF_UNENTERED:
     {
       StgPtr q, end;
       
@@ -2416,7 +2396,7 @@ scavenge_mut_once_list(generation *gen)
        * it from the mutable list if possible by promoting whatever it
        * points to.
        */
-      ASSERT(p->header.info == &MUT_CONS_info);
+      ASSERT(p->header.info == &stg_MUT_CONS_info);
       if (scavenge_one(((StgMutVar *)p)->var) == rtsTrue) {
        /* didn't manage to promote everything, so put the
         * MUT_CONS back on the list.
@@ -2426,35 +2406,6 @@ scavenge_mut_once_list(generation *gen)
       } 
       continue;
       
-    case CAF_ENTERED:
-      { 
-       StgCAF *caf = (StgCAF *)p;
-       caf->body  = evacuate(caf->body);
-       caf->value = evacuate(caf->value);
-       if (failed_to_evac) {
-         failed_to_evac = rtsFalse;
-         p->mut_link = new_list;
-         new_list = p;
-       } else {
-         p->mut_link = NULL;
-       }
-      }
-      continue;
-
-    case CAF_UNENTERED:
-      { 
-       StgCAF *caf = (StgCAF *)p;
-       caf->body  = evacuate(caf->body);
-       if (failed_to_evac) {
-         failed_to_evac = rtsFalse;
-         p->mut_link = new_list;
-         new_list = p;
-       } else {
-          p->mut_link = NULL;
-        }
-      }
-      continue;
-
     default:
       /* shouldn't have anything else on the mutables list */
       barf("scavenge_mut_once_list: strange object? %d", (int)(info->type));
@@ -2532,7 +2483,7 @@ scavenge_mutable_list(generation *gen)
        * it from the mutable list if possible by promoting whatever it
        * points to.
        */
-      ASSERT(p->header.info != &MUT_CONS_info);
+      ASSERT(p->header.info != &stg_MUT_CONS_info);
       ((StgMutVar *)p)->var = evacuate(((StgMutVar *)p)->var);
       p->mut_link = gen->mut_list;
       gen->mut_list = p;
@@ -2834,7 +2785,7 @@ scavenge_stack(StgPtr p, StgPtr stack_end)
          continue;
        } else {
          bdescr *bd = Bdescr((P_)frame->updatee);
-         step *step;
+         step *stp;
          if (bd->gen->no > N) { 
            if (bd->gen->no < evac_gen) {
              failed_to_evac = rtsTrue;
@@ -2843,22 +2794,22 @@ scavenge_stack(StgPtr p, StgPtr stack_end)
          }
 
          /* Don't promote blackholes */
-         step = bd->step;
-         if (!(step->gen->no == 0 && 
-               step->no != 0 &&
-               step->no == step->gen->n_steps-1)) {
-           step = step->to;
+         stp = bd->step;
+         if (!(stp->gen->no == 0 && 
+               stp->no != 0 &&
+               stp->no == stp->gen->n_steps-1)) {
+           stp = stp->to;
          }
 
          switch (type) {
          case BLACKHOLE:
          case CAF_BLACKHOLE:
            to = copyPart(frame->updatee, BLACKHOLE_sizeW(), 
-                         sizeofW(StgHeader), step);
+                         sizeofW(StgHeader), stp);
            frame->updatee = to;
            continue;
          case BLACKHOLE_BQ:
-           to = copy(frame->updatee, BLACKHOLE_sizeW(), step);
+           to = copy(frame->updatee, BLACKHOLE_sizeW(), stp);
            frame->updatee = to;
            recordMutable((StgMutClosure *)to);
            continue;
@@ -2943,7 +2894,7 @@ scavenge_stack(StgPtr p, StgPtr stack_end)
 //@cindex scavenge_large
 
 static void
-scavenge_large(step *step)
+scavenge_large(step *stp)
 {
   bdescr *bd;
   StgPtr p;
@@ -2951,17 +2902,17 @@ scavenge_large(step *step)
   nat saved_evac_gen = evac_gen; /* used for temporarily changing evac_gen */
 
   evac_gen = 0;                        /* most objects are mutable */
-  bd = step->new_large_objects;
+  bd = stp->new_large_objects;
 
-  for (; bd != NULL; bd = step->new_large_objects) {
+  for (; bd != NULL; bd = stp->new_large_objects) {
 
     /* take this object *off* the large objects list and put it on
      * the scavenged large objects list.  This is so that we can
      * treat new_large_objects as a stack and push new objects on
      * the front when evacuating.
      */
-    step->new_large_objects = bd->link;
-    dbl_link_onto(bd, &step->scavenged_large_objects);
+    stp->new_large_objects = bd->link;
+    dbl_link_onto(bd, &stp->scavenged_large_objects);
 
     p = bd->start;
     info  = get_itbl((StgClosure *)p);
@@ -3003,18 +2954,6 @@ scavenge_large(step *step)
        continue;
       }
 
-    case BCO:
-      {
-       StgBCO* bco = (StgBCO *)p;
-       nat i;
-       evac_gen = saved_evac_gen;
-       for (i = 0; i < bco->n_ptrs; i++) {
-         bcoConstCPtr(bco,i) = evacuate(bcoConstCPtr(bco,i));
-       }
-       evac_gen = 0;
-       continue;
-      }
-
     case TSO:
        scavengeTSO((StgTSO *)p);
        continue;
@@ -3061,7 +3000,6 @@ zero_static_object_list(StgClosure* first_static)
  * It doesn't do any harm to zero all the mutable link fields on the
  * mutable list.
  */
-//@cindex zero_mutable_list
 
 static void
 zero_mutable_list( StgMutClosure *first )
@@ -3074,43 +3012,37 @@ zero_mutable_list( StgMutClosure *first )
   }
 }
 
-//@node Reverting CAFs, Sanity code for CAF garbage collection, Scavenging
-//@subsection Reverting CAFs
-
 /* -----------------------------------------------------------------------------
    Reverting CAFs
    -------------------------------------------------------------------------- */
-//@cindex RevertCAFs
 
-void RevertCAFs(void)
+void
+revertCAFs( void )
 {
-#ifdef INTERPRETER
-   StgInt i;
-
-   /* Deal with CAFs created by compiled code. */
-   for (i = 0; i < usedECafTable; i++) {
-      SET_INFO( (StgInd*)(ecafTable[i].closure), ecafTable[i].origItbl );
-      ((StgInd*)(ecafTable[i].closure))->indirectee = 0;
-   }
-
-   /* Deal with CAFs created by the interpreter. */
-   while (ecafList != END_ECAF_LIST) {
-      StgCAF* caf  = ecafList;
-      ecafList     = caf->link;
-      ASSERT(get_itbl(caf)->type == CAF_ENTERED);
-      SET_INFO(caf,&CAF_UNENTERED_info);
-      caf->value   = (StgClosure *)0xdeadbeef;
-      caf->link    = (StgCAF *)0xdeadbeef;
-   }
-
-   /* Empty out both the table and the list. */
-   clearECafTable();
-   ecafList = END_ECAF_LIST;
-#endif
+    StgIndStatic *c;
+
+    for (c = (StgIndStatic *)caf_list; c != NULL; 
+        c = (StgIndStatic *)c->static_link) 
+    {
+       c->header.info = c->saved_info;
+       c->saved_info = NULL;
+       /* could, but not necessary: c->static_link = NULL; */
+    }
+    caf_list = NULL;
 }
 
-//@node Sanity code for CAF garbage collection, Lazy black holing, Reverting CAFs
-//@subsection Sanity code for CAF garbage collection
+void
+scavengeCAFs( void )
+{
+    StgIndStatic *c;
+
+    evac_gen = 0;
+    for (c = (StgIndStatic *)caf_list; c != NULL; 
+        c = (StgIndStatic *)c->static_link) 
+    {
+       c->indirectee = evacuate(c->indirectee);
+    }
+}
 
 /* -----------------------------------------------------------------------------
    Sanity code for CAF garbage collection.
@@ -3148,7 +3080,7 @@ gcCAFs(void)
     if (STATIC_LINK(info,p) == NULL) {
       IF_DEBUG(gccafs, fprintf(stderr, "CAF gc'd at 0x%04x\n", (int)p));
       /* black hole it */
-      SET_INFO(p,&BLACKHOLE_info);
+      SET_INFO(p,&stg_BLACKHOLE_info);
       p = STATIC_LINK2(info,p);
       *pp = p;
     }
@@ -3203,16 +3135,16 @@ threadLazyBlackHole(StgTSO *tso)
        * The blackhole made for a CAF is a CAF_BLACKHOLE, so they
        * don't interfere with this optimisation.
        */
-      if (bh->header.info == &BLACKHOLE_info) {
+      if (bh->header.info == &stg_BLACKHOLE_info) {
        return;
       }
 
-      if (bh->header.info != &BLACKHOLE_BQ_info &&
-         bh->header.info != &CAF_BLACKHOLE_info) {
+      if (bh->header.info != &stg_BLACKHOLE_BQ_info &&
+         bh->header.info != &stg_CAF_BLACKHOLE_info) {
 #if (!defined(LAZY_BLACKHOLING)) && defined(DEBUG)
         fprintf(stderr,"Unexpected lazy BHing required at 0x%04x\n",(int)bh);
 #endif
-       SET_INFO(bh,&BLACKHOLE_info);
+       SET_INFO(bh,&stg_BLACKHOLE_info);
       }
 
       update_frame = update_frame->link;
@@ -3292,16 +3224,20 @@ threadSqueezeStack(StgTSO *tso)
                    frame, prev_frame);
             })
     switch (get_itbl(frame)->type) {
-    case UPDATE_FRAME: upd_frames++;
-                       if (frame->updatee->header.info == &BLACKHOLE_info)
-                        bhs++;
-                       break;
-    case STOP_FRAME:  stop_frames++;
-                      break;
-    case CATCH_FRAME: catch_frames++;
-                      break;
-    case SEQ_FRAME: seq_frames++;
-                    break;
+    case UPDATE_FRAME:
+       upd_frames++;
+       if (frame->updatee->header.info == &stg_BLACKHOLE_info)
+           bhs++;
+       break;
+    case STOP_FRAME:
+       stop_frames++;
+       break;
+    case CATCH_FRAME:
+       catch_frames++;
+       break;
+    case SEQ_FRAME:
+       seq_frames++;
+       break;
     default:
       barf("Found non-frame during stack squeezing at %p (prev frame was %p)\n",
           frame, prev_frame);
@@ -3309,7 +3245,7 @@ threadSqueezeStack(StgTSO *tso)
     }
 #endif
     if (get_itbl(frame)->type == UPDATE_FRAME
-       && frame->updatee->header.info == &BLACKHOLE_info) {
+       && frame->updatee->header.info == &stg_BLACKHOLE_info) {
         break;
     }
   }
@@ -3375,11 +3311,11 @@ threadSqueezeStack(StgTSO *tso)
 #  if (!defined(LAZY_BLACKHOLING)) && defined(DEBUG)
 #    error Unimplemented lazy BH warning.  (KSW 1999-01)
 #  endif
-      if (GET_INFO(updatee_bypass) == BLACKHOLE_BQ_info
-         || GET_INFO(updatee_bypass) == CAF_BLACKHOLE_info
+      if (GET_INFO(updatee_bypass) == stg_BLACKHOLE_BQ_info
+         || GET_INFO(updatee_bypass) == stg_CAF_BLACKHOLE_info
          ) {
        /* Sigh.  It has one.  Don't lose those threads! */
-         if (GET_INFO(updatee_keep) == BLACKHOLE_BQ_info) {
+         if (GET_INFO(updatee_keep) == stg_BLACKHOLE_BQ_info) {
          /* Urgh.  Two queues.  Merge them. */
          P_ keep_tso = ((StgBlockingQueue *)updatee_keep)->blocking_queue;
          
@@ -3405,8 +3341,18 @@ threadSqueezeStack(StgTSO *tso)
       /* wasn't there something about update squeezing and ticky to be
        * sorted out?  oh yes: we aren't counting each enter properly
        * in this case.  See the log somewhere.  KSW 1999-04-21
+       *
+       * Check two things: that the two update frames don't point to
+       * the same object, and that the updatee_bypass isn't already an
+       * indirection.  Both of these cases only happen when we're in a
+       * block hole-style loop (and there are multiple update frames
+       * on the stack pointing to the same closure), but they can both
+       * screw us up if we don't check.
        */
-      UPD_IND_NOLOCK(updatee_bypass, updatee_keep); /* this wakes the threads up */
+      if (updatee_bypass != updatee_keep && !closure_IND(updatee_bypass)) {
+         /* this wakes the threads up */
+         UPD_IND_NOLOCK(updatee_bypass, updatee_keep);
+      }
       
       sp = (P_)frame - 1;      /* sp = stuff to slide */
       displacement += sizeofW(StgUpdateFrame);
@@ -3419,13 +3365,31 @@ threadSqueezeStack(StgTSO *tso)
        */
       if (is_update_frame) {
        StgBlockingQueue *bh = (StgBlockingQueue *)frame->updatee;
-       if (bh->header.info != &BLACKHOLE_info &&
-           bh->header.info != &BLACKHOLE_BQ_info &&
-           bh->header.info != &CAF_BLACKHOLE_info) {
+       if (bh->header.info != &stg_BLACKHOLE_info &&
+           bh->header.info != &stg_BLACKHOLE_BQ_info &&
+           bh->header.info != &stg_CAF_BLACKHOLE_info) {
 #if (!defined(LAZY_BLACKHOLING)) && defined(DEBUG)
           fprintf(stderr,"Unexpected lazy BHing required at 0x%04x\n",(int)bh);
 #endif
-         SET_INFO(bh,&BLACKHOLE_info);
+#ifdef DEBUG
+         /* zero out the slop so that the sanity checker can tell
+          * where the next closure is.
+          */
+         { 
+             StgInfoTable *info = get_itbl(bh);
+             nat np = info->layout.payload.ptrs, nw = info->layout.payload.nptrs, i;
+             /* don't zero out slop for a THUNK_SELECTOR, because it's layout
+              * info is used for a different purpose, and it's exactly the
+              * same size as a BLACKHOLE in any case.
+              */
+             if (info->type != THUNK_SELECTOR) {
+               for (i = np; i < np + nw; i++) {
+                 ((StgClosure *)bh)->payload[i] = 0;
+               }
+             }
+         }
+#endif
+         SET_INFO(bh,&stg_BLACKHOLE_info);
        }
       }
 
@@ -3538,8 +3502,7 @@ maybeLarge(StgClosure *closure)
   return (info->type == MUT_ARR_PTRS ||
          info->type == MUT_ARR_PTRS_FROZEN ||
          info->type == TSO ||
-         info->type == ARR_WORDS ||
-         info->type == BCO);
+         info->type == ARR_WORDS);
 }