[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-ID: <CANrsvRPQcWz-p_3TYfNf+Waek3bcNNPniXhFzyyS=7qbCqzGyg@mail.gmail.com>
Date: Wed, 13 Dec 2017 15:24:29 +0900
From: Byungchul Park <max.byungchul.park@...il.com>
To: Thomas Gleixner <tglx@...utronix.de>,
Peter Zijlstra <peterz@...radead.org>,
Ingo Molnar <mingo@...nel.org>, david@...morbit.com,
tytso@....edu, willy@...radead.org,
Linus Torvalds <torvalds@...ux-foundation.org>,
Amir Goldstein <amir73il@...il.com>, byungchul.park@....com,
linux-kernel@...r.kernel.org, linux-mm@...ck.org,
linux-block@...r.kernel.org, linux-fsdevel@...r.kernel.org,
oleg@...hat.com
Subject: About the try to remove cross-release feature entirely by Ingo
Lockdep works, based on the following:
(1) Classifying locks properly
(2) Checking relationship between the classes
If (1) is not good or (2) is not good, then we
might get false positives.
For (1), we don't have to classify locks 100%
properly but need as enough as lockdep works.
For (2), we should have a mechanism w/o
logical defects.
Cross-release added an additional capacity to
(2) and requires (1) to get more precisely classified.
Since the current classification level is too low for
cross-release to work, false positives are being
reported frequently with enabling cross-release.
Yes. It's a obvious problem. It needs to be off by
default until the classification is done by the level
that cross-release requires.
But, the logic (2) is valid and logically true. Please
keep the code, mechanism, and logic.
--
Thanks,
Byungchul
Powered by blists - more mailing lists