Thread (14 messages) 14 messages, 5 authors, 2007-08-02

Re: [PATCH] Merge the Sonics Silicon Backplane subsystem

From: Andrew Morton <hidden>
Date: 2007-07-27 20:15:19
Also in: linux-wireless, lkml

On Fri, 27 Jul 2007 21:43:59 +0200
Michael Buesch [off-list ref] wrote:
quoted
Sure, but why is the locking interruptible rather than plain old
mutex_lock()?
Hm, well. We hold this mutex for several seconds, as writing takes
this long. So I simply thought it was worth allowing the waiter
to interrupt here. If you say that's not an issue, I'll be happy
to use mutex_lock() and reduce code complexity in this area.
So..  is that what the _interruptible() is for?  To allow an impatient user to ^c
a read?

If so, that sounds reasonable.  It's worth a comment explaining these decisions
to future readers, because it is hard to work out this sort of thinking just
from the bare C code.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help