From mboxrd@z Thu Jan 1 00:00:00 1970 From: Bryan Schumaker Date: Tue, 31 Jul 2012 12:47:56 +0000 Subject: Re: [nfs:devel 46/51] fs/nfs/write.c:1592:5: error: redefinition of 'nfs_commit_inode' Message-Id: <5017D3FC.9010905@netapp.com> List-Id: References: <20120731024328.GA10480@localhost> <4FA345DA4F4AE44899BD2B03EEEC2FA934BE82@SACEXCMBX04-PRD.hq.netapp.com> In-Reply-To: <4FA345DA4F4AE44899BD2B03EEEC2FA934BE82@SACEXCMBX04-PRD.hq.netapp.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: "Myklebust, Trond" Cc: Fengguang Wu , "Schumaker, Bryan" , "kernel-janitors@vger.kernel.org" , "linux-nfs@vger.kernel.org" On 07/30/2012 11:14 PM, Myklebust, Trond wrote: > Bryan, > > Why did you switch to using IS_ENABLED in the first place, and why wasn't that put in a separate patch? IS_ENABLED() will check for CONFIG_NFS_V*_MODULE, last I checked the defined() macro doesn't. Putting this into a separate patch makes the change non-bisectable since gcc will be checking against CONFIG_NFS_V3 but CONFIG_NFS_V3_MODULE is set. Should I have changed the CONFIG_ variables instead of using IS_ENABLED()? > > > >> -----Original Message----- >> From: Fengguang Wu [mailto:fengguang.wu@intel.com] >> Sent: Monday, July 30, 2012 10:43 PM >> To: Schumaker, Bryan >> Cc: kernel-janitors@vger.kernel.org; Myklebust, Trond; linux- >> nfs@vger.kernel.org >> Subject: [nfs:devel 46/51] fs/nfs/write.c:1592:5: error: redefinition of >> 'nfs_commit_inode' >> >> Hi Bryan, >> >> Kernel build failed on >> >> tree: git://git.linux-nfs.org/projects/trondmy/linux-nfs.git devel >> head: 5c13c9e1c15ee2ca9ab2b953224001af53d9be09 >> commit: 1c606fb74c758beafd98cbad9a9133eadeec2371 [46/51] NFS: Convert >> v3 into a module >> config: blackfin-BF533-EZKIT_defconfig (attached as .config) >> >> All related error/warning messages: >> >> fs/nfs/write.c:1592:5: error: redefinition of 'nfs_commit_inode' >> In file included from fs/nfs/write.c:19:0: >> include/linux/nfs_fs.h:547:1: note: previous definition of 'nfs_commit_inode' >> was here >> >> vim +1592 fs/nfs/write.c >> 1589 return status; >> 1590 } >> 1591 >>> 1592 int nfs_commit_inode(struct inode *inode, int how) >> 1593 { >> 1594 LIST_HEAD(head); >> 1595 struct nfs_commit_info cinfo; >> >> --- >> 0-DAY kernel build testing backend Open Source Technology Centre >> Fengguang Wu Intel Corporation