From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from psmtp.com (na3sys010amx192.postini.com [74.125.245.192]) by kanga.kvack.org (Postfix) with SMTP id D44B36B0032 for ; Wed, 26 Jun 2013 04:20:34 -0400 (EDT) Date: Wed, 26 Jun 2013 17:20:40 +0900 From: Minchan Kim Subject: Re: [PATCH] vmpressure: implement strict mode Message-ID: <20130626082040.GI29127@bbox> References: <20130625175129.7c0d79e1@redhat.com> <20130626075051.GG29127@bbox> <20130626075921.GD28748@dhcp22.suse.cz> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20130626075921.GD28748@dhcp22.suse.cz> Sender: owner-linux-mm@kvack.org List-ID: To: Michal Hocko Cc: Luiz Capitulino , linux-mm@kvack.org, linux-kernel@vger.kernel.org, anton@enomsg.org, akpm@linux-foundation.org Hello Michal, On Wed, Jun 26, 2013 at 09:59:21AM +0200, Michal Hocko wrote: > On Wed 26-06-13 16:50:51, Minchan Kim wrote: > > On Tue, Jun 25, 2013 at 05:51:29PM -0400, Luiz Capitulino wrote: > > > Currently, applications are notified for the level they registered for > > > _plus_ higher levels. > > > > > > This is a problem if the application wants to implement different > > > actions for different levels. For example, an application might want > > > to release 10% of its cache on level low, 50% on medium and 100% on > > > critical. To do this, the application has to register a different fd > > > for each event. However, fd low is always going to be notified and > > > and all fds are going to be notified on level critical. > > > > > > Strict mode solves this problem by strictly notifiying the event > > > an fd has registered for. It's optional. By default we still notify > > > on higher levels. > > > > > > Signed-off-by: Luiz Capitulino > > Acked-by: Minchan Kim > > > > Shouldn't we make this default? > > The interface is not there for long but still, changing it is always > quite tricky. And the users who care can be modified really easily so I > would stick with the original default. Yeb, I am not strong against to stick old at a moment but at least, this patch makes more sense to me so I'd like to know why we didn't do it from the beginning. Surely, Anton has a answer. > > -- > Michal Hocko > SUSE Labs > > -- > To unsubscribe, send a message with 'unsubscribe linux-mm' in > the body to majordomo@kvack.org. For more info on Linux MM, > see: http://www.linux-mm.org/ . > Don't email: email@kvack.org -- Kind regards, Minchan Kim -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: email@kvack.org