Re: [PATCH] add some drop_caches documentation and info messsge

2012-11-01 Thread Pavel Machek
Hi! > > > hmpf. This patch worries me. If there are people out there who are > > > regularly using drop_caches because the VM sucks, it seems pretty > > > obnoxious of us to go dumping stuff into their syslog. What are they > > > supposed to do? Stop using drop_caches? > > > > People use drop

Re: [PATCH] add some drop_caches documentation and info messsge

2012-10-31 Thread Borislav Petkov
On Wed, Oct 31, 2012 at 06:31:54PM +0100, Pavel Machek wrote: > Hmm? When I resume from hibernate, I want to use my machine. Well, in my case with a workstation with 8 Gb, the only time the swapin is noticeable is when I try to use firefox with a couple of dozens tabs open. Once that thing is swap

Re: [PATCH] add some drop_caches documentation and info messsge

2012-10-31 Thread Pavel Machek
On Mon 2012-10-29 10:58:19, Borislav Petkov wrote: > On Mon, Oct 29, 2012 at 09:59:59AM +0100, Jiri Kosina wrote: > > You might or might not want to do that. Dropping caches around suspend > > makes the hibernation process itself faster, but the realtime response > > of the applications afterwards

Re: [PATCH] add some drop_caches documentation and info messsge

2012-10-29 Thread Borislav Petkov
On Mon, Oct 29, 2012 at 11:01:59AM +0100, Jiri Kosina wrote: > Well if the point of dropping caches is lowering the resume time, then > the point is rendered moot as soon as you switch to your browser and > have to wait noticeable amount of time until it starts reacting. Not the resume time - the

Re: [PATCH] add some drop_caches documentation and info messsge

2012-10-29 Thread Jiri Kosina
On Mon, 29 Oct 2012, Borislav Petkov wrote: > > You might or might not want to do that. Dropping caches around suspend > > makes the hibernation process itself faster, but the realtime response > > of the applications afterwards is worse, as everything touched by user > > has to be paged in again.

Re: [PATCH] add some drop_caches documentation and info messsge

2012-10-29 Thread Borislav Petkov
On Mon, Oct 29, 2012 at 09:59:59AM +0100, Jiri Kosina wrote: > You might or might not want to do that. Dropping caches around suspend > makes the hibernation process itself faster, but the realtime response > of the applications afterwards is worse, as everything touched by user > has to be paged i

Re: [PATCH] add some drop_caches documentation and info messsge

2012-10-29 Thread Jiri Kosina
On Wed, 24 Oct 2012, Andrew Morton wrote: > > > > I have drop_caches in my suspend-to-disk script so that the hibernation > > > > image is kept at minimum and suspend times are as small as possible. > > > > > > hm, that sounds smart. > > > > > > > Would that be a valid use-case? > > > > > > I'd

Re: [PATCH] add some drop_caches documentation and info messsge

2012-10-26 Thread Mika Boström
On Wed, Oct 24, 2012 at 01:48:36PM -0700, Andrew Morton wrote: > Dave Hansen wrote: > > What kind of interface _is_ it in the first place? Is it really a > > production-level thing that we expect users to be poking at? Or, is it > > a rarely-used debugging and benchmarking knob which is fair gam

Re: [PATCH] add some drop_caches documentation and info messsge

2012-10-25 Thread Rafael J. Wysocki
On Wednesday, October 24, 2012 06:17:52 PM Andrew Morton wrote: > On Thu, 25 Oct 2012 00:04:46 +0200 "Rafael J. Wysocki" wrote: > > > On Wednesday 24 of October 2012 14:13:03 Andrew Morton wrote: > > > On Wed, 24 Oct 2012 23:06:00 +0200 > > > Borislav Petkov wrote: > > > > > > > On Wed, Oct 24,

Re: [PATCH] add some drop_caches documentation and info messsge

2012-10-25 Thread Michal Hocko
On Thu 25-10-12 04:57:11, Dave Hansen wrote: [...] > Here's the problem: Joe Kernel Developer gets a bug report, usually > something like "the kernel is slow", or "the kernel is eating up all my > memory". We then start going and digging in to the problem with the > usual tools. We almost *ALWAYS

Re: [PATCH] add some drop_caches documentation and info messsge

2012-10-25 Thread Borislav Petkov
On Thu, Oct 25, 2012 at 04:57:11AM -0700, Dave Hansen wrote: > On 10/25/2012 02:24 AM, Borislav Petkov wrote: > > But let's discuss this a bit further. So, for the benchmarking aspect, > > you're either going to have to always require dmesg along with > > benchmarking results or /proc/vmstat, depen

Re: [PATCH] add some drop_caches documentation and info messsge

2012-10-25 Thread Michal Hocko
On Wed 24-10-12 18:35:43, KOSAKI Motohiro wrote: > >> I have drop_caches in my suspend-to-disk script so that the hibernation > >> image is kept at minimum and suspend times are as small as possible. > > > > hm, that sounds smart. > > > >> Would that be a valid use-case? > > > > I'd say so, unle

Re: [PATCH] add some drop_caches documentation and info messsge

2012-10-25 Thread Michal Hocko
On Wed 24-10-12 12:54:39, Andrew Morton wrote: > On Wed, 24 Oct 2012 08:29:45 +0200 > Michal Hocko wrote: [...] > hmpf. This patch worries me. If there are people out there who are > regularly using drop_caches because the VM sucks, it seems pretty > obnoxious of us to go dumping stuff into thei

Re: [PATCH] add some drop_caches documentation and info messsge

2012-10-25 Thread Dave Hansen
On 10/25/2012 02:24 AM, Borislav Petkov wrote: > But let's discuss this a bit further. So, for the benchmarking aspect, > you're either going to have to always require dmesg along with > benchmarking results or /proc/vmstat, depending on where the drop_caches > stats end up. > > Is this how you en

Re: [PATCH] add some drop_caches documentation and info messsge

2012-10-25 Thread Borislav Petkov
On Wed, Oct 24, 2012 at 08:56:45PM -0400, KOSAKI Motohiro wrote: > > That effectively means removing it from the kernel since distros ship > > with those config options off. We don't want to do that since there > > _are_ valid, occasional uses like benchmarking that we want to be > > consistent. >

Re: [PATCH] add some drop_caches documentation and info messsge

2012-10-24 Thread Andrew Morton
On Thu, 25 Oct 2012 00:04:46 +0200 "Rafael J. Wysocki" wrote: > On Wednesday 24 of October 2012 14:13:03 Andrew Morton wrote: > > On Wed, 24 Oct 2012 23:06:00 +0200 > > Borislav Petkov wrote: > > > > > On Wed, Oct 24, 2012 at 01:48:36PM -0700, Andrew Morton wrote: > > > > Well who knows. Could

Re: [PATCH] add some drop_caches documentation and info messsge

2012-10-24 Thread KOSAKI Motohiro
On Wed, Oct 24, 2012 at 6:57 PM, Dave Hansen wrote: > On 10/24/2012 03:48 PM, Borislav Petkov wrote: >> On Wed, Oct 24, 2012 at 02:18:38PM -0700, Dave Hansen wrote: >>> Sounds fairly valid to me. But, it's also one that would not be harmed >>> or disrupted in any way because of a single additional

Re: [PATCH] add some drop_caches documentation and info messsge

2012-10-24 Thread Dave Hansen
On 10/24/2012 03:48 PM, Borislav Petkov wrote: > On Wed, Oct 24, 2012 at 02:18:38PM -0700, Dave Hansen wrote: >> Sounds fairly valid to me. But, it's also one that would not be harmed >> or disrupted in any way because of a single additional printk() during >> each suspend-to-disk operation. > > b

Re: [PATCH] add some drop_caches documentation and info messsge

2012-10-24 Thread Borislav Petkov
On Wed, Oct 24, 2012 at 02:18:38PM -0700, Dave Hansen wrote: > Sounds fairly valid to me. But, it's also one that would not be harmed > or disrupted in any way because of a single additional printk() during > each suspend-to-disk operation. Btw, back to the drop_caches patch. How about we hide th

Re: [PATCH] add some drop_caches documentation and info messsge

2012-10-24 Thread KOSAKI Motohiro
>> I have drop_caches in my suspend-to-disk script so that the hibernation >> image is kept at minimum and suspend times are as small as possible. > > hm, that sounds smart. > >> Would that be a valid use-case? > > I'd say so, unless we change the kernel to do that internally. We do > have the

Re: [PATCH] add some drop_caches documentation and info messsge

2012-10-24 Thread Rafael J. Wysocki
On Wednesday 24 of October 2012 14:13:03 Andrew Morton wrote: > On Wed, 24 Oct 2012 23:06:00 +0200 > Borislav Petkov wrote: > > > On Wed, Oct 24, 2012 at 01:48:36PM -0700, Andrew Morton wrote: > > > Well who knows. Could be that people's vm *does* suck. Or they have > > > some particularly peculi

Re: [PATCH] add some drop_caches documentation and info messsge

2012-10-24 Thread Dave Hansen
On 10/24/2012 02:06 PM, Borislav Petkov wrote: > On Wed, Oct 24, 2012 at 01:48:36PM -0700, Andrew Morton wrote: >> Well who knows. Could be that people's vm *does* suck. Or they have >> some particularly peculiar worklosd or requirement[*]. Or their VM >> *used* to suck, and the drop_caches is not

Re: [PATCH] add some drop_caches documentation and info messsge

2012-10-24 Thread Andrew Morton
On Wed, 24 Oct 2012 23:06:00 +0200 Borislav Petkov wrote: > On Wed, Oct 24, 2012 at 01:48:36PM -0700, Andrew Morton wrote: > > Well who knows. Could be that people's vm *does* suck. Or they have > > some particularly peculiar worklosd or requirement[*]. Or their VM > > *used* to suck, and the dro

Re: [PATCH] add some drop_caches documentation and info messsge

2012-10-24 Thread Borislav Petkov
On Wed, Oct 24, 2012 at 01:48:36PM -0700, Andrew Morton wrote: > Well who knows. Could be that people's vm *does* suck. Or they have > some particularly peculiar worklosd or requirement[*]. Or their VM > *used* to suck, and the drop_caches is not really needed any more but > it's there in vendor-pr

Re: [PATCH] add some drop_caches documentation and info messsge

2012-10-24 Thread Andrew Morton
On Wed, 24 Oct 2012 13:28:19 -0700 Dave Hansen wrote: > On 10/24/2012 12:54 PM, Andrew Morton wrote: > > hmpf. This patch worries me. If there are people out there who are > > regularly using drop_caches because the VM sucks, it seems pretty > > obnoxious of us to go dumping stuff into their sy

Re: [PATCH] add some drop_caches documentation and info messsge

2012-10-24 Thread Dave Hansen
On 10/24/2012 12:54 PM, Andrew Morton wrote: > hmpf. This patch worries me. If there are people out there who are > regularly using drop_caches because the VM sucks, it seems pretty > obnoxious of us to go dumping stuff into their syslog. What are they > supposed to do? Stop using drop_caches?

Re: [PATCH] add some drop_caches documentation and info messsge

2012-10-24 Thread Andrew Morton
On Wed, 24 Oct 2012 08:29:45 +0200 Michal Hocko wrote: > > > > > > + printk(KERN_NOTICE "%s (%d): dropped kernel caches: %d\n", > > > + current->comm, task_pid_nr(current), > > > sysctl_drop_caches); > > > > urgh. Are we really sure we want to do this? The system opera

Re: [PATCH] add some drop_caches documentation and info messsge

2012-10-23 Thread Michal Hocko
st refreshed the original patch on top of the current mm tree > > but I could live with KERN_INFO as well if people think that KERN_NOTICE > > is too hysterical. > > --- > > >From 1f4058be9b089bc9d43d71bc63989335d7637d8d Mon Sep 17 00:00:00 2001 > > From: Dave Hansen > &

Re: [PATCH] add some drop_caches documentation and info messsge

2012-10-23 Thread Andrew Morton
sterical. > --- > >From 1f4058be9b089bc9d43d71bc63989335d7637d8d Mon Sep 17 00:00:00 2001 > From: Dave Hansen > Date: Fri, 12 Oct 2012 14:30:54 +0200 > Subject: [PATCH] add some drop_caches documentation and info messsge > > There is plenty of anecdotal evidence and a load

Re: [PATCH] add some drop_caches documentation and info messsge

2012-10-15 Thread Dave Hansen
On 10/12/2012 05:57 AM, Michal Hocko wrote: > I would like to resurrect the following Dave's patch. The last time it > has been posted was here https://lkml.org/lkml/2010/9/16/250 and there > didn't seem to be any strong opposition. > Kosaki was worried about possible excessive logging when somebo

Re: [PATCH] add some drop_caches documentation and info messsge

2012-10-15 Thread Kamezawa Hiroyuki
ri, 12 Oct 2012 14:30:54 +0200 Subject: [PATCH] add some drop_caches documentation and info messsge There is plenty of anecdotal evidence and a load of blog posts suggesting that using "drop_caches" periodically keeps your system running in "tip top shape". Perhaps adding some

Re: [PATCH] add some drop_caches documentation and info messsge

2012-10-12 Thread KOSAKI Motohiro
cal. > --- > From 1f4058be9b089bc9d43d71bc63989335d7637d8d Mon Sep 17 00:00:00 2001 > From: Dave Hansen > Date: Fri, 12 Oct 2012 14:30:54 +0200 > Subject: [PATCH] add some drop_caches documentation and info messsge > > There is plenty of anecdotal evidence and a load of blog pos

[PATCH] add some drop_caches documentation and info messsge

2012-10-12 Thread Michal Hocko
: [PATCH] add some drop_caches documentation and info messsge There is plenty of anecdotal evidence and a load of blog posts suggesting that using "drop_caches" periodically keeps your system running in "tip top shape". Perhaps adding some kernel documentation will increase