Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1674355 > unrolled thread

linux-next: build warning after merge of the cifs tree

Back to article view | Back to linux.kernel

Contents

  linux-next: build warning after merge of the cifs tree Stephen Rothwell <sfr@canb.auug.org.au> - 2017-06-26 02:10 +0200
    Re: linux-next: build warning after merge of the cifs tree Steve French <smfrench@gmail.com> - 2017-06-26 19:00 +0200

#1674355 — linux-next: build warning after merge of the cifs tree

Hi all,

After merging the cifs tree, today's linux-next build (x86_64
allmodconfig) produced this warning:

In file included from include/linux/kernel.h:13:0,
                 from include/linux/list.h:8,
                 from include/linux/wait.h:6,
                 from include/linux/fs.h:5,
                 from fs/cifs/smb2transport.c:25:
fs/cifs/smb2transport.c: In function 'generate_smb3signingkey':
fs/cifs/cifs_debug.h:55:24: warning: field width specifier '*' expects argument of type 'int', but argument 4 has type 'long unsigned int' [-Wformat=]
   pr_debug_ratelimited("%s: "    \
                        ^
include/linux/printk.h:285:21: note: in definition of macro 'pr_fmt'
 #define pr_fmt(fmt) fmt
                     ^
fs/cifs/cifs_debug.h:55:3: note: in expansion of macro 'pr_debug_ratelimited'
   pr_debug_ratelimited("%s: "    \
   ^
fs/cifs/smb2transport.c:352:2: note: in expansion of macro 'cifs_dbg'
  cifs_dbg(VFS, "Session Id    %*ph\n", sizeof(ses->Suid), &ses->Suid);
  ^
In file included from fs/cifs/smb2transport.c:39:0:
fs/cifs/smb2transport.c:352:16: warning: field width specifier '*' expects argument of type 'int', but argument 2 has type 'long unsigned int' [-Wformat=]
  cifs_dbg(VFS, "Session Id    %*ph\n", sizeof(ses->Suid), &ses->Suid);
                ^
fs/cifs/cifs_debug.h:58:16: note: in definition of macro 'cifs_dbg'
   cifs_vfs_err(fmt, ##__VA_ARGS__);   \
                ^
In file included from include/linux/kernel.h:13:0,
                 from include/linux/list.h:8,
                 from include/linux/wait.h:6,
                 from include/linux/fs.h:5,
                 from fs/cifs/smb2transport.c:25:
fs/cifs/smb2transport.c:352:16: warning: field width specifier '*' expects argument of type 'int', but argument 3 has type 'long unsigned int' [-Wformat=]
  cifs_dbg(VFS, "Session Id    %*ph\n", sizeof(ses->Suid), &ses->Suid);
                ^
include/linux/printk.h:285:21: note: in definition of macro 'pr_fmt'
 #define pr_fmt(fmt) fmt
                     ^
fs/cifs/cifs_debug.h:60:3: note: in expansion of macro 'pr_debug_ratelimited'
   pr_debug_ratelimited(fmt, ##__VA_ARGS__);  \
   ^
fs/cifs/smb2transport.c:352:2: note: in expansion of macro 'cifs_dbg'
  cifs_dbg(VFS, "Session Id    %*ph\n", sizeof(ses->Suid), &ses->Suid);
  ^

Introduced by commit

  ba4f2a8b205f ("CIFS: add CONFIG_CIFS_DEBUG_KEYS to dump encryption keys")

-- 
Cheers,
Stephen Rothwell

[toc] | [next] | [standalone]

#1674947

I just merged an updated version of this patch to remove the warning
to cifs-2.6.git for-next

On Sun, Jun 25, 2017 at 7:07 PM, Stephen Rothwell <sfr@canb.auug.org.au> wrote:
> Hi all,
>
> After merging the cifs tree, today's linux-next build (x86_64
> allmodconfig) produced this warning:
>
> In file included from include/linux/kernel.h:13:0,
>                  from include/linux/list.h:8,
>                  from include/linux/wait.h:6,
>                  from include/linux/fs.h:5,
>                  from fs/cifs/smb2transport.c:25:
> fs/cifs/smb2transport.c: In function 'generate_smb3signingkey':
> fs/cifs/cifs_debug.h:55:24: warning: field width specifier '*' expects argument of type 'int', but argument 4 has type 'long unsigned int' [-Wformat=]
>    pr_debug_ratelimited("%s: "    \
>                         ^
> include/linux/printk.h:285:21: note: in definition of macro 'pr_fmt'
>  #define pr_fmt(fmt) fmt
>                      ^
> fs/cifs/cifs_debug.h:55:3: note: in expansion of macro 'pr_debug_ratelimited'
>    pr_debug_ratelimited("%s: "    \
>    ^
> fs/cifs/smb2transport.c:352:2: note: in expansion of macro 'cifs_dbg'
>   cifs_dbg(VFS, "Session Id    %*ph\n", sizeof(ses->Suid), &ses->Suid);
>   ^
> In file included from fs/cifs/smb2transport.c:39:0:
> fs/cifs/smb2transport.c:352:16: warning: field width specifier '*' expects argument of type 'int', but argument 2 has type 'long unsigned int' [-Wformat=]
>   cifs_dbg(VFS, "Session Id    %*ph\n", sizeof(ses->Suid), &ses->Suid);
>                 ^
> fs/cifs/cifs_debug.h:58:16: note: in definition of macro 'cifs_dbg'
>    cifs_vfs_err(fmt, ##__VA_ARGS__);   \
>                 ^
> In file included from include/linux/kernel.h:13:0,
>                  from include/linux/list.h:8,
>                  from include/linux/wait.h:6,
>                  from include/linux/fs.h:5,
>                  from fs/cifs/smb2transport.c:25:
> fs/cifs/smb2transport.c:352:16: warning: field width specifier '*' expects argument of type 'int', but argument 3 has type 'long unsigned int' [-Wformat=]
>   cifs_dbg(VFS, "Session Id    %*ph\n", sizeof(ses->Suid), &ses->Suid);
>                 ^
> include/linux/printk.h:285:21: note: in definition of macro 'pr_fmt'
>  #define pr_fmt(fmt) fmt
>                      ^
> fs/cifs/cifs_debug.h:60:3: note: in expansion of macro 'pr_debug_ratelimited'
>    pr_debug_ratelimited(fmt, ##__VA_ARGS__);  \
>    ^
> fs/cifs/smb2transport.c:352:2: note: in expansion of macro 'cifs_dbg'
>   cifs_dbg(VFS, "Session Id    %*ph\n", sizeof(ses->Suid), &ses->Suid);
>   ^
>
> Introduced by commit
>
>   ba4f2a8b205f ("CIFS: add CONFIG_CIFS_DEBUG_KEYS to dump encryption keys")
>
> --
> Cheers,
> Stephen Rothwell



-- 
Thanks,

Steve

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web