Re: [PATCH 2/4] Convert epoll to a bitlock

From: Eric Dumazet
Date: Tue Feb 03 2009 - 17:16:34 EST


Andrew Morton a écrit :
> On Mon, 2 Feb 2009 11:20:09 -0700
> Jonathan Corbet <corbet@xxxxxxx> wrote:
>
>> Matt Mackall suggested converting epoll's ep_lock to a bitlock as a way of
>> saving space in struct file. This patch makes that change.
>
> hrm. bit_spin_lock() makes people upset (large penguiny people). iirc
> it doesn't have all the correct/well-understood memory/compiler
> ordering semantics which spinlocks have. And lockdep doesn't know about
> it.
>

In a previous attempt (2005), I suggested using a single global lock.

http://search.luky.org/linux-kernel.2005/msg50862.html

Probably an array of hashed spinlocks would be more than enough.



--
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/