Re: [PATCH] 9p: add missing locking around taking dentry fid list

From: Dominique Martinet
Date: Thu May 23 2024 - 07:38:58 EST


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@xxxxxxxxxxxxx>

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.

--
Dominique Martinet | Asmadeus