monotonic time for mount.cifs timeouts

2010-08-24 Thread Björn JACKE
Hi, attached are patches to spot the clock_gettime function and to use that and the monotonic clock for timeout handling if possible. If there are no objections, can you please apply them, Jeff? Thanks Björn From 250d5b5df23d93ebed66a4c39bccf3a3614c54ad Mon Sep 17 00:00:00 2001 From:

[linux-cifs-client][PATCH][Try 2] Eliminate sparse warning - bad constant expression

2010-08-24 Thread shirishpargaonkar
Eliminiate sparse warning during usage of crypto_shash_* APIs error: bad constant expression Allocate memory for shash descriptors once, so that we do not kmalloc/kfree it for every signature generation (shash descriptor for md5 hash). From ed753861981decc44b5660b52268077b74f3 Mon

Re: monotonic time for mount.cifs timeouts

2010-08-24 Thread Jeff Layton
On Tue, 24 Aug 2010 13:07:55 -0500 Steve French smfre...@gmail.com wrote: Any pointers to background on gettimeofday vs clock_gettime(CLOCK_MONOTONIC) and why the latter is better? IIUC, the reason for this is that gettimeofday is affected by wallclock changes (think NTP):

Re: monotonic time for mount.cifs timeouts

2010-08-24 Thread Jeff Layton
On Tue, 24 Aug 2010 14:22:48 -0400 Jeff Layton jlay...@samba.org wrote: On Tue, 24 Aug 2010 13:07:55 -0500 Steve French smfre...@gmail.com wrote: Any pointers to background on gettimeofday vs clock_gettime(CLOCK_MONOTONIC) and why the latter is better? IIUC, the reason for this is that

Re: monotonic time for mount.cifs timeouts

2010-08-24 Thread Jeff Layton
On Tue, 24 Aug 2010 22:08:09 +0200 Björn JACKE b...@sernet.de wrote: On 2010-08-24 at 14:34 -0400 Jeff Layton sent off: Any pointers to background on gettimeofday vs clock_gettime(CLOCK_MONOTONIC) and why the latter is better? IIUC, the reason for this is that gettimeofday is

Re: monotonic time for mount.cifs timeouts

2010-08-24 Thread Steve French
On Tue, Aug 24, 2010 at 6:30 PM, Jeff Layton jlay...@samba.org wrote: On Tue, 24 Aug 2010 22:08:09 +0200 Björn JACKE b...@sernet.de wrote: Ok, thanks for the clarification. Committed... This should make the 4.7 release. Thanks! Yes - makes sense - thanks -- Thanks, Steve -- To