All of lore.kernel.org
 help / color / mirror / Atom feed
From: Mel Gorman <mel@csn.ul.ie>
To: KOSAKI Motohiro <kosaki.motohiro@jp.fujitsu.com>
Cc: Wu Fengguang <fengguang.wu@intel.com>,
	Andrew Morton <akpm@linux-foundation.org>,
	stable@kernel.org, Rik van Riel <riel@redhat.com>,
	Christoph Hellwig <hch@infradead.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"linux-fsdevel@vger.kernel.org" <linux-fsdevel@vger.kernel.org>,
	"linux-mm@kvack.org" <linux-mm@kvack.org>,
	Dave Chinner <david@fromorbit.com>,
	Chris Mason <chris.mason@oracle.com>,
	Nick Piggin <npiggin@suse.de>,
	Johannes Weiner <hannes@cmpxchg.org>,
	KAMEZAWA Hiroyuki <kamezawa.hiroyu@jp.fujitsu.com>,
	Andrea Arcangeli <aarcange@redhat.com>,
	Minchan Kim <minchan.kim@gmail.com>, Andreas Mohr <andi@lisas.de>,
	Bill Davidsen <davidsen@tmr.com>,
	Ben Gamari <bgamari.foss@gmail.com>
Subject: Re: Why PAGEOUT_IO_SYNC stalls for a long time
Date: Wed, 4 Aug 2010 12:10:06 +0100	[thread overview]
Message-ID: <20100804111005.GA17745@csn.ul.ie> (raw)
In-Reply-To: <20100801174229.4B08.A69D9226@jp.fujitsu.com>

On Sun, Aug 01, 2010 at 05:47:08PM +0900, KOSAKI Motohiro wrote:
> > > side note: page lock contention is very common case.
> > > 
> > > For case (8), I don't think sleeping is right way. get_page() is used in really various place of
> > > our kernel. so we can't assume it's only temporary reference count increasing.
> > 
> > In what case is a munlocked pages reference count permanently increased and
> > why is this not a memory leak?
> 
> V4L, audio, GEM and/or other multimedia driver?
> 

Ok, that is quite likely. Have you made a start on a series related to
lumpy reclaim? I was holding off making a start on such a thing while I
reviewed the other writeback issues and travelling to MM Summit is going
to delay things for me. If you haven't started when I get back, I'll
make some sort of stab at it.

-- 
Mel Gorman
Part-time Phd Student                          Linux Technology Center
University of Limerick                         IBM Dublin Software Lab

WARNING: multiple messages have this Message-ID (diff)
From: Mel Gorman <mel@csn.ul.ie>
To: KOSAKI Motohiro <kosaki.motohiro@jp.fujitsu.com>
Cc: Wu Fengguang <fengguang.wu@intel.com>,
	Andrew Morton <akpm@linux-foundation.org>,
	stable@kernel.org, Rik van Riel <riel@redhat.com>,
	Christoph Hellwig <hch@infradead.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"linux-fsdevel@vger.kernel.org" <linux-fsdevel@vger.kernel.org>,
	"linux-mm@kvack.org" <linux-mm@kvack.org>,
	Dave Chinner <david@fromorbit.com>,
	Chris Mason <chris.mason@oracle.com>,
	Nick Piggin <npiggin@suse.de>,
	Johannes Weiner <hannes@cmpxchg.org>,
	KAMEZAWA Hiroyuki <kamezawa.hiroyu@jp.fujitsu.com>,
	Andrea Arcangeli <aarcange@redhat.com>,
	Minchan Kim <minchan.kim@gmail.com>, Andreas Mohr <andi@lisas.de>,
	Bill Davidsen <davidsen@tmr.com>,
	Ben Gamari <bgamari.foss@gmail.com>
Subject: Re: Why PAGEOUT_IO_SYNC stalls for a long time
Date: Wed, 4 Aug 2010 12:10:06 +0100	[thread overview]
Message-ID: <20100804111005.GA17745@csn.ul.ie> (raw)
In-Reply-To: <20100801174229.4B08.A69D9226@jp.fujitsu.com>

On Sun, Aug 01, 2010 at 05:47:08PM +0900, KOSAKI Motohiro wrote:
> > > side note: page lock contention is very common case.
> > > 
> > > For case (8), I don't think sleeping is right way. get_page() is used in really various place of
> > > our kernel. so we can't assume it's only temporary reference count increasing.
> > 
> > In what case is a munlocked pages reference count permanently increased and
> > why is this not a memory leak?
> 
> V4L, audio, GEM and/or other multimedia driver?
> 

Ok, that is quite likely. Have you made a start on a series related to
lumpy reclaim? I was holding off making a start on such a thing while I
reviewed the other writeback issues and travelling to MM Summit is going
to delay things for me. If you haven't started when I get back, I'll
make some sort of stab at it.

-- 
Mel Gorman
Part-time Phd Student                          Linux Technology Center
University of Limerick                         IBM Dublin Software Lab

--
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:[~2010-08-04 11:10 UTC|newest]

Thread overview: 69+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-07-28  7:17 [PATCH] vmscan: raise the bar to PAGEOUT_IO_SYNC stalls Wu Fengguang
2010-07-28  7:17 ` Wu Fengguang
2010-07-28  7:49 ` Minchan Kim
2010-07-28  7:49   ` Minchan Kim
2010-07-28  8:46   ` [PATCH] vmscan: remove wait_on_page_writeback() from pageout() Wu Fengguang
2010-07-28  8:46     ` Wu Fengguang
2010-07-28  9:10     ` Mel Gorman
2010-07-28  9:10       ` Mel Gorman
2010-07-28  9:30       ` Wu Fengguang
2010-07-28  9:30         ` Wu Fengguang
2010-07-28  9:45         ` Mel Gorman
2010-07-28  9:45           ` Mel Gorman
2010-07-28  9:43       ` KOSAKI Motohiro
2010-07-28  9:43         ` KOSAKI Motohiro
2010-07-28  9:50         ` Mel Gorman
2010-07-28  9:50           ` Mel Gorman
2010-07-28  9:59           ` KOSAKI Motohiro
2010-07-28  9:59             ` KOSAKI Motohiro
2010-08-01  5:27             ` Wu Fengguang
2010-08-01  5:27               ` Wu Fengguang
2010-08-01  5:49               ` Wu Fengguang
2010-08-01  8:32               ` KOSAKI Motohiro
2010-08-01  8:32                 ` KOSAKI Motohiro
2010-08-01  8:35                 ` Wu Fengguang
2010-08-01  8:35                   ` Wu Fengguang
2010-08-01  8:40                   ` KOSAKI Motohiro
2010-08-01  8:40                     ` KOSAKI Motohiro
2010-08-01  5:17         ` Wu Fengguang
2010-08-01  5:17           ` Wu Fengguang
2010-07-28 16:29     ` Minchan Kim
2010-07-28 16:29       ` Minchan Kim
2010-07-28 11:40 ` Why PAGEOUT_IO_SYNC stalls for a long time KOSAKI Motohiro
2010-07-28 11:40   ` KOSAKI Motohiro
2010-07-28 13:10   ` Mel Gorman
2010-07-28 13:10     ` Mel Gorman
2010-07-29 10:34     ` KOSAKI Motohiro
2010-07-29 10:34       ` KOSAKI Motohiro
2010-07-29 14:24       ` Mel Gorman
2010-07-29 14:24         ` Mel Gorman
2010-07-30  4:54         ` KOSAKI Motohiro
2010-07-30  4:54           ` KOSAKI Motohiro
2010-07-30 10:30           ` Mel Gorman
2010-07-30 10:30             ` Mel Gorman
2010-08-01  8:47             ` KOSAKI Motohiro
2010-08-01  8:47               ` KOSAKI Motohiro
2010-08-04 11:10               ` Mel Gorman [this message]
2010-08-04 11:10                 ` Mel Gorman
2010-08-05  6:20                 ` KOSAKI Motohiro
2010-08-05  6:20                   ` KOSAKI Motohiro
2010-08-05  8:09                   ` Andreas Mohr
2010-08-05  8:09                     ` Andreas Mohr
2010-07-28 17:30   ` Andrew Morton
2010-07-28 17:30     ` Andrew Morton
2010-07-29  1:01     ` KOSAKI Motohiro
2010-07-29  1:01       ` KOSAKI Motohiro
2010-07-30 13:17 ` [PATCH] vmscan: raise the bar to PAGEOUT_IO_SYNC stalls Andrea Arcangeli
2010-07-30 13:17   ` Andrea Arcangeli
2010-07-30 13:31   ` Mel Gorman
2010-07-30 13:31     ` Mel Gorman
2010-07-31 16:13 ` Wu Fengguang
2010-07-31 16:13   ` Wu Fengguang
2010-07-31 17:33   ` Christoph Hellwig
2010-07-31 17:33     ` Christoph Hellwig
2010-07-31 17:55     ` Pekka Enberg
2010-07-31 17:55       ` Pekka Enberg
2010-07-31 17:59       ` Christoph Hellwig
2010-07-31 17:59         ` Christoph Hellwig
2010-07-31 18:09         ` Pekka Enberg
2010-07-31 18:09           ` Pekka Enberg

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=20100804111005.GA17745@csn.ul.ie \
    --to=mel@csn.ul.ie \
    --cc=aarcange@redhat.com \
    --cc=akpm@linux-foundation.org \
    --cc=andi@lisas.de \
    --cc=bgamari.foss@gmail.com \
    --cc=chris.mason@oracle.com \
    --cc=david@fromorbit.com \
    --cc=davidsen@tmr.com \
    --cc=fengguang.wu@intel.com \
    --cc=hannes@cmpxchg.org \
    --cc=hch@infradead.org \
    --cc=kamezawa.hiroyu@jp.fujitsu.com \
    --cc=kosaki.motohiro@jp.fujitsu.com \
    --cc=linux-fsdevel@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    --cc=minchan.kim@gmail.com \
    --cc=npiggin@suse.de \
    --cc=riel@redhat.com \
    --cc=stable@kernel.org \
    /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.