Don't kill fossil or venti even if they are hogs! Move a comment about "no swap configured" to swap.c because it's not always correct(killbig() is called by ^T^Tk). Notes: Thu Jan 19 11:35:35 EST 2006 rsc The extra print in swap.c is unnecessary, since it will print "no physical memory" right after calling killbig. I'm not happy with the proliferation of magic process names. Instead, why not check if(p->procmode&0222). The startup script can then chmod -w /proc/importantguys/ctl. Reference: /n/sources/patch/sorry/killbig-nits Date: Thu Jan 19 05:44:47 CET 2006 Reviewed-by: rsc --- /sys/src/9/port/proc.c Thu Jan 19 05:40:56 2006 +++ /sys/src/9/port/proc.c Thu Jan 19 05:40:50 2006 @@ -1474,7 +1474,7 @@ if(s != 0) l += s->top - s->base; } - if(l > max && strcmp(p->text, "kfs") != 0){ + if(l > max && strcmp(p->text, "kfs") != 0 && strcmp(p->text, "fossil") != 0 && strcmp(p->text, "venti") != 0) { kp = p; max = l; } @@ -1487,7 +1487,7 @@ qunlock(&s->lk); } } - print("%lud: %s killed because no swap configured\n", kp->pid, kp->text); + print("%lud: %s killed by killbig()\n", kp->pid, kp->text); } /* --- /sys/src/9/port/swap.c Thu Jan 19 05:41:11 2006 +++ /sys/src/9/port/swap.c Thu Jan 19 05:41:08 2006 @@ -162,8 +162,10 @@ else { if(!cpuserver) freebroken(); /* can use the memory */ - else + else { + print("no swap configured, calling killbig()"); killbig(); + } /* Emulate the old system if no swap channel */ print("no physical memory\n");