On Tue, May 6, 2025 at 7:55 AM Vlastimil Babka <vbabka@xxxxxxx> wrote: > > On 5/6/25 14:56, Vlastimil Babka wrote: > > On 5/1/25 05:27, Alexei Starovoitov wrote: > >> @@ -81,7 +84,7 @@ do { \ > >> local_lock_debug_init(lock); \ > >> } while (0) > >> > >> -#define __local_trylock_init(lock) __local_lock_init(lock.llock) > >> +#define __local_trylock_init(lock) __local_lock_init((local_lock_t *)lock) > > > > This cast seems unnecessary. Better not hide mistakes when using the > > local_trylock_init() macro. > > Nevermind, tested the wrong config, it's necessary. Sigh. Yep. lockdep assumption is deep. I see no other way of doing it.