![]() | |||||||||||||
Messages in this thread |
On Wed, 7 May 2008, Ingo Molnar wrote: > > it was removed by me in the course of this discussion: > > http://lkml.org/lkml/2008/1/2/58 > > the whole discussion started IIRC because !CONFIG_PREEMPT_BKL [the > spinlock version] was broken for a longer period of time (it crashed > trivially), because nobody apparently used it. Hmm. I've generally used PREEMPT_NONE, and always thought PREEMPT_BKL was the known-flaky one. The thread you point to also says that it's PREEMPT_BKL=y that was the problem (ie "I've seen 1s+ desktop latencies due to PREEMPT_BKL when I was still using reiserfs."), not the plain spinlock approach. But it would definitely be interesting to see the crash reports. And the help message always said "Say N if you are unsure." even if it ended up being marked 'y' by default at some point (and then in January was made first unconditional, and then removed entirely) Because in many ways, the non-preempt BKL is the *much* simpler case. I don't see why it would crash - it just turns the BKL into a trivial counting spinlock that can sleep. Linus | ||||||||||||
| Last update: 2008-05-07 19:27 [from the cache] ©2003-2008 | |||||||||||||