[PATCH 2/2] smb: client: retry compound request without reusing lease
Paulo Alcantara
pc at manguebit.com
Wed Jan 3 14:37:11 UTC 2024
Meetakshi Setiya <meetakshisetiyaoss at gmail.com> writes:
> As per the discussion with Tom on the previous version of the changes, I
> conferred with Shyam and Steve about possible workarounds and this seemed like a
> choice which did the job without much perf drawbacks and code changes. One
> highlighted difference between the two could be that in the previous
> version, lease
> would not be reused for any file with hardlinks at all, even though the inode
> may hold the correct lease for that particular file. The current changes
> would take care of this by sending the lease at least once, irrespective of the
> number of hardlinks.
Thanks for the explanation. However, the code change size is no excuse
for providing workarounds rather than the actual fix.
A possible way to handle such case would be keeping a list of
pathname:lease_key pairs inside the inode, so in smb2_compound_op() you
could look up the lease key by using @dentry. I'm not sure if there's a
better way to handle it as I haven't looked into it further.
More information about the samba-technical
mailing list