On Sun, Feb 03, 2008 at 04:21:02PM +0100, Peter Zijlstra wrote: > On Fri, 2008-02-01 at 08:21 -0700, Dale Farnsworth wrote: > > Add each lock class to the all_lock_classes list when it is > > first registered. > > > > Previously, lock classes were added to all_lock_classes when > > the lock class was first used. > > > Since one of the uses of the list is to find unused locks, this didn't > > work well. > > You mean in lockdep_proc.c to generate the statistics?
Yes, that's where I noticed it. On my platform, there was one unused lock which was missed. > > Another potential issue might be count_matching_names() generating a > wrong class version. I haven't looked into that issue yet. -Dale -- 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/
