Thread (28 messages) 28 messages, 4 authors, 2012-08-31

Re: [PATCH -v12 01/15] resources: Split out __allocate_resource()

From: Yinghai Lu <yinghai@kernel.org>
Date: 2012-06-26 20:33:43
Also in: linux-pci, lkml

On Tue, Jun 26, 2012 at 12:01 PM, Linus Torvalds
[off-list ref] wrote:
On Tue, Jun 26, 2012 at 11:53 AM, Yinghai Lu [off-list ref] wrote:
quoted
It will take bool lock, so we could use it in other functions that
hold the resource lock already.
This is too damn ugly.

These kinds of "conditionally take lock" things are always just bugs
waiting to happen. Don't do it.

Just make the rule be that the caller of the __allocate_resource
helper has to hold the lock. Sure, that means that you need to then
use split reallocate_resource() into a helper function (ie a static
__reallocate_resource() that needs to have the lock taken by the
caller too), but dammit, that's definitely the right thing to do
anyway.

These kinds of "bool lock" crap things have to die. They are *wrong*.
They are a sign of bad locking rules.
You are right, please check updated one.

Thanks

Yinghai

Attachments

Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help