lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Sat, 29 Aug 2020 14:15:21 +0900 From: Masahiro Yamada <masahiroy@...nel.org> To: Tony Luck <tony.luck@...el.com>, Fenghua Yu <fenghua.yu@...el.com>, linux-ia64@...r.kernel.org Cc: Randy Dunlap <rdunlap@...radead.org>, linux-kbuild@...r.kernel.org, Masahiro Yamada <masahiroy@...nel.org>, Andrew Morton <akpm@...ux-foundation.org>, Ard Biesheuvel <ardb@...nel.org>, Arnd Bergmann <arnd@...db.de>, Dmitry Safonov <0x7f454c46@...il.com>, afzal mohammed <afzal.mohd.ma@...il.com>, linux-efi@...r.kernel.org, linux-kernel@...r.kernel.org Subject: [PATCH 0/3] ia64: clean-up header dependency and build process, fix build warning Randy Dunlap reports the following warning with CONFIG_IA64_PALINFO=m: ../scripts/Makefile.build:68: 'arch/ia64/kernel/palinfo.ko' will not be built even though obj-m is specified. ../scripts/Makefile.build:69: You cannot use subdir-y/m to visit a module Makefile. Use obj-y/m instead. This comes from the fact Kbuild descends into arch/ia64/kernel/ twice. First, to generate <generated/nr-irqs.h>, Second, to build kernel and module objects. The warning is emitted in the first descend because it is not the intended usage. I looked into the code closely, and noticed arch/ia64/kernel/nr-irqs.c was not needed in the first place. It was separated out of arch/ia64/kernel/asm-offsets.c just because <asm/mca.h> was including too many bogus headers. IA64 is not actively maintained, and there exists unneeded obsolete code. The first two patches are the outcome when I played with ARCH=ia64 builds, but not prerequisites for 3/3. Anyway I believe they are nice cleanups and folded in this patch set. 3/3 is the important one to fix the false positive warning, and it is a nice cleanup too. Masahiro Yamada (3): ia64: do not typedef struct pal_min_state_area_s ia64: remove unneeded header includes from <asm/mca.h> ia64: remove generated/nr-irqs.h generation to fix build warning arch/ia64/Makefile | 6 ------ arch/ia64/include/asm/irq.h | 4 +++- arch/ia64/include/asm/mca.h | 11 ++++------- arch/ia64/include/asm/pal.h | 4 ++-- arch/ia64/include/asm/sal.h | 2 +- arch/ia64/kernel/Makefile | 5 ----- arch/ia64/kernel/asm-offsets.c | 18 +++++++++--------- arch/ia64/kernel/efi.c | 1 + arch/ia64/kernel/mca.c | 5 +++-- arch/ia64/kernel/mca_drv.c | 2 +- arch/ia64/kernel/nr-irqs.c | 22 ---------------------- 11 files changed, 24 insertions(+), 56 deletions(-) delete mode 100644 arch/ia64/kernel/nr-irqs.c -- 2.25.1
Powered by blists - more mailing lists