diff mbox series

[RFT] ia64: Fix build error with !COREDUMP

Message ID 20200604201842.29482-1-krzk@kernel.org
State Superseded
Headers show
Series [RFT] ia64: Fix build error with !COREDUMP | expand

Commit Message

Krzysztof Kozlowski June 4, 2020, 8:18 p.m. UTC
Fix linkage error when CONFIG_BINFMT_ELF is selected but CONFIG_COREDUMP
is not:

    ia64-linux-ld: arch/ia64/kernel/elfcore.o: in function `elf_core_write_extra_phdrs':
    elfcore.c:(.text+0x172): undefined reference to `dump_emit'
    ia64-linux-ld: arch/ia64/kernel/elfcore.o: in function `elf_core_write_extra_data':
    elfcore.c:(.text+0x2b2): undefined reference to `dump_emit'

Cc: <stable@vger.kernel.org>
Fixes: 1fcccbac89f5 ("elf coredump: replace ELF_CORE_EXTRA_* macros by functions")
Reported-by: kernel test robot <lkp@intel.com>
Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>

---

Please let kbuild test it for a while before applying. I built it only
on few configurations.

This is similar fix to commit 42d91f612c87 ("um: Fix build error and
kconfig for i386") although I put different fixes tag - the commit which
introduced this part of code.
---
 arch/ia64/kernel/Makefile | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Krzysztof Kozlowski July 7, 2020, 6:34 p.m. UTC | #1
On Thu, Jun 04, 2020 at 10:18:42PM +0200, Krzysztof Kozlowski wrote:
> Fix linkage error when CONFIG_BINFMT_ELF is selected but CONFIG_COREDUMP

> is not:

> 

>     ia64-linux-ld: arch/ia64/kernel/elfcore.o: in function `elf_core_write_extra_phdrs':

>     elfcore.c:(.text+0x172): undefined reference to `dump_emit'

>     ia64-linux-ld: arch/ia64/kernel/elfcore.o: in function `elf_core_write_extra_data':

>     elfcore.c:(.text+0x2b2): undefined reference to `dump_emit'

> 

> Cc: <stable@vger.kernel.org>

> Fixes: 1fcccbac89f5 ("elf coredump: replace ELF_CORE_EXTRA_* macros by functions")

> Reported-by: kernel test robot <lkp@intel.com>

> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>

> 

> ---

> 

> Please let kbuild test it for a while before applying. I built it only

> on few configurations.

> 

> This is similar fix to commit 42d91f612c87 ("um: Fix build error and

> kconfig for i386") although I put different fixes tag - the commit which

> introduced this part of code.


Kernel-test-robot did not complain anymore so I think build tests
passed.

Any comments here?

Best regards,
Krzysztof
diff mbox series

Patch

diff --git a/arch/ia64/kernel/Makefile b/arch/ia64/kernel/Makefile
index 1a8df6669eee..18d6008b151f 100644
--- a/arch/ia64/kernel/Makefile
+++ b/arch/ia64/kernel/Makefile
@@ -41,7 +41,7 @@  obj-y				+= esi_stub.o	# must be in kernel proper
 endif
 obj-$(CONFIG_INTEL_IOMMU)	+= pci-dma.o
 
-obj-$(CONFIG_BINFMT_ELF)	+= elfcore.o
+obj-$(CONFIG_ELF_CORE)		+= elfcore.o
 
 # fp_emulate() expects f2-f5,f16-f31 to contain the user-level state.
 CFLAGS_traps.o  += -mfixed-range=f2-f5,f16-f31