Re: [PATCH] kexec: update manpage with explicit mention of clean kexec

2023-08-27 Thread Hari Bathini
Any comments on this, please.. On 07/07/23 10:59 am, Hari Bathini wrote: While the manpage does mention about kexec boot with a clean shutdown, it is not explicit about it. Make it explicit. Signed-off-by: Hari Bathini --- kexec/kexec.8 | 12 ++-- 1 file changed, 10 insertions(+), 2

Re: [PATCH 8/8] crash_core.c: remove unneeded functions

2023-08-27 Thread Samuel Holland
Hi Baoquan, On 2023-08-27 5:11 AM, Baoquan He wrote: > So far, nobody calls functions parse_crashkernel_high() and > parse_crashkernel_high(), remove both of them. minor typo: should be parse_crashkernel_low(). Regards, Samuel > Signed-off-by: Baoquan He > --- > include/linux/crash_core.h |

Re: [PATCH 5/8] crash_core: add generic function to do reservation

2023-08-27 Thread kernel test robot
us a note. And when submitting patch, we suggest to use '--base' as documented in https://git-scm.com/docs/git-format-patch#_base_tree_information] url: https://github.com/intel-lab-lkp/linux/commits/Baoquan-He/crash_core-c-remove-unnecessary-parameter-of-function/20230827-181555 base:

Re: [RFC PATCH 0/4] kdump: add generic functions to simplify crashkernel crashkernel in architecture

2023-08-27 Thread Baoquan He
Hi Dave, Philipp On 07/10/23 at 07:20pm, Philipp Rudo wrote: > Hi Baoquan, > Hi Dave, > > On Sat, 8 Jul 2023 10:15:53 +0800 > Dave Young wrote: > > > On 06/19/23 at 01:59pm, Baoquan He wrote: > > > In the current arm64, crashkernel=,high support has been finished after > > > several rounds of p

[PATCH 8/8] crash_core.c: remove unneeded functions

2023-08-27 Thread Baoquan He
So far, nobody calls functions parse_crashkernel_high() and parse_crashkernel_high(), remove both of them. Signed-off-by: Baoquan He --- include/linux/crash_core.h | 4 kernel/crash_core.c| 18 -- 2 files changed, 22 deletions(-) diff --git a/include/linux/crash_co

[PATCH 6/8] arm64: kdump: use generic interface to simplify crashkernel reservation

2023-08-27 Thread Baoquan He
With the help of newly changed function parse_crashkernel() and generic reserve_crashkernel_generic(), crashkernel reservation can be simplified by steps: 1) Provide CRASH_ALIGN, CRASH_ADDR_LOW_MAX, CRASH_ADDR_HIGH_MAX and DEFAULT_CRASH_KERNEL_LOW_SIZE in ; 2) Add arch_reserve_crashkernel() to

[PATCH 7/8] x86: kdump: use generic interface to simplify crashkernel reservation code

2023-08-27 Thread Baoquan He
With the help of newly changed function parse_crashkernel() and generic reserve_crashkernel_generic(), crashkernel reservation can be simplified by steps: 1) Provide CRASH_ALIGN, CRASH_ADDR_LOW_MAX, CRASH_ADDR_HIGH_MAX and DEFAULT_CRASH_KERNEL_LOW_SIZE in ; 2) Add arch_reserve_crashkernel() to

[PATCH 5/8] crash_core: add generic function to do reservation

2023-08-27 Thread Baoquan He
In architecture like x86_64, arm64 and riscv, they have vast virtual address space and usually have huge physical memory RAM. Their crashkernel reservation doesn't have to be limited under 4G RAM, but can be extended to the whole physical memory via crashkernel=,high support. Now add function rese

[PATCH 2/8] crash_core: change the prototype of function parse_crashkernel()

2023-08-27 Thread Baoquan He
Add two parameters 'low_size' and 'high' to function parse_crashkernel(), later crashkernel=,high|low parsing will be added. Make adjustments in all call sites of parse_crashkernel() in arch. Signed-off-by: Baoquan He --- arch/arm/kernel/setup.c | 3 ++- arch/arm64/mm/init.c

[PATCH 4/8] crash_core: change parse_crashkernel() to support crashkernel=,high|low parsing

2023-08-27 Thread Baoquan He
Now parse_crashkernel() is a real entry point for all kinds of crahskernel parsing on any architecture. And wrap the crahskernel=,high|low handling inside CONFIG_ARCH_HAS_GENERIC_CRASHKERNEL_RESERVATION ifdeffery scope. Signed-off-by: Baoquan He --- include/linux/crash_core.h | 6 ++ kerne

[PATCH 3/8] include/linux/kexec.h: move down crash_core.h including

2023-08-27 Thread Baoquan He
Later generic macros related to crash_core will be added into , and the corresponding arch specific macros will be added to override them if needed. And Kconfig option KEXEC_CORE selects CRASH_CORE. So move including after . And also move the crash_res|low_res and crash_notes delcarations after

[PATCH 0/8] kdump: use generic functions to simplify crashkernel reservation in architectures

2023-08-27 Thread Baoquan He
In the current arm64, crashkernel=,high support has been finished after several rounds of posting and careful reviewing. The code in arm64 which parses crashkernel kernel parameters firstly, then reserve memory can be a good example for other ARCH to refer to. Whereas in x86_64, the code mixing cr

[PATCH 1/8] crash_core.c: remove unnecessary parameter of function

2023-08-27 Thread Baoquan He
In all call sites of __parse_crashkernel(), the parameter 'name' is hardcoded as "crashkernel=". So remove the unnecessary parameter 'name', add local varibale 'name' inside __parse_crashkernel() instead. Signed-off-by: Baoquan He --- kernel/crash_core.c | 8 1 file changed, 4 insertion