0.pre7.56:
[sbcl.git] / src / runtime / gc.c
index b751687..e8061a3 100644 (file)
@@ -68,31 +68,33 @@ static int scav_lose(lispobj *where, lispobj object);
 #define CEILING(x,y) (((x) + ((y) - 1)) & (~((y) - 1)))
 
 \f
-/* Predicates */
+/* predicates */
 
 #if defined(DEBUG_SPACE_PREDICATES)
 
-boolean from_space_p(lispobj object)
+boolean
+from_space_p(lispobj object)
 {
        lispobj *ptr;
 
        /* this can be called for untagged pointers as well as for 
           descriptors, so this assertion's not applicable
-          gc_assert(Pointerp(object));
+          gc_assert(is_lisp_pointer(object));
        */
-       ptr = (lispobj *) PTR(object);
+       ptr = (lispobj *) native_pointer(object);
 
        return ((from_space <= ptr) &&
                (ptr < from_space_free_pointer));
 }          
 
-boolean new_space_p(lispobj object)
+boolean
+new_space_p(lispobj object)
 {
        lispobj *ptr;
 
-       gc_assert(Pointerp(object));
+       gc_assert(is_lisp_pointer(object));
 
-       ptr = (lispobj *) PTR(object);
+       ptr = (lispobj *) native_pointer(object);
                
        return ((new_space <= ptr) &&
                (ptr < new_space_free_pointer));
@@ -111,7 +113,7 @@ boolean new_space_p(lispobj object)
 #endif
 
 \f
-/* Copying Objects */
+/* copying objects */
 
 static lispobj
 copy_object(lispobj object, int nwords)
@@ -120,19 +122,19 @@ copy_object(lispobj object, int nwords)
        lispobj *new;
        lispobj *source, *dest;
 
-       gc_assert(Pointerp(object));
+       gc_assert(is_lisp_pointer(object));
        gc_assert(from_space_p(object));
        gc_assert((nwords & 0x01) == 0);
 
        /* get tag of object */
-       tag = LowtagOf(object);
+       tag = lowtagof(object);
 
        /* allocate space */
        new = new_space_free_pointer;
        new_space_free_pointer += nwords;
 
        dest = new;
-       source = (lispobj *) PTR(object);
+       source = (lispobj *) native_pointer(object);
 
 #ifdef DEBUG_COPY_VERBOSE
        fprintf(stderr,"Copying %d words from %p to %p\n", nwords,source,new);
@@ -151,10 +153,11 @@ copy_object(lispobj object, int nwords)
 }
 
 \f
-/* Collect Garbage */
+/* collecting garbage */
 
 #ifdef PRINTNOISE
-static double tv_diff(struct timeval *x, struct timeval *y)
+static double
+tv_diff(struct timeval *x, struct timeval *y)
 {
     return (((double) x->tv_sec + (double) x->tv_usec * 1.0e-6) -
            ((double) y->tv_sec + (double) y->tv_usec * 1.0e-6));
@@ -168,7 +171,8 @@ static double tv_diff(struct timeval *x, struct timeval *y)
 #else
 #define U32 unsigned long
 #endif
-static void zero_stack(void)
+static void
+zero_stack(void)
 {
     U32 *ptr = (U32 *)current_control_stack_pointer;
  search:
@@ -188,10 +192,11 @@ static void zero_stack(void)
 #undef U32
 
 
-/* this is not generational.  It's called with a last_gen arg, which we shun.
- */
-
-void collect_garbage(unsigned ignore)
+/* Note: The generic GC interface we're implementing passes us a
+ * last_generation argument. That's meaningless for us, since we're
+ * not a generational GC. So we ignore it. */
+void
+collect_garbage(unsigned ignore)
 {
 #ifdef PRINTNOISE
 struct timeval start_tv, stop_tv;
@@ -225,14 +230,12 @@ struct timeval start_tv, stop_tv;
        /* Set up from space and new space pointers. */
 
        from_space = current_dynamic_space;
-#ifndef ibmrt
        from_space_free_pointer = dynamic_space_free_pointer;
-#else
-       from_space_free_pointer = (lispobj *)SymbolValue(ALLOCATION_POINTER);
-#endif
 
+#ifdef PRINTNOISE
        fprintf(stderr,"from_space = %lx\n",
                (unsigned long) current_dynamic_space);
+#endif
        if (current_dynamic_space == (lispobj *) DYNAMIC_0_SPACE_START)
            new_space = (lispobj *)DYNAMIC_1_SPACE_START;
        else if (current_dynamic_space == (lispobj *) DYNAMIC_1_SPACE_START)
@@ -272,14 +275,9 @@ struct timeval start_tv, stop_tv;
        scavenge(((lispobj *)CONTROL_STACK_START), control_stack_size);
                 
 
-#ifdef ibmrt
-       binding_stack_size =
-           (lispobj *)SymbolValue(BINDING_STACK_POINTER) - binding_stack;
-#else
        binding_stack_size = 
          current_binding_stack_pointer - 
            (lispobj *)BINDING_STACK_START;
-#endif
 #ifdef PRINTNOISE
        printf("Scavenging the binding stack %x - %x (%d words) ...\n",
               BINDING_STACK_START,current_binding_stack_pointer,
@@ -324,11 +322,7 @@ struct timeval start_tv, stop_tv;
                (os_vm_size_t) DYNAMIC_SPACE_SIZE);
 
        current_dynamic_space = new_space;
-#ifndef ibmrt
        dynamic_space_free_pointer = new_space_free_pointer;
-#else
-       SetSymbolValue(ALLOCATION_POINTER, (lispobj)new_space_free_pointer);
-#endif
 
 #ifdef PRINTNOISE
        size_discarded = (from_space_free_pointer - from_space) * sizeof(lispobj);
@@ -377,9 +371,7 @@ struct timeval start_tv, stop_tv;
 }
 
 \f
-/* Scavenging */
-
-#define DIRECT_SCAV 0
+/* scavenging */
 
 static void
 scavenge(lispobj *start, u32 nwords)
@@ -396,18 +388,16 @@ scavenge(lispobj *start, u32 nwords)
                       (unsigned long) start, (unsigned long) object, type);
 #endif
 
-#if DIRECT_SCAV
-               words_scavenged = (scavtab[type])(start, object);
-#else
-                if (Pointerp(object)) {
+                if (is_lisp_pointer(object)) {
                    /* It be a pointer. */
                    if (from_space_p(object)) {
                         /* It currently points to old space.  Check for a */
                         /* forwarding pointer. */
                         lispobj first_word;
 
-                        first_word = *((lispobj *)PTR(object));
-                        if (Pointerp(first_word) && new_space_p(first_word)) {
+                        first_word = *((lispobj *)native_pointer(object));
+                        if (is_lisp_pointer(first_word) &&
+                           new_space_p(first_word)) {
                             /* Yep, there be a forwarding pointer. */
                             *start = first_word;
                             words_scavenged = 1;
@@ -449,14 +439,15 @@ scavenge(lispobj *start, u32 nwords)
                     words_scavenged = (scavtab[type])(start, object);
 
                 }
-#endif
+
                start += words_scavenged;
                nwords -= words_scavenged;
        }
        gc_assert(nwords == 0);
 }
 
-static void scavenge_newspace(void)
+static void
+scavenge_newspace(void)
 {
     lispobj *here, *next;
 
@@ -470,13 +461,13 @@ static void scavenge_newspace(void)
     }
     /* printf("done with newspace\n"); */
 }
-
 \f
-/* Scavenging Interrupt Contexts */
+/* scavenging interrupt contexts */
 
 static int boxed_registers[] = BOXED_REGISTERS;
 
-static void scavenge_interrupt_context(os_context_t *context)
+static void
+scavenge_interrupt_context(os_context_t *context)
 {
        int i;
 #ifdef reg_LIP
@@ -504,7 +495,7 @@ static void scavenge_interrupt_context(os_context_t *context)
                index = boxed_registers[i];
                reg = *os_context_register_addr(context, index);
                /* would be using PTR if not for integer length issues */
-               if ((reg & ~((1L<<lowtag_Bits)-1)) <= lip) {
+               if ((reg & ~((1L<<N_LOWTAG_BITS)-1)) <= lip) {
                        offset = lip - reg;
                        if (offset < lip_offset) {
                                lip_offset = offset;
@@ -557,22 +548,22 @@ static void scavenge_interrupt_context(os_context_t *context)
 
 void scavenge_interrupt_contexts(void)
 {
-       int i, index;
-       os_context_t *context;
+    int i, index;
+    os_context_t *context;
 
-       index = fixnum_value(SymbolValue(FREE_INTERRUPT_CONTEXT_INDEX));
-       printf("Number of active contexts: %d\n", index);
+    index = fixnum_value(SymbolValue(FREE_INTERRUPT_CONTEXT_INDEX));
 
-       for (i = 0; i < index; i++) {
-               context = lisp_interrupt_contexts[i];
-               scavenge_interrupt_context(context); 
-       }
+    for (i = 0; i < index; i++) {
+       context = lisp_interrupt_contexts[i];
+       scavenge_interrupt_context(context); 
+    }
 }
 
 \f
-/* Debugging Code */
+/* debugging code */
 
-void print_garbage(lispobj *from_space, lispobj *from_space_free_pointer)
+void
+print_garbage(lispobj *from_space, lispobj *from_space_free_pointer)
 {
        lispobj *start;
        int total_words_not_copied;
@@ -587,27 +578,27 @@ void print_garbage(lispobj *from_space, lispobj *from_space_free_pointer)
                lispobj header;
 
                object = *start;
-               forwardp = Pointerp(object) && new_space_p(object);
+               forwardp = is_lisp_pointer(object) && new_space_p(object);
 
                if (forwardp) {
                        int tag;
                        lispobj *pointer;
 
-                       tag = LowtagOf(object);
+                       tag = lowtagof(object);
 
                        switch (tag) {
-                       case type_ListPointer:
+                       case LIST_POINTER_LOWTAG:
                                nwords = 2;
                                break;
-                       case type_InstancePointer:
+                       case INSTANCE_POINTER_LOWTAG:
                                printf("Don't know about instances yet!\n");
                                nwords = 1;
                                break;
-                       case type_FunctionPointer:
+                       case FUN_POINTER_LOWTAG:
                                nwords = 1;
                                break;
-                       case type_OtherPointer:
-                               pointer = (lispobj *) PTR(object);
+                       case OTHER_POINTER_LOWTAG:
+                               pointer = (lispobj *) native_pointer(object);
                                header = *pointer;
                                type = TypeOf(header);
                                nwords = (sizetab[type])(pointer);
@@ -627,69 +618,26 @@ void print_garbage(lispobj *from_space, lispobj *from_space_free_pointer)
 }
 
 \f
-/* Code and Code-Related Objects */
+/* code and code-related objects */
 
-#define RAW_ADDR_OFFSET (6*sizeof(lispobj) - type_FunctionPointer)
+/* FIXME: Shouldn't this be defined in sbcl.h? */
+#define FUN_RAW_ADDR_OFFSET (6*sizeof(lispobj) - FUN_POINTER_LOWTAG)
 
-static lispobj trans_function_header(lispobj object);
+static lispobj trans_fun_header(lispobj object);
 static lispobj trans_boxed(lispobj object);
 
-#if DIRECT_SCAV
 static int
-scav_function_pointer(lispobj *where, lispobj object)
-{
-       gc_assert(Pointerp(object));
-
-       if (from_space_p(object)) {
-               lispobj first, *first_pointer;
-
-               /* object is a pointer into from space.  check to see */
-               /* if it has been forwarded */
-               first_pointer = (lispobj *) PTR(object);
-               first = *first_pointer;
-               
-               if (!(Pointerp(first) && new_space_p(first))) {
-                       int type;
-                       lispobj copy;
-
-                       /* must transport object -- object may point */
-                       /* to either a function header, a closure */
-                       /* function header, or to a closure header. */
-                       
-                       type = TypeOf(first);
-                       switch (type) {
-                         case type_FunctionHeader:
-                         case type_ClosureFunctionHeader:
-                           copy = trans_function_header(object);
-                           break;
-                         default:
-                           copy = trans_boxed(object);
-                           break;
-                       }
-
-                       first = *first_pointer = copy;
-               }
-
-               gc_assert(Pointerp(first));
-               gc_assert(!from_space_p(first));
-
-               *where = first;
-       }
-       return 1;
-}
-#else
-static int
-scav_function_pointer(lispobj *where, lispobj object)
+scav_fun_pointer(lispobj *where, lispobj object)
 {
   lispobj  *first_pointer;
   lispobj copy;
   lispobj first;
   int type;
 
-  gc_assert(Pointerp(object));
+  gc_assert(is_lisp_pointer(object));
       
   /* object is a pointer into from space. Not a FP */
-  first_pointer = (lispobj *) PTR(object);
+  first_pointer = (lispobj *) native_pointer(object);
   first = *first_pointer;
                
   /* must transport object -- object may point */
@@ -698,9 +646,9 @@ scav_function_pointer(lispobj *where, lispobj object)
   
   type = TypeOf(first);
   switch (type) {
-  case type_FunctionHeader:
-  case type_ClosureFunctionHeader:
-    copy = trans_function_header(object);
+  case type_SimpleFunHeader:
+  case type_ClosureFunHeader:
+    copy = trans_fun_header(object);
     break;
   default:
     copy = trans_boxed(object);
@@ -709,13 +657,12 @@ scav_function_pointer(lispobj *where, lispobj object)
   
   first = *first_pointer = copy;
 
-  gc_assert(Pointerp(first));
+  gc_assert(is_lisp_pointer(first));
   gc_assert(!from_space_p(first));
 
   *where = first;
   return 1;
 }
-#endif
 
 static struct code *
 trans_code(struct code *code)
@@ -733,17 +680,17 @@ trans_code(struct code *code)
 
        /* if object has already been transported, just return pointer */
        first = code->header;
-       if (Pointerp(first) && new_space_p(first)) {
+       if (is_lisp_pointer(first) && new_space_p(first)) {
 #ifdef DEBUG_CODE_GC
            printf("Was already transported\n");
 #endif
-           return (struct code *) PTR(first);
+           return (struct code *) native_pointer(first);
        }
        
        gc_assert(TypeOf(first) == type_CodeHeader);
 
        /* prepare to transport the code vector */
-       l_code = (lispobj) LOW_WORD(code) | type_OtherPointer;
+       l_code = (lispobj) LOW_WORD(code) | OTHER_POINTER_LOWTAG;
 
        ncode_words = fixnum_value(code->code_size);
        nheader_words = HeaderValue(code->header);
@@ -751,7 +698,7 @@ trans_code(struct code *code)
        nwords = CEILING(nwords, 2);
 
        l_new_code = copy_object(l_code, nwords);
-       new_code = (struct code *) PTR(l_new_code);
+       new_code = (struct code *) native_pointer(l_new_code);
 
        displacement = l_new_code - l_code;
 
@@ -771,16 +718,16 @@ trans_code(struct code *code)
        prev_pointer = &new_code->entry_points;
 
        while (fheaderl != NIL) {
-               struct function *fheaderp, *nfheaderp;
+               struct simple_fun *fheaderp, *nfheaderp;
                lispobj nfheaderl;
                
-               fheaderp = (struct function *) PTR(fheaderl);
-               gc_assert(TypeOf(fheaderp->header) == type_FunctionHeader);
+               fheaderp = (struct simple_fun *) native_pointer(fheaderl);
+               gc_assert(TypeOf(fheaderp->header) == type_SimpleFunHeader);
 
                /* calcuate the new function pointer and the new */
                /* function header */
                nfheaderl = fheaderl + displacement;
-               nfheaderp = (struct function *) PTR(nfheaderl);
+               nfheaderp = (struct simple_fun *) native_pointer(nfheaderl);
 
                /* set forwarding pointer */
 #ifdef DEBUG_CODE_GC
@@ -811,7 +758,7 @@ scav_code_header(lispobj *where, lispobj object)
        struct code *code;
        int nheader_words, ncode_words, nwords;
        lispobj fheaderl;
-       struct function *fheaderp;
+       struct simple_fun *fheaderp;
 
        code = (struct code *) where;
        ncode_words = fixnum_value(code->code_size);
@@ -834,12 +781,12 @@ scav_code_header(lispobj *where, lispobj object)
        /* code data block */
        fheaderl = code->entry_points;
        while (fheaderl != NIL) {
-               fheaderp = (struct function *) PTR(fheaderl);
-               gc_assert(TypeOf(fheaderp->header) == type_FunctionHeader);
+               fheaderp = (struct simple_fun *) native_pointer(fheaderl);
+               gc_assert(TypeOf(fheaderp->header) == type_SimpleFunHeader);
                
 #if defined(DEBUG_CODE_GC)
                printf("Scavenging boxed section of entry point located at 0x%08x.\n",
-                      (unsigned long) PTR(fheaderl));
+                      (unsigned long) native_pointer(fheaderl));
 #endif
                scavenge(&fheaderp->name, 1);
                scavenge(&fheaderp->arglist, 1);
@@ -856,8 +803,8 @@ trans_code_header(lispobj object)
 {
        struct code *ncode;
 
-       ncode = trans_code((struct code *) PTR(object));
-       return (lispobj) LOW_WORD(ncode) | type_OtherPointer;
+       ncode = trans_code((struct code *) native_pointer(object));
+       return (lispobj) LOW_WORD(ncode) | OTHER_POINTER_LOWTAG;
 }
 
 static int
@@ -890,11 +837,11 @@ scav_return_pc_header(lispobj *where, lispobj object)
 static lispobj
 trans_return_pc_header(lispobj object)
 {
-       struct function *return_pc;
+       struct simple_fun *return_pc;
        unsigned long offset;
        struct code *code, *ncode;
        lispobj ret;
-       return_pc = (struct function *) PTR(object);
+       return_pc = (struct simple_fun *) native_pointer(object);
        offset = HeaderValue(return_pc->header)  * 4 ;
 
        /* Transport the whole code object */
@@ -904,21 +851,21 @@ trans_return_pc_header(lispobj object)
 #endif
        ncode = trans_code(code);
        if(object==0x304748d7) {
-           /* ldb_monitor(); */
+           /* monitor_or_something(); */
        }
-       ret= ((lispobj) LOW_WORD(ncode) + offset) | type_OtherPointer;
+       ret= ((lispobj) LOW_WORD(ncode) + offset) | OTHER_POINTER_LOWTAG;
 #ifdef DEBUG_CODE_GC
        printf("trans_return_pc_header returning %x\n",ret);
 #endif
        return ret;
 }
 
-/* On the 386, closures hold a pointer to the raw address instead of the
-   function object, so we can use CALL [$FDEFN+const] to invoke the function
-   without loading it into a register.  Given that code objects don't move,
-   we don't need to update anything, but we do have to figure out that the
-   function is still live. */
-#ifdef i386
+/* On the 386, closures hold a pointer to the raw address instead of
+ * the function object, so we can use CALL [$FDEFN+const] to invoke
+ * the function without loading it into a register. Given that code
+ * objects don't move, we don't need to update anything, but we do
+ * have to figure out that the function is still live. */
+#ifdef __i386__
 static
 scav_closure_header(where, object)
 lispobj *where, object;
@@ -927,7 +874,7 @@ lispobj *where, object;
        lispobj fun;
 
        closure = (struct closure *)where;
-       fun = closure->function - RAW_ADDR_OFFSET;
+       fun = closure->fun - FUN_RAW_ADDR_OFFSET;
        scavenge(&fun, 1);
 
        return 2;
@@ -935,7 +882,7 @@ lispobj *where, object;
 #endif
 
 static int
-scav_function_header(lispobj *where, lispobj object)
+scav_fun_header(lispobj *where, lispobj object)
 {
     fprintf(stderr, "GC lossage.  Should not be scavenging a ");
     fprintf(stderr, "Function Header.\n");
@@ -946,107 +893,61 @@ scav_function_header(lispobj *where, lispobj object)
 }
 
 static lispobj
-trans_function_header(lispobj object)
+trans_fun_header(lispobj object)
 {
-       struct function *fheader;
+       struct simple_fun *fheader;
        unsigned long offset;
        struct code *code, *ncode;
        
-       fheader = (struct function *) PTR(object);
+       fheader = (struct simple_fun *) native_pointer(object);
        offset = HeaderValue(fheader->header) * 4;
 
        /* Transport the whole code object */
        code = (struct code *) ((unsigned long) fheader - offset);
        ncode = trans_code(code);
 
-       return ((lispobj) LOW_WORD(ncode) + offset) | type_FunctionPointer;
+       return ((lispobj) LOW_WORD(ncode) + offset) | FUN_POINTER_LOWTAG;
 }
 
 
 \f
-/* Instances */
-
-#if DIRECT_SCAV
-static int
-scav_instance_pointer(lispobj *where, lispobj object)
-{
-    if (from_space_p(object)) {
-       lispobj first, *first_pointer;
+/* instances */
 
-       /* object is a pointer into from space.  check to see */
-       /* if it has been forwarded */
-       first_pointer = (lispobj *) PTR(object);
-       first = *first_pointer;
-               
-       if (!(Pointerp(first) && new_space_p(first)))
-           first = *first_pointer = trans_boxed(object);
-       *where = first;
-    }
-    return 1;
-}
-#else
 static int
 scav_instance_pointer(lispobj *where, lispobj object)
 {
   lispobj  *first_pointer;
   
   /* object is a pointer into from space.  Not a FP */
-  first_pointer = (lispobj *) PTR(object);
+  first_pointer = (lispobj *) native_pointer(object);
   
   *where = *first_pointer = trans_boxed(object);
   return 1;
 }
-#endif
 
 \f
-/* Lists and Conses */
+/* lists and conses */
 
 static lispobj trans_list(lispobj object);
 
-#if DIRECT_SCAV
-static int
-scav_list_pointer(lispobj *where, lispobj object)
-{
-       gc_assert(Pointerp(object));
-
-       if (from_space_p(object)) {
-               lispobj first, *first_pointer;
-
-               /* object is a pointer into from space.  check to see */
-               /* if it has been forwarded */
-               first_pointer = (lispobj *) PTR(object);
-               first = *first_pointer;
-               
-               if (!(Pointerp(first) && new_space_p(first)))
-                       first = *first_pointer = trans_list(object);
-
-               gc_assert(Pointerp(first));
-               gc_assert(!from_space_p(first));
-       
-               *where = first;
-       }
-       return 1;
-}
-#else
 static int
 scav_list_pointer(lispobj *where, lispobj object)
 {
   lispobj first, *first_pointer;
 
-  gc_assert(Pointerp(object));
+  gc_assert(is_lisp_pointer(object));
 
   /* object is a pointer into from space.  Not a FP. */
-  first_pointer = (lispobj *) PTR(object);
+  first_pointer = (lispobj *) native_pointer(object);
   
   first = *first_pointer = trans_list(object);
   
-  gc_assert(Pointerp(first));
+  gc_assert(is_lisp_pointer(first));
   gc_assert(!from_space_p(first));
   
   *where = first;
   return 1;
 }
-#endif
 
 static lispobj
 trans_list(lispobj object)
@@ -1054,11 +955,11 @@ trans_list(lispobj object)
        lispobj new_list_pointer;
        struct cons *cons, *new_cons;
        
-       cons = (struct cons *) PTR(object);
+       cons = (struct cons *) native_pointer(object);
 
        /* ### Don't use copy_object here. */
        new_list_pointer = copy_object(object, 2);
-       new_cons = (struct cons *) PTR(new_list_pointer);
+       new_cons = (struct cons *) native_pointer(new_list_pointer);
 
        /* Set forwarding pointer. */
        cons->car = new_list_pointer;
@@ -1072,17 +973,17 @@ trans_list(lispobj object)
 
                cdr = cons->cdr;
 
-                if (LowtagOf(cdr) != type_ListPointer ||
+                if (lowtagof(cdr) != LIST_POINTER_LOWTAG ||
                     !from_space_p(cdr) ||
-                    (Pointerp(first = *(lispobj *)PTR(cdr)) &&
-                     new_space_p(first)))
+                    (is_lisp_pointer(first = *(lispobj *)native_pointer(cdr))
+                    && new_space_p(first)))
                        break;
 
-               cdr_cons = (struct cons *) PTR(cdr);
+               cdr_cons = (struct cons *) native_pointer(cdr);
 
                /* ### Don't use copy_object here */
                new_cdr = copy_object(cdr, 2);
-               new_cdr_cons = (struct cons *) PTR(new_cdr);
+               new_cdr_cons = (struct cons *) native_pointer(new_cdr);
 
                /* Set forwarding pointer */
                cdr_cons->car = new_cdr;
@@ -1100,55 +1001,28 @@ trans_list(lispobj object)
 }
 
 \f
-/* Scavenging and Transporting Other Pointers */
+/* scavenging and transporting other pointers */
 
-#if DIRECT_SCAV
-static int
-scav_other_pointer(lispobj *where, lispobj object)
-{
-       gc_assert(Pointerp(object));
-
-       if (from_space_p(object)) {
-               lispobj first, *first_pointer;
-
-               /* object is a pointer into from space.  check to see */
-               /* if it has been forwarded */
-               first_pointer = (lispobj *) PTR(object);
-               first = *first_pointer;
-               
-               if (!(Pointerp(first) && new_space_p(first)))
-                       first = *first_pointer = 
-                               (transother[TypeOf(first)])(object);
-
-               gc_assert(Pointerp(first));
-               gc_assert(!from_space_p(first));
-
-               *where = first;
-       }
-       return 1;
-}
-#else
 static int
 scav_other_pointer(lispobj *where, lispobj object)
 {
   lispobj first, *first_pointer;
 
-  gc_assert(Pointerp(object));
+  gc_assert(is_lisp_pointer(object));
 
   /* Object is a pointer into from space - not a FP */
-  first_pointer = (lispobj *) PTR(object);
+  first_pointer = (lispobj *) native_pointer(object);
   first = *first_pointer = (transother[TypeOf(*first_pointer)])(object);
 
-  gc_assert(Pointerp(first));
+  gc_assert(is_lisp_pointer(first));
   gc_assert(!from_space_p(first));
 
   *where = first;
   return 1;
 }
-#endif
 
 \f
-/* Immediate, Boxed, and Unboxed Objects */
+/* immediate, boxed, and unboxed objects */
 
 static int
 size_pointer(lispobj *where)
@@ -1189,9 +1063,9 @@ trans_boxed(lispobj object)
        lispobj header;
        unsigned long length;
 
-       gc_assert(Pointerp(object));
+       gc_assert(is_lisp_pointer(object));
 
-       header = *((lispobj *) PTR(object));
+       header = *((lispobj *) native_pointer(object));
        length = HeaderValue(header) + 1;
        length = CEILING(length, 2);
 
@@ -1212,7 +1086,7 @@ size_boxed(lispobj *where)
 }
 
 /* Note: on the sparc we don't have to do anything special for fdefns, */
-/* cause the raw-addr has a function lowtag. */
+/* 'cause the raw-addr has a function lowtag. */
 #ifndef sparc
 static int
 scav_fdefn(lispobj *where, lispobj object)
@@ -1221,10 +1095,11 @@ scav_fdefn(lispobj *where, lispobj object)
 
     fdefn = (struct fdefn *)where;
     
-    if ((char *)(fdefn->function + RAW_ADDR_OFFSET) 
+    if ((char *)(fdefn->fun + FUN_RAW_ADDR_OFFSET) 
        == (char *)((unsigned long)(fdefn->raw_addr))) {
         scavenge(where + 1, sizeof(struct fdefn)/sizeof(lispobj) - 1);
-        fdefn->raw_addr = (u32)  ((char *) LOW_WORD(fdefn->function)) + RAW_ADDR_OFFSET;
+        fdefn->raw_addr =
+           (u32)  ((char *) LOW_WORD(fdefn->fun)) + FUN_RAW_ADDR_OFFSET;
         return sizeof(struct fdefn) / sizeof(lispobj);
     }
     else
@@ -1250,9 +1125,9 @@ trans_unboxed(lispobj object)
        unsigned long length;
 
 
-       gc_assert(Pointerp(object));
+       gc_assert(is_lisp_pointer(object));
 
-       header = *((lispobj *) PTR(object));
+       header = *((lispobj *) native_pointer(object));
        length = HeaderValue(header) + 1;
        length = CEILING(length, 2);
 
@@ -1273,7 +1148,7 @@ size_unboxed(lispobj *where)
 }
 
 \f
-/* Vector-Like Objects */
+/* vector-like objects */
 
 #define NWORDS(x,y) (CEILING((x),(y)) / (y))
 
@@ -1299,12 +1174,12 @@ trans_string(lispobj object)
        struct vector *vector;
        int length, nwords;
 
-       gc_assert(Pointerp(object));
+       gc_assert(is_lisp_pointer(object));
 
        /* NOTE: Strings contain one more byte of data than the length */
        /* slot indicates. */
 
-       vector = (struct vector *) PTR(object);
+       vector = (struct vector *) native_pointer(object);
        length = fixnum_value(vector->length) + 1;
        nwords = CEILING(NWORDS(length, 4) + 2, 2);
 
@@ -1331,7 +1206,7 @@ static int
 scav_vector(lispobj *where, lispobj object)
 {
     if (HeaderValue(object) == subtype_VectorValidHashing)
-        *where = (subtype_VectorMustRehash<<type_Bits) | type_SimpleVector;
+        *where = (subtype_VectorMustRehash<<N_TYPE_BITS) | type_SimpleVector;
 
     return 1;
 }
@@ -1343,9 +1218,9 @@ trans_vector(lispobj object)
        struct vector *vector;
        int length, nwords;
 
-       gc_assert(Pointerp(object));
+       gc_assert(is_lisp_pointer(object));
 
-       vector = (struct vector *) PTR(object);
+       vector = (struct vector *) native_pointer(object);
 
        length = fixnum_value(vector->length);
        nwords = CEILING(length + 2, 2);
@@ -1386,9 +1261,9 @@ trans_vector_bit(lispobj object)
        struct vector *vector;
        int length, nwords;
 
-       gc_assert(Pointerp(object));
+       gc_assert(is_lisp_pointer(object));
 
-       vector = (struct vector *) PTR(object);
+       vector = (struct vector *) native_pointer(object);
        length = fixnum_value(vector->length);
        nwords = CEILING(NWORDS(length, 32) + 2, 2);
 
@@ -1428,9 +1303,9 @@ trans_vector_unsigned_byte_2(lispobj object)
        struct vector *vector;
        int length, nwords;
 
-       gc_assert(Pointerp(object));
+       gc_assert(is_lisp_pointer(object));
 
-       vector = (struct vector *) PTR(object);
+       vector = (struct vector *) native_pointer(object);
        length = fixnum_value(vector->length);
        nwords = CEILING(NWORDS(length, 16) + 2, 2);
 
@@ -1470,9 +1345,9 @@ trans_vector_unsigned_byte_4(lispobj object)
        struct vector *vector;
        int length, nwords;
 
-       gc_assert(Pointerp(object));
+       gc_assert(is_lisp_pointer(object));
 
-       vector = (struct vector *) PTR(object);
+       vector = (struct vector *) native_pointer(object);
        length = fixnum_value(vector->length);
        nwords = CEILING(NWORDS(length, 8) + 2, 2);
 
@@ -1512,9 +1387,9 @@ trans_vector_unsigned_byte_8(lispobj object)
        struct vector *vector;
        int length, nwords;
 
-       gc_assert(Pointerp(object));
+       gc_assert(is_lisp_pointer(object));
 
-       vector = (struct vector *) PTR(object);
+       vector = (struct vector *) native_pointer(object);
        length = fixnum_value(vector->length);
        nwords = CEILING(NWORDS(length, 4) + 2, 2);
 
@@ -1554,9 +1429,9 @@ trans_vector_unsigned_byte_16(lispobj object)
        struct vector *vector;
        int length, nwords;
 
-       gc_assert(Pointerp(object));
+       gc_assert(is_lisp_pointer(object));
 
-       vector = (struct vector *) PTR(object);
+       vector = (struct vector *) native_pointer(object);
        length = fixnum_value(vector->length);
        nwords = CEILING(NWORDS(length, 2) + 2, 2);
 
@@ -1596,9 +1471,9 @@ trans_vector_unsigned_byte_32(lispobj object)
        struct vector *vector;
        int length, nwords;
 
-       gc_assert(Pointerp(object));
+       gc_assert(is_lisp_pointer(object));
 
-       vector = (struct vector *) PTR(object);
+       vector = (struct vector *) native_pointer(object);
        length = fixnum_value(vector->length);
        nwords = CEILING(length + 2, 2);
 
@@ -1638,9 +1513,9 @@ trans_vector_single_float(lispobj object)
        struct vector *vector;
        int length, nwords;
 
-       gc_assert(Pointerp(object));
+       gc_assert(is_lisp_pointer(object));
 
-       vector = (struct vector *) PTR(object);
+       vector = (struct vector *) native_pointer(object);
        length = fixnum_value(vector->length);
        nwords = CEILING(length + 2, 2);
 
@@ -1680,9 +1555,9 @@ trans_vector_double_float(lispobj object)
        struct vector *vector;
        int length, nwords;
 
-       gc_assert(Pointerp(object));
+       gc_assert(is_lisp_pointer(object));
 
-       vector = (struct vector *) PTR(object);
+       vector = (struct vector *) native_pointer(object);
        length = fixnum_value(vector->length);
        nwords = CEILING(length * 2 + 2, 2);
 
@@ -1725,9 +1600,9 @@ trans_vector_long_float(lispobj object)
        struct vector *vector;
        int length, nwords;
 
-       gc_assert(Pointerp(object));
+       gc_assert(is_lisp_pointer(object));
 
-       vector = (struct vector *) PTR(object);
+       vector = (struct vector *) native_pointer(object);
        length = fixnum_value(vector->length);
 #ifdef sparc
        nwords = CEILING(length * 4 + 2, 2);
@@ -1773,9 +1648,9 @@ trans_vector_complex_single_float(lispobj object)
        struct vector *vector;
        int length, nwords;
 
-       gc_assert(Pointerp(object));
+       gc_assert(is_lisp_pointer(object));
 
-       vector = (struct vector *) PTR(object);
+       vector = (struct vector *) native_pointer(object);
        length = fixnum_value(vector->length);
        nwords = CEILING(length * 2 + 2, 2);
 
@@ -1816,9 +1691,9 @@ trans_vector_complex_double_float(lispobj object)
        struct vector *vector;
        int length, nwords;
 
-       gc_assert(Pointerp(object));
+       gc_assert(is_lisp_pointer(object));
 
-       vector = (struct vector *) PTR(object);
+       vector = (struct vector *) native_pointer(object);
        length = fixnum_value(vector->length);
        nwords = CEILING(length * 4 + 2, 2);
 
@@ -1861,9 +1736,9 @@ trans_vector_complex_long_float(lispobj object)
        struct vector *vector;
        int length, nwords;
 
-       gc_assert(Pointerp(object));
+       gc_assert(is_lisp_pointer(object));
 
-       vector = (struct vector *) PTR(object);
+       vector = (struct vector *) native_pointer(object);
        length = fixnum_value(vector->length);
 #ifdef sparc
        nwords = CEILING(length * 8 + 2, 2);
@@ -1889,7 +1764,7 @@ size_vector_complex_long_float(lispobj *where)
 #endif
 
 \f
-/* Weak Pointers */
+/* weak pointers */
 
 #define WEAK_POINTER_NWORDS \
        CEILING((sizeof(struct weak_pointer) / sizeof(lispobj)), 2)
@@ -1910,7 +1785,7 @@ trans_weak_pointer(lispobj object)
        lispobj copy;
        struct weak_pointer *wp;
 
-       gc_assert(Pointerp(object));
+       gc_assert(is_lisp_pointer(object));
 
 #if defined(DEBUG_WEAK)
        printf("Transporting weak pointer from 0x%08x\n", object);
@@ -1920,7 +1795,7 @@ trans_weak_pointer(lispobj object)
        /* been transported so they can be fixed up in a post-GC pass. */
 
        copy = copy_object(object, WEAK_POINTER_NWORDS);
-       wp = (struct weak_pointer *) PTR(copy);
+       wp = (struct weak_pointer *) native_pointer(copy);
        
 
        /* Push the weak pointer onto the list of weak pointers. */
@@ -1952,7 +1827,7 @@ void scan_weak_pointers(void)
                printf("Value: 0x%08x\n", (unsigned int) value);
 #endif         
 
-               if (!(Pointerp(value) && from_space_p(value)))
+               if (!(is_lisp_pointer(value) && from_space_p(value)))
                        continue;
 
                /* Now, we need to check if the object has been */
@@ -1960,14 +1835,14 @@ void scan_weak_pointers(void)
                /* still good and needs to be updated.  Otherwise, the */
                /* weak pointer needs to be nil'ed out. */
 
-               first_pointer = (lispobj *) PTR(value);
+               first_pointer = (lispobj *) native_pointer(value);
                first = *first_pointer;
                
 #if defined(DEBUG_WEAK)
                printf("First: 0x%08x\n", (unsigned long) first);
 #endif         
 
-               if (Pointerp(first) && new_space_p(first))
+               if (is_lisp_pointer(first) && new_space_p(first))
                        wp->value = first;
                else {
                        wp->value = NIL;
@@ -1978,7 +1853,7 @@ void scan_weak_pointers(void)
 
 
 \f
-/* Initialization */
+/* initialization */
 
 static int
 scav_lose(lispobj *where, lispobj object)
@@ -2008,24 +1883,30 @@ size_lose(lispobj *where)
        return 1;
 }
 
-void gc_init(void)
+/* KLUDGE: SBCL already has two GC implementations, and if someday the
+ * precise generational GC is revived, it might have three. It would
+ * be nice to share the scavtab[] data set up here, and perhaps other
+ * things too, between all of them, rather than trying to maintain
+ * multiple copies. -- WHN 2001-05-09 */
+void
+gc_init(void)
 {
        int i;
 
-       /* Scavenge Table */
+       /* scavenge table */
        for (i = 0; i < 256; i++)
            scavtab[i] = scav_lose; 
        /* scavtab[i] = scav_immediate; */
 
        for (i = 0; i < 32; i++) {
-               scavtab[type_EvenFixnum|(i<<3)] = scav_immediate;
-               scavtab[type_FunctionPointer|(i<<3)] = scav_function_pointer;
-               /* OtherImmediate0 */
-               scavtab[type_ListPointer|(i<<3)] = scav_list_pointer;
-               scavtab[type_OddFixnum|(i<<3)] = scav_immediate;
-               scavtab[type_InstancePointer|(i<<3)] = scav_instance_pointer;
-               /* OtherImmediate1 */
-               scavtab[type_OtherPointer|(i<<3)] = scav_other_pointer;
+               scavtab[EVEN_FIXNUM_LOWTAG|(i<<3)] = scav_immediate;
+               scavtab[FUN_POINTER_LOWTAG|(i<<3)] = scav_fun_pointer;
+               /* skipping OTHER_IMMEDIATE_0_LOWTAG */
+               scavtab[LIST_POINTER_LOWTAG|(i<<3)] = scav_list_pointer;
+               scavtab[ODD_FIXNUM_LOWTAG|(i<<3)] = scav_immediate;
+               scavtab[INSTANCE_POINTER_LOWTAG|(i<<3)] =scav_instance_pointer;
+               /* skipping OTHER_IMMEDIATE_1_LOWTAG */
+               scavtab[OTHER_POINTER_LOWTAG|(i<<3)] = scav_other_pointer;
        }
 
        scavtab[type_Bignum] = scav_unboxed;
@@ -2085,21 +1966,15 @@ void gc_init(void)
        scavtab[type_ComplexVector] = scav_boxed;
        scavtab[type_ComplexArray] = scav_boxed;
        scavtab[type_CodeHeader] = scav_code_header;
-       scavtab[type_FunctionHeader] = scav_function_header;
-       scavtab[type_ClosureFunctionHeader] = scav_function_header;
+       scavtab[type_SimpleFunHeader] = scav_fun_header;
+       scavtab[type_ClosureFunHeader] = scav_fun_header;
        scavtab[type_ReturnPcHeader] = scav_return_pc_header;
-#ifdef i386
+#ifdef __i386__
        scavtab[type_ClosureHeader] = scav_closure_header;
        scavtab[type_FuncallableInstanceHeader] = scav_closure_header;
-       scavtab[type_ByteCodeFunction] = scav_closure_header;
-       scavtab[type_ByteCodeClosure] = scav_closure_header;
-       /*      scavtab[type_DylanFunctionHeader] = scav_closure_header; */
 #else
        scavtab[type_ClosureHeader] = scav_boxed;
        scavtab[type_FuncallableInstanceHeader] = scav_boxed;
-       scavtab[type_ByteCodeFunction] = scav_boxed;
-       scavtab[type_ByteCodeClosure] = scav_boxed;
-       /* scavtab[type_DylanFunctionHeader] = scav_boxed; */
 #endif
        scavtab[type_ValueCellHeader] = scav_boxed;
         scavtab[type_SymbolHeader] = scav_boxed;
@@ -2175,13 +2050,11 @@ void gc_init(void)
        transother[type_ComplexVector] = trans_boxed;
        transother[type_ComplexArray] = trans_boxed;
        transother[type_CodeHeader] = trans_code_header;
-       transother[type_FunctionHeader] = trans_function_header;
-       transother[type_ClosureFunctionHeader] = trans_function_header;
+       transother[type_SimpleFunHeader] = trans_fun_header;
+       transother[type_ClosureFunHeader] = trans_fun_header;
        transother[type_ReturnPcHeader] = trans_return_pc_header;
        transother[type_ClosureHeader] = trans_boxed;
        transother[type_FuncallableInstanceHeader] = trans_boxed;
-       transother[type_ByteCodeFunction] = trans_boxed;
-       transother[type_ByteCodeClosure] = trans_boxed;
        transother[type_ValueCellHeader] = trans_boxed;
        transother[type_SymbolHeader] = trans_boxed;
        transother[type_BaseChar] = trans_immediate;
@@ -2197,14 +2070,14 @@ void gc_init(void)
                sizetab[i] = size_lose;
 
        for (i = 0; i < 32; i++) {
-               sizetab[type_EvenFixnum|(i<<3)] = size_immediate;
-               sizetab[type_FunctionPointer|(i<<3)] = size_pointer;
-               /* OtherImmediate0 */
-               sizetab[type_ListPointer|(i<<3)] = size_pointer;
-               sizetab[type_OddFixnum|(i<<3)] = size_immediate;
-               sizetab[type_InstancePointer|(i<<3)] = size_pointer;
-               /* OtherImmediate1 */
-               sizetab[type_OtherPointer|(i<<3)] = size_pointer;
+               sizetab[EVEN_FIXNUM_LOWTAG|(i<<3)] = size_immediate;
+               sizetab[FUN_POINTER_LOWTAG|(i<<3)] = size_pointer;
+               /* skipping OTHER_IMMEDIATE_0_LOWTAG */
+               sizetab[LIST_POINTER_LOWTAG|(i<<3)] = size_pointer;
+               sizetab[ODD_FIXNUM_LOWTAG|(i<<3)] = size_immediate;
+               sizetab[INSTANCE_POINTER_LOWTAG|(i<<3)] = size_pointer;
+               /* skipping OTHER_IMMEDIATE_1_LOWTAG */
+               sizetab[OTHER_POINTER_LOWTAG|(i<<3)] = size_pointer;
        }
 
        sizetab[type_Bignum] = size_unboxed;
@@ -2266,8 +2139,8 @@ void gc_init(void)
        sizetab[type_CodeHeader] = size_code_header;
 #if 0
        /* Shouldn't see these so just lose if it happens */
-       sizetab[type_FunctionHeader] = size_function_header;
-       sizetab[type_ClosureFunctionHeader] = size_function_header;
+       sizetab[type_SimpleFunHeader] = size_function_header;
+       sizetab[type_ClosureFunHeader] = size_function_header;
        sizetab[type_ReturnPcHeader] = size_return_pc_header;
 #endif
        sizetab[type_ClosureHeader] = size_boxed;
@@ -2281,12 +2154,8 @@ void gc_init(void)
         sizetab[type_InstanceHeader] = size_boxed;
        sizetab[type_Fdefn] = size_boxed;
 }
-
-
 \f
-/* Noise to manipulate the gc trigger stuff. */
-
-#ifndef ibmrt
+/* noise to manipulate the gc trigger stuff */
 
 void set_auto_gc_trigger(os_vm_size_t dynamic_usage)
 {
@@ -2340,5 +2209,3 @@ void clear_auto_gc_trigger(void)
        current_auto_gc_trigger = NULL;
     }
 }
-
-#endif