Skip to content

Commit

Permalink
a few other nits
Browse files Browse the repository at this point in the history
  • Loading branch information
kaashoek committed Sep 17, 2006
1 parent b438eff commit f8ac639
Showing 1 changed file with 12 additions and 2 deletions.
14 changes: 12 additions & 2 deletions BUGS
Original file line number Diff line number Diff line change
@@ -1,11 +1,16 @@
missing bootmain.c from printout
funny \ alignment in mmu.h
runoff.list:
bootmain.c missing

mmu.h:
funny \ alignment in mmu.h

main.c:
should use static int bcpu to avoid having live
stack variable across the asm volatile that
changes %esp and %ebp.

cpus[0] -> cpus[bcpu]

init.c:
should while(wait() >= 0);
not just wait();
Expand All @@ -30,7 +35,12 @@ proc.c:
proc_exit should wake up proc[1] if any
reparenting was done.

sched -> switch2scheduler? or just switch?

trap.c
if spin locks are ever held without cli,
trap should call yield() only if nlock == 0

main.c, proc.c, trap.c:
use a constant for 3 (USER).

0 comments on commit f8ac639

Please sign in to comment.