Re: [patch] remove the BKL (Big Kernel Lock), this time for real

From: William Lee Irwin III
Date: Fri Sep 17 2004 - 09:27:08 EST


On Fri, Sep 17, 2004 at 06:47:37AM -0700, William Lee Irwin III wrote:
>> as the BKL is acquired before spinlocks in all instances. There are

On Fri, Sep 17, 2004 at 03:56:57PM +0200, Andrea Arcangeli wrote:
> that reinforces my argument, that's another case that can break with
> this patch.
> I don't think it has a chance to fix any bug, if something it will
> trigger some deadlock or race condition, but OTOH I agree it should work
> fine (all code I can recall by memory takes the BKL before any inner
> spinlock too, and I don't think we left anything that depends on
> smp_processor_id()), but again this is the kind of change that requires
> some testing and cannot be shipped by default in a stable tree, it
> requires config option at the very least.

But the corner cases I outlined don't exist; the case that does exist
is where the BKL is dropped by mistake by e.g. a call to a function
that may sleep. c.f. fs/fcntl.c:setfl().


-- wli
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/