On 12/01/2010 06:39 PM, Jeff Layton wrote: > On Wed, 1 Dec 2010 18:30:36 +0530 > Suresh Jayaraman <sjayaraman@xxxxxxx> wrote: > >> >> Signed-off-by: Suresh Jayaraman <sjayaraman@xxxxxxx> >> --- >> mount.cifs.8 | 18 ++++++++++++++++++ >> 1 files changed, 18 insertions(+), 0 deletions(-) >> >> diff --git a/mount.cifs.8 b/mount.cifs.8 >> index 7d2bf69..15dc2dc 100644 >> --- a/mount.cifs.8 >> +++ b/mount.cifs.8 >> @@ -403,6 +403,24 @@ Map user accesses to individual credentials when accessing the server\&. By defa >> With this change, it's feasible for the server to handle permissions enforcement, so this option also implies "noperm"\&. Furthermore, when unix extensions aren't in use and the administrator has not overriden ownership using the uid= or gid= options, ownership of files is presented as the current user accessing the share\&. >> .RE >> .PP >> +actimeo=\fIarg\fR >> +.RS 4 >> +The time (in seconds) that the CIFS client caches attributes of a file or >> +directory before it requests attribute information from a server. During this >> +period the changes that occur on the server remain undetected until the client >> +checks the server again. >> +.sp >> +By default, the attribute cache timeout is set to 1 second. This means more >> +frequent on-the-wire calls to the server to check whether attributes have >> +changed which could impact performance. With this option the users can make > > "With this option, users can make a" > >> +tradeoff between performance and cache metadata correctness depending on the > "correctness, depending" > >> +workload needs. Shorter timeouts mean better the cache coherency, but frequent > > "Shorter timeouts mean better cache coherency, but an" > >> +increased number of calls to the server. Longer timeouts mean reduced number >> +of calls to the server at the expense of stricter cache coherency\&. The > > A little unclear...maybe "Longer timeouts mean a reduced number of calls to the server but looser cache coherency." > >> +actimeo value is a positive integral that can hold values between 0 and a Doh, this word "integral" was borrowed from nfs man page. But, I agree "integer" is correct and the nfs man page perhaps needs to be fixed too. > "integer" > >> +maximum value of 2^30 * HZ (frequency of timer interrupt) setting\&. >> +.RE >> +.PP >> \-\-verbose >> .RS 4 >> Print additional debugging information for the mount\&. Note that this parameter must be specified before the \-o\&. For example: > > If those edits look ok, I'll add them in when I merge the patch, which > should be soon after Steve merges the kernel patch. No need to resend. > All of them look fine. Thanks for getting those finer bits correct. -- Suresh Jayaraman -- To unsubscribe from this list: send the line "unsubscribe linux-cifs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html