[linux-stable-rc:linux-4.4.y 9997/9999] fs/9p/vfs_dir.c:226:2: note: in expansion of macro 'p9_debug'

[linux-stable-rc:linux-4.4.y 9997/9999] fs/9p/vfs_dir.c:226:2: note: in expansion of macro 'p9_debug'

[kernel test robot]

[-- Attachment #1: Type: text/plain, Size: 4786 bytes --]

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git linux-4.4.y
head:   95a3867e897abd7811196123f81a119a75aba863
commit: b60b53d4980f879884740e672d83155980d74445 [9997/9999] printk: help pr_debug and pr_devel to optimize out arguments
config: arm64-defconfig (attached as .config)
compiler: aarch64-linux-gcc (GCC) 7.5.0
reproduce (this is a W=1 build):
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        git checkout b60b53d4980f879884740e672d83155980d74445
        # save the attached .config to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-7.5.0 make.cross ARCH=arm64 

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>

All warnings (new ones prefixed by >>, old ones prefixed by <<):

In file included from arch/arm64/include/asm/hw_breakpoint.h:20:0,
from arch/arm64/include/asm/processor.h:33,
from arch/arm64/include/asm/spinlock.h:21,
from include/linux/spinlock.h:87,
from include/linux/seqlock.h:35,
from include/linux/time.h:5,
from include/uapi/linux/timex.h:56,
from include/linux/timex.h:56,
from include/linux/sched.h:19,
from arch/arm64/include/asm/compat.h:25,
from arch/arm64/include/asm/stat.h:23,
from include/linux/stat.h:5,
from include/linux/module.h:10,
from fs/9p/vfs_dir.c:26:
arch/arm64/include/asm/cpufeature.h: In function 'cpuid_feature_extract_field':
arch/arm64/include/asm/cpufeature.h:156:55: warning: signed and unsigned type in conditional expression [-Wsign-compare]
cpuid_feature_extract_signed_field(features, field) :
^
In file included from include/linux/blkdev.h:14:0,
from include/linux/backing-dev.h:14,
from fs/9p/v9fs.h:26,
from fs/9p/vfs_dir.c:40:
include/linux/pagemap.h: In function 'fault_in_multipages_readable':
include/linux/pagemap.h:632:16: warning: variable 'c' set but not used [-Wunused-but-set-variable]
volatile char c;
^
In file included from include/linux/blkdev.h:18:0,
from include/linux/backing-dev.h:14,
from fs/9p/v9fs.h:26,
from fs/9p/vfs_dir.c:40:
include/linux/bio.h: In function 'bio_next_split':
include/linux/bio.h:424:14: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
if (sectors >= bio_sectors(bio))
^~
In file included from include/linux/kernel.h:13:0,
from include/linux/list.h:8,
from include/linux/module.h:9,
from fs/9p/vfs_dir.c:26:
fs/9p/vfs_dir.c: In function 'v9fs_dir_release':
fs/9p/vfs_dir.c:227:32: warning: signed and unsigned type in conditional expression [-Wsign-compare]
inode, filp, fid ? fid->fid : -1);
^
include/linux/printk.h:114:17: note: in definition of macro 'no_printk'
printk(fmt, ##__VA_ARGS__);                     ^~~~~~~~~~~
>> fs/9p/vfs_dir.c:226:2: note: in expansion of macro 'p9_debug'
p9_debug(P9_DEBUG_VFS, "inode: %p filp: %p fid: %dn",
^~~~~~~~

vim +/p9_debug +226 fs/9p/vfs_dir.c

7751bdb3a095ad Sripathi Kodi       2010-06-04  212  
e69e7fe5b0c86b Eric Van Hensbergen 2005-09-09  213  
e69e7fe5b0c86b Eric Van Hensbergen 2005-09-09  214  /**
e69e7fe5b0c86b Eric Van Hensbergen 2005-09-09  215   * v9fs_dir_release - close a directory
e69e7fe5b0c86b Eric Van Hensbergen 2005-09-09  216   * @inode: inode of the directory
e69e7fe5b0c86b Eric Van Hensbergen 2005-09-09  217   * @filp: file pointer to a directory
e69e7fe5b0c86b Eric Van Hensbergen 2005-09-09  218   *
e69e7fe5b0c86b Eric Van Hensbergen 2005-09-09  219   */
e69e7fe5b0c86b Eric Van Hensbergen 2005-09-09  220  
e69e7fe5b0c86b Eric Van Hensbergen 2005-09-09  221  int v9fs_dir_release(struct inode *inode, struct file *filp)
e69e7fe5b0c86b Eric Van Hensbergen 2005-09-09  222  {
bd238fb431f319 Latchesar Ionkov    2007-07-10  223  	struct p9_fid *fid;
e69e7fe5b0c86b Eric Van Hensbergen 2005-09-09  224  
bd238fb431f319 Latchesar Ionkov    2007-07-10  225  	fid = filp->private_data;
5d3851530d6d68 Joe Perches         2011-11-28 @226  	p9_debug(P9_DEBUG_VFS, "inode: %p filp: %p fid: %d\n",
62726a7ab3a6a3 jvrao               2010-08-25  227  		 inode, filp, fid ? fid->fid : -1);
62726a7ab3a6a3 jvrao               2010-08-25  228  	if (fid)
bd238fb431f319 Latchesar Ionkov    2007-07-10  229  		p9_client_clunk(fid);
e69e7fe5b0c86b Eric Van Hensbergen 2005-09-09  230  	return 0;
e69e7fe5b0c86b Eric Van Hensbergen 2005-09-09  231  }
e69e7fe5b0c86b Eric Van Hensbergen 2005-09-09  232  

:::::: The code at line 226 was first introduced by commit
:::::: 5d3851530d6d68564e4e0ce04d0547d4d106fc72 9p: Reduce object size with CONFIG_NET_9P_DEBUG

:::::: TO: Joe Perches <joe@perches.com>
:::::: CC: Eric Van Hensbergen <ericvh@gmail.com>

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org

[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 18688 bytes --]