On Fri, 10 Mar 2023 21:38:43 +0800 Kefeng Wang <wangkefeng.wang@xxxxxxxxxx> wrote: > Use IS_ENABLED(CONFIG_HWPOISON_INJECT) to check whether or not to > enable HWPoison injector module. This is quite clear from the patch itself. What is unclear is "why". Does it fix a build error? If so, please describe and add a Fixes: if appropriate. Is it just a cleanup? etcetera. Thanks. > --- a/mm/memory-failure.c > +++ b/mm/memory-failure.c > @@ -168,7 +168,7 @@ static bool page_handle_poison(struct page *page, bool hugepage_or_freepage, boo > return true; > } > > -#if defined(CONFIG_HWPOISON_INJECT) || defined(CONFIG_HWPOISON_INJECT_MODULE) > +#if IS_ENABLED(CONFIG_HWPOISON_INJECT) > > u32 hwpoison_filter_enable = 0; > u32 hwpoison_filter_dev_major = ~0U; > -- > 2.35.3
