Hi Michal and David, AFAICS, the null pointer is the task_struct of p in the dump_header. Why don't we just add a if before printing the victim task's informaiton.
diff --git a/mm/oom_kill.c b/mm/oom_kill.c --- a/mm/oom_kill.c +++ b/mm/oom_kill.c @@ -438,14 +438,6 @@ static void dump_header(struct oom_control *oc, struct task_struct *p) cpuset_print_current_mems_allowed(); mem_cgroup_print_oom_context(oc->memcg, p); - pr_cont(",task=%s,pid=%d,uid=%d\n", p->comm, p->pid, - from_kuid(&init_user_ns, task_uid(p))); + if (p) + pr_cont(",task=%s,pid=%d,uid=%d", p->comm, p->pid, + from_kuid(&init_user_ns, task_uid(p))); + pr_cont("\n"); if (is_memcg_oom(oc)) mem_cgroup_print_oom_meminfo(oc->memcg); Thanks Zhoujian