Messages in this thread | | | Date | Thu, 22 Jan 2015 18:44:06 +0300 | From | Stefan Strogin <> | Subject | Re: [PATCH 0/3] mm: cma: /proc/cmainfo |
| |
Hello Pavel,
On 02/01/15 08:11, Pavel Machek wrote: > On Mon 2014-12-29 11:36:39, Minchan Kim wrote: >> Hello, >> >> On Fri, Dec 26, 2014 at 05:39:01PM +0300, Stefan I. Strogin wrote: >>> Hello all, >>> >>> Here is a patch set that adds /proc/cmainfo. >>> >>> When compiled with CONFIG_CMA_DEBUG /proc/cmainfo will contain information >>> about about total, used, maximum free contiguous chunk and all currently >>> allocated contiguous buffers in CMA regions. The information about allocated >>> CMA buffers includes pid, comm, allocation latency and stacktrace at the >>> moment of allocation. > We should not add new non-process related files in > /proc. So... NAK. Should this go to debugfs instead?
As you say, I'll move it to debugfs and also split it by CMA region. Something like: /sys/kernel/debug/cma/*/allocated Thanks.
| |