[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <op.v9mt58ch3l0zgt@mpn-glaptop>
Date: Mon, 13 Feb 2012 11:38:22 -0800
From: "Michal Nazarewicz" <mina86@...a86.com>
To: "Marek Szyprowski" <m.szyprowski@...sung.com>,
"Robert Nelson" <robertcnelson@...il.com>
Cc: linux-kernel@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
linux-media@...r.kernel.org, linux-mm@...ck.org,
linaro-mm-sig@...ts.linaro.org, "Ohad Ben-Cohen" <ohad@...ery.com>,
"Daniel Walker" <dwalker@...eaurora.org>,
"Russell King" <linux@....linux.org.uk>,
"Arnd Bergmann" <arnd@...db.de>,
"Jonathan Corbet" <corbet@....net>, "Mel Gorman" <mel@....ul.ie>,
"Dave Hansen" <dave@...ux.vnet.ibm.com>,
"Jesse Barker" <jesse.barker@...aro.org>,
"Kyungmin Park" <kyungmin.park@...sung.com>,
"Andrew Morton" <akpm@...ux-foundation.org>,
"Rob Clark" <rob.clark@...aro.org>,
"KAMEZAWA Hiroyuki" <kamezawa.hiroyu@...fujitsu.com>
Subject: Re: [Linaro-mm-sig] [PATCHv21 12/16] mm: trigger page reclaim in
alloc_contig_range() to stabilise watermarks
> On Fri, Feb 10, 2012 at 11:32 AM, Marek Szyprowski
> <m.szyprowski@...sung.com> wrote:
>> @@ -5637,6 +5642,56 @@ static int __alloc_contig_migrate_range(unsigned long start, unsigned long end)
>> return ret > 0 ? 0 : ret;
>> }
>>
>> +/*
>> + * Update zone's cma pages counter used for watermark level calculation.
>> + */
>> +static inline void __update_cma_wmark_pages(struct zone *zone, int count)
>> +{
>> + unsigned long flags;
>> + spin_lock_irqsave(&zone->lock, flags);
>> + zone->min_cma_pages += count;
>> + spin_unlock_irqrestore(&zone->lock, flags);
>> + setup_per_zone_wmarks();
>> +}
>> +
>> +/*
>> + * Trigger memory pressure bump to reclaim some pages in order to be able to
>> + * allocate 'count' pages in single page units. Does similar work as
>> + *__alloc_pages_slowpath() function.
>> + */
>> +static int __reclaim_pages(struct zone *zone, gfp_t gfp_mask, int count)
>> +{
>> + enum zone_type high_zoneidx = gfp_zone(gfp_mask);
>> + struct zonelist *zonelist = node_zonelist(0, gfp_mask);
>> + int did_some_progress = 0;
>> + int order = 1;
>> + unsigned long watermark;
>> +
>> + /*
>> + * Increase level of watermarks to force kswapd do his job
>> + * to stabilise at new watermark level.
>> + */
>> + __modify_min_cma_pages(zone, count);
On Mon, 13 Feb 2012 10:57:58 -0800, Robert Nelson <robertcnelson@...il.com> wrote:
> Hi Marek, This ^^^ function doesn't seem to exist in this patchset,
> is it in another set posted to lkml?
This should read __update_cma_wmark_pages(). Sorry for the incorrect patch.
--
Best regards, _ _
.o. | Liege of Serenely Enlightened Majesty of o' \,=./ `o
..o | Computer Science, Michał “mina86” Nazarewicz (o o)
ooo +----<email/xmpp: mpn@...gle.com>--------------ooO--(_)--Ooo--
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists