I see a memory leak of dclose in cifs_close. I hope that's the one you were referring to earlier? Also, we'll need to avoid cifs_close_all_deferred_files in the future. Maybe not for this patch. Other than that, looks good to me. Reviewed-by: Shyam Prasad N <sprasad@xxxxxxxxxxxxx> On Mon, Aug 9, 2021 at 3:54 PM Rohith Surabattula <rohiths.msft@xxxxxxxxx> wrote: > > Hi Steve, All, > > Have updated the patch slightly with null check to avoid kernel oops. > > Regards, > Rohith > > On Wed, Aug 4, 2021 at 11:13 PM Steve French <smfrench@xxxxxxxxx> wrote: > > > > Lightly updated version of Rohith's newer version of his deferred close fix (fixed minor checkpatch issue and added cc:stable #5.13 > > > > -- > > Thanks, > > > > Steve -- Regards, Shyam