From patchwork Tue Apr 21 09:52:13 2009 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Suresh Jayaraman X-Patchwork-Id: 19153 Received: from lists.samba.org (mail.samba.org [66.70.73.150]) by demeter.kernel.org (8.14.2/8.14.2) with ESMTP id n3L9qtVM007716 for ; Tue, 21 Apr 2009 09:52:55 GMT Received: from dp.samba.org (localhost [127.0.0.1]) by lists.samba.org (Postfix) with ESMTP id CA0B9163BD7 for ; Tue, 21 Apr 2009 09:52:33 +0000 (GMT) X-Spam-Checker-Version: SpamAssassin 3.1.7 (2006-10-05) on dp.samba.org X-Spam-Level: X-Spam-Status: No, score=-2.6 required=3.8 tests=AWL, BAYES_00 autolearn=ham version=3.1.7 X-Original-To: linux-cifs-client@lists.samba.org Delivered-To: linux-cifs-client@lists.samba.org Received: from victor.provo.novell.com (victor.provo.novell.com [137.65.250.26]) by lists.samba.org (Postfix) with ESMTP id 76003163ADD for ; Tue, 21 Apr 2009 09:52:13 +0000 (GMT) Received: from [192.168.2.100] (prv-ext-foundry1.gns.novell.com [137.65.251.240]) by victor.provo.novell.com with ESMTP (TLS encrypted); Tue, 21 Apr 2009 03:52:27 -0600 Message-ID: <49ED974D.9000905@suse.de> Date: Tue, 21 Apr 2009 15:22:13 +0530 From: Suresh Jayaraman User-Agent: Thunderbird 2.0.0.19 (X11/20081227) MIME-Version: 1.0 To: Jeff Layton Subject: Re: [linux-cifs-client] [PATCH] doc: clarify that mount.cifs never uses smb.conf References: <49E46916.2040103@suse.de> <49EBFC70.5060803@suse.de> <20090421042357.648196e9@tupile.poochiereds.net> In-Reply-To: <20090421042357.648196e9@tupile.poochiereds.net> X-Enigmail-Version: 0.95.7 Cc: Steve French , "linux-cifs-client@lists.samba.org" X-BeenThere: linux-cifs-client@lists.samba.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: The Linux CIFS VFS client List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: linux-cifs-client-bounces+patchwork-cifs-client=patchwork.kernel.org@lists.samba.org Errors-To: linux-cifs-client-bounces+patchwork-cifs-client=patchwork.kernel.org@lists.samba.org Jeff Layton wrote: > On Mon, 20 Apr 2009 10:09:12 +0530 > Suresh Jayaraman wrote: > >> Hi Steve, Jeff, >> >> If the following mount.cifs doc patch makes sense, can this be >> considered for merging? I have seen atleast a couple of bug reports that >> reports that cifs is not honoring configuration setting in smb.conf. >> >> Thanks, >> >> Suresh Jayaraman wrote: >>> It seems that for some users that are used to smbclient, it is not clear >>> that mount.cifs never uses smb.conf for configuring client specific >>> parameters. So, let's add this information to mount.cifs man page. >>> >>> Signed-off-by: Suresh Jayaraman >>> --- >>> >>> docs-xml/manpages-3/mount.cifs.8.xml | 6 ++++++ >>> 1 files changed, 6 insertions(+), 0 deletions(-) >>> >>> diff --git a/docs-xml/manpages-3/mount.cifs.8.xml b/docs-xml/manpages-3/mount.cifs.8.xml >>> index d0affea..32b2970 100644 >>> --- a/docs-xml/manpages-3/mount.cifs.8.xml >>> +++ b/docs-xml/manpages-3/mount.cifs.8.xml >>> @@ -65,6 +65,12 @@ kernel log. >>> the mounted resource is unmounted (usually via the umount utility). >>> >>> >>> + >>> + Some samba client tools like smbclient(8) honours client-side > ^^^^^^^ > Should be "honour" Ok >>> + configuration parameters present in smb.conf. Unlike those client tools, >>> + mount.cifs ignores smb.conf completely. >>> + >>> + >>> >>> >>> > > Maybe this should go in the NOTES section? It seems more appropriate to add in the NOTES section. Here's the updated patch. docs-xml/manpages-3/mount.cifs.8.xml | 7 +++++++ 1 files changed, 7 insertions(+), 0 deletions(-) diff --git a/docs-xml/manpages-3/mount.cifs.8.xml b/docs-xml/manpages-3/mount.cifs.8.xml index d0affea..c5b516b 100644 --- a/docs-xml/manpages-3/mount.cifs.8.xml +++ b/docs-xml/manpages-3/mount.cifs.8.xml @@ -544,6 +544,13 @@ read and used as the password. NOTES This command may be used only by root, unless installed setuid, in which case the noeexec and nosuid mount flags are enabled. + + + Some samba client tools like smbclient(8) honour client-side + configuration parameters present in smb.conf. Unlike those client tools, + mount.cifs ignores smb.conf completely. + +