From: Vasiliy Kulikov <segooon@xxxxxxxxx> Use IS_ERR() instead of strict checking. Signed-off-by: Vasiliy Kulikov <segooon@xxxxxxxxx> --- Compile tested. mm/oom_kill.c | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/mm/oom_kill.c b/mm/oom_kill.c index fc81cb2..2ee3350 100644 --- a/mm/oom_kill.c +++ b/mm/oom_kill.c @@ -514,7 +514,7 @@ void mem_cgroup_out_of_memory(struct mem_cgroup *mem, gfp_t gfp_mask) read_lock(&tasklist_lock); retry: p = select_bad_process(&points, limit, mem, NULL); - if (!p || PTR_ERR(p) == -1UL) + if (IS_ERR_OR_NULL(p)) goto out; if (oom_kill_process(p, gfp_mask, 0, points, limit, mem, NULL, @@ -691,7 +691,7 @@ retry: p = select_bad_process(&points, totalpages, NULL, constraint == CONSTRAINT_MEMORY_POLICY ? nodemask : NULL); - if (PTR_ERR(p) == -1UL) + if (IS_ERR(p)) goto out; /* Found nothing?!?! Either we hang forever, or we panic. */ -- 1.7.0.4 -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html