2006-06-22 15:51:57 +00:00
|
|
|
bochs 2.2.6:
|
|
|
|
./configure --enable-smp --enable-disasm --enable-debugger --enable-all-optimizations --enable-4meg-pages --enable-global-pages --enable-pae --disable-reset-on-triple-fault
|
Changes to allow use of native x86 ELF compilers, which on my
Linux 2.4 box using gcc 3.4.6 don't seem to follow the same
conventions as the i386-jos-elf-gcc compilers.
Can run make 'TOOLPREFIX=' or edit the Makefile.
curproc[cpu()] can now be NULL, indicating that no proc is running.
This seemed safer to me than having curproc[0] and curproc[1]
both pointing at proc[0] potentially.
The old implementation of swtch depended on the stack frame layout
used inside swtch being okay to return from on the other stack
(exactly the V6 you are not expected to understand this).
It also could be called in two contexts: at boot time, to schedule
the very first process, and later, on behalf of a process, to sleep
or schedule some other process.
I split this into two functions: scheduler and swtch.
The scheduler is now a separate never-returning function, invoked
by each cpu once set up. The scheduler looks like:
scheduler() {
setjmp(cpu.context);
pick proc to schedule
blah blah blah
longjmp(proc.context)
}
The new swtch is intended to be called only when curproc[cpu()] is not NULL,
that is, only on behalf of a user proc. It does:
swtch() {
if(setjmp(proc.context) == 0)
longjmp(cpu.context)
}
to save the current proc context and then jump over to the scheduler,
running on the cpu stack.
Similarly the system call stubs are now in assembly in usys.S to avoid
needing to know the details of stack frame layout used by the compiler.
Also various changes in the debugging prints.
2006-07-11 01:07:40 +00:00
|
|
|
bochs CVS after 2.2.6:
|
|
|
|
./configure --enable-smp --enable-disasm --enable-debugger --enable-all-optimizations --enable-4meg-pages --enable-global-pages --enable-pae
|
2006-06-22 15:51:57 +00:00
|
|
|
|
2006-06-12 15:22:12 +00:00
|
|
|
bootmain.c doesn't work right if the ELF sections aren't
|
|
|
|
sector-aligned. so you can't use ld -N. and the sections may also need
|
|
|
|
to be non-zero length, only really matters for tiny "kernels".
|
|
|
|
|
|
|
|
kernel loaded at 1 megabyte. stack same place that bootasm.S left it.
|
|
|
|
|
|
|
|
kinit() should find real mem size
|
|
|
|
and rescue useable memory below 1 meg
|
|
|
|
|
|
|
|
no paging, no use of page table hardware, just segments
|
|
|
|
|
|
|
|
no user area: no magic kernel stack mapping
|
|
|
|
so no copying of kernel stack during fork
|
|
|
|
though there is a kernel stack page for each process
|
|
|
|
|
|
|
|
no kernel malloc(), just kalloc() for user core
|
|
|
|
|
|
|
|
user pointers aren't valid in the kernel
|
|
|
|
|
|
|
|
setting up first process
|
|
|
|
we do want a process zero, as template
|
|
|
|
but not runnable
|
|
|
|
just set up return-from-trap frame on new kernel stack
|
|
|
|
fake user program that calls exec
|
|
|
|
|
|
|
|
map text read-only?
|
|
|
|
shared text?
|
|
|
|
|
|
|
|
what's on the stack during a trap or sys call?
|
|
|
|
PUSHA before scheduler switch? for callee-saved registers.
|
|
|
|
segment contents?
|
|
|
|
what does iret need to get out of the kernel?
|
|
|
|
how does INT know what kernel stack to use?
|
|
|
|
|
|
|
|
are interrupts turned on in the kernel? probably.
|
|
|
|
|
|
|
|
per-cpu curproc
|
|
|
|
one tss per process, or one per cpu?
|
|
|
|
one segment array per cpu, or per process?
|
|
|
|
|
|
|
|
pass curproc explicitly, or implicit from cpu #?
|
|
|
|
e.g. argument to newproc()?
|
2006-06-15 16:02:20 +00:00
|
|
|
hmm, you need a global curproc[cpu] for trap() &c
|
2006-06-12 15:22:12 +00:00
|
|
|
|
|
|
|
test stack expansion
|
|
|
|
test running out of memory, process slots
|
|
|
|
|
|
|
|
we can't really use a separate stack segment, since stack addresses
|
|
|
|
need to work correctly as ordinary pointers. the same may be true of
|
|
|
|
data vs text. how can we have a gap between data and stack, so that
|
|
|
|
both can grow, without committing 4GB of physical memory? does this
|
|
|
|
mean we need paging?
|
|
|
|
|
|
|
|
what's the simplest way to add the paging we need?
|
|
|
|
one page table, re-write it each time we leave the kernel?
|
|
|
|
page table per process?
|
|
|
|
probably need to use 0-0xffffffff segments, so that
|
|
|
|
both data and stack pointers always work
|
|
|
|
so is it now worth it to make a process's phys mem contiguous?
|
|
|
|
or could use segment limits and 4 meg pages?
|
|
|
|
but limits would prevent using stack pointers as data pointers
|
|
|
|
how to write-protect text? not important?
|
|
|
|
|
|
|
|
perhaps have fixed-size stack, put it in the data segment?
|
|
|
|
|
|
|
|
oops, if kernel stack is in contiguous user phys mem, then moving
|
|
|
|
users' memory (e.g. to expand it) will wreck any pointers into the
|
|
|
|
kernel stack.
|
2006-06-13 15:50:06 +00:00
|
|
|
|
|
|
|
do we need to set fs and gs? so user processes can't abuse them?
|
|
|
|
|
|
|
|
setupsegs() may modify current segment table, is that legal?
|
|
|
|
|
|
|
|
trap() ought to lgdt on return, since currently only done in swtch()
|
|
|
|
|
|
|
|
protect hardware interrupt vectors from user INT instructions?
|
2006-06-13 22:08:20 +00:00
|
|
|
|
2006-06-27 14:35:53 +00:00
|
|
|
test out-of-fd cases for creating pipe.
|
2006-07-11 17:39:45 +00:00
|
|
|
test pipe reader closes then write
|
|
|
|
test two readers, two writers.
|
|
|
|
test children being inherited by grandparent &c
|
|
|
|
|
2006-07-12 01:48:35 +00:00
|
|
|
some sleep()s should be interruptible by kill()
|
2006-07-11 17:39:45 +00:00
|
|
|
|
|
|
|
cli/sti in acquire/release should nest!
|
|
|
|
in case you acquire two locks
|
2006-07-12 01:48:35 +00:00
|
|
|
|
|
|
|
what would need fixing if we got rid of kernel_lock?
|
|
|
|
console output
|
|
|
|
proc_exit() needs lock on proc *array* to deallocate
|
|
|
|
kill() needs lock on proc *array*
|
|
|
|
allocator's free list
|
|
|
|
global fd table (really free-ness)
|
|
|
|
sys_close() on fd table
|
|
|
|
fork on proc list, also next pid
|
|
|
|
hold lock until public slots in proc struct initialized
|
|
|
|
|
|
|
|
locks
|
|
|
|
init_lock
|
|
|
|
sequences CPU startup
|
|
|
|
proc_table_lock
|
|
|
|
also protects next_pid
|
|
|
|
per-fd lock *just* protects count read-modify-write
|
|
|
|
also maybe freeness?
|
|
|
|
memory allocator
|
|
|
|
printf
|
|
|
|
|
|
|
|
wakeup needs proc_table_lock
|
|
|
|
so we need recursive locks?
|
|
|
|
or you must hold the lock to call wakeup?
|
|
|
|
|
|
|
|
in general, the table locks protect both free-ness and
|
|
|
|
public variables of table elements
|
|
|
|
in many cases you can use table elements w/o a lock
|
|
|
|
e.g. if you are the process, or you are using an fd
|
|
|
|
|
|
|
|
lock code shouldn't call cprintf...
|
2006-07-12 15:35:33 +00:00
|
|
|
|
|
|
|
nasty hack to allow locks before first process,
|
|
|
|
and to allow them in interrupts when curproc may be zero
|
|
|
|
|
|
|
|
race between release and sleep in sys_wait()
|
|
|
|
race between sys_exit waking up parent and setting state=ZOMBIE
|
2006-07-15 12:03:57 +00:00
|
|
|
race in pipe code when full/empty
|
|
|
|
|
|
|
|
lock order
|
|
|
|
per-pipe lock
|
|
|
|
proc_table_lock fd_table_lock kalloc_lock
|
|
|
|
console_lock
|
|
|
|
|
|
|
|
condition variable + mutex that protects it
|
|
|
|
proc * (for wait()), proc_table_lock
|
|
|
|
pipe structure, pipe lock
|
|
|
|
|
|
|
|
systematic way to test sleep races?
|
|
|
|
print something at the start of sleep?
|
|
|
|
|
|
|
|
do you have to be holding the mutex in order to call wakeup()?
|
|
|
|
|
|
|
|
device interrupts don't clear FL_IF
|
|
|
|
so a recursive timer interrupt is possible
|
|
|
|
|
2006-07-21 22:10:40 +00:00
|
|
|
what does inode->busy mean?
|
|
|
|
might be held across disk reads
|
|
|
|
no-one is allowed to do anything to the inode
|
|
|
|
protected by inode_table_lock
|
|
|
|
inode->count counts in-memory pointers to the struct
|
|
|
|
prevents inode[] element from being re-used
|
|
|
|
protected by inode_table_lock
|
|
|
|
|
|
|
|
blocks and inodes have ad-hoc sleep-locks
|
|
|
|
provide a single mechanism?
|
|
|
|
|
|
|
|
need to lock bufs in bio between bread and brelse
|
2006-07-15 12:03:57 +00:00
|
|
|
|
2006-07-21 22:10:40 +00:00
|
|
|
test 14-character file names
|
|
|
|
and file arguments longer than 14
|
|
|
|
and directories longer than one sector
|
2006-07-28 22:33:07 +00:00
|
|
|
|
|
|
|
kalloc() can return 0; do callers handle this right?
|
2006-08-08 19:58:06 +00:00
|
|
|
|
|
|
|
why directing interrupts to cpu 1 causes trouble
|
|
|
|
cpu 1 turns on interrupts with no tss!
|
|
|
|
and perhaps a stale gdt (from boot)
|
|
|
|
since it has never run a process, never called setupsegs()
|
|
|
|
but does cpu really need the tss?
|
|
|
|
not switching stacks
|
|
|
|
fake process per cpu, just for tss?
|
|
|
|
seems like a waste
|
|
|
|
move tss to cpu[]?
|
|
|
|
but tss points to per-process kernel stack
|
|
|
|
would also give us a gdt
|
|
|
|
OOPS that wasn't the problem
|
|
|
|
|
|
|
|
wait for other cpu to finish starting before enabling interrupts?
|
|
|
|
some kind of crash in ide_init ioapic_enable cprintf
|
|
|
|
move ide_init before mp_start?
|
|
|
|
didn't do any good
|
|
|
|
maybe cpu0 taking ide interrupt, cpu1 getting a nested lock error
|
|
|
|
|
|
|
|
cprintfs are screwed up if locking is off
|
|
|
|
often loops forever
|
|
|
|
hah, just use lpt alone
|
|
|
|
|
|
|
|
looks like cpu0 took the ide interrupt and was the last to hold
|
|
|
|
the lock, but cpu1 thinks it is nested
|
|
|
|
cpu0 is in load_icode / printf / cons_putc
|
|
|
|
probably b/c cpu1 cleared use_console_lock
|
|
|
|
cpu1 is in scheduler() / printf / acquire
|
|
|
|
|
|
|
|
1: init timer
|
|
|
|
0: init timer
|
|
|
|
cpu 1 initial nlock 1
|
|
|
|
ne0s:t iidd el_occnkt rc
|
|
|
|
onsole cpu 1 old caller stack 1001A5 10071D 104DFF 1049FE
|
|
|
|
panic: acquire
|
|
|
|
^CNext at t=33002418
|
|
|
|
(0) [0x00100091] 0008:0x00100091 (unk. ctxt): jmp .+0xfffffffe ; ebfe
|
|
|
|
(1) [0x00100332] 0008:0x00100332 (unk. ctxt): jmp .+0xfffffffe
|
|
|
|
|
|
|
|
why is output interleaved even before panic?
|
|
|
|
|
|
|
|
does release turn on interrupts even inside an interrupt handler?
|
|
|
|
|
|
|
|
overflowing cpu[] stack?
|
|
|
|
probably not, change from 512 to 4096 didn't do anything
|
|
|
|
|
|
|
|
|
|
|
|
1: init timer
|
|
|
|
0: init timer
|
|
|
|
cnpeus te11 linnitki aclo nnoolleek cp1u
|
|
|
|
ss oarltd sccahleldeul esrt aocnk cpu 0111 Ej6 buf1 01A3140 C5118
|
|
|
|
0
|
|
|
|
la anic1::7 0a0c0 uuirr e
|
|
|
|
^CNext at t=31691050
|
|
|
|
(0) [0x00100373] 0008:0x00100373 (unk. ctxt): jmp .+0xfffffffe ; ebfe
|
|
|
|
(1) [0x00100091] 0008:0x00100091 (unk. ctxt): jmp .+0xfffffffe ; ebfe
|
|
|
|
|
|
|
|
cpu0:
|
|
|
|
|
|
|
|
0: init timer
|
|
|
|
nested lock console cpu 0 old caller stack 1001e6 101a34 1 0
|
|
|
|
(that's mpmain)
|
|
|
|
panic: acquire
|
|
|
|
|
|
|
|
cpu1:
|
|
|
|
|
|
|
|
1: init timer
|
|
|
|
cpu 1 initial nlock 1
|
|
|
|
start scheduler on cpu 1 jmpbuf ...
|
|
|
|
la 107000 lr ...
|
|
|
|
that is, nlock != 0
|
|
|
|
|
|
|
|
maybe a race; acquire does
|
|
|
|
locked = 1
|
|
|
|
cpu = cpu()
|
|
|
|
what if another acquire calls holding w/ locked = 1 but
|
|
|
|
before cpu is set?
|