altos/lisp: Change GC to do moves in batches of 32
authorKeith Packard <keithp@keithp.com>
Tue, 15 Nov 2016 02:45:12 +0000 (18:45 -0800)
committerKeith Packard <keithp@keithp.com>
Mon, 20 Feb 2017 19:16:50 +0000 (11:16 -0800)
This should make it quite a bit faster than doing one at a time.

Signed-off-by: Keith Packard <keithp@keithp.com>
src/lisp/ao_lisp.h
src/lisp/ao_lisp_atom.c
src/lisp/ao_lisp_cons.c
src/lisp/ao_lisp_eval.c
src/lisp/ao_lisp_frame.c
src/lisp/ao_lisp_lambda.c
src/lisp/ao_lisp_mem.c
src/lisp/ao_lisp_read.c
src/lisp/ao_lisp_string.c

index ea3d2a0905f8d5e8e9deab93523404f3bd0ecb61..906bae19f8f7dc79f21b463d14e0e6b143ac4de2 100644 (file)
@@ -134,6 +134,7 @@ struct ao_lisp_type {
        int     (*size)(void *addr);
        void    (*mark)(void *addr);
        void    (*move)(void *addr);
+       char    name[];
 };
 
 struct ao_lisp_cons {
@@ -304,11 +305,17 @@ ao_lisp_other_poly(const void *other)
 }
 
 static inline int
-ao_lisp_mem_round(int size)
+ao_lisp_size_round(int size)
 {
        return (size + 3) & ~3;
 }
 
+static inline int
+ao_lisp_size(const struct ao_lisp_type *type, void *addr)
+{
+       return ao_lisp_size_round(type->size(addr));
+}
+
 #define AO_LISP_OTHER_POLY(other) ((ao_poly)(other) + AO_LISP_OTHER)
 
 static inline int ao_lisp_poly_base_type(ao_poly poly) {
@@ -389,7 +396,7 @@ ao_lisp_mark(const struct ao_lisp_type *type, void *addr);
 
 /* returns 1 if the object was already marked */
 int
-ao_lisp_mark_memory(void *addr, int size);
+ao_lisp_mark_memory(const struct ao_lisp_type *type, void *addr);
 
 void *
 ao_lisp_move_map(void *addr);
@@ -400,7 +407,7 @@ ao_lisp_move(const struct ao_lisp_type *type, void **ref);
 
 /* returns 1 if the object was already moved */
 int
-ao_lisp_move_memory(void **ref, int size);
+ao_lisp_move_memory(const struct ao_lisp_type *type, void **ref);
 
 void *
 ao_lisp_alloc(int size);
@@ -408,14 +415,23 @@ ao_lisp_alloc(int size);
 void
 ao_lisp_collect(void);
 
-int
-ao_lisp_root_add(const struct ao_lisp_type *type, void *addr);
+void
+ao_lisp_cons_stash(int id, struct ao_lisp_cons *cons);
 
-int
-ao_lisp_root_poly_add(ao_poly *p);
+struct ao_lisp_cons *
+ao_lisp_cons_fetch(int id);
 
 void
-ao_lisp_root_clear(void *addr);
+ao_lisp_string_stash(int id, char *string);
+
+char *
+ao_lisp_string_fetch(int id);
+
+void
+ao_lisp_poly_stash(int id, ao_poly poly);
+
+ao_poly
+ao_lisp_poly_fetch(int id);
 
 /* cons */
 extern const struct ao_lisp_type ao_lisp_cons_type;
@@ -435,9 +451,6 @@ ao_lisp_cons_length(struct ao_lisp_cons *cons);
 /* string */
 extern const struct ao_lisp_type ao_lisp_string_type;
 
-char *
-ao_lisp_string_new(int len);
-
 char *
 ao_lisp_string_copy(char *a);
 
@@ -529,6 +542,10 @@ char *
 ao_lisp_args_name(uint8_t args);
 
 /* read */
+extern struct ao_lisp_cons     *ao_lisp_read_cons;
+extern struct ao_lisp_cons     *ao_lisp_read_cons_tail;
+extern struct ao_lisp_cons     *ao_lisp_read_stack;
+
 ao_poly
 ao_lisp_read(void);
 
@@ -585,6 +602,8 @@ ao_lisp_restore(struct ao_lisp_cons *cons);
 
 /* error */
 
+extern const struct ao_lisp_type ao_lisp_stack_type;
+
 void
 ao_lisp_stack_print(void);
 
@@ -631,4 +650,32 @@ ao_lisp_frames_dump(void)
 #define DBG_FRAMES()
 #endif
 
+#define DBG_MEM                1
+#define DBG_MEM_START  1
+
+#if DBG_MEM
+
+#include <assert.h>
+extern int dbg_move_depth;
+#define MDBG_DUMP 1
+#define MDBG_OFFSET(a) ((int) ((uint8_t *) (a) - ao_lisp_pool))
+
+extern int dbg_mem;
+
+#define MDBG_DO(a)     a
+#define MDBG_MOVE(...) do { if (dbg_mem) { int d; for (d = 0; d < dbg_move_depth; d++) printf ("  "); printf(__VA_ARGS__); } } while (0)
+#define MDBG_MORE(...) do { if (dbg_mem) printf(__VA_ARGS__); } while (0)
+#define MDBG_MOVE_IN() (dbg_move_depth++)
+#define MDBG_MOVE_OUT()        (assert(--dbg_move_depth >= 0))
+
+#else
+
+#define MDBG_DO(a)
+#define MDBG_MOVE(...)
+#define MDBG_MORE(...)
+#define MDBG_MOVE_IN()
+#define MDBG_MOVE_OUT()
+
+#endif
+
 #endif /* _AO_LISP_H_ */
index e1d9b0821949f5624ab3fd1eb40e9427a85eccf7..6705f14036c07748a9b3a9004936c67b8bac3cdc 100644 (file)
@@ -38,7 +38,7 @@ static void atom_mark(void *addr)
                atom = ao_lisp_poly_atom(atom->next);
                if (!atom)
                        break;
-               if (ao_lisp_mark_memory(atom, atom_size(atom)))
+               if (ao_lisp_mark_memory(&ao_lisp_atom_type, atom))
                        break;
        }
 }
@@ -53,7 +53,7 @@ static void atom_move(void *addr)
 
                if (!next)
                        break;
-               ret = ao_lisp_move_memory((void **) &next, atom_size(next));
+               ret = ao_lisp_move_memory(&ao_lisp_atom_type, (void **) &next);
                if (next != ao_lisp_poly_atom(atom->next))
                        atom->next = ao_lisp_atom_poly(next);
                if (ret)
@@ -66,6 +66,7 @@ const struct ao_lisp_type ao_lisp_atom_type = {
        .mark = atom_mark,
        .size = atom_size,
        .move = atom_move,
+       .name = "atom"
 };
 
 struct ao_lisp_atom    *ao_lisp_atoms;
@@ -85,12 +86,12 @@ ao_lisp_atom_intern(char *name)
                        return atom;
        }
 #endif
+       ao_lisp_string_stash(0, name);
        atom = ao_lisp_alloc(name_size(name));
+       name = ao_lisp_string_fetch(0);
        if (atom) {
                atom->type = AO_LISP_ATOM;
                atom->next = ao_lisp_atom_poly(ao_lisp_atoms);
-               if (!ao_lisp_atoms)
-                       ao_lisp_root_add(&ao_lisp_atom_type, &ao_lisp_atoms);
                ao_lisp_atoms = atom;
                strcpy(atom->name, name);
        }
@@ -103,11 +104,8 @@ struct ao_lisp_frame       *ao_lisp_frame_current;
 static void
 ao_lisp_atom_init(void)
 {
-       if (!ao_lisp_frame_global) {
+       if (!ao_lisp_frame_global)
                ao_lisp_frame_global = ao_lisp_frame_new(0);
-               ao_lisp_root_add(&ao_lisp_frame_type, &ao_lisp_frame_global);
-               ao_lisp_root_add(&ao_lisp_frame_type, &ao_lisp_frame_current);
-       }
 }
 
 static ao_poly *
index c7d8382f79d19c1d563516997fee1dce9b3dad18..311d63ab8dc3dd0497f1edb3385d219eeda42e14 100644 (file)
@@ -14,8 +14,6 @@
 
 #include "ao_lisp.h"
 
-#define OFFSET(a)      ((int) ((uint8_t *) (a) - ao_lisp_const))
-
 static void cons_mark(void *addr)
 {
        struct ao_lisp_cons     *cons = addr;
@@ -25,7 +23,7 @@ static void cons_mark(void *addr)
                cons = ao_lisp_poly_cons(cons->cdr);
                if (!cons)
                        break;
-               if (ao_lisp_mark_memory(cons, sizeof (struct ao_lisp_cons)))
+               if (ao_lisp_mark_memory(&ao_lisp_cons_type, cons))
                        break;
        }
 }
@@ -47,13 +45,17 @@ static void cons_move(void *addr)
                struct ao_lisp_cons     *cdr;
                int                     ret;
 
+               MDBG_MOVE("cons_move start %d (%d, %d)\n",
+                         MDBG_OFFSET(cons), MDBG_OFFSET(ao_lisp_ref(cons->car)), MDBG_OFFSET(ao_lisp_ref(cons->cdr)));
                (void) ao_lisp_poly_move(&cons->car, 1);
                cdr = ao_lisp_poly_cons(cons->cdr);
                if (!cdr)
                        break;
-               ret = ao_lisp_move_memory((void **) &cdr, sizeof (struct ao_lisp_cons));
+               ret = ao_lisp_move_memory(&ao_lisp_cons_type, (void **) &cdr);
                if (cdr != ao_lisp_poly_cons(cons->cdr))
                        cons->cdr = ao_lisp_cons_poly(cdr);
+               MDBG_MOVE("cons_move end %d (%d, %d)\n",
+                         MDBG_OFFSET(cons), MDBG_OFFSET(ao_lisp_ref(cons->car)), MDBG_OFFSET(ao_lisp_ref(cons->cdr)));
                if (ret)
                        break;
                cons = cdr;
@@ -64,31 +66,23 @@ const struct ao_lisp_type ao_lisp_cons_type = {
        .mark = cons_mark,
        .size = cons_size,
        .move = cons_move,
+       .name = "cons",
 };
 
-static ao_poly cons_car;
-static struct ao_lisp_cons *cons_cdr;
-static int been_here;
-
 struct ao_lisp_cons *
 ao_lisp_cons_cons(ao_poly car, struct ao_lisp_cons *cdr)
 {
        struct ao_lisp_cons     *cons;
 
-       if (!been_here) {
-               ao_lisp_root_add(&ao_lisp_cons_type, &cons_cdr);
-               ao_lisp_root_poly_add(&cons_car);
-               been_here = 1;
-       }
-       cons_car = car;
-       cons_cdr = cdr;
+       ao_lisp_poly_stash(0, car);
+       ao_lisp_cons_stash(0, cdr);
        cons = ao_lisp_alloc(sizeof (struct ao_lisp_cons));
+       car = ao_lisp_poly_fetch(0);
+       cdr = ao_lisp_cons_fetch(0);
        if (!cons)
                return NULL;
-       cons->car = cons_car;
-       cons->cdr = ao_lisp_cons_poly(cons_cdr);
-       cons_car = AO_LISP_NIL;
-       cons_cdr = NULL;
+       cons->car = car;
+       cons->cdr = ao_lisp_cons_poly(cdr);
        return cons;
 }
 
index f945bc163a35cbef1925ca6aa1c124e1376af971..04d0e70a2f79eeeee957ee01df3ced52575478db 100644 (file)
@@ -16,6 +16,8 @@
 #include "ao_lisp.h"
 #include <assert.h>
 
+const struct ao_lisp_type ao_lisp_stack_type;
+
 static int
 stack_size(void *addr)
 {
@@ -34,13 +36,11 @@ stack_mark(void *addr)
                ao_lisp_poly_mark(stack->frame, 0);
                ao_lisp_poly_mark(stack->list, 0);
                stack = ao_lisp_poly_stack(stack->prev);
-               if (ao_lisp_mark_memory(stack, sizeof (struct ao_lisp_stack)))
+               if (ao_lisp_mark_memory(&ao_lisp_stack_type, stack))
                        break;
        }
 }
 
-static const struct ao_lisp_type ao_lisp_stack_type;
-
 static void
 stack_move(void *addr)
 {
@@ -57,8 +57,7 @@ stack_move(void *addr)
                prev = ao_lisp_poly_stack(stack->prev);
                if (!prev)
                        break;
-               ret = ao_lisp_move_memory((void **) &prev,
-                                         sizeof (struct ao_lisp_stack));
+               ret = ao_lisp_move_memory(&ao_lisp_stack_type, (void **) &prev);
                if (prev != ao_lisp_poly_stack(stack->prev))
                        stack->prev = ao_lisp_stack_poly(prev);
                if (ret)
@@ -67,10 +66,11 @@ stack_move(void *addr)
        }
 }
 
-static const struct ao_lisp_type ao_lisp_stack_type = {
+const struct ao_lisp_type ao_lisp_stack_type = {
        .size = stack_size,
        .mark = stack_mark,
-       .move = stack_move
+       .move = stack_move,
+       .name = "stack"
 };
 
 struct ao_lisp_stack           *ao_lisp_stack;
@@ -567,14 +567,7 @@ ao_lisp_eval_restart(void)
 ao_poly
 ao_lisp_eval(ao_poly _v)
 {
-       static uint8_t been_here;
-
        ao_lisp_v = _v;
-       if (!been_here) {
-               been_here = 1;
-               ao_lisp_root_add(&ao_lisp_stack_type, &ao_lisp_stack);
-               ao_lisp_root_poly_add(&ao_lisp_v);
-       }
 
        if (!ao_lisp_stack_push())
                return AO_LISP_NIL;
index 082860ee7176c4ee00be785a3fa0377ab4aa2535..e23a641384b74781349f4dae712bb2bb21b1f910 100644 (file)
 
 #include "ao_lisp.h"
 
-#if 0
-#define DBG(...)       printf(__VA_ARGS__)
-#else
-#define DBG(...)
-#endif
-
 static inline int
 frame_num_size(int num)
 {
@@ -33,8 +27,6 @@ frame_size(void *addr)
        return frame_num_size(frame->num);
 }
 
-#define OFFSET(a)      ((int) ((uint8_t *) (ao_lisp_ref(a)) - ao_lisp_const))
-
 static void
 frame_mark(void *addr)
 {
@@ -42,22 +34,23 @@ frame_mark(void *addr)
        int                     f;
 
        for (;;) {
-               DBG("frame mark %p\n", frame);
+               MDBG_MOVE("frame mark %d\n", MDBG_OFFSET(frame));
                if (!AO_LISP_IS_POOL(frame))
                        break;
                for (f = 0; f < frame->num; f++) {
                        struct ao_lisp_val      *v = &frame->vals[f];
 
                        ao_lisp_poly_mark(v->val, 0);
-                       DBG ("\tframe mark atom %s %d val %d at %d\n",
-                            ao_lisp_poly_atom(v->atom)->name,
-                            OFFSET(v->atom), OFFSET(v->val), f);
+                       MDBG_MOVE("frame mark atom %s %d val %d at %d\n",
+                                 ao_lisp_poly_atom(v->atom)->name,
+                                 MDBG_OFFSET(ao_lisp_ref(v->atom)),
+                                 MDBG_OFFSET(ao_lisp_ref(v->val)), f);
                }
                frame = ao_lisp_poly_frame(frame->next);
-               DBG("frame next %p\n", frame);
+               MDBG_MOVE("frame next %d\n", MDBG_OFFSET(frame));
                if (!frame)
                        break;
-               if (ao_lisp_mark_memory(frame, frame_size(frame)))
+               if (ao_lisp_mark_memory(&ao_lisp_frame_type, frame))
                        break;
        }
 }
@@ -72,22 +65,29 @@ frame_move(void *addr)
                struct ao_lisp_frame    *next;
                int                     ret;
 
-               DBG("frame move %p\n", frame);
+               MDBG_MOVE("frame move %d\n", MDBG_OFFSET(frame));
                if (!AO_LISP_IS_POOL(frame))
                        break;
                for (f = 0; f < frame->num; f++) {
                        struct ao_lisp_val      *v = &frame->vals[f];
 
                        ao_lisp_poly_move(&v->atom, 0);
-                       DBG("moved atom %s\n", ao_lisp_poly_atom(v->atom)->name);
                        ao_lisp_poly_move(&v->val, 0);
+                       MDBG_MOVE("frame move atom %s %d val %d at %d\n",
+                                 ao_lisp_poly_atom(v->atom)->name,
+                                 MDBG_OFFSET(ao_lisp_ref(v->atom)),
+                                 MDBG_OFFSET(ao_lisp_ref(v->val)), f);
                }
                next = ao_lisp_poly_frame(frame->next);
                if (!next)
                        break;
-               ret = ao_lisp_move_memory((void **) &next, frame_size(next));
-               if (next != ao_lisp_poly_frame(frame->next))
+               ret = ao_lisp_move_memory(&ao_lisp_frame_type, (void **) &next);
+               if (next != ao_lisp_poly_frame(frame->next)) {
+                       MDBG_MOVE("frame next moved from %d to %d\n",
+                                 MDBG_OFFSET(ao_lisp_poly_frame(frame->next)),
+                                 MDBG_OFFSET(next));
                        frame->next = ao_lisp_frame_poly(next);
+               }
                if (ret)
                        break;
                frame = next;
@@ -97,7 +97,8 @@ frame_move(void *addr)
 const struct ao_lisp_type ao_lisp_frame_type = {
        .mark = frame_mark,
        .size = frame_size,
-       .move = frame_move
+       .move = frame_move,
+       .name = "frame",
 };
 
 void
@@ -206,8 +207,8 @@ ao_lisp_frame_add(struct ao_lisp_frame **frame_ref, ao_poly atom, ao_poly val)
 
        if (!ref) {
                int f;
-               ao_lisp_root_poly_add(&atom);
-               ao_lisp_root_poly_add(&val);
+               ao_lisp_poly_stash(0, atom);
+               ao_lisp_poly_stash(1, val);
                if (frame) {
                        f = frame->num;
                        frame = ao_lisp_frame_realloc(frame_ref, f + 1);
@@ -215,12 +216,11 @@ ao_lisp_frame_add(struct ao_lisp_frame **frame_ref, ao_poly atom, ao_poly val)
                        f = 0;
                        frame = ao_lisp_frame_new(1);
                }
-               ao_lisp_root_clear(&atom);
-               ao_lisp_root_clear(&val);
+               atom = ao_lisp_poly_fetch(0);
+               val = ao_lisp_poly_fetch(1);
                if (!frame)
                        return 0;
                *frame_ref = frame;
-               DBG ("add atom %s %d, val %d at %d\n", ao_lisp_poly_atom(atom)->name, OFFSET(atom), OFFSET(val), f);
                frame->vals[f].atom = atom;
                ref = &frame->vals[f].val;
        }
index c53a38fd687f99533d4e7075082967ce99e87bde..6020a8b8b1a8482e970d891c49306171d7e593b5 100644 (file)
@@ -47,6 +47,7 @@ const struct ao_lisp_type ao_lisp_lambda_type = {
        .size = lambda_size,
        .mark = lambda_mark,
        .move = lambda_move,
+       .name = "lambda",
 };
 
 void
@@ -68,7 +69,9 @@ ao_lisp_lambda_print(ao_poly poly)
 ao_poly
 ao_lisp_lambda_alloc(struct ao_lisp_cons *code, int args)
 {
+       ao_lisp_cons_stash(0, code);
        struct ao_lisp_lambda   *lambda = ao_lisp_alloc(sizeof (struct ao_lisp_lambda));
+       code = ao_lisp_cons_fetch(0);
        struct ao_lisp_cons     *arg;
        int                     f;
 
index 0373f0151448f87144b961a622c094799eae0f44..60f4bbeeed0b8bd8a2fa1033b8811ac549d1edbb 100644 (file)
 #include <stdio.h>
 
 #ifdef AO_LISP_MAKE_CONST
+
+/*
+ * When building the constant table, it is the
+ * pool for allocations.
+ */
+
 #include <stdlib.h>
 uint8_t ao_lisp_const[AO_LISP_POOL_CONST] __attribute__((aligned(4)));
 #define ao_lisp_pool ao_lisp_const
 #undef AO_LISP_POOL
 #define AO_LISP_POOL AO_LISP_POOL_CONST
+
 #else
+
 uint8_t        ao_lisp_pool[AO_LISP_POOL + AO_LISP_POOL_EXTRA] __attribute__((aligned(4)));
-#endif
 
-#if 0
-#define MDBG_COLLECT_ALWAYS
 #endif
 
 #if 0
 #define MDBG_POOL
 #endif
 
-#if 0
-#include <assert.h>
-#define MDBG_INCLUDE
-#if 1
-#define MDBG_MOVE(...) do { int d; for (d = 0; d < move_depth; d++) printf ("  "); printf(__VA_ARGS__); } while (0)
-#endif
-#define MDBG_OFFSET(a) ((int) ((uint8_t *) (a) - ao_lisp_pool))
-#define MDBG(...) printf(__VA_ARGS__)
-#define MDBG_DO(a)     a
-static int move_depth;
-#define MDBG_MOVE_IN() (move_depth++)
-#define MDBG_MOVE_OUT()        (assert(--move_depth >= 0))
+#if DBG_MEM
+int dbg_move_depth;
+int dbg_mem = DBG_MEM_START;
+int dbg_collects = 0;
+int dbg_validate = 0;
+
+struct ao_lisp_record {
+       struct ao_lisp_record           *next;
+       const struct ao_lisp_type       *type;
+       void                            *addr;
+       int                             size;
+};
+
+static struct ao_lisp_record   *record_head, **record_tail;
+
+static void
+ao_lisp_record_free(struct ao_lisp_record *record)
+{
+       while (record) {
+               struct ao_lisp_record *next = record->next;
+               free(record);
+               record = next;
+       }
+}
+
+static void
+ao_lisp_record_reset(void)
+{
+       ao_lisp_record_free(record_head);
+       record_head = NULL;
+       record_tail = &record_head;
+}
+
+static void
+ao_lisp_record(const struct ao_lisp_type       *type,
+              void                             *addr,
+              int                              size)
+{
+       struct ao_lisp_record   *r = malloc(sizeof (struct ao_lisp_record));
+
+       r->next = NULL;
+       r->type = type;
+       r->addr = addr;
+       r->size = size;
+       *record_tail = r;
+       record_tail = &r->next;
+}
+
+static struct ao_lisp_record *
+ao_lisp_record_save(void)
+{
+       struct ao_lisp_record *r = record_head;
+
+       record_head = NULL;
+       record_tail = &record_head;
+       return r;
+}
+
+static void
+ao_lisp_record_compare(char *where,
+                      struct ao_lisp_record *a,
+                      struct ao_lisp_record *b)
+{
+       while (a && b) {
+               if (a->type != b->type || a->size != b->size) {
+                       printf("%s record difers %d %s %d -> %d %s %d\n",
+                              where,
+                              MDBG_OFFSET(a->addr),
+                              a->type->name,
+                              a->size,
+                              MDBG_OFFSET(b->addr),
+                              b->type->name,
+                              b->size);
+                       ao_lisp_abort();
+               }
+               a = a->next;
+               b = b->next;
+       }
+       if (a) {
+               printf("%s record differs %d %s %d -> NULL\n",
+                      where,
+                      MDBG_OFFSET(a->addr),
+                      a->type->name,
+                      a->size);
+               ao_lisp_abort();
+       }
+       if (b) {
+               printf("%s record differs NULL -> %d %s %d\n",
+                      where,
+                      MDBG_OFFSET(b->addr),
+                      b->type->name,
+                      b->size);
+               ao_lisp_abort();
+       }
+}
+
 #else
-#define MDBG(...)
-#define MDBG_DO(a)
-#define MDBG_MOVE(...)
-#define MDBG_MOVE_IN()
-#define MDBG_MOVE_OUT()
+#define ao_lisp_record_reset()
 #endif
 
 uint8_t        ao_lisp_exception;
 
 struct ao_lisp_root {
-       void                            **addr;
        const struct ao_lisp_type       *type;
+       void                            **addr;
 };
 
-#define AO_LISP_ROOT   16
+static struct ao_lisp_cons     *save_cons[2];
+static char                    *save_string[2];
+static ao_poly                 save_poly[2];
+
+static const struct ao_lisp_root       ao_lisp_root[] = {
+       {
+               .type = &ao_lisp_cons_type,
+               .addr = (void **) &save_cons[0],
+       },
+       {
+               .type = &ao_lisp_cons_type,
+               .addr = (void **) &save_cons[1],
+       },
+       {
+               .type = &ao_lisp_string_type,
+               .addr = (void **) &save_string[0]
+       },
+       {
+               .type = &ao_lisp_string_type,
+               .addr = (void **) &save_string[1]
+       },
+       {
+               .type = NULL,
+               .addr = (void **) &save_poly[0]
+       },
+       {
+               .type = NULL,
+               .addr = (void **) &save_poly[1]
+       },
+       {
+               .type = &ao_lisp_atom_type,
+               .addr = (void **) &ao_lisp_atoms
+       },
+       {
+               .type = &ao_lisp_frame_type,
+               .addr = (void **) &ao_lisp_frame_global,
+       },
+       {
+               .type = &ao_lisp_frame_type,
+               .addr = (void **) &ao_lisp_frame_current,
+       },
+       {
+               .type = &ao_lisp_stack_type,
+               .addr = (void **) &ao_lisp_stack,
+       },
+       {
+               .type = NULL,
+               .addr = (void **) &ao_lisp_v,
+       },
+       {
+               .type = &ao_lisp_cons_type,
+               .addr = (void **) &ao_lisp_read_cons,
+       },
+       {
+               .type = &ao_lisp_cons_type,
+               .addr = (void **) &ao_lisp_read_cons_tail,
+       },
+       {
+               .type = &ao_lisp_cons_type,
+               .addr = (void **) &ao_lisp_read_stack,
+       },
+};
 
-static struct ao_lisp_root     ao_lisp_root[AO_LISP_ROOT];
+#define AO_LISP_ROOT   (sizeof (ao_lisp_root) / sizeof (ao_lisp_root[0]))
 
 #define AO_LISP_BUSY_SIZE      ((AO_LISP_POOL + 31) / 32)
 
 static uint8_t ao_lisp_busy[AO_LISP_BUSY_SIZE];
-static uint8_t ao_lisp_moving[AO_LISP_BUSY_SIZE];
 static uint8_t ao_lisp_cons_note[AO_LISP_BUSY_SIZE];
 static uint8_t ao_lisp_cons_last[AO_LISP_BUSY_SIZE];
 static uint8_t ao_lisp_cons_noted;
 
 uint16_t       ao_lisp_top;
 
+struct ao_lisp_chunk {
+       uint16_t                old_addr;
+       union {
+               uint16_t        size;
+               uint16_t        new_addr;
+       };
+};
+
+#define AO_LISP_NCHUNK 32
+
+static struct ao_lisp_chunk ao_lisp_chunk[AO_LISP_NCHUNK];
+
+/* Offset of an address within the pool. */
+static inline uint16_t pool_offset(void *addr) {
+       if (!AO_LISP_IS_POOL(addr))
+               ao_lisp_abort();
+       return ((uint8_t *) addr) - ao_lisp_pool;
+}
+
+/*
+ * Convert back and forth between 'poly's used
+ * as short addresses in the pool and addresses.
+ * These are used in the chunk code.
+ */
+static inline ao_poly pool_poly(void *addr) {
+       if (!AO_LISP_IS_POOL(addr))
+               ao_lisp_abort();
+       return ((uint8_t *) addr) - AO_LISP_POOL_BASE;
+}
+
+static inline void *pool_ref(ao_poly p) {
+       return AO_LISP_POOL_BASE + p;
+}
+
 static inline void mark(uint8_t *tag, int offset) {
        int     byte = offset >> 5;
        int     bit = (offset >> 2) & 7;
@@ -101,24 +280,28 @@ static inline int limit(int offset) {
        return min(AO_LISP_POOL, max(offset, 0));
 }
 
+static int total_marked;
+
+/*
+ * Mark a range of addresses
+ */
 static int
 mark_object(uint8_t *tag, void *addr, int size) {
        int     base;
        int     bound;
 
-       if (!addr)
-               return 1;
+       MDBG_DO(if (!AO_LISP_IS_POOL((uint8_t *) addr + size - 1))
+                       ao_lisp_abort());
 
-       if ((uint8_t *) addr < ao_lisp_pool || ao_lisp_pool + AO_LISP_POOL <= (uint8_t*) addr)
-               return 1;
-
-       base = (uint8_t *) addr - ao_lisp_pool;
+       base = pool_offset(addr);
        bound = base + size;
 
-       base = limit(base);
-       bound = limit(bound);
+       MDBG_DO(if (bound > ao_lisp_top) ao_lisp_abort());
+
        if (busy(tag, base))
                return 1;
+       if (tag == ao_lisp_busy)
+               total_marked += size;
        while (base < bound) {
                mark(tag, base);
                base += 4;
@@ -126,12 +309,14 @@ mark_object(uint8_t *tag, void *addr, int size) {
        return 0;
 }
 
+MDBG_DO(
 static int
 clear_object(uint8_t *tag, void *addr, int size) {
        int     base;
        int     bound;
-       if (!addr)
-               return 1;
+
+       MDBG_DO(if (!AO_LISP_IS_POOL((uint8_t *) addr + size - 1))
+                       ao_lisp_abort());
 
        base = (uint8_t *) addr - ao_lisp_pool;
        bound = base + size;
@@ -140,29 +325,13 @@ clear_object(uint8_t *tag, void *addr, int size) {
        bound = limit(bound);
        if (!busy(tag, base))
                return 1;
+       total_marked -= size;
        while (base < bound) {
                clear(tag, base);
                base += 4;
        }
        return 0;
-}
-
-static int
-busy_object(uint8_t *tag, void *addr) {
-       int     base;
-
-       if (!addr)
-               return 1;
-
-       if ((uint8_t *) addr < ao_lisp_pool || ao_lisp_pool + AO_LISP_POOL <= (uint8_t*) addr)
-               return 1;
-
-       base = (uint8_t *) addr - ao_lisp_pool;
-       base = limit(base);
-       if (busy(tag, base))
-               return 1;
-       return 0;
-}
+})
 
 static void
 note_cons(void *addr)
@@ -175,31 +344,63 @@ note_cons(void *addr)
        }
 }
 
+static uint16_t        chunk_low;
+static uint16_t        chunk_first, chunk_last;
+
+static void
+note_chunk(uint16_t addr, uint16_t size)
+{
+       int i;
+
+       if (addr < chunk_low)
+               return;
+
+       for (i = 0; i < AO_LISP_NCHUNK; i++) {
+               if (ao_lisp_chunk[i].size && ao_lisp_chunk[i].old_addr == addr) {
+                       if (ao_lisp_chunk[i].size != size)
+                               ao_lisp_abort();
+                       return;
+               }
+               if (ao_lisp_chunk[i].old_addr > addr) {
+                       memmove(&ao_lisp_chunk[i+1],
+                               &ao_lisp_chunk[i],
+                               (AO_LISP_NCHUNK - (i+1)) * sizeof (struct ao_lisp_chunk));
+                       ao_lisp_chunk[i].size = 0;
+               }
+               if (ao_lisp_chunk[i].size == 0) {
+                       ao_lisp_chunk[i].old_addr = addr;
+                       ao_lisp_chunk[i].size = size;
+                       return;
+               }
+       }
+}
+
 /*
  * Walk all referenced objects calling functions on each one
  */
 
 static void
-walk(uint8_t *tag,
-     int (*visit_addr)(const struct ao_lisp_type *type, void **addr),
+walk(int (*visit_addr)(const struct ao_lisp_type *type, void **addr),
      int (*visit_poly)(ao_poly *p, uint8_t do_note_cons))
 {
        int i;
 
-       memset(tag, '\0', sizeof (ao_lisp_busy));
+       total_marked = 0;
+       ao_lisp_record_reset();
+       memset(ao_lisp_busy, '\0', sizeof (ao_lisp_busy));
        memset(ao_lisp_cons_note, '\0', sizeof (ao_lisp_cons_note));
        ao_lisp_cons_noted = 0;
        for (i = 0; i < AO_LISP_ROOT; i++) {
                if (ao_lisp_root[i].type) {
                        void **a = ao_lisp_root[i].addr, *v;
                        if (a && (v = *a)) {
-                               MDBG("root ptr %d\n", MDBG_OFFSET(v));
+                               MDBG_MOVE("root ptr %d\n", MDBG_OFFSET(v));
                                visit_addr(ao_lisp_root[i].type, a);
                        }
                } else {
                        ao_poly *a = (ao_poly *) ao_lisp_root[i].addr, p;
                        if (a && (p = *a)) {
-                               MDBG("root poly %d\n", MDBG_OFFSET(ao_lisp_ref(p)));
+                               MDBG_MOVE("root poly %d\n", MDBG_OFFSET(ao_lisp_ref(p)));
                                visit_poly(a, 0);
                        }
                }
@@ -211,33 +412,32 @@ walk(uint8_t *tag,
                for (i = 0; i < AO_LISP_POOL; i += 4) {
                        if (busy(ao_lisp_cons_last, i)) {
                                void *v = ao_lisp_pool + i;
-                               MDBG("root cons %d\n", MDBG_OFFSET(v));
+                               MDBG_MOVE("root cons %d\n", MDBG_OFFSET(v));
                                visit_addr(&ao_lisp_cons_type, &v);
                        }
                }
        }
 }
 
-static void    *move_old, *move_new;
-static int     move_size;
-
 #if MDBG_DUMP
 static void
 dump_busy(void)
 {
        int     i;
-       printf("busy:");
+       MDBG_MOVE("busy:");
        for (i = 0; i < ao_lisp_top; i += 4) {
-               if ((i & 0xff) == 0)
-                       printf("\n");
+               if ((i & 0xff) == 0) {
+                       MDBG_MORE("\n");
+                       MDBG_MOVE("%s", "");
+               }
                else if ((i & 0x1f) == 0)
-                       printf(" ");
+                       MDBG_MORE(" ");
                if (busy(ao_lisp_busy, i))
-                       putchar('*');
+                       MDBG_MORE("*");
                else
-                       putchar('-');
+                       MDBG_MORE("-");
        }
-       printf ("\n");
+       MDBG_MORE ("\n");
 }
 #define DUMP_BUSY()    dump_busy()
 #else
@@ -272,183 +472,241 @@ ao_lisp_collect(void)
 {
        int     i;
        int     top;
+#if DBG_MEM
+       int     loops = 0;
+       int     marked;
+       int     moved;
+       struct ao_lisp_record   *mark_record = NULL, *move_record = NULL;
+
+       ++dbg_collects;
+       MDBG_MOVE("collect %d\n", dbg_collects);
+       marked = moved = 0;
+#endif
+       chunk_low = 0;
+       top = 0;
+       for (;;) {
+               MDBG_DO(loops++);
+               MDBG_MOVE("move chunks from %d to %d\n", chunk_low, top);
+               /* Find the sizes of the first chunk of objects to move */
+               memset(ao_lisp_chunk, '\0', sizeof (ao_lisp_chunk));
+               walk(ao_lisp_mark_ref, ao_lisp_poly_mark_ref);
+#if DBG_MEM
+               marked = total_marked;
+
+               ao_lisp_record_free(mark_record);
+               mark_record = ao_lisp_record_save();
+               if (mark_record && move_record)
+                       ao_lisp_record_compare("mark", move_record, mark_record);
+
+               if (moved && moved != marked)
+                       ao_lisp_abort();
+#endif
 
-       MDBG("collect\n");
-       /* Mark */
-       walk(ao_lisp_busy, ao_lisp_mark_ref, ao_lisp_poly_mark_ref);
+               DUMP_BUSY();
 
-       DUMP_BUSY();
-       /* Compact */
-       MDBG("find first busy\n");
-       for (i = 0; i < ao_lisp_top; i += 4) {
-               if (!busy(ao_lisp_busy, i))
-                       break;
-       }
-       top = i;
-       while(i < ao_lisp_top) {
-               if (busy(ao_lisp_busy, i)) {
-                       MDBG("busy %d -> %d\n", i, top);
-                       MDBG_MOVE_IN();
-                       move_old = &ao_lisp_pool[i];
-                       move_new = &ao_lisp_pool[top];
-                       move_size = 0;
-                       walk(ao_lisp_moving, ao_lisp_move, ao_lisp_poly_move);
-                       MDBG("\tbusy size %d\n", move_size);
-                       if (move_size == 0)
+               /* Find the first moving object */
+               for (i = 0; i < AO_LISP_NCHUNK; i++) {
+                       uint16_t        size = ao_lisp_chunk[i].size;
+
+                       if (!size)
+                               break;
+
+                       if (ao_lisp_chunk[i].old_addr > top)
+                               break;
+                       if (ao_lisp_chunk[i].old_addr != top)
+                               ao_lisp_abort();
+
+                       top += size;
+                       MDBG_MOVE("chunk %d %d not moving\n",
+                                 ao_lisp_chunk[i].old_addr,
+                                 ao_lisp_chunk[i].size);
+                       chunk_low = ao_lisp_chunk[i].old_addr + size;
+               }
+
+               chunk_first = i;
+               /* Copy all of the objects */
+               for (; i < AO_LISP_NCHUNK; i++) {
+                       uint16_t        size = ao_lisp_chunk[i].size;
+
+                       if (!size)
+                               break;
+
+                       MDBG_MOVE("chunk %d %d -> %d\n",
+                                 ao_lisp_chunk[i].old_addr,
+                                 size,
+                                 top);
+                       ao_lisp_chunk[i].new_addr = top;
+                       memmove(&ao_lisp_pool[top],
+                               &ao_lisp_pool[ao_lisp_chunk[i].old_addr],
+                               size);
+                       MDBG_DO(clear_object(ao_lisp_busy, &ao_lisp_pool[ao_lisp_chunk[i].old_addr], size));
+                       MDBG_DO(mark_object(ao_lisp_busy, &ao_lisp_pool[top], size));
+                       top += size;
+                       chunk_low = ao_lisp_chunk[i].old_addr + size;
+               }
+
+               MDBG_MOVE("after moving objects, busy is now:\n");
+               DUMP_BUSY();
+               chunk_last = i;
+
+               if (chunk_first < chunk_last) {
+                       /* Relocate all references to the objects */
+                       walk(ao_lisp_move, ao_lisp_poly_move);
+
+#if DBG_MEM
+                       ao_lisp_record_free(move_record);
+                       move_record = ao_lisp_record_save();
+                       if (mark_record && move_record)
+                               ao_lisp_record_compare("move", mark_record, move_record);
+
+                       moved = total_marked;
+                       if (moved != marked)
                                ao_lisp_abort();
-                       clear_object(ao_lisp_busy, move_old, move_size);
-                       mark_object(ao_lisp_busy, move_new, move_size);
-                       if (busy_object(ao_lisp_cons_note, move_old)) {
-                               clear_object(ao_lisp_cons_note, move_old, move_size);
-                               mark_object(ao_lisp_cons_note, move_new, move_size);
-                       }
-                       i += move_size;
-                       top += move_size;
-#if MDBG_MOVE
-                       DUMP_BUSY();
 #endif
-                       MDBG_MOVE_OUT();
-               } else {
-                       i += 4;
                }
+
+               if (chunk_last != AO_LISP_NCHUNK)
+                       break;
        }
        ao_lisp_top = top;
+
+       MDBG_DO(memset(ao_lisp_chunk, '\0', sizeof (ao_lisp_chunk));
+               walk(ao_lisp_mark_ref, ao_lisp_poly_mark_ref));
+
+//     printf ("collect. top %d loops %d\n", top, loops);
 }
 
+/*
+ * Mark interfaces for objects
+ *
+ * Note a reference to memory and
+ * collect information about a few object sizes
+ * at a time
+ */
 
 int
-ao_lisp_mark(const struct ao_lisp_type *type, void *addr)
+ao_lisp_mark_memory(const struct ao_lisp_type *type, void *addr)
 {
-       if (!addr)
+       int size;
+       if (!AO_LISP_IS_POOL(addr))
                return 1;
-       MDBG_MOVE_IN();
+
+       size = ao_lisp_size(type, addr);
+       MDBG_MOVE("mark memory %d\n", MDBG_OFFSET(addr));
+       if (!mark_object(ao_lisp_busy, addr, size)) {
+               note_chunk(pool_offset(addr), size);
+               MDBG_DO(ao_lisp_record(type, addr, size));
+               return 0;
+       }
+       MDBG_MOVE("already marked\n");
+       return 1;
+}
+
+int
+ao_lisp_mark(const struct ao_lisp_type *type, void *addr)
+{
+       int ret;
        MDBG_MOVE("mark %d\n", MDBG_OFFSET(addr));
-       if (mark_object(ao_lisp_busy, addr, type->size(addr))) {
-               MDBG_MOVE("already marked\n");
-               MDBG_MOVE_OUT();
-               return 1;
+       MDBG_MOVE_IN();
+       ret = ao_lisp_mark_memory(type, addr);
+       if (!ret) {
+               MDBG_MOVE("mark recurse\n");
+               type->mark(addr);
        }
-       type->mark(addr);
        MDBG_MOVE_OUT();
-       return 0;
+       return ret;
 }
 
 int
 ao_lisp_poly_mark(ao_poly p, uint8_t do_note_cons)
 {
-       uint8_t type = ao_lisp_poly_type(p);
+       uint8_t type;
+       void    *addr;
 
        if (!p)
                return 1;
+
+       type = ao_lisp_poly_base_type(p);
+       addr = ao_lisp_ref(p);
+
+       if (!AO_LISP_IS_POOL(addr))
+               return 1;
+
        if (type == AO_LISP_CONS && do_note_cons) {
-               MDBG_MOVE("note cons %d\n", MDBG_OFFSET(ao_lisp_ref(p)));
                note_cons(ao_lisp_ref(p));
-               return 0;
-       } else {
-               const struct ao_lisp_type *lisp_type = ao_lisp_types[ao_lisp_poly_type(p)];
-               if (lisp_type)
-                       return ao_lisp_mark(lisp_type, ao_lisp_ref(p));
                return 1;
-       }
-}
+       } else {
+               const struct ao_lisp_type       *lisp_type;
 
-int
-ao_lisp_mark_memory(void *addr, int size)
-{
-       return mark_object(ao_lisp_busy, addr, size);
-}
+               if (type == AO_LISP_OTHER) {
+                       type = ao_lisp_other_type(ao_lisp_poly_other(p));
+                       if (type <= AO_LISP_OTHER || AO_LISP_NUM_TYPE <= type)
+                               ao_lisp_abort();
+               }
 
-/*
- * After the object has been moved, we have to reference it
- * in the new location. This is only relevant for ao_lisp_poly_move
- * as it needs to fetch the type byte from the object, which
- * may have been overwritten by the copy
- */
-void *
-ao_lisp_move_map(void *addr)
-{
-       if (addr == move_old) {
-               if (move_size != 0)
-                       return move_new;
+               lisp_type = ao_lisp_types[ao_lisp_poly_type(p)];
+               if (!lisp_type)
+                       return 1;
+               return ao_lisp_mark(lisp_type, ao_lisp_ref(p));
        }
-       return addr;
 }
 
 static void *
-check_move(void *addr, int size)
+move_map(void *addr)
 {
-       if (addr == move_old) {
-               MDBG_MOVE("mapping %d -> %d\n", MDBG_OFFSET(addr), MDBG_OFFSET(move_new));
-               if (move_size && move_size != ((size + 3) & ~3))
-                       ao_lisp_abort();
-
-               /* Only copy the object once, otherwise we may
-                * smash stuff
-                */
-               if (move_size == 0) {
-                       MDBG_MOVE("  copy %d\n", size);
-                       memmove(move_new, move_old, size);
-                       move_size = (size + 3) & ~3;
+       uint16_t        offset = pool_offset(addr);
+       int             i;
+
+       for (i = chunk_first; i < chunk_last; i++) {
+               if (ao_lisp_chunk[i].old_addr == offset) {
+                       MDBG_MOVE("move %d -> %d\n",
+                                 ao_lisp_chunk[i].old_addr,
+                                 ao_lisp_chunk[i].new_addr);
+                       return ao_lisp_pool + ao_lisp_chunk[i].new_addr;
                }
-               addr = move_new;
        }
        return addr;
 }
 
 int
-ao_lisp_move(const struct ao_lisp_type *type, void **ref)
+ao_lisp_move_memory(const struct ao_lisp_type *type, void **ref)
 {
        void            *addr = *ref;
-       uint8_t         *a = addr;
-       int             size = type->size(ao_lisp_move_map(addr));
+       int             size;
 
-       if (!addr)
+       if (!AO_LISP_IS_POOL(addr))
                return 1;
 
-#ifndef AO_LISP_MAKE_CONST
-       if (AO_LISP_IS_CONST(addr))
-               return 1;
-#endif
-       MDBG_MOVE("object %d\n", MDBG_OFFSET(addr));
-       if (!AO_LISP_IS_POOL(a))
-               ao_lisp_abort();
-       MDBG_MOVE_IN();
-       addr = check_move(addr, size);
-       if (addr != *ref)
+       MDBG_MOVE("move memory %d\n", MDBG_OFFSET(addr));
+       addr = move_map(addr);
+       size = ao_lisp_size(type, addr);
+       if (addr != *ref) {
+               MDBG_MOVE("update ref %d %d -> %d\n",
+                         AO_LISP_IS_POOL(ref) ? MDBG_OFFSET(ref) : -1,
+                         MDBG_OFFSET(*ref), MDBG_OFFSET(addr));
                *ref = addr;
-       if (mark_object(ao_lisp_moving, addr, size)) {
-               MDBG_MOVE("already moved\n");
-               MDBG_MOVE_OUT();
-               return 1;
        }
-       MDBG_MOVE_OUT();
-       MDBG_MOVE("recursing...\n");
-       MDBG_MOVE_IN();
-       type->move(addr);
-       MDBG_MOVE_OUT();
-       MDBG_MOVE("done %d\n", MDBG_OFFSET(addr));
-       return 0;
+       if (!mark_object(ao_lisp_busy, addr, size)) {
+               MDBG_DO(ao_lisp_record(type, addr, size));
+               return 0;
+       }
+       MDBG_MOVE("already moved\n");
+       return 1;
 }
 
 int
-ao_lisp_move_memory(void **ref, int size)
+ao_lisp_move(const struct ao_lisp_type *type, void **ref)
 {
-       void *addr = *ref;
-       if (!addr)
-               return 1;
-
-       MDBG_MOVE("memory %d\n", MDBG_OFFSET(addr));
+       int ret;
+       MDBG_MOVE("move object %d\n", MDBG_OFFSET(*ref));
        MDBG_MOVE_IN();
-       addr = check_move(addr, size);
-       if (addr != *ref)
-               *ref = addr;
-       if (mark_object(ao_lisp_moving, addr, size)) {
-               MDBG_MOVE("already moved\n");
-               MDBG_MOVE_OUT();
-               return 1;
+       ret = ao_lisp_move_memory(type, ref);
+       if (!ret) {
+               MDBG_MOVE("move recurse\n");
+               type->move(*ref);
        }
        MDBG_MOVE_OUT();
-       return 0;
+       return ret;
 }
 
 int
@@ -456,7 +714,6 @@ ao_lisp_poly_move(ao_poly *ref, uint8_t do_note_cons)
 {
        uint8_t                         type;
        ao_poly                         p = *ref;
-       const struct ao_lisp_type       *lisp_type;
        int                             ret;
        void                            *addr;
 
@@ -466,20 +723,24 @@ ao_lisp_poly_move(ao_poly *ref, uint8_t do_note_cons)
        type = ao_lisp_poly_base_type(p);
        addr = ao_lisp_ref(p);
 
-       if ((uint8_t *) addr < ao_lisp_pool || ao_lisp_pool + AO_LISP_POOL <= (uint8_t*) addr)
+       if (!AO_LISP_IS_POOL(addr))
                return 1;
 
        if (type == AO_LISP_CONS && do_note_cons) {
-               addr = check_move(addr, sizeof (struct ao_lisp_cons));
+//             addr = move_map(addr);
+               MDBG_DO(if (addr != move_map(addr)) MDBG_MOVE("noting cons at old addr %d instead of new addr %d\n", MDBG_OFFSET(addr), MDBG_OFFSET(move_map(addr))););
+
                note_cons(addr);
+               addr = move_map(addr);
                ret = 1;
        } else {
+               const struct ao_lisp_type       *lisp_type;
 
-               if (type == AO_LISP_OTHER)
-                       type = ao_lisp_other_type(ao_lisp_move_map(ao_lisp_poly_other(p)));
-
-               if (type >= AO_LISP_NUM_TYPE)
-                       ao_lisp_abort();
+               if (type == AO_LISP_OTHER) {
+                       type = ao_lisp_other_type(move_map(ao_lisp_poly_other(p)));
+                       if (type <= AO_LISP_OTHER || AO_LISP_NUM_TYPE <= type)
+                               ao_lisp_abort();
+               }
 
                lisp_type = ao_lisp_types[type];
                if (!lisp_type)
@@ -487,10 +748,11 @@ ao_lisp_poly_move(ao_poly *ref, uint8_t do_note_cons)
                ret = ao_lisp_move(lisp_type, &addr);
        }
 
+       /* Re-write the poly value */
        if (addr != ao_lisp_ref(p)) {
                ao_poly np = ao_lisp_poly(addr, p & AO_LISP_TYPE_MASK);
-               MDBG("poly %d moved %d -> %d\n",
-                    type, MDBG_OFFSET(ao_lisp_ref(p)), MDBG_OFFSET(ao_lisp_ref(np)));
+               MDBG_MOVE("poly %d moved %d -> %d\n",
+                         type, MDBG_OFFSET(ao_lisp_ref(p)), MDBG_OFFSET(ao_lisp_ref(np)));
                *ref = np;
        }
        return ret;
@@ -535,15 +797,28 @@ ao_lisp_poison(void)
 #define AO_LISP_POOL_CUR AO_LISP_POOL
 #endif
 
+#if DBG_MEM
+void
+ao_lisp_validate(void)
+{
+       chunk_low = 0;
+       memset(ao_lisp_chunk, '\0', sizeof (ao_lisp_chunk));
+       walk(ao_lisp_mark_ref, ao_lisp_poly_mark_ref);
+}
+
+int dbg_allocs;
+
+#endif
+
+
 void *
 ao_lisp_alloc(int size)
 {
        void    *addr;
 
-       size = ao_lisp_mem_round(size);
-#ifdef MDBG_COLLECT_ALWAYS
-       ao_lisp_collect();
-#endif
+       MDBG_DO(++dbg_allocs);
+       MDBG_DO(if (dbg_validate) ao_lisp_validate());
+       size = ao_lisp_size_round(size);
        if (ao_lisp_top + size > AO_LISP_POOL_CUR) {
 #ifdef MDBG_POOL
                if (AO_LISP_POOL_CUR < AO_LISP_POOL) {
@@ -573,37 +848,47 @@ ao_lisp_alloc(int size)
        return addr;
 }
 
-int
-ao_lisp_root_add(const struct ao_lisp_type *type, void *addr)
+void
+ao_lisp_cons_stash(int id, struct ao_lisp_cons *cons)
 {
-       int     i;
-       MDBG("add root type %p addr %p\n", type, addr);
-       for (i = 0; i < AO_LISP_ROOT; i++) {
-               if (!ao_lisp_root[i].addr) {
-                       ao_lisp_root[i].addr = addr;
-                       ao_lisp_root[i].type = type;
-                       return 1;
-               }
-       }
-       ao_lisp_abort();
-       return 0;
+       if (save_cons[id] != NULL)
+               ao_lisp_abort();
+       save_cons[id] = cons;
 }
 
-int
-ao_lisp_root_poly_add(ao_poly *p)
+struct ao_lisp_cons *
+ao_lisp_cons_fetch(int id)
 {
-       return ao_lisp_root_add(NULL, p);
+       struct ao_lisp_cons *cons = save_cons[id];
+       save_cons[id] = NULL;
+       return cons;
 }
 
 void
-ao_lisp_root_clear(void *addr)
+ao_lisp_string_stash(int id, char *string)
 {
-       int     i;
-       for (i = 0; i < AO_LISP_ROOT; i++) {
-               if (ao_lisp_root[i].addr == addr) {
-                       ao_lisp_root[i].addr = 0;
-                       ao_lisp_root[i].type = 0;
-                       break;
-               }
-       }
+       if (save_cons[id] != NULL)
+               ao_lisp_abort();
+       save_string[id] = string;
+}
+
+char *
+ao_lisp_string_fetch(int id)
+{
+       char *string = save_string[id];
+       save_string[id] = NULL;
+       return string;
+}
+void
+ao_lisp_poly_stash(int id, ao_poly poly)
+{
+       save_poly[id] = poly;
+}
+
+ao_poly
+ao_lisp_poly_fetch(int id)
+{
+       ao_poly poly = save_poly[id];
+       save_poly[id] = AO_LISP_NIL;
+       return poly;
 }
index 7a5751cef0315a7ec4662b3a49ab6ab0490a6592..b792c2f19a123f55138df67da8557d635dc5de99 100644 (file)
@@ -357,25 +357,25 @@ lex(void)
 }
 
 static int parse_token;
-static uint8_t                 been_here;
-static struct ao_lisp_cons     *read_cons;
-static struct ao_lisp_cons     *read_cons_tail;
-static struct ao_lisp_cons     *read_stack;
+
+struct ao_lisp_cons    *ao_lisp_read_cons;
+struct ao_lisp_cons    *ao_lisp_read_cons_tail;
+struct ao_lisp_cons    *ao_lisp_read_stack;
 
 static int
 push_read_stack(int cons, int in_quote)
 {
-       DBGI("push read stack %p %d\n", read_cons, in_quote);
+       DBGI("push read stack %p %d\n", ao_lisp_read_cons, in_quote);
        DBG_IN();
        if (cons) {
-               read_stack = ao_lisp_cons_cons(ao_lisp_cons_poly(read_cons),
+               ao_lisp_read_stack = ao_lisp_cons_cons(ao_lisp_cons_poly(ao_lisp_read_cons),
                                               ao_lisp_cons_cons(ao_lisp_int_poly(in_quote),
-                                                                read_stack));
-               if (!read_stack)
+                                                                ao_lisp_read_stack));
+               if (!ao_lisp_read_stack)
                        return 0;
        }
-       read_cons = NULL;
-       read_cons_tail = NULL;
+       ao_lisp_read_cons = NULL;
+       ao_lisp_read_cons_tail = NULL;
        return 1;
 }
 
@@ -384,21 +384,21 @@ pop_read_stack(int cons)
 {
        int     in_quote = 0;
        if (cons) {
-               read_cons = ao_lisp_poly_cons(read_stack->car);
-               read_stack = ao_lisp_poly_cons(read_stack->cdr);
-               in_quote = ao_lisp_poly_int(read_stack->car);
-               read_stack = ao_lisp_poly_cons(read_stack->cdr);
-               for (read_cons_tail = read_cons;
-                    read_cons_tail && read_cons_tail->cdr;
-                    read_cons_tail = ao_lisp_poly_cons(read_cons_tail->cdr))
+               ao_lisp_read_cons = ao_lisp_poly_cons(ao_lisp_read_stack->car);
+               ao_lisp_read_stack = ao_lisp_poly_cons(ao_lisp_read_stack->cdr);
+               in_quote = ao_lisp_poly_int(ao_lisp_read_stack->car);
+               ao_lisp_read_stack = ao_lisp_poly_cons(ao_lisp_read_stack->cdr);
+               for (ao_lisp_read_cons_tail = ao_lisp_read_cons;
+                    ao_lisp_read_cons_tail && ao_lisp_read_cons_tail->cdr;
+                    ao_lisp_read_cons_tail = ao_lisp_poly_cons(ao_lisp_read_cons_tail->cdr))
                        ;
        } else {
-               read_cons = 0;
-               read_cons_tail = 0;
-               read_stack = 0;
+               ao_lisp_read_cons = 0;
+               ao_lisp_read_cons_tail = 0;
+               ao_lisp_read_stack = 0;
        }
        DBG_OUT();
-       DBGI("pop read stack %p %d\n", read_cons, in_quote);
+       DBGI("pop read stack %p %d\n", ao_lisp_read_cons, in_quote);
        return in_quote;
 }
 
@@ -411,18 +411,12 @@ ao_lisp_read(void)
        int                     in_quote;
        ao_poly                 v;
 
-       if (!been_here) {
-               ao_lisp_root_add(&ao_lisp_cons_type, &read_cons);
-               ao_lisp_root_add(&ao_lisp_cons_type, &read_cons_tail);
-               ao_lisp_root_add(&ao_lisp_cons_type, &read_stack);
-               been_here = 1;
-       }
        parse_token = lex();
        DBGI("token %d (%s)\n", parse_token, token_string);
 
        cons = 0;
        in_quote = 0;
-       read_cons = read_cons_tail = read_stack = 0;
+       ao_lisp_read_cons = ao_lisp_read_cons_tail = ao_lisp_read_stack = 0;
        for (;;) {
                while (parse_token == OPEN) {
                        if (!push_read_stack(cons, in_quote))
@@ -469,7 +463,7 @@ ao_lisp_read(void)
                                v = AO_LISP_NIL;
                                break;
                        }
-                       v = ao_lisp_cons_poly(read_cons);
+                       v = ao_lisp_cons_poly(ao_lisp_read_cons);
                        --cons;
                        in_quote = pop_read_stack(cons);
                        break;
@@ -484,16 +478,16 @@ ao_lisp_read(void)
                        if (!read)
                                return AO_LISP_NIL;
 
-                       if (read_cons_tail)
-                               read_cons_tail->cdr = ao_lisp_cons_poly(read);
+                       if (ao_lisp_read_cons_tail)
+                               ao_lisp_read_cons_tail->cdr = ao_lisp_cons_poly(read);
                        else
-                               read_cons = read;
-                       read_cons_tail = read;
+                               ao_lisp_read_cons = read;
+                       ao_lisp_read_cons_tail = read;
 
-                       if (!in_quote || !read_cons->cdr)
+                       if (!in_quote || !ao_lisp_read_cons->cdr)
                                break;
 
-                       v = ao_lisp_cons_poly(read_cons);
+                       v = ao_lisp_cons_poly(ao_lisp_read_cons);
                        --cons;
                        in_quote = pop_read_stack(cons);
                }
index 9ee1a7dd0b0e595a2b5a431fe7bc3ca361b2b671..207d4f3be8eae13c90e6e4869a8327ba64f0793c 100644 (file)
@@ -38,23 +38,17 @@ const struct ao_lisp_type ao_lisp_string_type = {
        .mark = string_mark,
        .size = string_size,
        .move = string_move,
+       .name = "string",
 };
 
-char *
-ao_lisp_string_new(int len) {
-       char    *a = ao_lisp_alloc(len + 1);
-       if (!a)
-               return NULL;
-       a[len] = '\0';
-       return a;
-}
-
 char *
 ao_lisp_string_copy(char *a)
 {
        int     alen = strlen(a);
 
+       ao_lisp_string_stash(0, a);
        char    *r = ao_lisp_alloc(alen + 1);
+       a = ao_lisp_string_fetch(0);
        if (!r)
                return NULL;
        strcpy(r, a);
@@ -66,7 +60,12 @@ ao_lisp_string_cat(char *a, char *b)
 {
        int     alen = strlen(a);
        int     blen = strlen(b);
+
+       ao_lisp_string_stash(0, a);
+       ao_lisp_string_stash(1, b);
        char    *r = ao_lisp_alloc(alen + blen + 1);
+       a = ao_lisp_string_fetch(0);
+       b = ao_lisp_string_fetch(1);
        if (!r)
                return NULL;
        strcpy(r, a);
@@ -78,7 +77,9 @@ ao_poly
 ao_lisp_string_pack(struct ao_lisp_cons *cons)
 {
        int     len = ao_lisp_cons_length(cons);
+       ao_lisp_cons_stash(0, cons);
        char    *r = ao_lisp_alloc(len + 1);
+       cons = ao_lisp_cons_fetch(0);
        char    *s = r;
 
        while (cons) {
@@ -96,11 +97,17 @@ ao_lisp_string_unpack(char *a)
 {
        struct ao_lisp_cons     *cons = NULL, *tail = NULL;
        int                     c;
+       int                     i;
 
-       ao_lisp_root_add(&ao_lisp_cons_type, &cons);
-       ao_lisp_root_add(&ao_lisp_cons_type, &tail);
-       while ((c = *a++)) {
+       for (i = 0; (c = a[i]); i++) {
+               ao_lisp_cons_stash(0, cons);
+               ao_lisp_cons_stash(1, tail);
+               ao_lisp_string_stash(0, a);
                struct ao_lisp_cons     *n = ao_lisp_cons_cons(ao_lisp_int_poly(c), NULL);
+               cons = ao_lisp_cons_fetch(0);
+               tail = ao_lisp_cons_fetch(1);
+               a = ao_lisp_string_fetch(0);
+
                if (!n) {
                        cons = NULL;
                        break;
@@ -111,8 +118,6 @@ ao_lisp_string_unpack(char *a)
                        cons = n;
                tail = n;
        }
-       ao_lisp_root_clear(&cons);
-       ao_lisp_root_clear(&tail);
        return ao_lisp_cons_poly(cons);
 }