Re: [TCWG CI] Regression caused by linux: parisc: Declare pci_iounmap() parisc version only when CONFIG_PCI enabled

From: Nathan Chancellor <nathan@kernel.org>
To: ci_notify@linaro.org
Cc: Helge Deller <deller@gmx.de>,
	arnd@linaro.org, linaro-toolchain@lists.linaro.org,
	llvm@lists.linux.dev
Subject: Re: [TCWG CI] Regression caused by linux: parisc: Declare pci_iounmap() parisc version only when CONFIG_PCI enabled
Date: Tue, 21 Sep 2021 08:45:53 -0700	[thread overview]
Message-ID: <YUn+MRw4dkxrMouZ@archlinux-ax161> (raw)
In-Reply-To: <342384227.4073.1632234888093@localhost>

On Tue, Sep 21, 2021 at 02:34:47PM +0000, ci_notify@linaro.org wrote:
> [TCWG CI] Regression caused by linux: parisc: Declare pci_iounmap() parisc version only when CONFIG_PCI enabled:
> commit 9caea0007601d3bc6debec04f8b4cd6f4c2394be
> Author: Helge Deller <deller@gmx.de>
> 
>     parisc: Declare pci_iounmap() parisc version only when CONFIG_PCI enabled
> 
> Results regressed to
> # reset_artifacts:
> -10
> # build_abe binutils:
> -9
> # build_llvm:
> -5
> # build_abe qemu:
> -2
> # linux_n_obj:
> 37
> # First few build errors in logs:
> 
> from
> # reset_artifacts:
> -10
> # build_abe binutils:
> -9
> # build_llvm:
> -5
> # build_abe qemu:
> -2
> # linux_n_obj:
> 20151
> 
> THIS IS THE END OF INTERESTING STUFF.  BELOW ARE LINKS TO BUILDS, REPRODUCTION INSTRUCTIONS, AND THE RAW COMMIT.
> 
> This commit has regressed these CI configurations:
>  - tcwg_kernel/llvm-master-aarch64-mainline-allyesconfig
> 
> First_bad build: https://ci.linaro.org/job/tcwg_kernel-llvm-bisect-llvm-master-aarch64-mainline-allyesconfig/17/artifact/artifacts/build-9caea0007601d3bc6debec04f8b4cd6f4c2394be/
> Last_good build: https://ci.linaro.org/job/tcwg_kernel-llvm-bisect-llvm-master-aarch64-mainline-allyesconfig/17/artifact/artifacts/build-31ad37bd6faf871c070650f72ac9488ceeeceeb0/
> Baseline build: https://ci.linaro.org/job/tcwg_kernel-llvm-bisect-llvm-master-aarch64-mainline-allyesconfig/17/artifact/artifacts/build-baseline/
> Even more details: https://ci.linaro.org/job/tcwg_kernel-llvm-bisect-llvm-master-aarch64-mainline-allyesconfig/17/artifact/artifacts/

00:00:38 In file included from arch/arm64/kernel/asm-offsets.c:10:
00:00:38 In file included from ./include/linux/arm_sdei.h:8:
00:00:38 In file included from ./include/acpi/ghes.h:5:
00:00:38 In file included from ./include/acpi/apei.h:9:
00:00:38 In file included from ./include/linux/acpi.h:35:
00:00:38 In file included from ./include/acpi/acpi_io.h:5:
00:00:38 In file included from ./include/linux/io.h:13:
00:00:38 In file included from ./arch/arm64/include/asm/io.h:185:
00:00:38 ./include/asm-generic/io.h:1060:20: error: static declaration of 'pci_iounmap' follows non-static declaration
00:00:38 static inline void pci_iounmap(struct pci_dev *dev, void __iomem *p)
00:00:38                    ^
00:00:38 ./include/asm-generic/io.h:1059:21: note: expanded from macro 'pci_iounmap'
00:00:38 #define pci_iounmap pci_iounmap
00:00:38                     ^
00:00:38 ./include/asm-generic/pci_iomap.h:21:13: note: previous declaration is here
00:00:38 extern void pci_iounmap(struct pci_dev *dev, void __iomem *);
00:00:38             ^
00:00:38 1 error generated.
00:00:38 make[1]: *** [scripts/Makefile.build:121: arch/arm64/kernel/asm-offsets.s] Error 1
00:00:38 make[1]: Target '__build' not remade because of errors.
00:00:38 make: *** [Makefile:1219: prepare0] Error 2
00:00:38 make: Target '__all' not remade because of errors.

Already reported and fixed:

https://lore.kernel.org/r/YUeriU9EIJ5hiFjL@archlinux-ax161/
316e8d79a095 ("pci_iounmap'2: Electric Boogaloo: try to make sense of it all")

Cheers,
Nathan