From: kernel test robot <lkp@intel.com>
To: kbuild-all@lists.01.org
Subject: Re: [PATCH v5 1/9] MIPS: Avoid future duplicate elf core header reservation
Date: Wed, 11 Aug 2021 22:02:08 +0800 [thread overview]
Message-ID: <202108112108.DkNswavL-lkp@intel.com> (raw)
In-Reply-To: <92b6718f5618d5469f67b48fbea189cca0c12f4b.1628670468.git.geert+renesas@glider.be>
[-- Attachment #1: Type: text/plain, Size: 3328 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: mips-randconfig-r022-20210809 (attached as .config)
compiler: mips64-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/e3a0dd2c8001096543634dfedc74e52f819bd528
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 e3a0dd2c8001096543634dfedc74e52f819bd528
# 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=mips SHELL=/bin/bash
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 >>):
arch/mips/kernel/setup.c: In function 'mips_reserve_vmcore':
>> arch/mips/kernel/setup.c:432:68: error: expected ')' before 'memblock_reserve'
432 | if (!memblock_is_region_reserved(elfcorehdr_addr, elfcorehdr_size)
| ~ ^
| )
433 | memblock_reserve(elfcorehdr_addr, elfcorehdr_size);
| ~~~~~~~~~~~~~~~~
>> arch/mips/kernel/setup.c:435:1: error: expected expression before '}' token
435 | }
| ^
vim +432 arch/mips/kernel/setup.c
408
409 static void __init mips_reserve_vmcore(void)
410 {
411 #ifdef CONFIG_PROC_VMCORE
412 phys_addr_t start, end;
413 u64 i;
414
415 if (!elfcorehdr_size) {
416 for_each_mem_range(i, &start, &end) {
417 if (elfcorehdr_addr >= start && elfcorehdr_addr < end) {
418 /*
419 * Reserve from the elf core header to the end of
420 * the memory segment, that should all be kdump
421 * reserved memory.
422 */
423 elfcorehdr_size = end - elfcorehdr_addr;
424 break;
425 }
426 }
427 }
428
429 pr_info("Reserving %ldKB of memory@%ldKB for kdump\n",
430 (unsigned long)elfcorehdr_size >> 10, (unsigned long)elfcorehdr_addr >> 10);
431
> 432 if (!memblock_is_region_reserved(elfcorehdr_addr, elfcorehdr_size)
433 memblock_reserve(elfcorehdr_addr, elfcorehdr_size);
434 #endif
> 435 }
436
---
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: 29617 bytes --]
next prev parent reply other threads:[~2021-08-11 14:02 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 [this message]
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
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=202108112108.DkNswavL-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.