Re: [PATCH 1/1] lockd: Change nsm_use_hostnames from bool to u32

From: kbuild test robot <lkp@intel.com>
To: Jia He <hejianet@gmail.com>
Cc: kbuild-all@01.org, linux-nfs@vger.kernel.org,
	"J. Bruce Fields" <bfields@fieldses.org>,
	Jeff Layton <jlayton@poochiereds.net>,
	Trond Myklebust <trond.myklebust@primarydata.com>,
	Anna Schumaker <anna.schumaker@netapp.com>,
	linux-kernel@vger.kernel.org, Jia He <hejianet@gmail.com>
Subject: Re: [PATCH 1/1] lockd: Change nsm_use_hostnames from bool to u32
Date: Sun, 11 Dec 2016 23:04:13 +0800	[thread overview]
Message-ID: <201612112341.nzl2YBQ4%fengguang.wu@intel.com> (raw)
In-Reply-To: <1481467752-30728-2-git-send-email-hejianet@gmail.com>

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

Hi Jia,

[auto build test ERROR on nfsd/nfsd-next]
[also build test ERROR on v4.9-rc8 next-20161209]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]

url:    https://github.com/0day-ci/linux/commits/Jia-He/lockd-Change-nsm_use_hostnames-from-bool-to-u32/20161211-225133
base:   git://linux-nfs.org/~bfields/linux.git nfsd-next
config: i386-randconfig-x006-201650 (attached as .config)
compiler: gcc-6 (Debian 6.2.0-3) 6.2.0 20160901
reproduce:
        # save the attached .config to linux build tree
        make ARCH=i386 

All error/warnings (new ones prefixed by >>):

   In file included from include/linux/module.h:18:0,
                    from fs/lockd/svc.c:15:
>> include/linux/moduleparam.h:146:27: error: expected ')' before '&' token
     param_check_##type(name, &(value));       \
                              ^
>> include/linux/moduleparam.h:126:2: note: in expansion of macro 'module_param_named'
     module_param_named(name, name, type, perm)
     ^~~~~~~~~~~~~~~~~~
>> fs/lockd/svc.c:661:1: note: in expansion of macro 'module_param'
    module_param(nsm_use_hostnames, u32, 0644);
    ^~~~~~~~~~~~
>> include/linux/moduleparam.h:147:25: error: 'param_ops_u32' undeclared here (not in a function)
     module_param_cb(name, &param_ops_##type, &value, perm);     \
                            ^
   include/linux/moduleparam.h:224:39: note: in definition of macro '__module_param_call'
     = { __param_str_##name, THIS_MODULE, ops,   \
                                          ^~~
>> include/linux/moduleparam.h:147:2: note: in expansion of macro 'module_param_cb'
     module_param_cb(name, &param_ops_##type, &value, perm);     \
     ^~~~~~~~~~~~~~~
>> include/linux/moduleparam.h:126:2: note: in expansion of macro 'module_param_named'
     module_param_named(name, name, type, perm)
     ^~~~~~~~~~~~~~~~~~
>> fs/lockd/svc.c:661:1: note: in expansion of macro 'module_param'
    module_param(nsm_use_hostnames, u32, 0644);
    ^~~~~~~~~~~~
--
>> fs/lockd/mon.c:60:20: error: conflicting types for 'nsm_use_hostnames'
    u32 __read_mostly  nsm_use_hostnames;
                       ^~~~~~~~~~~~~~~~~
   In file included from fs/lockd/mon.c:18:0:
   include/linux/lockd/lockd.h:199:15: note: previous declaration of 'nsm_use_hostnames' was here
    extern bool   nsm_use_hostnames;
                  ^~~~~~~~~~~~~~~~~

vim +/nsm_use_hostnames +60 fs/lockd/mon.c

    54	static				DEFINE_SPINLOCK(nsm_lock);
    55	
    56	/*
    57	 * Local NSM state
    58	 */
    59	u32	__read_mostly		nsm_local_state;
  > 60	u32	__read_mostly		nsm_use_hostnames;
    61	
    62	static inline struct sockaddr *nsm_addr(const struct nsm_handle *nsm)
    63	{

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

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