Buildresult: linus/um-allyesconfig/um-x86_64 built on Jun 10 2022, 09:10

kisskb

Revisions | Branches | Compilers | Configs | Build Results | Build Failures |

Status:	Failed
Date/Time:	Jun 10 2022, 09:10
Duration:	0:04:10.367646
Builder:	ka7
Revision:	netfs: Fix gcc-12 warning by embedding vfs inode in netfs_i_context (`874c8ca1e60b2c564a48f7e7acc40d328d5c8733)`
Target:	linus/um-allyesconfig/um-x86_64
Branch:	linus
Compiler:	um-x86_64 (x86_64-linux-gcc.br_real (Buildroot 2017.05) 5.4.0 / GNU ld (GNU Binutils) 2.27)
Config:	allyesconfig (download)
Log:	Download original Possible errors drivers/nvdimm/pmem.c:48:20: error: conflicting types for 'to_phys' arch/um/include/asm/page.h:105:20: error: too few arguments to function 'to_phys' drivers/nvdimm/pmem.c:324:1: error: control reaches end of non-void function [-Werror=return-type] cc1: all warnings being treated as errors make[3]: * [scripts/Makefile.build:249: drivers/nvdimm/pmem.o] Error 1 make[2]: * [scripts/Makefile.build:466: drivers/nvdimm] Error 2 arch/um/include/asm/processor-generic.h:103:18: error: called object is not a function or function pointer drivers/gpu/drm/amd/amdgpu/../amdkfd/kfd_topology.c:1614:1: error: control reaches end of non-void function [-Werror=return-type] cc1: all warnings being treated as errors make[5]: * [scripts/Makefile.build:249: drivers/gpu/drm/amd/amdgpu/../amdkfd/kfd_topology.o] Error 1 make[4]: * [scripts/Makefile.build:466: drivers/gpu/drm/amd/amdgpu] Error 2 make[3]: * [scripts/Makefile.build:466: drivers/gpu/drm] Error 2 make[2]: * [scripts/Makefile.build:466: drivers/gpu] Error 2 make[1]: * [Makefile:1843: drivers] Error 2 make: * [Makefile:219: __sub-make] Error 2 Possible warnings (5) .config:12576:warning: override: reassigning to symbol GCC_PLUGIN_SANCOV .config:12581:warning: override: reassigning to symbol UML_NET_ETHERTAP .config:12583:warning: override: reassigning to symbol UML_NET_SLIP .config:12586:warning: override: reassigning to symbol UML_NET_MCAST .config:12588:warning: override: reassigning to symbol UML_NET_SLIRP Full Log # git rev-parse -q --verify 874c8ca1e60b2c564a48f7e7acc40d328d5c8733^{commit} 874c8ca1e60b2c564a48f7e7acc40d328d5c8733 already have revision, skipping fetch # git checkout -q -f -B kisskb 874c8ca1e60b2c564a48f7e7acc40d328d5c8733 # git clean -qxdf # < git log -1 # commit 874c8ca1e60b2c564a48f7e7acc40d328d5c8733 # Author: David Howells <dhowells@redhat.com> # Date: Thu Jun 9 21:46:04 2022 +0100 # # netfs: Fix gcc-12 warning by embedding vfs inode in netfs_i_context # # While randstruct was satisfied with using an open-coded "void " offset # cast for the netfs_i_context <-> inode casting, __builtin_object_size() as # used by FORTIFY_SOURCE was not as easily fooled. This was causing the # following complaint[1] from gcc v12: # # In file included from include/linux/string.h:253, # from include/linux/ceph/ceph_debug.h:7, # from fs/ceph/inode.c:2: # In function 'fortify_memset_chk', # inlined from 'netfs_i_context_init' at include/linux/netfs.h:326:2, # inlined from 'ceph_alloc_inode' at fs/ceph/inode.c:463:2: # include/linux/fortify-string.h:242:25: warning: call to '__write_overflow_field' declared with attribute warning: detected write beyond size of field (1st parameter); maybe use struct_group()? [-Wattribute-warning] # 242 \| __write_overflow_field(p_size_field, size); # \| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ # # Fix this by embedding a struct inode into struct netfs_i_context (which # should perhaps be renamed to struct netfs_inode). The struct inode # vfs_inode fields are then removed from the 9p, afs, ceph and cifs inode # structs and vfs_inode is then simply changed to "netfs.inode" in those # filesystems. # # Further, rename netfs_i_context to netfs_inode, get rid of the # netfs_inode() function that converted a netfs_i_context pointer to an # inode pointer (that can now be done with &ctx->inode) and rename the # netfs_i_context() function to netfs_inode() (which is now a wrapper # around container_of()). # # Most of the changes were done with: # # perl -p -i -e 's/vfs_inode/netfs.inode/'g \ # `git grep -l 'vfs_inode' -- fs/{9p,afs,ceph,cifs}/.[ch]` # # Kees suggested doing it with a pair structure[2] and a special # declarator to insert that into the network filesystem's inode # wrapper[3], but I think it's cleaner to embed it - and then it doesn't # matter if struct randomisation reorders things. # # Dave Chinner suggested using a filesystem-specific VFS_I() function in # each filesystem to convert that filesystem's own inode wrapper struct # into the VFS inode struct[4]. # # Version #2: # - Fix a couple of missed name changes due to a disabled cifs option. # - Rename nfs_i_context to nfs_inode # - Use "netfs" instead of "nic" as the member name in per-fs inode wrapper # structs. # # [ This also undoes commit 507160f46c55 ("netfs: gcc-12: temporarily # disable '-Wattribute-warning' for now") that is no longer needed ] # # Fixes: bc899ee1c898 ("netfs: Add a netfs inode context") # Reported-by: Jeff Layton <jlayton@kernel.org> # Signed-off-by: David Howells <dhowells@redhat.com> # Reviewed-by: Jeff Layton <jlayton@kernel.org> # Reviewed-by: Kees Cook <keescook@chromium.org> # Reviewed-by: Xiubo Li <xiubli@redhat.com> # cc: Jonathan Corbet <corbet@lwn.net> # cc: Eric Van Hensbergen <ericvh@gmail.com> # cc: Latchesar Ionkov <lucho@ionkov.net> # cc: Dominique Martinet <asmadeus@codewreck.org> # cc: Christian Schoenebeck <linux_oss@crudebyte.com> # cc: Marc Dionne <marc.dionne@auristor.com> # cc: Ilya Dryomov <idryomov@gmail.com> # cc: Steve French <smfrench@gmail.com> # cc: William Kucharski <william.kucharski@oracle.com> # cc: "Matthew Wilcox (Oracle)" <willy@infradead.org> # cc: Dave Chinner <david@fromorbit.com> # cc: linux-doc@vger.kernel.org # cc: v9fs-developer@lists.sourceforge.net # cc: linux-afs@lists.infradead.org # cc: ceph-devel@vger.kernel.org # cc: linux-cifs@vger.kernel.org # cc: samba-technical@lists.samba.org # cc: linux-fsdevel@vger.kernel.org # cc: linux-hardening@vger.kernel.org # Link: https://lore.kernel.org/r/d2ad3a3d7bdd794c6efb562d2f2b655fb67756b9.camel@kernel.org/ [1] # Link: https://lore.kernel.org/r/20220517210230.864239-1-keescook@chromium.org/ [2] # Link: https://lore.kernel.org/r/20220518202212.2322058-1-keescook@chromium.org/ [3] # Link: https://lore.kernel.org/r/20220524101205.GI2306852@dread.disaster.area/ [4] # Link: https://lore.kernel.org/r/165296786831.3591209.12111293034669289733.stgit@warthog.procyon.org.uk/ # v1 # Link: https://lore.kernel.org/r/165305805651.4094995.7763502506786714216.stgit@warthog.procyon.org.uk # v2 # Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org> # < /opt/cross/kisskb/fe-x86-64-core-i7-2017.05/bin/x86_64-linux-gcc --version # < /opt/cross/kisskb/fe-x86-64-core-i7-2017.05/bin/x86_64-linux-ld --version # < git log --format=%s --max-count=1 874c8ca1e60b2c564a48f7e7acc40d328d5c8733 # < make -s -j 32 ARCH=um O=/kisskb/build/linus_um-allyesconfig_um-x86_64 CROSS_COMPILE=/opt/cross/kisskb/fe-x86-64-core-i7-2017.05/bin/x86_64-linux- SUBARCH=x86_64 allyesconfig # Added to kconfig CONFIG_STANDALONE=y # Added to kconfig CONFIG_KCOV=n # Added to kconfig CONFIG_GCC_PLUGINS=n # Added to kconfig CONFIG_GCC_PLUGIN_CYC_COMPLEXITY=n # Added to kconfig CONFIG_GCC_PLUGIN_SANCOV=n # Added to kconfig CONFIG_GCC_PLUGIN_LATENT_ENTROPY=n # Added to kconfig CONFIG_GCC_PLUGIN_STRUCTLEAK=n # Added to kconfig CONFIG_GCC_PLUGIN_RANDSTRUCT=n # Added to kconfig CONFIG_UML_NET=n # Added to kconfig CONFIG_UML_NET_ETHERTAP=n # Added to kconfig CONFIG_UML_NET_TUNTAP=n # Added to kconfig CONFIG_UML_NET_SLIP=n # Added to kconfig CONFIG_UML_NET_DAEMON=n # Added to kconfig CONFIG_UML_NET_VDE=n # Added to kconfig CONFIG_UML_NET_MCAST=n # Added to kconfig CONFIG_UML_NET_PCAP=n # Added to kconfig CONFIG_UML_NET_SLIRP=n # Added to kconfig CONFIG_GCOV_KERNEL=n # Added to kconfig CONFIG_DEBUG_INFO_BTF=n # Added to kconfig CONFIG_BPF_PRELOAD=n # < make -s -j 32 ARCH=um O=/kisskb/build/linus_um-allyesconfig_um-x86_64 CROSS_COMPILE=/opt/cross/kisskb/fe-x86-64-core-i7-2017.05/bin/x86_64-linux- SUBARCH=x86_64 help # make -s -j 32 ARCH=um O=/kisskb/build/linus_um-allyesconfig_um-x86_64 CROSS_COMPILE=/opt/cross/kisskb/fe-x86-64-core-i7-2017.05/bin/x86_64-linux- SUBARCH=x86_64 olddefconfig .config:12576:warning: override: reassigning to symbol GCC_PLUGIN_SANCOV .config:12581:warning: override: reassigning to symbol UML_NET_ETHERTAP .config:12583:warning: override: reassigning to symbol UML_NET_SLIP .config:12586:warning: override: reassigning to symbol UML_NET_MCAST .config:12588:warning: override: reassigning to symbol UML_NET_SLIRP # make -s -j 32 ARCH=um O=/kisskb/build/linus_um-allyesconfig_um-x86_64 CROSS_COMPILE=/opt/cross/kisskb/fe-x86-64-core-i7-2017.05/bin/x86_64-linux- SUBARCH=x86_64 /kisskb/src/drivers/nvdimm/pmem.c:48:20: error: conflicting types for 'to_phys' static phys_addr_t to_phys(struct pmem_device pmem, phys_addr_t offset) ^ In file included from /kisskb/src/arch/um/include/asm/page.h:98:0, from /kisskb/src/arch/um/include/asm/thread_info.h:15, from /kisskb/src/include/linux/thread_info.h:60, from /kisskb/src/include/asm-generic/preempt.h:5, from ./arch/um/include/generated/asm/preempt.h:1, from /kisskb/src/include/linux/preempt.h:78, from /kisskb/src/include/linux/spinlock.h:55, from /kisskb/src/include/linux/wait.h:9, from /kisskb/src/include/linux/wait_bit.h:8, from /kisskb/src/include/linux/fs.h:6, from /kisskb/src/include/linux/highmem.h:5, from /kisskb/src/include/linux/bvec.h:10, from /kisskb/src/include/linux/blk_types.h:10, from /kisskb/src/include/linux/blkdev.h:9, from /kisskb/src/drivers/nvdimm/pmem.c:10: /kisskb/src/arch/um/include/shared/mem.h:12:29: note: previous definition of 'to_phys' was here static inline unsigned long to_phys(void virt) ^ In file included from /kisskb/src/arch/um/include/asm/page.h:114:0, from /kisskb/src/arch/um/include/asm/thread_info.h:15, from /kisskb/src/include/linux/thread_info.h:60, from /kisskb/src/include/asm-generic/preempt.h:5, from ./arch/um/include/generated/asm/preempt.h:1, from /kisskb/src/include/linux/preempt.h:78, from /kisskb/src/include/linux/spinlock.h:55, from /kisskb/src/include/linux/wait.h:9, from /kisskb/src/include/linux/wait_bit.h:8, from /kisskb/src/include/linux/fs.h:6, from /kisskb/src/include/linux/highmem.h:5, from /kisskb/src/include/linux/bvec.h:10, from /kisskb/src/include/linux/blk_types.h:10, from /kisskb/src/include/linux/blkdev.h:9, from /kisskb/src/drivers/nvdimm/pmem.c:10: /kisskb/src/drivers/nvdimm/pmem.c: In function 'pmem_dax_zero_page_range': /kisskb/src/arch/um/include/asm/page.h:105:20: error: too few arguments to function 'to_phys' #define __pa(virt) to_phys((void ) (unsigned long) (virt)) ^ /kisskb/src/include/asm-generic/memory_model.h:18:41: note: in definition of macro '__pfn_to_page' #define __pfn_to_page(pfn) (mem_map + ((pfn) - ARCH_PFN_OFFSET)) ^ /kisskb/src/arch/um/include/asm/pgtable.h:276:40: note: in expansion of macro 'phys_to_pfn' #define phys_to_page(phys) pfn_to_page(phys_to_pfn(phys)) ^ /kisskb/src/arch/um/include/asm/pgtable.h:277:30: note: in expansion of macro 'phys_to_page' #define __virt_to_page(virt) phys_to_page(__pa(virt)) ^ /kisskb/src/arch/um/include/asm/pgtable.h:277:43: note: in expansion of macro '__pa' #define __virt_to_page(virt) phys_to_page(__pa(virt)) ^ /kisskb/src/arch/um/include/asm/pgtable.h:279:28: note: in expansion of macro '__virt_to_page' #define virt_to_page(addr) __virt_to_page((const unsigned long) addr) ^ /kisskb/src/arch/um/include/asm/pgtable.h:93:26: note: in expansion of macro 'virt_to_page' #define ZERO_PAGE(vaddr) virt_to_page(empty_zero_page) ^ /kisskb/src/drivers/nvdimm/pmem.c:321:49: note: in expansion of macro 'ZERO_PAGE' return blk_status_to_errno(pmem_do_write(pmem, ZERO_PAGE(0), 0, ^ /kisskb/src/drivers/nvdimm/pmem.c:48:20: note: declared here static phys_addr_t to_phys(struct pmem_device pmem, phys_addr_t offset) ^ /kisskb/src/drivers/nvdimm/pmem.c:324:1: error: control reaches end of non-void function [-Werror=return-type] } ^ cc1: all warnings being treated as errors make[3]: * [/kisskb/src/scripts/Makefile.build:249: drivers/nvdimm/pmem.o] Error 1 make[3]: * Waiting for unfinished jobs.... make[2]: * [/kisskb/src/scripts/Makefile.build:466: drivers/nvdimm] Error 2 make[2]: * Waiting for unfinished jobs.... In file included from /kisskb/src/arch/x86/um/asm/processor.h:41:0, from /kisskb/src/include/linux/mutex.h:19, from /kisskb/src/include/linux/kernfs.h:11, from /kisskb/src/include/linux/sysfs.h:16, from /kisskb/src/include/linux/kobject.h:20, from /kisskb/src/include/linux/pci.h:35, from /kisskb/src/drivers/gpu/drm/amd/amdgpu/../amdkfd/kfd_topology.c:26: /kisskb/src/drivers/gpu/drm/amd/amdgpu/../amdkfd/kfd_topology.c: In function 'kfd_cpumask_to_apic_id': /kisskb/src/arch/um/include/asm/processor-generic.h:103:18: error: called object is not a function or function pointer #define cpu_data (&boot_cpu_data) ^ /kisskb/src/drivers/gpu/drm/amd/amdgpu/../amdkfd/kfd_topology.c:1610:9: note: in expansion of macro 'cpu_data' return cpu_data(first_cpu_of_numa_node).apicid; ^ /kisskb/src/drivers/gpu/drm/amd/amdgpu/../amdkfd/kfd_topology.c:1614:1: error: control reaches end of non-void function [-Werror=return-type] } ^ cc1: all warnings being treated as errors make[5]: * [/kisskb/src/scripts/Makefile.build:249: drivers/gpu/drm/amd/amdgpu/../amdkfd/kfd_topology.o] Error 1 make[5]: * Waiting for unfinished jobs.... make[4]: * [/kisskb/src/scripts/Makefile.build:466: drivers/gpu/drm/amd/amdgpu] Error 2 make[4]: * Waiting for unfinished jobs.... make[3]: * [/kisskb/src/scripts/Makefile.build:466: drivers/gpu/drm] Error 2 make[2]: * [/kisskb/src/scripts/Makefile.build:466: drivers/gpu] Error 2 make[1]: * [/kisskb/src/Makefile:1843: drivers] Error 2 make: * [Makefile:219: __sub-make] Error 2 Command 'make -s -j 32 ARCH=um O=/kisskb/build/linus_um-allyesconfig_um-x86_64 CROSS_COMPILE=/opt/cross/kisskb/fe-x86-64-core-i7-2017.05/bin/x86_64-linux- SUBARCH=x86_64 ' returned non-zero exit status 2 # rm -rf /kisskb/build/linus_um-allyesconfig_um-x86_64 # Build took: 0:04:10.367646

© Michael Ellerman 2006-2018.