[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <eb77e210-1088-d140-043f-088141deb94d@i-love.sakura.ne.jp>
Date: Sat, 12 Jun 2021 09:23:03 +0900
From: Tetsuo Handa <penguin-kernel@...ove.sakura.ne.jp>
To: Waiman Long <llong@...hat.com>, Aaron Tomlin <atomlin@...hat.com>,
linux-mm@...ck.org
Cc: akpm@...ux-foundation.org, vbabka@...e.cz, mhocko@...e.com,
linux-kernel@...r.kernel.org
Subject: Re: [RFC PATCH] mm/oom_kill: show oom eligibility when displaying the
current memory state of all tasks
On 2021/06/12 2:42, Waiman Long wrote:
> A minor nit:
>
> "oom eligible?" has 13 characters. The field width is 15. Maybe you should pad 2 more spaces to make the proper alignment.
Maybe single character (e.g. 'S' for OOM_SCORE_ADJ_MIN, 'R' for MMF_OOM_SKIP, 'V' for in_vfork(),
none for eligible) is better because dump_task() might print many hundreds of lines?
Powered by blists - more mailing lists