On Sun, Jun 30, 2024 at 06:08:21PM GMT, Waiman Long wrote:
On 6/30/24 01:10, Kent Overstreet wrote:novalidate just switches off checking of lock ordering, but the fact
Add a new helper to disable lockdep tracking entirely for a given class.Could you explain a bit more what the current novalidate_class is lacking
This is needed for bcachefs, which takes too many btree node locks for
lockdep to track. Instead, we have a single lockdep_map for "btree_trans
has any btree nodes locked", which makes more since given that we have
centralized lock management and a cycle detector.
WRT to the bcachefs lock? Is it excessive performance overhead or some bogus
lockdep warning?
that the locks are held is still tracked.
bcachefs takes more btree node locks than lockdep can track, so I'm
switching to a single lockdep map for "this btree_trans has any btree
nodes locked" instead.