From patchwork Fri Jun 29 03:37:55 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Steve French X-Patchwork-Id: 10495573 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork.web.codeaurora.org (Postfix) with ESMTP id B8ACA601C7 for ; Fri, 29 Jun 2018 03:38:09 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 99DB02A168 for ; Fri, 29 Jun 2018 03:38:09 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 8DD822A17E; Fri, 29 Jun 2018 03:38:09 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-7.8 required=2.0 tests=BAYES_00, DKIM_ADSP_CUSTOM_MED, DKIM_SIGNED, FREEMAIL_FROM, MAILING_LIST_MULTI, RCVD_IN_DNSWL_HI, T_DKIM_INVALID, T_TVD_MIME_EPI autolearn=ham version=3.3.1 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 83ED62A168 for ; Fri, 29 Jun 2018 03:38:08 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753441AbeF2DiI (ORCPT ); Thu, 28 Jun 2018 23:38:08 -0400 Received: from mail-pl0-f65.google.com ([209.85.160.65]:37940 "EHLO mail-pl0-f65.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751466AbeF2DiH (ORCPT ); Thu, 28 Jun 2018 23:38:07 -0400 Received: by mail-pl0-f65.google.com with SMTP id d10-v6so3761028plo.5 for ; Thu, 28 Jun 2018 20:38:07 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=KHaLETT+092JsxpZRtfDoE5Gw/ztkS8xeJJk2MLmtC8=; b=Y437eIknnNMq+HbZCcVYCL/RNgYIEtNLcGrqc9znjXGNn1+KTwPCEb8wGNcJ0naONI U1yx+2dt7MDbYnuRf/cM/GnpxLiIY4FA3tCcSR+qEjjjzzSmeoqEjCaaaQ7YfaLMuoaM NGzi4+YXRn8jujRwj40PXylLQYTlQXe2pPgmwFDdmf2II355iR4Rr0ak6eCLStoG7e+q KCLoQSbeb0nkjfNZtVYa2lg71G85vxf26oBi5ZbX4tIzhCL2BxchqoS8wczGn2iddlLB FW5AtN0mKj03uSb7QoRxhZKdYDwUvl6nvGjKaLdjNKEWLZxwia6qvdGimYgDMYo5d9fc u+MQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=KHaLETT+092JsxpZRtfDoE5Gw/ztkS8xeJJk2MLmtC8=; b=Dc929uoSxsFjhXDYcxm2U3UIZA3UmWG2FG/ZvoMek3aH8r963pUsUaUkFKcx8MPaP9 5ovLFtiHG/MDqbMDVzFH5QaoJCzd4mky3rDsOJj2RSQOxGhI40B4LKSVrWV5j9pErxqC W9UjB+8B52aQ4eEvsivHo368YKYm+dUx5Gl8jMRU4HYYhPC8CTlr1UFF608NDcSU1VqB jgA+cYM22KkcWVbiKHm9OPntQ5YSCJvc3/2pOPz9rxwxuEEp9CXApPntZxrgx9F/Iiwe vkfD8r6pQYnZBr0KxaeDOVyyuOwJL4dGxRt0BEf8no2/S8h2p85OOJtCiTNCAhPSSZRB 658Q== X-Gm-Message-State: APt69E1II0rIOSFDwNYbKU+EJ+b19qNYMkX08WWWKEKIS4UQ2IIXYUPw ChhVOxLMzpyj4gfXSn/X+CuhUI6haQhnntwcqPk= X-Google-Smtp-Source: ADUXVKKOv2A5rKPAh0t20aM7ldedjSxljWiGexhPkJG5dKv+Z2+8UZAR9efSzdbxwmQ9zxIuqeeYmcI2cJmZh9lp7tU= X-Received: by 2002:a17:902:42c3:: with SMTP id h61-v6mr13045604pld.319.1530243486841; Thu, 28 Jun 2018 20:38:06 -0700 (PDT) MIME-Version: 1.0 References: <201806290913.BXjNzT4V%fengguang.wu@intel.com> In-Reply-To: <201806290913.BXjNzT4V%fengguang.wu@intel.com> From: Steve French Date: Thu, 28 Jun 2018 22:37:55 -0500 Message-ID: Subject: Re: [cifs:for-next 15/16] fs//cifs/cifs_debug.c:178:9: warning: "CONFIG_CIFS_DEBUG" is not defined, evaluates to 0 To: kbuild test robot Cc: Steve French , Paulo Alcantara , CIFS , samba-technical , kbuild-all@01.org, Ronnie Sahlberg Sender: linux-cifs-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-cifs@vger.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP Updated patch On Thu, Jun 28, 2018 at 8:34 PM kbuild test robot via samba-technical wrote: > > tree: git://git.samba.org/sfrench/cifs-2.6.git for-next > head: 739943e0de624a14da460d879f714b9607beda78 > commit: fcff23e72c225da6a22bc0061d9fbeebf1f7fd48 [15/16] cifs: add missing debug entries for kconfig options > config: x86_64-randconfig-x014-201825 (attached as .config) > compiler: gcc-7 (Debian 7.3.0-16) 7.3.0 > reproduce: > git checkout fcff23e72c225da6a22bc0061d9fbeebf1f7fd48 > # save the attached .config to linux build tree > make ARCH=x86_64 > > All warnings (new ones prefixed by >>): > > fs//cifs/cifs_debug.c: In function 'cifs_debug_data_proc_show': > >> fs//cifs/cifs_debug.c:178:9: warning: "CONFIG_CIFS_DEBUG" is not defined, evaluates to 0 [-Wundef] > #elif CONFIG_CIFS_DEBUG > ^~~~~~~~~~~~~~~~~ > > vim +/CONFIG_CIFS_DEBUG +178 fs//cifs/cifs_debug.c > > 147 > 148 static int cifs_debug_data_proc_show(struct seq_file *m, void *v) > 149 { > 150 struct list_head *tmp1, *tmp2, *tmp3; > 151 struct mid_q_entry *mid_entry; > 152 struct TCP_Server_Info *server; > 153 struct cifs_ses *ses; > 154 struct cifs_tcon *tcon; > 155 int i, j; > 156 > 157 seq_puts(m, > 158 "Display Internal CIFS Data Structures for Debugging\n" > 159 "---------------------------------------------------\n"); > 160 seq_printf(m, "CIFS Version %s\n", CIFS_VERSION); > 161 seq_printf(m, "Features:"); > 162 #ifdef CONFIG_CIFS_DFS_UPCALL > 163 seq_printf(m, " DFS"); > 164 #endif > 165 #ifdef CONFIG_CIFS_FSCACHE > 166 seq_printf(m, ",FSCACHE"); > 167 #endif > 168 #ifdef CONFIG_CIFS_SMB_DIRECT > 169 seq_printf(m, ",SMB_DIRECT"); > 170 #endif > 171 #ifdef CONFIG_CIFS_STATS2 > 172 seq_printf(m, ",STATS2"); > 173 #elif CONFIG_CIFS_STATS > 174 seq_printf(m, ",STATS"); > 175 #endif > 176 #ifdef CONFIG_CIFS_DEBUG2 > 177 seq_printf(m, ",DEBUG2"); > > 178 #elif CONFIG_CIFS_DEBUG > 179 seq_printf(m, ",DEBUG"); > 180 #endif > 181 #ifdef CONFIG_CIFS_ALLOW_INSECURE_LEGACY > 182 seq_printf(m, ",ALLOW_INSECURE_LEGACY"); > 183 #endif > 184 #ifdef CONFIG_CIFS_WEAK_PW_HASH > 185 seq_printf(m, ",WEAK_PW_HASH"); > 186 #endif > 187 #ifdef CONFIG_CIFS_POSIX > 188 seq_printf(m, ",CIFS_POSIX"); > 189 #endif > 190 #ifdef CONFIG_CIFS_UPCALL > 191 seq_printf(m, ",UPCALL(SPNEGO)"); > 192 #endif > 193 #ifdef CONFIG_CIFS_XATTR > 194 seq_printf(m, ",XATTR"); > 195 #endif > 196 #ifdef CONFIG_CIFS_ACL > 197 seq_printf(m, ",ACL"); > 198 #endif > 199 seq_putc(m, '\n'); > 200 seq_printf(m, "Active VFS Requests: %d\n", GlobalTotalActiveXid); > 201 seq_printf(m, "Servers:"); > 202 > 203 i = 0; > 204 spin_lock(&cifs_tcp_ses_lock); > 205 list_for_each(tmp1, &cifs_tcp_ses_list) { > 206 server = list_entry(tmp1, struct TCP_Server_Info, > 207 tcp_ses_list); > 208 > > --- > 0-DAY kernel test infrastructure Open Source Technology Center > https://lists.01.org/pipermail/kbuild-all Intel Corporation From fcff23e72c225da6a22bc0061d9fbeebf1f7fd48 Mon Sep 17 00:00:00 2001 From: Steve French Date: Thu, 28 Jun 2018 18:46:40 -0500 Subject: [PATCH 1/2] cifs: add missing debug entries for kconfig options /proc/fs/cifs/DebugData displays the features (Kconfig options) used to build cifs.ko but it was missing some, and needed comma separator. These can be useful in debugging certain problems so we know which optional features were enabled in the user's build. Also clarify them, by making them more closely match the corresponding CONFIG_CIFS_* parm. Old format: Features: dfs fscache posix spnego xattr acl New format: Features: DFS,FSCACHE,SMB_DIRECT,STATS,DEBUG2,ALLOW_INSECURE_LEGACY,CIFS_POSIX,UPCALL(SPNEGO),XATTR,ACL Signed-off-by: Steve French Reviewed-by: Ronnie Sahlberg Reviewed-by: Paulo Alcantara CC: Stable --- fs/cifs/cifs_debug.c | 30 +++++++++++++++++++++++------- 1 file changed, 23 insertions(+), 7 deletions(-) diff --git a/fs/cifs/cifs_debug.c b/fs/cifs/cifs_debug.c index bfe999505815..72f72d1935b1 100644 --- a/fs/cifs/cifs_debug.c +++ b/fs/cifs/cifs_debug.c @@ -160,25 +160,41 @@ static int cifs_debug_data_proc_show(struct seq_file *m, void *v) seq_printf(m, "CIFS Version %s\n", CIFS_VERSION); seq_printf(m, "Features:"); #ifdef CONFIG_CIFS_DFS_UPCALL - seq_printf(m, " dfs"); + seq_printf(m, " DFS"); #endif #ifdef CONFIG_CIFS_FSCACHE - seq_printf(m, " fscache"); + seq_printf(m, ",FSCACHE"); +#endif +#ifdef CONFIG_CIFS_SMB_DIRECT + seq_printf(m, ",SMB_DIRECT"); +#endif +#ifdef CONFIG_CIFS_STATS2 + seq_printf(m, ",STATS2"); +#elif defined(CONFIG_CIFS_STATS) + seq_printf(m, ",STATS"); +#endif +#ifdef CONFIG_CIFS_DEBUG2 + seq_printf(m, ",DEBUG2"); +#elif defined(CONFIG_CIFS_DEBUG) + seq_printf(m, ",DEBUG"); +#endif +#ifdef CONFIG_CIFS_ALLOW_INSECURE_LEGACY + seq_printf(m, ",ALLOW_INSECURE_LEGACY"); #endif #ifdef CONFIG_CIFS_WEAK_PW_HASH - seq_printf(m, " lanman"); + seq_printf(m, ",WEAK_PW_HASH"); #endif #ifdef CONFIG_CIFS_POSIX - seq_printf(m, " posix"); + seq_printf(m, ",CIFS_POSIX"); #endif #ifdef CONFIG_CIFS_UPCALL - seq_printf(m, " spnego"); + seq_printf(m, ",UPCALL(SPNEGO)"); #endif #ifdef CONFIG_CIFS_XATTR - seq_printf(m, " xattr"); + seq_printf(m, ",XATTR"); #endif #ifdef CONFIG_CIFS_ACL - seq_printf(m, " acl"); + seq_printf(m, ",ACL"); #endif seq_putc(m, '\n'); seq_printf(m, "Active VFS Requests: %d\n", GlobalTotalActiveXid); -- 2.17.1