lkml.org 
[lkml]   [2022]   [Jan]   [20]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH v2 0/5] kexec: use IS_ENABLED(CONFIG_KEXEC_CORE) instead of #ifdef
On 12/07/21 at 12:05am, Jisheng Zhang wrote:
> Replace the conditional compilation using "#ifdef CONFIG_KEXEC_CORE"
> by a check for "IS_ENABLED(CONFIG_KEXEC_CORE)", to simplify the code
> and increase compile coverage.

Only checked the x86 patch, but the whole patchset looks good to me,
thanks, Jisheng.

Acked-by: Baoquan He <bhe@redhat.com>

Maybe Andrew can help pick this whole series lest each patch need be taken
care of by its own ARCH maintainer.

>
> I only modify x86, arm, arm64 and riscv, other architectures such as
> sh, powerpc and s390 are better to be kept kexec code as-is so they
> are not touched.
>
> Since v1:
> - collect Reviewed-by tag
> - fix misleading commit msg.
>
> Jisheng Zhang (5):
> kexec: make crashk_res, crashk_low_res and crash_notes symbols always
> visible
> riscv: mm: init: use IS_ENABLED(CONFIG_KEXEC_CORE) instead of #ifdef
> x86/setup: use IS_ENABLED(CONFIG_KEXEC_CORE) instead of #ifdef
> arm64: mm: use IS_ENABLED(CONFIG_KEXEC_CORE) instead of #ifdef
> arm: use IS_ENABLED(CONFIG_KEXEC_CORE) instead of #ifdef
>
> arch/arm/kernel/setup.c | 7 +++----
> arch/arm64/mm/init.c | 9 +++------
> arch/riscv/mm/init.c | 6 ++----
> arch/x86/kernel/setup.c | 10 +++-------
> include/linux/kexec.h | 12 ++++++------
> 5 files changed, 17 insertions(+), 27 deletions(-)
>
> --
> 2.34.1
>

\
 
 \ /
  Last update: 2022-01-20 10:51    [W:0.113 / U:1.928 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site