[linux-cifs-client] Re: [PATCH 0/2]: Introduction: Helper utilities for CIFS VFS client

[Steve French]

The code looks fine, but I think the utilities would be better if they
allowed you to do things in an easier way than e.g. what you must do
now

"echo 1 > /proc/fs/cifs/traceSMB"
and
"cat /proc/fs/cifs/Stats"

This seems to match too closely to the syntax of proc and not save
much typing/looking up of things - but perhaps could if we exposed the
"sec=" options in more detail.  We also could use a utility to
translate traceSMB data in dmesg into wireshark format (so we could
read dmesg smb data in the wireshark gui easily) and it would also be
useful to have easier mount syntax for new users (perhaps interactive
series of Q&A options if mount.cifs is compiled a certain way??) - but
a GUI might be the best for this since there are so many options -
some of which are sensitive to the presence/absence of other mount
options.   I also think it would also be better if the help for the
options (from fs/cifs/README) were included in the help (although not
sure how SuSE and RedHat like to package translatable text these days
for utilities).

On Jan 7, 2008 9:37 AM, Suresh Jayaraman <sjayaraman at suse.de> wrote:
> This patchset provides utilities to configure/display cifs proc settings
> and display cifs Debug information and Statistics.
>
> There are two patches:
>
> [PATCH 1/2]: Utility program that allows configuration of cifs client
> settings
> [PATCH 2/2]: Utility program to dump cifs Debug data and Statistics
>
> This can be considered for inclusion along with mount.cifs helper and
> others. I have done some amount of testing and it works without issues.
>
> TODO
> ----
> * Man page for cifsconfig and cifsinfo.
>
> Comments welcome!
>
> Thanks,
>
> --
> Suresh Jayaraman
>



-- 
Thanks,

Steve