]> git.ipfire.org Git - thirdparty/kernel/linux.git/commit
eventfs: Use children field for rcu head and add memory barriers
authorSteven Rostedt <rostedt@goodmis.org>
Sat, 8 Aug 2026 13:42:15 +0000 (09:42 -0400)
committerSteven Rostedt <rostedt@goodmis.org>
Sat, 8 Aug 2026 14:36:05 +0000 (10:36 -0400)
commitf0ece16ffca7384787b692431961ce202907acf5
tree0f6054475864edce7bb89437448b32034dccadb4
parentfd73b691702170d37d66f4b0278530cea8ed419a
eventfs: Use children field for rcu head and add memory barriers

When an eventfs inode is freed, it sets ei->is_freed and then uses its
ei->list to add it to the srcu link list as the list field is a union with
the rcu list head. As the ei->list is used to iterate over an SRCU
protected list without taking the eventfs_mutex, there's nothing stopping
the iteration over that list to see the ei->rcu instead of the ei->list
and it will read a corrupt target.

To fix this, change the union of the rcu list head with the children list.
On freeing the eventfs inode, set the is_free and execute a smp_wmb()
before adding the eventfs inode to the SRCU list.

On iteration of the ei->children list, at the start, execute a smp_rmb()
and then read the is_freed of the ei to see if the children list is still
valid. If is_freed is set, then the ei_child read is not valid and the
loop should exit immediately.

Cc: stable@vger.kernel.org
Link: https://patch.msgid.link/20260808094215.4252430d@robin
Fixes: 704f960dbee2f ("eventfs: Read ei->entries before ei->children in eventfs_iterate()")
Reported-by: Sashiko <sashiko-bot@kernel.org>
Closes: https://sashiko.dev/#/patchset/20260806022719.375354-1-shuangpeng.kernel%40gmail.com
Reviewed-by: Masami Hiramatsu (Google) <mhiramat@kernel.org>
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
fs/tracefs/event_inode.c
fs/tracefs/internal.h