All of lore.kernel.org
 help / color / mirror / Atom feed
From: Vlastimil Babka <vbabka@suse.cz>
To: Andrew Morton <akpm@linux-foundation.org>, linux-mm@kvack.org
Cc: linux-kernel@vger.kernel.org, Christoph Lameter <cl@linux.com>,
	David Rientjes <rientjes@google.com>,
	Joonsoo Kim <iamjoonsoo.kim@lge.com>,
	Mel Gorman <mgorman@suse.de>,
	Michal Nazarewicz <mina86@mina86.com>,
	Minchan Kim <minchan@kernel.org>,
	Naoya Horiguchi <n-horiguchi@ah.jp.nec.com>,
	Rik van Riel <riel@redhat.com>
Subject: Re: [PATCH v2 0/5] Assorted compaction cleanups and optimizations
Date: Fri, 31 Jul 2015 17:30:59 +0200	[thread overview]
Message-ID: <55BB94B3.7090603@suse.cz> (raw)
In-Reply-To: <1438356487-7082-1-git-send-email-vbabka@suse.cz>

On 07/31/2015 05:28 PM, Vlastimil Babka wrote:
> v2 changes:
>   - dropped Patch 6 as adjusting to Joonsoo's objection would be too
>     complicated and the results didn't justify it
>   - don't check for compound order > 0 in patches 4 and 5 as suggested by
>     Michal Nazarewicz. Negative values are handled by converting to unsinged
>     int, the pfn calculations work fine with 0 and it's unlikely to see 0
>     due to a race when we just checked PageCompound().

ah, also patch 3 calls reset_cached_positions() from 
__reset_isolation_suitable() as v1 missed a callsite by doing it 
separately (spotted by Joonsoo)

--
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: Vlastimil Babka <vbabka@suse.cz>
To: Andrew Morton <akpm@linux-foundation.org>, linux-mm@kvack.org
Cc: linux-kernel@vger.kernel.org, Christoph Lameter <cl@linux.com>,
	David Rientjes <rientjes@google.com>,
	Joonsoo Kim <iamjoonsoo.kim@lge.com>,
	Mel Gorman <mgorman@suse.de>,
	Michal Nazarewicz <mina86@mina86.com>,
	Minchan Kim <minchan@kernel.org>,
	Naoya Horiguchi <n-horiguchi@ah.jp.nec.com>,
	Rik van Riel <riel@redhat.com>
Subject: Re: [PATCH v2 0/5] Assorted compaction cleanups and optimizations
Date: Fri, 31 Jul 2015 17:30:59 +0200	[thread overview]
Message-ID: <55BB94B3.7090603@suse.cz> (raw)
In-Reply-To: <1438356487-7082-1-git-send-email-vbabka@suse.cz>

On 07/31/2015 05:28 PM, Vlastimil Babka wrote:
> v2 changes:
>   - dropped Patch 6 as adjusting to Joonsoo's objection would be too
>     complicated and the results didn't justify it
>   - don't check for compound order > 0 in patches 4 and 5 as suggested by
>     Michal Nazarewicz. Negative values are handled by converting to unsinged
>     int, the pfn calculations work fine with 0 and it's unlikely to see 0
>     due to a race when we just checked PageCompound().

ah, also patch 3 calls reset_cached_positions() from 
__reset_isolation_suitable() as v1 missed a callsite by doing it 
separately (spotted by Joonsoo)


  parent reply	other threads:[~2015-07-31 15:31 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-07-31 15:28 [PATCH v2 0/5] Assorted compaction cleanups and optimizations Vlastimil Babka
2015-07-31 15:28 ` Vlastimil Babka
2015-07-31 15:28 ` [PATCH v2 1/5] mm, compaction: more robust check for scanners meeting Vlastimil Babka
2015-07-31 15:28   ` Vlastimil Babka
2015-07-31 15:28 ` [PATCH v2 2/5] mm, compaction: simplify handling restart position in free pages scanner Vlastimil Babka
2015-07-31 15:28   ` Vlastimil Babka
2015-08-03 17:00   ` Michal Nazarewicz
2015-08-03 17:00     ` Michal Nazarewicz
2015-07-31 15:28 ` [PATCH v2 3/5] mm, compaction: encapsulate resetting cached scanner positions Vlastimil Babka
2015-07-31 15:28   ` Vlastimil Babka
2015-07-31 15:28 ` [PATCH v2 4/5] mm, compaction: always skip compound pages by order in migrate scanner Vlastimil Babka
2015-07-31 15:28   ` Vlastimil Babka
2015-08-07  9:18   ` Vlastimil Babka
2015-08-07  9:18     ` Vlastimil Babka
2015-07-31 15:28 ` [PATCH v2 5/5] mm, compaction: skip compound pages by order in free scanner Vlastimil Babka
2015-07-31 15:28   ` Vlastimil Babka
2015-07-31 15:30 ` Vlastimil Babka [this message]
2015-07-31 15:30   ` [PATCH v2 0/5] Assorted compaction cleanups and optimizations Vlastimil Babka

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=55BB94B3.7090603@suse.cz \
    --to=vbabka@suse.cz \
    --cc=akpm@linux-foundation.org \
    --cc=cl@linux.com \
    --cc=iamjoonsoo.kim@lge.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    --cc=mgorman@suse.de \
    --cc=mina86@mina86.com \
    --cc=minchan@kernel.org \
    --cc=n-horiguchi@ah.jp.nec.com \
    --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 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.