lkml.org 
[lkml]   [2012]   [Feb]   [22]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] oom: add sysctl to enable slab memory dump
On Wed, 22 Feb 2012, Rafael Aquini wrote:

> Adds a new sysctl, 'oom_dump_slabs', that enables the kernel to produce a
> dump of all eligible system slab caches when performing an OOM-killing.
> Information includes per cache active objects, total objects, object size,
> cache name and cache size.
>
> The eligibility for being reported is given by an auxiliary sysctl,
> 'oom_dump_slabs_ratio', which express (in percentage) the memory committed
> ratio between a particular cache size and the total slab size.
>
> This, alongside with all other data dumped in OOM events, is very helpful
> information in diagnosing why there was an OOM condition specially when
> kernel code is under investigation.
>

I don't like this because it duplicates what is given by /proc/slabinfo
that can easily be read at the time of oom and is unnecessary to dump to
the kernel log. We display the meminfo (which includes the amount of
slab, just not broken down by cache) because it's absolutely necessary to
understand why the oom was triggered. The tasklist dump is allowed
because it's difficult to attain all that information easily and to
determine which threads are eligible in the oom context (global, memcg,
cpuset, mempolicy) so they matter to the oom condition. The per-cache
slabinfo fits neither of that criteria and just duplicates code in the
slab allocators that is attainable elsewhere.

I think this also gives another usecase for a possible /dev/mem_notify in
the future: userspace could easily poll on an eventfd and wait for an oom
to occur and then cat /proc/slabinfo to attain all this. In other words,
if we had this functionality (which I think we undoubtedly will in the
future), this patch would be obsoleted.


\
 
 \ /
  Last update: 2012-02-23 01:47    [W:0.098 / U:0.160 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site