> -----Original Message----- > From: Andrew Morton [mailto:akpm@xxxxxxxxxxxxxxxxxxxx] > Sent: Tuesday, February 11, 2014 5:51 AM > To: Johannes Weiner > Cc: Rik van Riel; Dave Hansen; Michal Hocko; Motohiro Kosaki JP; KAMEZAWA > Hiroyuki; linux-mm@xxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx > Subject: Re: [patch] drop_caches: add some documentation and info > message > > On Fri, 7 Feb 2014 16:26:01 -0500 Johannes Weiner <hannes@xxxxxxxxxxx> > wrote: > > > On Fri, Feb 07, 2014 at 12:31:29PM -0800, Andrew Morton wrote: > > > On Fri, 7 Feb 2014 13:13:32 -0500 Johannes Weiner > <hannes@xxxxxxxxxxx> wrote: > > > > > > > @@ -63,6 +64,9 @@ int drop_caches_sysctl_handler(ctl_table *table, > int write, > > > > iterate_supers(drop_pagecache_sb, NULL); > > > > if (sysctl_drop_caches & 2) > > > > drop_slab(); > > > > + printk_ratelimited(KERN_INFO "%s (%d): dropped kernel > caches: %d\n", > > > > + current->comm, task_pid_nr(current), > > > > + sysctl_drop_caches); > > > > } > > > > return 0; > > > > } > > > > > > My concern with this is that there may be people whose > > > other-party-provided software uses drop_caches. Their machines will > > > now sit there emitting log messages and there's nothing they can do > > > about it, apart from whining at their vendors. > > > > Ironically, we have a customer that is complaining that we currently > > do not log these events, and they want to know who in their stack is > > being idiotic. > > Right. But if we release a kernel which goes blah on every write to > drop_caches, that customer has logs full of blahs which they are now totally > uninterested in. Please let me know if I misunderstand something. This patch uses KERN_INFO. Then, any log shouldn't be emitted by default. Moreover, if someone change syslog log level to INFO, they are going to see much prenty annoying and too much logs even if we reject this patch. -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@xxxxxxxxx. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: <a href