mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Vlastimil Babka <vbabka@suse.cz>
To: Mel Gorman <mgorman@techsingularity.net>
Cc: Andrew Morton <akpm@linux-foundation.org>,
	Linux Kernel <linux-kernel@vger.kernel.org>,
	Linux-MM <linux-mm@kvack.org>,
	Hillf Danton <hillf.zj@alibaba-inc.com>,
	Jesper Dangaard Brouer <brouer@redhat.com>
Subject: Re: [PATCH 1/3] mm, page_alloc: Split buffered_rmqueue
Date: Fri, 13 Jan 2017 14:27:08 +0100	[thread overview]
Message-ID: <c24ac47f-2abb-1e46-d828-da4e26a6c9dc@suse.cz> (raw)
In-Reply-To: <20170112172131.wd64o44kqg6e4nou@techsingularity.net>

On 01/12/2017 06:21 PM, Mel Gorman wrote:
>
>> > Signed-off-by: Mel Gorman <mgorman@techsingularity.net>
>> > Acked-by: Hillf Danton <hillf.zj@alibaba-inc.com>
>> > ---
>> >  mm/page_alloc.c | 126 ++++++++++++++++++++++++++++++++++----------------------
>> >  1 file changed, 77 insertions(+), 49 deletions(-)
>> >
>> > diff --git a/mm/page_alloc.c b/mm/page_alloc.c
>> > index 2c6d5f64feca..d8798583eaf8 100644
>> > --- a/mm/page_alloc.c
>> > +++ b/mm/page_alloc.c
>> > @@ -2610,68 +2610,96 @@ static inline void zone_statistics(struct zone *preferred_zone, struct zone *z,
>> >  #endif
>> >  }
>> >
>> > +/* Remote page from the per-cpu list, caller must protect the list */
>>
>>     ^ Remove
>>
>> > +static struct page *__rmqueue_pcplist(struct zone *zone, unsigned int order,
>> > +			gfp_t gfp_flags, int migratetype, bool cold,
>>
>> order and gfp_flags seem unused here
>>
>
> This on top?

Yeah, thanks!

  reply	other threads:[~2017-01-13 13:27 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-01-12 10:42 [PATCH 0/3] Use per-cpu allocator for !irq requests and prepare for a bulk allocator Mel Gorman
2017-01-12 10:42 ` [PATCH 1/3] mm, page_alloc: Split buffered_rmqueue Mel Gorman
2017-01-12 15:44   ` Vlastimil Babka
2017-01-12 17:21     ` Mel Gorman
2017-01-13 13:27       ` Vlastimil Babka [this message]
2017-01-12 10:42 ` [PATCH 2/3] mm, page_alloc: Split alloc_pages_nodemask Mel Gorman
2017-01-12 10:43 ` [PATCH 3/3] mm, page_allocator: Only use per-cpu allocator for irq-safe requests Mel Gorman
2017-01-12 17:02   ` Vlastimil Babka
2017-01-13 10:18     ` Mel Gorman

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=c24ac47f-2abb-1e46-d828-da4e26a6c9dc@suse.cz \
    --to=vbabka@suse.cz \
    --cc=akpm@linux-foundation.org \
    --cc=brouer@redhat.com \
    --cc=hillf.zj@alibaba-inc.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    --cc=mgorman@techsingularity.net \
    /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

all inboxes | Powered by JetHome®