Messages in this thread | | | Date | Fri, 21 Aug 2015 09:38:58 -0700 | From | Jörn Engel <> | Subject | Re: [PATCH v5 2/2] mm: hugetlb: proc: add HugetlbPages field to /proc/PID/status |
| |
On Fri, Aug 21, 2015 at 08:32:33AM +0200, Michal Hocko wrote: > > Both mmotm and linus tree have > REG("smaps", S_IRUGO, proc_pid_smaps_operations), > > and opening the file requires PTRACE_MODE_READ. So I do not see any > requirement for root here. Or did you mean that you need root to examine > all processes? That would be true but I am wondering why would be a regular > user interested in this break out numbers. Hugetlb management sounds > pretty much like an administrative or very specialized thing. > > From my understanding of the discussion there is no usecase to have this > information world readable. Is this correct?
Well, tools like top currently display rss. Once we have some interface, I would like a version of top that displays the true rss including hugepages (hrss maybe?).
If we make such a tool impossible today, someone will complain about it in the future and we created a new mess for ourselves. I think it is trouble enough to deal with the old one.
Jörn
-- Denying any reality for any laudable political goal is a bad strategy. When the facts come out, the discovery of the facts will undermine the laudable political goals. -- Jared Diamond
| |