Re: kswapd craziness round 2

From: Hillf Danton
Date: Thu Feb 21 2013 - 07:07:10 EST


On Thu, Feb 21, 2013 at 6:14 AM, Jiri Slaby <jslaby@xxxxxxx> wrote:
>>
>> Does Ingo's revert help? https://lkml.org/lkml/2013/2/15/168
>
> Not at all...
>
Then mind taking a try?

--- a/mm/vmscan.c Thu Feb 21 20:01:02 2013
+++ b/mm/vmscan.c Thu Feb 21 20:05:58 2013
@@ -1715,7 +1715,7 @@ static void get_scan_count(struct lruvec
* to swap. Better start now and leave the - probably heavily
* thrashing - remaining file pages alone.
*/
- if (global_reclaim(sc)) {
+ if (global_reclaim(sc) && sc->priority >= DEF_PRIORITY - 2) {
free = zone_page_state(zone, NR_FREE_PAGES);
if (unlikely(file + free <= high_wmark_pages(zone))) {
scan_balance = SCAN_ANON;
@@ -2840,9 +2840,10 @@ out:
* reclaim if they wish.
*/
if (sc.nr_reclaimed < SWAP_CLUSTER_MAX)
- order = sc.order = 0;
-
- goto loop_again;
+ if (order != 0) {
+ sc.order = order = 0;
+ goto loop_again;
+ }
}

/*
--
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/