Exercise: locking

There is no class today. You don’t need to turn in anything for this exercise. Do try them for yourself though!

Double acquires

Make sure you understand what would happen if the xv6 kernel executed the following code snippet:

struct spinlock lk;
initlock(&lk, "test lock");
acquire(&lk);
acquire(&lk);

Feel free to use QEMU to find out; acquire is in spinlock.c.

Race in ide.c

An acquire ensures interrupts are off on the local processor using the cli instruction (via pushcli()), and interrupts remain off until the release of the last lock held by that processor (at which point they are enabled using sti).

Let’s see what happens if we turn on interrupts while holding the ide lock. In iderw in ide.c, add a call to sti() after the acquire(), and a call to cli() just before the release(). Rebuild the kernel and boot it in QEMU. Chances are the kernel will panic soon after boot; try booting QEMU a few times if it doesn’t. Try running QEMU on Linux.

Make sure you understand why the kernel panicked. You may find it useful to look up the stack trace (the sequence of %eip values printed by panic) in the kernel.asm listing.

Race in file.c

Remove the sti() and cli() you added, rebuild the kernel, and make sure it works again.

Now let’s see what happens if we turn on interrupts while holding the ftable.lock. This lock protects the table of file descriptors, which the kernel modifies when an application opens or closes a file. In filealloc() in file.c, add a call to sti() after the call to acquire(), and a cli() just before each of the release()es. You will also need to add #include "x86.h" at the top of the file after the other #include lines. Rebuild the kernel and boot it in QEMU. It will not panic.

Why didn’t the kernel panic? Why do ftable.lock and ide_lock have different behavior in this respect?

You do not need to understand anything about the details of the IDE hardware to answer this question, but you may find it helpful to look at which functions acquire each lock, and then at when those functions get called.

There is a very small but non-zero chance that the kernel will panic with the extra sti() in filealloc(). If the kernel does panic, make doubly sure that you removed the sti() call from iderw.

xv6 lock implementation

Why does release() clear lk->pcs[0] and lk->cpu before clearing lk->locked? Why not wait until after?