linux-mm.kvack.org archive mirror
 help / color / mirror / Atom feed
From: Minchan Kim <minchan.kim@gmail.com>
To: KOSAKI Motohiro <kosaki.motohiro@jp.fujitsu.com>
Cc: Christoph Lameter <cl@linux-foundation.org>,
	Wu Fengguang <fengguang.wu@intel.com>,
	LKML <linux-kernel@vger.kernel.org>,
	linux-mm <linux-mm@kvack.org>,
	Andrew Morton <akpm@linux-foundation.org>,
	David Rientjes <rientjes@google.com>,
	Rik van Riel <riel@redhat.com>
Subject: Re: [PATCH 5/5] add NR_ANON_PAGES to OOM log
Date: Thu, 9 Jul 2009 19:27:52 +0900	[thread overview]
Message-ID: <28c262360907090327w63b0cdfdn5bc2ad628239e53b@mail.gmail.com> (raw)
In-Reply-To: <20090709161712.23B0.A69D9226@jp.fujitsu.com>

On Thu, Jul 9, 2009 at 4:20 PM, KOSAKI
Motohiro<kosaki.motohiro@jp.fujitsu.com> wrote:
>> > On Mon, 6 Jul 2009, Minchan Kim wrote:
>> >
>> > > Anyway, I think it's not a big cost in normal system.
>> > > So If you want to add new accounting, I don't have any objection. :)
>> >
>> > Lets keep the counters to a mininum. If we can calculate the values from
>> > something else then there is no justification for a new counter.
>> >
>> > A new counter increases the size of the per cpu structures that exist for
>> > each zone and each cpu. 1 byte gets multiplies by the number of cpus and
>> > that gets multiplied by the number of zones.
>>
>> OK. I'll implement this idea.
>
> Grr, sorry I cancel this opinion. Shem pages can't be calculated
> by minchan's formula.
>
> if those page are mlocked, the page move to unevictable lru. then
> this calculation don't account mlocked page. However mlocked tmpfs pages
> also make OOM issue.

Absolutely. You're right.
But In my opinion,  mlocked shmem pages are important ?
Now we care only number of unevictable pages but don't care what kinds
of pages there are in unevictable list.

What we need is to decode OOM more easily.
I think what kinds of pages there area unevictable lur list is not important.

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

  reply	other threads:[~2009-07-09 10:12 UTC|newest]

Thread overview: 53+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-07-05  9:22 [PATCH 0/5] OOM analysis helper patches KOSAKI Motohiro
2009-07-05  9:23 ` [PATCH 1/5] add per-zone statistics to show_free_areas() KOSAKI Motohiro
2009-07-05 11:05   ` Wu Fengguang
2009-07-05 11:09     ` KOSAKI Motohiro
2009-07-05 11:14       ` Wu Fengguang
2009-07-07 16:33   ` Christoph Lameter
2009-07-05  9:24 ` [PATCH 2/5] add buffer cache information " KOSAKI Motohiro
2009-07-05 11:21   ` Wu Fengguang
2009-07-05 11:31     ` KOSAKI Motohiro
2009-07-05 12:07       ` Wu Fengguang
2009-07-05 14:16   ` Minchan Kim
2009-07-06  4:24     ` KOSAKI Motohiro
2009-07-05  9:24 ` [PATCH 3/5] Show kernel stack usage to /proc/meminfo and OOM log KOSAKI Motohiro
2009-07-07 16:37   ` Christoph Lameter
2009-07-09  2:13     ` KOSAKI Motohiro
2009-07-09 21:00       ` Christoph Lameter
2009-07-05  9:25 ` [PATCH 4/5] add isolate pages vmstat KOSAKI Motohiro
2009-07-05 12:10   ` Wu Fengguang
2009-07-05 12:23     ` KOSAKI Motohiro
2009-07-05 14:51       ` Minchan Kim
2009-07-06  9:28         ` KOSAKI Motohiro
2009-07-06 11:55           ` KOSAKI Motohiro
2009-07-07  0:01             ` Minchan Kim
2009-07-07  0:07               ` KOSAKI Motohiro
2009-07-07  1:19                 ` KOSAKI Motohiro
2009-07-07  1:48                   ` Minchan Kim
2009-07-07  2:12                     ` KOSAKI Motohiro
2009-07-07  2:31                       ` Minchan Kim
2009-07-07  3:02                     ` Rik van Riel
2009-07-07 13:51                   ` Wu Fengguang
2009-07-07 16:49                   ` Christoph Lameter
2009-07-07 17:23                     ` Rik van Riel
2009-07-07 23:32                       ` Christoph Lameter
2009-07-09  2:19                     ` KOSAKI Motohiro
2009-07-09 21:00                       ` Christoph Lameter
2009-07-07 16:46   ` Christoph Lameter
2009-07-08  1:44     ` Wu Fengguang
2009-07-05  9:26 ` [PATCH 5/5] add NR_ANON_PAGES to OOM log KOSAKI Motohiro
2009-07-05 12:13   ` Wu Fengguang
2009-07-05 12:21     ` KOSAKI Motohiro
2009-07-05 13:02       ` Wu Fengguang
2009-07-05 13:19         ` KOSAKI Motohiro
2009-07-05 15:04           ` Minchan Kim
2009-07-05 15:16             ` Wu Fengguang
2009-07-05 15:27               ` Minchan Kim
2009-07-07 16:53                 ` Christoph Lameter
2009-07-09  5:50                   ` KOSAKI Motohiro
2009-07-09  7:20                     ` KOSAKI Motohiro
2009-07-09 10:27                       ` Minchan Kim [this message]
2009-07-07  1:22         ` KOSAKI Motohiro
2009-07-07 10:54           ` Wu Fengguang
2009-07-07 13:56           ` Wu Fengguang
2009-07-09  5:13             ` KOSAKI Motohiro

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=28c262360907090327w63b0cdfdn5bc2ad628239e53b@mail.gmail.com \
    --to=minchan.kim@gmail.com \
    --cc=akpm@linux-foundation.org \
    --cc=cl@linux-foundation.org \
    --cc=fengguang.wu@intel.com \
    --cc=kosaki.motohiro@jp.fujitsu.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    --cc=riel@redhat.com \
    --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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).