Re: [Openipmi-developer] [PATCH] ipmi: fix suspicious RCU usage warning
Brought to you by:
cminyard
|
From: Breno L. <le...@de...> - 2025-03-17 09:33:51
|
On Wed, Mar 12, 2025 at 01:19:32PM -0400, Rik van Riel wrote: > On recent kernels this warning fires: > > drivers/char/ipmi/ipmi_msghandler.c:1238 RCU-list traversed in non-reader section!! > > This looks like a fairly simple lockdep trigger, where > list_for_each_entry_rcu and list_for_each_entry_srcu are > functionally identical, but the lockdep annotation in > the former has an extra check. > > That extra check is whether the RCU read lock is held, > which is not true when the code uses srcu_read_lock. > > Get rid of the warning by using the properly annotated > list traversal macro. Thanks for looking at this one. There was a discussion about this issue a few years ago, with a different approach that never landed upstream. https://lore.kernel.org/all/202...@mi.../#r |