1index: 2 SEG_NULL, SEG_ASM not in the index 3 4mmu.h: 5 funny \ alignment in mmu.h 6 7main.c: 8 should use static int bcpu to avoid having live 9 stack variable across the asm volatile that 10 changes %esp and %ebp. 11 12 cpus[0] -> cpus[bcpu] 13 14init.c: 15 should while(wait() >= 0); 16 not just wait(); 17 18proc.c: 19 comment at top of scheduler() should say 20 via longjmp back to the scheduler 21 not 22 via longjmp back to the top of scheduler 23 24 sched should panic if state == RUNNING. 25 26 forkret comment should say "Return" not "return" 27 28 as a consequence of the implementation of proc_kill, 29 any loop calling sleep should check for p->killed 30 and be able to break out with an error return. 31 32 can swap procdump up after proc_kill 33 and then have proc_exit and proc_wait on same sheet 34 35 proc_exit should wake up proc[1] if any 36 reparenting was done. 37 38 sched -> switch2scheduler? or just switch? 39 40 factor out switching and scheduling code from process code 41 42trap.c 43 if spin locks are ever held without cli, 44 trap should call yield() only if nlock == 0 45 46main.c, proc.c, trap.c: 47 use a constant for 3 (USER). 48 49ide.c: synchronous disk write -> polling disk write. search for 50 (a)synchronous; xv6 doesn't have asynchronous writes. 51 52fs.c: split all name operations off in name.c? (starting with namei but move 53 wdir keep in fs.c) 54 55 unlink . and .. should be disallowed 56 57ls.c: apply tim's patch 58 59