From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 6A3D61991B8 for ; Sat, 15 Feb 2025 16:51:13 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1739638278; cv=none; b=ocDc+FzPiq+flZU6wfTS84Co/fwrvOsW1XRCo4aZNQit6d6lQNxiCLml8ijU4uQYBzBo8w0M5As2xNNswjE8JbgRIvhk8hAIUPqppmD5N4je3k/4zxmIrZ5er7Z0O0yHho5OFSgJZko0uoTE0g5LoON3zNLv+jPzizZxT0nLjAE= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1739638278; c=relaxed/simple; bh=Jo9cF7pAhN2YFZUmdxCbT4J0O+zQ5h1EjLAFlcSJa9U=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=aRqrXvDlqxvQ9LuGQzK+FhQJPDy1DgghP1dJwVVlGTpEGRcy+LCBenOhmc1KnKmcZWac6+dYfugwpYP93ke1jg6hWPuNTbqteknor4pM/F43SWjxV7Gh/RiPrLnWAjnIsMuUrw4Y5W7MnYib+5C/oLK7wUFW/zBW84CkxGlsfC8= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=UjroG7Lm; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="UjroG7Lm" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 894BCC4CEDF; Sat, 15 Feb 2025 16:51:13 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1739638273; bh=Jo9cF7pAhN2YFZUmdxCbT4J0O+zQ5h1EjLAFlcSJa9U=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=UjroG7Lm5+CEGKjQk35eUkAh2aO1qGrE6WwlE46EhgAbOykaNbxx4Qjx2kevGnryz KgEP8sg/Gf4ZduKIpXDL70rrpLo4AIJF2VO40lmfduoGaKNbHXKOSW5Rw8TCYcyM3P Nybt2vFaQcEwQ73Cot/e1KeA5N9ihh3u5kAe+Wj9DO8qRKq4sCPgOwRPfuBpVR+dwI zgzQPT8hivD51cfRvT8ZGOoYl2DTyzuzYSHP1oWt0kuVnFxSVlLXu5mwPlJ1wGgnBI 8xYDLRQCUsYYBQkHeq+AN8JuLkGsDHlCS6Df7LDo3P31i7Nyw4fpBpiZlgXeyIacbT s4kIDpXOfjpTA== Received: by pali.im (Postfix) id D8F4A676; Sat, 15 Feb 2025 17:51:00 +0100 (CET) Date: Sat, 15 Feb 2025 17:51:00 +0100 From: Pali =?utf-8?B?Um9ow6Fy?= To: Chuck Lever Cc: Mike Snitzer , linux-nfs@vger.kernel.org Subject: Re: nfs compile error nfslocalio.o and localio.o since v6.14-rc1 Message-ID: <20250215165100.jlibe46qwwdfgau5@pali> References: <20250215120054.mfvr2fzs5426bthx@pali> <4c790142-7126-413d-a2f3-bb080bb26ce6@oracle.com> <20250215163800.v4qdyum6slbzbmts@pali> Precedence: bulk X-Mailing-List: linux-nfs@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: User-Agent: NeoMutt/20180716 On Saturday 15 February 2025 11:41:25 Chuck Lever wrote: > On 2/15/25 11:38 AM, Pali Rohár wrote: > > On Saturday 15 February 2025 11:29:45 Chuck Lever wrote: > >> Hi Pali - > >> > >> On 2/15/25 7:00 AM, Pali Rohár wrote: > >>> Hello, since v6.14-rc1, file nfslocalio.c cannot be compiled with > >>> gcc-8.3 and attached .config file. Same problem is with localio.c. > >> > >> If the interwebs are correct, gcc-8.3 was released in 2014. ISTR that > >> recent releases of the Linux kernel no longer support gcc versions that > >> old. > > > > Hello, I know that this is old version, and I specially used it just to > > check if everything compiles correctly. And it failed. > > > > Per https://docs.kernel.org/process/changes.html the minimal version of > > gcc is 5.1, so I think that compilation with gcc 8.3 should still be > > supported. > > > >> It appears to be snagging on kernel-wide utility helpers, not code > >> specific to NFS. > > > > It looks like that, but only those two nfs files cause compile errors. > > Everything else compiles without problem. So it is quite suspicious and > > maybe it could signal that those helper are used incorrectly in nfs > > code? I'm not sure, I have not investigated it. > > A bisect would be helpful. > > Also, what is the CPU platform architecture? x86_64? Yes, it is x86_64, I hope that all details/configuration is in the .config file. I took generic gcc 8.3 version which was distributed by some debian version. So nothing special. > > >> If that's the case, it might not be possible for us to address this > >> breakage. > >> > >> Adding Mike, who contributed this code. > >> > >>> Error is: > >>> > >>> $ make bzImage > >>> CALL scripts/checksyscalls.sh > >>> DESCEND objtool > >>> INSTALL libsubcmd_headers > >>> CC fs/nfs_common/nfslocalio.o > >>> In file included from ./include/linux/rbtree.h:24, > >>> from ./include/linux/mm_types.h:11, > >>> from ./include/linux/mmzone.h:22, > >>> from ./include/linux/gfp.h:7, > >>> from ./include/linux/umh.h:4, > >>> from ./include/linux/kmod.h:9, > >>> from ./include/linux/module.h:17, > >>> from fs/nfs_common/nfslocalio.c:7: > >>> fs/nfs_common/nfslocalio.c: In function ‘nfs_close_local_fh’: > >>> ./include/linux/rcupdate.h:531:9: error: dereferencing pointer to incomplete type ‘struct nfsd_file’ > >>> typeof(*p) *local = (typeof(*p) *__force)READ_ONCE(p); \ > >>> ^ > >>> ./include/linux/rcupdate.h:650:31: note: in expansion of macro ‘__rcu_access_pointer’ > >>> #define rcu_access_pointer(p) __rcu_access_pointer((p), __UNIQUE_ID(rcu), __rcu) > >>> ^~~~~~~~~~~~~~~~~~~~ > >>> fs/nfs_common/nfslocalio.c:288:10: note: in expansion of macro ‘rcu_access_pointer’ > >>> ro_nf = rcu_access_pointer(nfl->ro_file); > >>> ^~~~~~~~~~~~~~~~~~ > >>> make[4]: *** [scripts/Makefile.build:207: fs/nfs_common/nfslocalio.o] Error 1 > >>> make[3]: *** [scripts/Makefile.build:465: fs/nfs_common] Error 2 > >>> make[2]: *** [scripts/Makefile.build:465: fs] Error 2 > >>> make[1]: *** [/home/pali/develop/kernel.org/linux/Makefile:1994: .] Error 2 > >>> make: *** [Makefile:251: __sub-make] Error 2 > >>> > >>> > >>> $ make fs/nfs/localio.o > >>> CALL scripts/checksyscalls.sh > >>> DESCEND objtool > >>> INSTALL libsubcmd_headers > >>> CC fs/nfs/localio.o > >>> In file included from ./include/linux/rbtree.h:24, > >>> from ./include/linux/mm_types.h:11, > >>> from ./include/linux/mmzone.h:22, > >>> from ./include/linux/gfp.h:7, > >>> from ./include/linux/umh.h:4, > >>> from ./include/linux/kmod.h:9, > >>> from ./include/linux/module.h:17, > >>> from fs/nfs/localio.c:11: > >>> fs/nfs/localio.c: In function ‘nfs_local_open_fh’: > >>> ./include/linux/rcupdate.h:538:9: error: dereferencing pointer to incomplete type ‘struct nfsd_file’ > >>> typeof(*p) *local = (typeof(*p) *__force)READ_ONCE(p); \ > >>> ^ > >>> ./include/linux/rcupdate.h:686:2: note: in expansion of macro ‘__rcu_dereference_check’ > >>> __rcu_dereference_check((p), __UNIQUE_ID(rcu), \ > >>> ^~~~~~~~~~~~~~~~~~~~~~~ > >>> ./include/linux/rcupdate.h:758:28: note: in expansion of macro ‘rcu_dereference_check’ > >>> #define rcu_dereference(p) rcu_dereference_check(p, 0) > >>> ^~~~~~~~~~~~~~~~~~~~~ > >>> fs/nfs/localio.c:275:7: note: in expansion of macro ‘rcu_dereference’ > >>> nf = rcu_dereference(*pnf); > >>> ^~~~~~~~~~~~~~~ > >>> make[4]: *** [scripts/Makefile.build:207: fs/nfs/localio.o] Error 1 > >>> make[3]: *** [scripts/Makefile.build:465: fs/nfs] Error 2 > >>> make[2]: *** [scripts/Makefile.build:465: fs] Error 2 > >>> make[1]: *** [/home/pali/develop/kernel.org/linux/Makefile:1994: .] Error 2 > >>> make: *** [Makefile:251: __sub-make] Error 2 > >>> > >>> > >>> Reproduced from commit 7ff71e6d9239 ("Merge tag 'alpha-fixes-v6.14-rc2' > >>> of git://git.kernel.org/pub/scm/linux/kernel/git/mattst88/alpha"). > >> > >> > >> -- > >> Chuck Lever > > > -- > Chuck Lever