From: kernel test robot <lkp@intel.com>
To: kbuild-all@lists.01.org
Subject: Re: [PATCH v5 3/9] of: fdt: Add generic support for handling elf core headers property
Date: Thu, 12 Aug 2021 04:46:22 +0800 [thread overview]
Message-ID: <202108120456.1UML5wZM-lkp@intel.com> (raw)
In-Reply-To: <c7e46e50aaf87ef49bdaa61358d25b122f32b7df.1628670468.git.geert+renesas@glider.be>
[-- Attachment #1: Type: text/plain, Size: 4218 bytes --]
Hi Geert,
I love your patch! Yet something to improve:
[auto build test ERROR on robh/for-next]
[also build test ERROR on arm64/for-next/core arm/for-next linus/master v5.14-rc5 next-20210811]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]
url: https://github.com/0day-ci/linux/commits/Geert-Uytterhoeven/Add-generic-support-for-kdump-DT-properties/20210811-165455
base: https://git.kernel.org/pub/scm/linux/kernel/git/robh/linux.git for-next
config: ia64-allmodconfig (attached as .config)
compiler: ia64-linux-gcc (GCC) 10.3.0
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# https://github.com/0day-ci/linux/commit/34ef57fdde286c4999f0deea8ee16d56d2ac3bae
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Geert-Uytterhoeven/Add-generic-support-for-kdump-DT-properties/20210811-165455
git checkout 34ef57fdde286c4999f0deea8ee16d56d2ac3bae
# save the attached .config to linux build tree
mkdir build_dir
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-10.3.0 make.cross O=build_dir ARCH=ia64 SHELL=/bin/bash drivers/
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
All errors (new ones prefixed by >>):
>> drivers/of/fdt.c:593:20: error: conflicting types for 'reserve_elfcorehdr'
593 | static void __init reserve_elfcorehdr(void)
| ^~~~~~~~~~~~~~~~~~
In file included from arch/ia64/include/asm/mmzone.h:16,
from arch/ia64/include/asm/nodedata.h:17,
from arch/ia64/include/asm/processor.h:81,
from arch/ia64/include/asm/thread_info.h:12,
from include/linux/thread_info.h:60,
from include/asm-generic/preempt.h:5,
from ./arch/ia64/include/generated/asm/preempt.h:1,
from include/linux/preempt.h:78,
from include/linux/rcupdate.h:27,
from include/linux/rculist.h:11,
from include/linux/pid.h:5,
from include/linux/sched.h:14,
from include/linux/ptrace.h:6,
from arch/ia64/include/asm/user.h:32,
from include/linux/user.h:1,
from include/linux/elfcore.h:5,
from include/linux/crash_core.h:6,
from include/linux/kexec.h:18,
from include/linux/crash_dump.h:5,
from drivers/of/fdt.c:11:
arch/ia64/include/asm/meminit.h:43:12: note: previous declaration of 'reserve_elfcorehdr' was here
43 | extern int reserve_elfcorehdr(u64 *start, u64 *end);
| ^~~~~~~~~~~~~~~~~~
vim +/reserve_elfcorehdr +593 drivers/of/fdt.c
584
585 /*
586 * reserve_elfcorehdr() - reserves memory for elf core header
587 *
588 * This function reserves the memory occupied by an elf core header
589 * described in the device tree. This region contains all the
590 * information about primary kernel's core image and is used by a dump
591 * capture kernel to access the system memory on primary kernel.
592 */
> 593 static void __init reserve_elfcorehdr(void)
594 {
595 if (!IS_ENABLED(CONFIG_CRASH_DUMP) || !elfcorehdr_size)
596 return;
597
598 if (memblock_is_region_reserved(elfcorehdr_addr, elfcorehdr_size)) {
599 pr_warn("elfcorehdr is overlapped\n");
600 return;
601 }
602
603 memblock_reserve(elfcorehdr_addr, elfcorehdr_size);
604
605 pr_info("Reserving %llu KiB of memory at 0x%llx for elfcorehdr\n",
606 elfcorehdr_size >> 10, elfcorehdr_addr);
607 }
608
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org
[-- Attachment #2: config.gz --]
[-- Type: application/gzip, Size: 64456 bytes --]
next prev parent reply other threads:[~2021-08-11 20:46 UTC|newest]
Thread overview: 95+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-08-11 8:50 [PATCH v5 0/9] Add generic support for kdump DT properties Geert Uytterhoeven
2021-08-11 8:50 ` Geert Uytterhoeven
2021-08-11 8:50 ` Geert Uytterhoeven
2021-08-11 8:50 ` Geert Uytterhoeven
2021-08-11 8:50 ` [PATCH v5 1/9] MIPS: Avoid future duplicate elf core header reservation Geert Uytterhoeven
2021-08-11 8:50 ` Geert Uytterhoeven
2021-08-11 8:50 ` Geert Uytterhoeven
2021-08-11 8:50 ` Geert Uytterhoeven
2021-08-11 14:02 ` kernel test robot
2021-08-11 15:35 ` Geert Uytterhoeven
2021-08-11 15:35 ` Geert Uytterhoeven
2021-08-11 15:35 ` Geert Uytterhoeven
2021-08-11 15:35 ` Geert Uytterhoeven
2021-08-16 5:52 ` Mike Rapoport
2021-08-16 5:52 ` Mike Rapoport
2021-08-16 5:52 ` Mike Rapoport
2021-08-16 5:52 ` Mike Rapoport
2021-08-23 10:17 ` Geert Uytterhoeven
2021-08-23 10:17 ` Geert Uytterhoeven
2021-08-23 10:17 ` Geert Uytterhoeven
2021-08-23 10:17 ` Geert Uytterhoeven
2021-08-23 13:09 ` Mike Rapoport
2021-08-23 13:09 ` Mike Rapoport
2021-08-23 13:09 ` Mike Rapoport
2021-08-23 13:09 ` Mike Rapoport
2021-08-23 14:44 ` Rob Herring
2021-08-23 14:44 ` Rob Herring
2021-08-23 14:44 ` Rob Herring
2021-08-23 14:44 ` Rob Herring
2021-08-23 15:20 ` Mike Rapoport
2021-08-23 15:20 ` Mike Rapoport
2021-08-23 15:20 ` Mike Rapoport
2021-08-23 15:20 ` Mike Rapoport
2021-08-11 8:51 ` [PATCH v5 2/9] crash_dump: Make elfcorehdr address/size symbols always visible Geert Uytterhoeven
2021-08-11 8:51 ` Geert Uytterhoeven
2021-08-11 8:51 ` Geert Uytterhoeven
2021-08-11 8:51 ` Geert Uytterhoeven
2021-08-11 8:51 ` [PATCH v5 3/9] of: fdt: Add generic support for handling elf core headers property Geert Uytterhoeven
2021-08-11 8:51 ` Geert Uytterhoeven
2021-08-11 8:51 ` Geert Uytterhoeven
2021-08-11 8:51 ` Geert Uytterhoeven
2021-08-11 20:46 ` kernel test robot [this message]
2021-08-12 8:16 ` Geert Uytterhoeven
2021-08-11 8:51 ` [PATCH v5 4/9] of: fdt: Add generic support for handling usable memory range property Geert Uytterhoeven
2021-08-11 8:51 ` Geert Uytterhoeven
2021-08-11 8:51 ` Geert Uytterhoeven
2021-08-11 8:51 ` Geert Uytterhoeven
2021-08-11 8:51 ` [PATCH v5 5/9] of: fdt: Use IS_ENABLED(CONFIG_BLK_DEV_INITRD) instead of #ifdef Geert Uytterhoeven
2021-08-11 8:51 ` Geert Uytterhoeven
2021-08-11 8:51 ` Geert Uytterhoeven
2021-08-11 8:51 ` Geert Uytterhoeven
2021-08-11 8:51 ` [PATCH v5 6/9] riscv: Remove non-standard linux,elfcorehdr handling Geert Uytterhoeven
2021-08-11 8:51 ` Geert Uytterhoeven
2021-08-11 8:51 ` Geert Uytterhoeven
2021-08-11 8:51 ` Geert Uytterhoeven
2021-08-11 8:51 ` [PATCH v5 7/9] arm64: kdump: Remove custom " Geert Uytterhoeven
2021-08-11 8:51 ` Geert Uytterhoeven
2021-08-11 8:51 ` Geert Uytterhoeven
2021-08-11 8:51 ` Geert Uytterhoeven
2021-08-23 12:55 ` Catalin Marinas
2021-08-23 12:55 ` Catalin Marinas
2021-08-23 12:55 ` Catalin Marinas
2021-08-23 12:55 ` Catalin Marinas
2021-08-11 8:51 ` [PATCH v5 8/9] arm64: kdump: Remove custom linux,usable-memory-range handling Geert Uytterhoeven
2021-08-11 8:51 ` [PATCH v5 8/9] arm64: kdump: Remove custom linux, usable-memory-range handling Geert Uytterhoeven
2021-08-11 8:51 ` Geert Uytterhoeven
2021-08-11 8:51 ` Geert Uytterhoeven
2021-08-23 12:55 ` [PATCH v5 8/9] arm64: kdump: Remove custom linux,usable-memory-range handling Catalin Marinas
2021-08-23 12:55 ` Catalin Marinas
2021-08-23 12:55 ` Catalin Marinas
2021-08-23 12:55 ` Catalin Marinas
2021-08-11 8:51 ` [PATCH v5 9/9] ARM: uncompress: Parse "linux,usable-memory-range" DT property Geert Uytterhoeven
2021-08-11 8:51 ` [PATCH v5 9/9] ARM: uncompress: Parse "linux, usable-memory-range" " Geert Uytterhoeven
2021-08-11 8:51 ` Geert Uytterhoeven
2021-08-11 8:51 ` Geert Uytterhoeven
2021-08-15 15:25 ` [PATCH v5 0/9] Add generic support for kdump DT properties Rob Herring
2021-08-15 15:25 ` Rob Herring
2021-08-15 15:25 ` Rob Herring
2021-08-15 15:25 ` Rob Herring
2021-08-23 10:13 ` Geert Uytterhoeven
2021-08-23 10:13 ` Geert Uytterhoeven
2021-08-23 10:13 ` Geert Uytterhoeven
2021-08-23 10:13 ` Geert Uytterhoeven
2021-08-23 14:52 ` Rob Herring
2021-08-23 14:52 ` Rob Herring
2021-08-23 14:52 ` Rob Herring
2021-08-23 14:52 ` Rob Herring
2021-08-24 11:55 ` Geert Uytterhoeven
2021-08-24 11:55 ` Geert Uytterhoeven
2021-08-24 11:55 ` Geert Uytterhoeven
2021-08-24 11:55 ` Geert Uytterhoeven
2021-08-24 22:43 ` Rob Herring
2021-08-24 22:43 ` Rob Herring
2021-08-24 22:43 ` Rob Herring
2021-08-24 22:43 ` Rob Herring
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=202108120456.1UML5wZM-lkp@intel.com \
--to=lkp@intel.com \
--cc=kbuild-all@lists.01.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.