On Mon, Jul 20, 2020 at 7:20 AM Tetsuo Handa <penguin-kernel@xxxxxxxxxxxxxxxxxxx> wrote: > > On 2020/07/19 22:53, Yafang Shao wrote: > > p = find_lock_task_mm(victim); > > if (!p) { > > + pr_info("Process %d (%s) is already exiting\n", > > + task_pid_nr(victim), victim->comm); > > Maybe > > + pr_info("%s: Process %d (%s) is already exiting\n", > + message, task_pid_nr(victim), victim->comm); > > line is easier to compare with > > pr_err("%s: Killed process %d (%s) total-vm:%lukB, anon-rss:%lukB, file-rss:%lukB, shmem-rss:%lukB, UID:%u pgtables:%lukB oom_score_adj:%hd\n", > message, task_pid_nr(victim), victim->comm, K(mm->total_vm), > > line? > Seems so. Thanks for the suggestion. > > put_task_struct(victim); > > return; -- Thanks Yafang