пн, 24 февр. 2020 г. в 21:16, Murphy Zhou <jencce.kernel@xxxxxxxxx>: > > On Mon, Feb 24, 2020 at 11:39:27AM -0800, Pavel Shilovsky wrote: > > вт, 18 февр. 2020 г. в 18:10, Murphy Zhou <jencce.kernel@xxxxxxxxx>: > > > > > > On Fri, Feb 14, 2020 at 11:03:00AM -0800, Pavel Shilovsky wrote: > > > > Also, please make sure that resulting patch works against Windows file > > > > share since the locking semantics may be different there. > > > > > > OK. > > > > > > > > > > > Depending on a kind of lease we have on a file, locks may be cached or > > > > not. We probably don't want to have different behavior for cached and > > > > non-cached locks. Especially given the fact that a lease may be broken > > > > in the middle of app execution and the different behavior will be > > > > applied immediately. > > > > > > Testing new patch with and without cache=none option, both samba > > > and Win2019 server. > > > > > > Thanks very much for reviewing! > > > > > > > cache=none only affects IO and doesn't change the client behavior > > regarding locks. "nolease" mount option can be used to turn off leases > > and make all locks go to the server. > > Great to know! I can't find it in any man page. Doing more tests. > Good catch, it is missing in the man pages. Now added: https://github.com/piastry/cifs-utils/commit/4b8b2e2680e7e4aa9cc8bd4278d04e5fe07d885e Thanks! -- Best regards, Pavel Shilovsky
