On Tue, Jan 15, 2008 at 09:48:51AM -0500, J. Bruce Fields wrote: > On Mon, Jan 14, 2008 at 09:28:30PM -0700, Matthew Wilcox wrote: > > interruptible_sleep_on_locked() is just an open-coded > > wait_event_interruptible_timeout() with a few assumptions since we know > > we hold the BKL. locks_block_on_timeout() is only used in one place, so > > it's actually simpler to inline it into its caller. > > Makes sense, thanks. So the assumption we were depending on the BKL for > was that we could count on the wake-up not coming till after we block, > so we could skip a check ->fl_next that's normally needed to resolve the > usual sleeping-on-some-condition race? That's right. -- Intel are signing my paycheques ... these opinions are still mine "Bill, look, we understand that you're interested in selling us this operating system, but compare it to ours. We can't possibly take such a retrograde step." - To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html