Lockdep is less useful than it was
From: Matthew Wilcox
Date: Thu Dec 07 2017 - 17:38:33 EST
On Thu, Dec 07, 2017 at 11:06:34AM -0500, Theodore Ts'o wrote:
> The problem is that if it has too many false positives --- and it's
> gotten *way* worse with the completion callback "feature", people will
> just stop using Lockdep as being too annyoing and a waste of developer
> time when trying to figure what is a legitimate locking bug versus
> lockdep getting confused.
>
> <Rant>I can't even disable the new Lockdep feature which is throwing
> lots of new false positives --- it's just all or nothing.</Rant>
You *can* ... but it's way more hacking Kconfig than you ought to have
to do (which is a separate rant ...)
You need to get LOCKDEP_CROSSRELEASE off. I'd revert patches
e26f34a407aec9c65bce2bc0c838fabe4f051fc6 and
b483cf3bc249d7af706390efa63d6671e80d1c09
I think it was a mistake to force these on for everybody; they have a
much higher false-positive rate than the rest of lockdep, so as you say
forcing them on leads to fewer people using *any* of lockdep.
The bug you're hitting isn't Byungchul's fault; it's an annotation
problem. The same kind of annotation problem that we used to have with
dozens of other places in the kernel which are now fixed. If you didn't
have to hack Kconfig to get rid of this problem, you'd be happier, right?