Re: [PATCH] Merge the Sonics Silicon Backplane subsystem
From: Andrew Morton <hidden>
Date: 2007-07-27 20:15:19
Also in:
linux-wireless, lkml
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.