Re: brlock_is_locked()?

[David S. Miller]

   From: Brad Chapman <[e-mail]>
   Date: Wed, 22 Aug 2001 11:17:55 -0700 (PDT)

   	At this section of the code, it doesn't really matter who in the
   netfilter/network stack has BR_NETPROTO_LOCK, just that we need to know if
   it's already locked by someone else, or unlocked for us to use. Is what I'm
   asking for physically possible?

As a reader, the BR_NETPROTO_LOCK nests.  So no deadlock.

If the situation involves a writer, you must make sure that locking
rules are well defined and abided by.  No matter what you do, if you
try to conditionalize locking by testing if the lock is held, another
SMP can always get in there after you check it and corrupt your data.
The lock is basically pointless if you start testing it's locked
state.

Basically, "fix your code" ;-)

It's often easy to do this.  If you have two paths, one which is going
to already have the lock and one which won't, just make two functions
like so:

void __func(void)
{
	do_stuff();
}

void func(void)
{
	lock();
	__func();
	unlock();
}

I don't see why this is such a big problem.

Later,
David S. Miller
[e-mail]