Re: [PATCH] cifs: document 'fsc' mount option

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, 27 Jul 2010 10:13:14 -0500
Steve French <smfrench@xxxxxxxxx> wrote:

> Yes - I agree with Jeff, need to add the mention of the config option
> and perhaps how to tell if fsc feature is available in kernel.
> 

How would one tell? :)

> On Tue, Jul 27, 2010 at 9:33 AM, Jeff Layton <jlayton@xxxxxxxxxx> wrote:
> > On Tue, 27 Jul 2010 19:20:00 +0530
> > Suresh Jayaraman <sjayaraman@xxxxxxx> wrote:
> >
> >> Document the newly added local caching feature using FS-Cache. This patch
> >> could be queued and considered once the local caching patches gets merged
> >> upstream.
> >>
> >> Signed-off-by: Suresh Jayaraman <sjayaraman@xxxxxxx>
> >> ---
> >>  doc/mount.cifs.8.xml |   17 +++++++++++++++++
> >>  1 files changed, 17 insertions(+), 0 deletions(-)
> >>
> >> diff --git a/doc/mount.cifs.8.xml b/doc/mount.cifs.8.xml
> >> index 4937550..2200746 100644
> >> --- a/doc/mount.cifs.8.xml
> >> +++ b/doc/mount.cifs.8.xml
> >> @@ -534,6 +534,23 @@ permissions in memory that can't be stored on the server. This information can d
> >>                  maximum wsize currently allowed by CIFS is 57344 (fourteen
> >>                  4096 byte pages)</para></listitem>
> >>       </varlistentry>
> >> +     <varlistentry>
> >> +             <term>fsc</term>
> >> +
> >> +             <listitem><para>Enable local disk caching using FS-Cache
> >> +             for cifs. This option could be useful to improve performance
> >> +             on a slow link, heavily loaded server and/or network
> >> +             where reading from the disk is faster than reading from the
> >> +             server (over the network). This could also impact the
> >> +             scalability positively as the number of calls to the server
> >> +             are reduced. But, be warned that local caching is not suitable
> >> +             for all workloads, for e.g., read-once type workloads. So
> >> +             you need to consider carefully the situation/workload before
> >> +             using this option.
> >> +             NOTE: You also need to have cachefilesd daemon installed and
> >> +             running to make the cache operational.
> >> +                </para></listitem>
> >> +     </varlistentry>
> >
> > You may want to mention somewhere that this is only available in recent
> > kernels that have been built with fsc turned on.
> >
> > Other than that, this and the mount.cifs patch look fine and I'll plan
> > to commit them before the upcoming 4.6 release.
> >
> >>        <varlistentry>
> >>                  <term>--verbose</term>
> >>                  <listitem><para>Print additional debugging information for the mount. Note that this parameter must be specified before the -o. For example:</para><para>mount -t cifs //server/share /mnt --verbose -o user=username</para></listitem>
> >
> > --
> > Jeff Layton <jlayton@xxxxxxxxxx>
> >
> 
> 
> 


-- 
Jeff Layton <jlayton@xxxxxxxxxx>
--
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


[Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux