All of lore.kernel.org
 help / color / mirror / Atom feed
From: Anton Vorontsov <anton.vorontsov@linaro.org>
To: Minchan Kim <minchan@kernel.org>
Cc: David Rientjes <rientjes@google.com>,
	Pekka Enberg <penberg@kernel.org>, Mel Gorman <mgorman@suse.de>,
	Glauber Costa <glommer@parallels.com>,
	Michal Hocko <mhocko@suse.cz>,
	"Kirill A. Shutemov" <kirill@shutemov.name>,
	Luiz Capitulino <lcapitulino@redhat.com>,
	Andrew Morton <akpm@linux-foundation.org>,
	Greg Thelen <gthelen@google.com>,
	Leonid Moiseichuk <leonid.moiseichuk@nokia.com>,
	KOSAKI Motohiro <kosaki.motohiro@gmail.com>,
	Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>,
	John Stultz <john.stultz@linaro.org>,
	linux-mm@kvack.org, linux-kernel@vger.kernel.org,
	linaro-kernel@lists.linaro.org, patches@linaro.org,
	kernel-team@android.com
Subject: Re: [PATCH 1/2] Add mempressure cgroup
Date: Thu, 10 Jan 2013 22:09:59 -0800	[thread overview]
Message-ID: <20130111060959.GA22981@lizard.gateway.2wire.net> (raw)
In-Reply-To: <20130111055614.GD6183@blaptop>

On Fri, Jan 11, 2013 at 02:56:15PM +0900, Minchan Kim wrote:
[...]
> > Ahh. You're talking about the shrinker interface. Yes, there is no way to
> > tell if the freed memory will be actually "released" (and if not, then
> > yes, we released it unnecessary).
> 
> I don't tell about actually "released" or not.
> I assume application actually release pages but the pages would be another
> zones, NOT targetted zone from kernel. In case of that, kernel could ask
> continuously until target zone has enough free memory.
[...]
> > isolate task to only some nodes/zones, if we really care about precise
> > accounting?). But I'm surely open for ideas. :)
> 
> My dumb idea is only notify to user when reclaim is triggered by
> __GFP_HIGHMEM|__GFP_MOVABLE which is most gfp_t for application memory. :)

Ah, I see. Sure, that will help a lot. I'll try to incorporate this into
the next iteration. But there are still unresolved accounting issues that
I outlined, and I don't think that they are this easy to solve. :)

Thanks!

Anton

--
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: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

WARNING: multiple messages have this Message-ID (diff)
From: Anton Vorontsov <anton.vorontsov@linaro.org>
To: Minchan Kim <minchan@kernel.org>
Cc: David Rientjes <rientjes@google.com>,
	Pekka Enberg <penberg@kernel.org>, Mel Gorman <mgorman@suse.de>,
	Glauber Costa <glommer@parallels.com>,
	Michal Hocko <mhocko@suse.cz>,
	"Kirill A. Shutemov" <kirill@shutemov.name>,
	Luiz Capitulino <lcapitulino@redhat.com>,
	Andrew Morton <akpm@linux-foundation.org>,
	Greg Thelen <gthelen@google.com>,
	Leonid Moiseichuk <leonid.moiseichuk@nokia.com>,
	KOSAKI Motohiro <kosaki.motohiro@gmail.com>,
	Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>,
	John Stultz <john.stultz@linaro.org>,
	linux-mm@kvack.org, linux-kernel@vger.kernel.org,
	linaro-kernel@lists.linaro.org, patches@linaro.org,
	kernel-team@android.com
Subject: Re: [PATCH 1/2] Add mempressure cgroup
Date: Thu, 10 Jan 2013 22:09:59 -0800	[thread overview]
Message-ID: <20130111060959.GA22981@lizard.gateway.2wire.net> (raw)
In-Reply-To: <20130111055614.GD6183@blaptop>

On Fri, Jan 11, 2013 at 02:56:15PM +0900, Minchan Kim wrote:
[...]
> > Ahh. You're talking about the shrinker interface. Yes, there is no way to
> > tell if the freed memory will be actually "released" (and if not, then
> > yes, we released it unnecessary).
> 
> I don't tell about actually "released" or not.
> I assume application actually release pages but the pages would be another
> zones, NOT targetted zone from kernel. In case of that, kernel could ask
> continuously until target zone has enough free memory.
[...]
> > isolate task to only some nodes/zones, if we really care about precise
> > accounting?). But I'm surely open for ideas. :)
> 
> My dumb idea is only notify to user when reclaim is triggered by
> __GFP_HIGHMEM|__GFP_MOVABLE which is most gfp_t for application memory. :)

Ah, I see. Sure, that will help a lot. I'll try to incorporate this into
the next iteration. But there are still unresolved accounting issues that
I outlined, and I don't think that they are this easy to solve. :)

Thanks!

Anton

  reply	other threads:[~2013-01-11  6:13 UTC|newest]

Thread overview: 64+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-01-04  8:27 [PATCH 0/2] Mempressure cgroup Anton Vorontsov
2013-01-04  8:27 ` Anton Vorontsov
2013-01-04  8:29 ` [PATCH 1/2] Add mempressure cgroup Anton Vorontsov
2013-01-04  8:29   ` Anton Vorontsov
2013-01-04 15:05   ` Kirill A. Shutemov
2013-01-04 15:05     ` Kirill A. Shutemov
2013-01-07  8:51   ` Kamezawa Hiroyuki
2013-01-07  8:51     ` Kamezawa Hiroyuki
2013-01-08  7:29     ` Anton Vorontsov
2013-01-08  7:29       ` Anton Vorontsov
2013-01-08  7:57       ` leonid.moiseichuk
2013-01-08  7:57         ` leonid.moiseichuk
2013-01-08  8:24       ` Kamezawa Hiroyuki
2013-01-08  8:24         ` Kamezawa Hiroyuki
2013-01-08  8:49   ` Minchan Kim
2013-01-08  8:49     ` Minchan Kim
2013-01-09 22:14     ` Anton Vorontsov
2013-01-09 22:14       ` Anton Vorontsov
2013-01-11  5:12       ` Minchan Kim
2013-01-11  5:12         ` Minchan Kim
2013-01-11  5:38         ` Anton Vorontsov
2013-01-11  5:38           ` Anton Vorontsov
2013-01-11  5:56           ` Minchan Kim
2013-01-11  5:56             ` Minchan Kim
2013-01-11  6:09             ` Anton Vorontsov [this message]
2013-01-11  6:09               ` Anton Vorontsov
2013-01-08 21:44   ` Andrew Morton
2013-01-08 21:44     ` Andrew Morton
2013-01-09 14:10     ` Glauber Costa
2013-01-09 14:10       ` Glauber Costa
2013-01-09 20:28       ` Andrew Morton
2013-01-09 20:28         ` Andrew Morton
2013-01-09  8:56   ` Glauber Costa
2013-01-09  8:56     ` Glauber Costa
2013-01-09  9:15     ` Andrew Morton
2013-01-09  9:15       ` Andrew Morton
2013-01-09 13:43       ` Glauber Costa
2013-01-09 13:43         ` Glauber Costa
2013-01-09 20:37   ` Tejun Heo
2013-01-09 20:37     ` Tejun Heo
2013-01-09 20:39     ` Tejun Heo
2013-01-09 20:39       ` Tejun Heo
2013-01-09 21:20     ` Glauber Costa
2013-01-09 21:20       ` Glauber Costa
2013-01-09 21:36       ` Anton Vorontsov
2013-01-09 21:36         ` Anton Vorontsov
2013-01-09 21:55         ` Tejun Heo
2013-01-09 21:55           ` Tejun Heo
2013-01-09 22:04           ` Tejun Heo
2013-01-09 22:04             ` Tejun Heo
2013-01-09 22:06           ` Anton Vorontsov
2013-01-09 22:06             ` Anton Vorontsov
2013-01-09 22:21             ` Tejun Heo
2013-01-09 22:21               ` Tejun Heo
2013-01-10  7:18             ` Glauber Costa
2013-01-10  7:18               ` Glauber Costa
2013-01-13  8:50   ` Simon Jeons
2013-01-13  8:50     ` Simon Jeons
2013-01-13  8:52   ` Wanpeng Li
2013-01-13  8:52   ` Wanpeng Li
2013-01-04  8:29 ` [PATCH 2/2] Add shrinker interface for " Anton Vorontsov
2013-01-04  8:29   ` Anton Vorontsov
2013-01-11 19:13 ` [PATCH 0/2] Mempressure cgroup Luiz Capitulino
2013-01-11 19:13   ` Luiz Capitulino

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20130111060959.GA22981@lizard.gateway.2wire.net \
    --to=anton.vorontsov@linaro.org \
    --cc=akpm@linux-foundation.org \
    --cc=b.zolnierkie@samsung.com \
    --cc=glommer@parallels.com \
    --cc=gthelen@google.com \
    --cc=john.stultz@linaro.org \
    --cc=kernel-team@android.com \
    --cc=kirill@shutemov.name \
    --cc=kosaki.motohiro@gmail.com \
    --cc=lcapitulino@redhat.com \
    --cc=leonid.moiseichuk@nokia.com \
    --cc=linaro-kernel@lists.linaro.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    --cc=mgorman@suse.de \
    --cc=mhocko@suse.cz \
    --cc=minchan@kernel.org \
    --cc=patches@linaro.org \
    --cc=penberg@kernel.org \
    --cc=rientjes@google.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.