All of lore.kernel.org
 help / color / mirror / Atom feed
From: Rik van Riel <riel@redhat.com>
To: Mel Gorman <mgorman@suse.de>
Cc: Linux-MM <linux-mm@kvack.org>,
	Andrea Arcangeli <aarcange@redhat.com>,
	Minchan Kim <minchan.kim@gmail.com>, Jan Kara <jack@suse.cz>,
	Andy Isaacson <adi@hexapodia.org>,
	Johannes Weiner <jweiner@redhat.com>,
	LKML <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH 2/5] mm: compaction: Use synchronous compaction for /proc/sys/vm/compact_memory
Date: Mon, 21 Nov 2011 16:46:40 -0500	[thread overview]
Message-ID: <4ECAC6C0.3090202@redhat.com> (raw)
In-Reply-To: <1321635524-8586-3-git-send-email-mgorman@suse.de>

On 11/18/2011 11:58 AM, Mel Gorman wrote:
> When asynchronous compaction was introduced, the
> /proc/sys/vm/compact_memory handler should have been updated to always
> use synchronous compaction. This did not happen so this patch addresses
> it. The assumption is if a user writes to /proc/sys/vm/compact_memory,
> they are willing for that process to stall.
>
> Signed-off-by: Mel Gorman<mgorman@suse.de>

Reviewed-by: Rik van Riel <riel@redhat.com>

--
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/ .
Fight unfair telecom internet charges in Canada: sign http://stopthemeter.ca/
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

WARNING: multiple messages have this Message-ID (diff)
From: Rik van Riel <riel@redhat.com>
To: Mel Gorman <mgorman@suse.de>
Cc: Linux-MM <linux-mm@kvack.org>,
	Andrea Arcangeli <aarcange@redhat.com>,
	Minchan Kim <minchan.kim@gmail.com>, Jan Kara <jack@suse.cz>,
	Andy Isaacson <adi@hexapodia.org>,
	Johannes Weiner <jweiner@redhat.com>,
	LKML <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH 2/5] mm: compaction: Use synchronous compaction for /proc/sys/vm/compact_memory
Date: Mon, 21 Nov 2011 16:46:40 -0500	[thread overview]
Message-ID: <4ECAC6C0.3090202@redhat.com> (raw)
In-Reply-To: <1321635524-8586-3-git-send-email-mgorman@suse.de>

On 11/18/2011 11:58 AM, Mel Gorman wrote:
> When asynchronous compaction was introduced, the
> /proc/sys/vm/compact_memory handler should have been updated to always
> use synchronous compaction. This did not happen so this patch addresses
> it. The assumption is if a user writes to /proc/sys/vm/compact_memory,
> they are willing for that process to stall.
>
> Signed-off-by: Mel Gorman<mgorman@suse.de>

Reviewed-by: Rik van Riel <riel@redhat.com>


  parent reply	other threads:[~2011-11-21 21:46 UTC|newest]

Thread overview: 80+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-11-18 16:58 [RFC PATCH 0/5] Reduce compaction-related stalls and improve asynchronous migration of dirty pages v3 Mel Gorman
2011-11-18 16:58 ` Mel Gorman
2011-11-18 16:58 ` [PATCH 1/5] mm: compaction: Allow compaction to isolate dirty pages Mel Gorman
2011-11-18 16:58   ` Mel Gorman
2011-11-18 17:28   ` Andrea Arcangeli
2011-11-18 17:28     ` Andrea Arcangeli
2011-11-21 17:16   ` Rik van Riel
2011-11-21 17:16     ` Rik van Riel
2011-11-18 16:58 ` [PATCH 2/5] mm: compaction: Use synchronous compaction for /proc/sys/vm/compact_memory Mel Gorman
2011-11-18 16:58   ` Mel Gorman
2011-11-18 17:27   ` Andrea Arcangeli
2011-11-18 17:27     ` Andrea Arcangeli
2011-11-21 21:46   ` Rik van Riel [this message]
2011-11-21 21:46     ` Rik van Riel
2011-11-18 16:58 ` [PATCH 3/5] mm: Do not stall in synchronous compaction for THP allocations Mel Gorman
2011-11-18 16:58   ` Mel Gorman
2011-11-18 17:34   ` Andrea Arcangeli
2011-11-18 17:34     ` Andrea Arcangeli
2011-11-18 16:58 ` [PATCH 4/5] mm: compaction: Determine if dirty pages can be migreated without blocking within ->migratepage Mel Gorman
2011-11-18 16:58   ` Mel Gorman
2011-11-18 21:35   ` Andrea Arcangeli
2011-11-18 21:35     ` Andrea Arcangeli
2011-11-21 11:17     ` Mel Gorman
2011-11-21 11:17       ` Mel Gorman
2011-11-21 22:45       ` Andrea Arcangeli
2011-11-21 22:45         ` Andrea Arcangeli
2011-11-22  0:55         ` [PATCH] mm: compaction: make buffer cache __GFP_MOVABLE Rik van Riel
2011-11-22  0:55           ` Rik van Riel
2011-11-22 12:59         ` [PATCH 4/5] mm: compaction: Determine if dirty pages can be migreated without blocking within ->migratepage Mel Gorman
2011-11-22 12:59           ` Mel Gorman
2011-11-24  1:19           ` Andrea Arcangeli
2011-11-24  1:19             ` Andrea Arcangeli
2011-11-24 12:21             ` Mel Gorman
2011-11-24 12:21               ` Mel Gorman
2011-11-26  6:51               ` Andy Isaacson
2011-11-26  6:51                 ` Andy Isaacson
2011-11-27 20:50               ` Rik van Riel
2011-11-27 20:50                 ` Rik van Riel
2011-11-19  8:59   ` Nai Xia
2011-11-19  8:59     ` Nai Xia
2011-11-19  9:48     ` Nai Xia
2011-11-19  9:48       ` Nai Xia
2011-11-21 11:19     ` Mel Gorman
2011-11-21 11:19       ` Mel Gorman
2011-11-18 16:58 ` [PATCH 5/5] mm: compaction: make isolate_lru_page() filter-aware again Mel Gorman
2011-11-18 16:58   ` Mel Gorman
2011-11-19 19:54 ` [RFC PATCH 0/5] Reduce compaction-related stalls Andrea Arcangeli
2011-11-19 19:54   ` Andrea Arcangeli
2011-11-19 19:54 ` [PATCH 1/8] mm: compaction: Allow compaction to isolate dirty pages Andrea Arcangeli
2011-11-19 19:54   ` Andrea Arcangeli
2011-11-19 19:54 ` [PATCH 2/8] mm: compaction: Use synchronous compaction for /proc/sys/vm/compact_memory Andrea Arcangeli
2011-11-19 19:54   ` Andrea Arcangeli
2011-11-19 19:54 ` [PATCH 3/8] mm: check if we isolated a compound page during lumpy scan Andrea Arcangeli
2011-11-19 19:54   ` Andrea Arcangeli
2011-11-21 11:51   ` Mel Gorman
2011-11-21 11:51     ` Mel Gorman
2011-11-19 19:54 ` [PATCH 4/8] mm: compaction: defer compaction only with sync_migration Andrea Arcangeli
2011-11-19 19:54   ` Andrea Arcangeli
2011-11-21 12:36   ` Mel Gorman
2011-11-21 12:36     ` Mel Gorman
2011-11-19 19:54 ` [PATCH 5/8] mm: compaction: avoid overwork in migrate sync mode Andrea Arcangeli
2011-11-19 19:54   ` Andrea Arcangeli
2011-11-21 21:59   ` Rik van Riel
2011-11-21 21:59     ` Rik van Riel
2011-11-22  9:51     ` Mel Gorman
2011-11-22  9:51       ` Mel Gorman
2011-11-19 19:54 ` [PATCH 6/8] Revert "mm: compaction: make isolate_lru_page() filter-aware" Andrea Arcangeli
2011-11-19 19:54   ` Andrea Arcangeli
2011-11-21 12:57   ` Mel Gorman
2011-11-21 12:57     ` Mel Gorman
2011-11-19 19:54 ` [PATCH 7/8] Revert "vmscan: abort reclaim/compaction if compaction can proceed" Andrea Arcangeli
2011-11-19 19:54   ` Andrea Arcangeli
2011-11-21 13:09   ` Mel Gorman
2011-11-21 13:09     ` Mel Gorman
2011-11-21 15:37     ` Rik van Riel
2011-11-21 15:37       ` Rik van Riel
2011-11-19 19:54 ` [PATCH 8/8] Revert "vmscan: limit direct reclaim for higher order allocations" Andrea Arcangeli
2011-11-19 19:54   ` Andrea Arcangeli
2011-11-21 21:57   ` Rik van Riel
2011-11-21 21:57     ` Rik van Riel

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=4ECAC6C0.3090202@redhat.com \
    --to=riel@redhat.com \
    --cc=aarcange@redhat.com \
    --cc=adi@hexapodia.org \
    --cc=jack@suse.cz \
    --cc=jweiner@redhat.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    --cc=mgorman@suse.de \
    --cc=minchan.kim@gmail.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.