[lkml]   [2012]   [Feb]   [24]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: [PATCH] oom: add sysctl to enable slab memory dump
On Thu, 23 Feb 2012, David Rientjes wrote:
> > Great! So, why not letting the time tell us if this feature will be obsoleted
> > or not? I'd rather have this patch obsoleted by another one proven better, than
> > just stay still waiting for something that might, or might not, happen in the
> > future.
> Because (1) you're adding a sysctl that we don't want to obsolete and
> remove from the kernel that someone will come to depend on and then have
> to find an alternative solution like /dev/mem_notify, and (2) people parse
> messages like this that are emitted to the kernel log that we don't want
> to break in the future.
> So NACK on this approach.

Right. We should drop the sysctl and make it into a kernel command line
debugging option instead.


 \ /
  Last update: 2012-02-24 07:59    [W:0.215 / U:0.336 seconds]
©2003-2018 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site