Tunable timeout in cifs module.

2015-10-20 Thread Sachin Prabhu
Hello Steve/linux-cifs, We have received a request to make the timeout in the cifs module tunable. I have gone through the code to understand timeouts in the cifs client. http://sprabhu.blogspot.in/2015/08/investigation-into-effects-of-server .html The way to allow user settable timeout values i

Re: [PATCH 2/2] cifs: Drop cached dentry if its metadata changed

2015-10-20 Thread Ross Lagerwall
On 10/19/2015 05:21 PM, Steve French wrote: Could you verify this over SMB2 (vers=3.0) as well? It looks like it should fail because cifs_all_info_to_fattr doesn't see the inode attribute change and fill in the new inode number (IndexNuber). I am suspicious that your patch is overkill (sends a

Re: [PATCH v11 21/48] ext4: Add richacl feature flag

2015-10-20 Thread Austin S Hemmelgarn
On 2015-10-19 16:20, Andreas Gruenbacher wrote: On Mon, Oct 19, 2015 at 8:45 PM, Austin S Hemmelgarn wrote: On 2015-10-19 13:33, Andreas Gruenbacher wrote: Please spare me with all that nonsense. Compared to mount options, filesystem feature flags in this case simplify things (you don't have t

Re: Tunable timeout in cifs module.

2015-10-20 Thread Steve French
On Tue, Oct 20, 2015 at 3:20 AM, Sachin Prabhu wrote: > Hello Steve/linux-cifs, > > We have received a request to make the timeout in the cifs module > tunable. > > I have gone through the code to understand timeouts in the cifs client. > http://sprabhu.blogspot.in/2015/08/investigation-into-effec

Re: [PATCH 2/2] cifs: Drop cached dentry if its metadata changed

2015-10-20 Thread Steve French
On Tue, Oct 20, 2015 at 3:55 AM, Ross Lagerwall wrote: > On 10/19/2015 05:21 PM, Steve French wrote: >> >> Could you verify this over SMB2 (vers=3.0) as well? It looks like it >> should fail because cifs_all_info_to_fattr doesn't see the inode >> attribute change and fill in the new inode number

Re: [PATCH 2/2] cifs: Drop cached dentry if its metadata changed

2015-10-20 Thread Steve French
On Tue, Oct 20, 2015 at 12:22 PM, Steve French wrote: > On Tue, Oct 20, 2015 at 3:55 AM, Ross Lagerwall > wrote: >> On 10/19/2015 05:21 PM, Steve French wrote: >>> >>> Could you verify this over SMB2 (vers=3.0) as well? It looks like it >>> should fail because cifs_all_info_to_fattr doesn't see

Re: Tunable timeout in cifs module.

2015-10-20 Thread Steve French
This reconnect discussion should probably be broadened to see how it would change (the retry/timeout/reconnect) when we are tied into the (samba, user space) Witness protocol client that Guenther and others have been experimenting with. Perhaps ioctls down to cifs.ko from the witness protocol clie