Re: [PATCH 0/3] Synaptics - fix lockdep warnings

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 9/14/06, Jiri Kosina <[email protected]> wrote:
On Thu, 14 Sep 2006, Dmitry Torokhov wrote:

> Not yet ;) Is there a way to hide the depth in the spinlock/mutex
> structure itself so that initialization code could do
> spin_lock_init_nested() and spare the rest of the code from that
> knowledge?

(shortened CC list a bit)

In fact I am not sure what you mean. On every lock and unlock operation,
in case of recursive locking (which our case is), you have to provide
class identifier, which is used to distinguish if the lock is of the same
instance, or a different one (deeper or higher in the locking hierarchy).
There is no way how spin_lock() or mutex_lock() can know this
"automatically", you always have to provide the nesting level from
outside, as it depends on the ordering hierarchy, which locking primitives
are totally unaware of.


Well, we do not really care about nestiness do we? What we trying to
achieve is to teach lockdep that 2 locks while appear as the lame lock
in fact are different and protect 2 different structures. Ideally
lockdep should track every lock individually (based for example on its
address) but that would be too taxing so we need to help it. In your
implementation you embed this data into structure/code using lock, but
this information could be instilled into the lock itself upon
initialization and spin_[un]lock() implementation could be taught to
use this data thus making specialized spin_[un]lock*_nested()
functions unnecessary.

--
Dmitry
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]
  Powered by Linux