mm/memory-failure.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-)
From: Ye Liu <liuye@kylinos.cn>
'forcekill' is used as a boolean flag to control whether
processes should be forcibly killed. It is only assigned
from boolean expressions and never used in arithmetic or
bitmask operations.
Convert it from int to bool.
No functional change intended.
Signed-off-by: Ye Liu <liuye@kylinos.cn>
---
Changes in v2:
- Explicitly boolify the MF_MUST_KILL flag test at the kill_procs()
call site
- Link to v1:https://lore.kernel.org/all/20260407020715.2269255-3-ye.liu@linux.dev/
mm/memory-failure.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/mm/memory-failure.c b/mm/memory-failure.c
index d25adb390c3e..3babddadff04 100644
--- a/mm/memory-failure.c
+++ b/mm/memory-failure.c
@@ -459,7 +459,7 @@ void add_to_kill_ksm(struct task_struct *tsk, const struct page *p,
* Only do anything when FORCEKILL is set, otherwise just free the
* list (this is used for clean pages which do not need killing)
*/
-static void kill_procs(struct list_head *to_kill, int forcekill,
+static void kill_procs(struct list_head *to_kill, bool forcekill,
unsigned long pfn, int flags)
{
struct to_kill *tk, *next;
@@ -1582,7 +1582,7 @@ static bool hwpoison_user_mappings(struct folio *folio, struct page *p,
{
LIST_HEAD(tokill);
bool unmap_success;
- int forcekill;
+ bool forcekill;
bool mlocked = folio_test_mlocked(folio);
/*
@@ -1703,7 +1703,7 @@ static void unmap_and_kill(struct list_head *to_kill, unsigned long pfn,
unmap_mapping_range(mapping, start, size, 0);
}
- kill_procs(to_kill, flags & MF_MUST_KILL, pfn, flags);
+ kill_procs(to_kill, !!(flags & MF_MUST_KILL), pfn, flags);
}
/*
--
2.43.0
On 2026/4/10 15:47, Ye Liu wrote: > From: Ye Liu <liuye@kylinos.cn> > > 'forcekill' is used as a boolean flag to control whether > processes should be forcibly killed. It is only assigned > from boolean expressions and never used in arithmetic or > bitmask operations. > > Convert it from int to bool. > > No functional change intended. > > Signed-off-by: Ye Liu <liuye@kylinos.cn> Acked-by: Miaohe Lin <linmiaohe@huawei.com> Thanks. .
On Fri, 10 Apr 2026 15:47:39 +0800 Ye Liu <ye.liu@linux.dev> wrote: > From: Ye Liu <liuye@kylinos.cn> > > 'forcekill' is used as a boolean flag to control whether > processes should be forcibly killed. It is only assigned > from boolean expressions and never used in arithmetic or > bitmask operations. > > Convert it from int to bool. > > No functional change intended. > > Signed-off-by: Ye Liu <liuye@kylinos.cn> Reviewed-by: SeongJae Park <sj@kernel.org> Thanks, SJ [...]
© 2016 - 2026 Red Hat, Inc.