p_top must be set in process.c as well for bank16k

I'm not exactly sure, but I think I found a bug in the bank16k
implementation.  When ptab_alloc() calls pagemap_alloc(), p_top is not
set yet.  pagemap_alloc, then proceeds to assign only one bank to the
new processes.  Assuming normal copy forking, should p_top be set to
the parent's p_top *before* calling pagemap_alloc() ?

This fix seems to make my coco3 UDATA mods start working

diff --git a/Kernel/process.c b/Kernel/process.c
index b9a4670..0020d74 100644 (file)
--- a/Kernel/process.c
+++ b/Kernel/process.c
@@ -272,6 +272,7 @@ ptptr ptab_alloc(void)
                                    && p->p_pid == nextpid)
                                        newp->p_pid = 0;        /* try again */
                }
+               newp->p_top = udata.u_top;
                if (pagemap_alloc(newp) == 0) {
                        newp->p_status = P_FORKING;
                        nproc++;