Christian Schoenebeck wrote on Thu, May 23, 2024 at 12:05:44PM +0200:
I really think it's safe, but I do agree that it's hard to read, happy to move the `h = &dentry->d_fsdata` inside the lock if you prefer -- it compiles to the same code for me (x86_64/gcc 13.2.0)
No need, you can add my RB. Thanks for the clarification!
Reviewed-by: Christian Schoenebeck linux_oss@crudebyte.com
Thanks! I've fixed the typo in the commit message and queued it up in -next, will send this patch and the trace uninit fix to Linus early next week.