During an AIM7 run on a 16GB system, fork started failing around
32000 threads, despite the system having plenty of free swap and
15GB of pageable memory.
If normal pageout does not result in contiguous free pages for
kernel stacks, fall back to lumpy reclaim instead of failing fork
or doing excessive pageout IO.
I do not know whether this change is needed due to the extreme
stress test or because the inactive list is a smaller fraction
of system memory on huge systems.
Signed-off-by: Rik van Riel <riel@redhat.com>
Index: linux-2.6.25-rc2-mm1/mm/vmscan.c
===================================================================
--- linux-2.6.25-rc2-mm1.orig/mm/vmscan.c 2008-02-28 00:29:46.000000000 -0500
+++ linux-2.6.25-rc2-mm1/mm/vmscan.c 2008-02-28 00:29:55.000000000 -0500
@@ -870,7 +870,8 @@ int isolate_lru_page(struct page *page)
* of reclaimed pages
*/
static unsigned long shrink_inactive_list(unsigned long max_scan,
- struct zone *zone, struct scan_control *sc, int file)
+ struct zone *zone, struct scan_control *sc,
+ int priority, int file)
{
LIST_HEAD(page_list);
struct pagevec pvec;
@@ -888,8 +889,19 @@ static unsigned long shrink_inactive_lis
unsigned long nr_freed;
unsigned long nr_active;
unsigned int count[NR_LRU_LISTS] = { 0, };
- int mode = (sc->order > PAGE_ALLOC_COSTLY_ORDER) ?
- ISOLATE_BOTH : ISOLATE_INACTIVE;
+ int mode = ISOLATE_INACTIVE;
+
+ /*
+ * If we need a large contiguous chunk of memory, or have
+ * trouble getting a small set of contiguous pages, we
+ * will reclaim both active and inactive pages.
+ *
+ * We use the same threshold as pageout congestion_wait below.
+ */
+ if (sc->order > PAGE_ALLOC_COSTLY_ORDER)
+ mode = ISOLATE_BOTH;
+ else if (sc->order && priority < DEF_PRIORITY - 2)
+ mode = ISOLATE_BOTH;
nr_taken = sc->isolate_pages(sc->swap_cluster_max,
&page_list, &nr_scan, sc->order, mode,
@@ -1178,7 +1190,7 @@ static unsigned long shrink_list(enum lr
shrink_active_list(nr_to_scan, zone, sc, priority, file);
return 0;
}
- return shrink_inactive_list(nr_to_scan, zone, sc, file);
+ return shrink_inactive_list(nr_to_scan, zone, sc, priority, file);
}
/*
--
All Rights Reversed
--
| David Newall | Re: Slow DOWN, please!!! |
| Linus Torvalds | Re: O_DIRECT question |
| Ingo Molnar | Re: 2.6.24-rc4-git5: Reported regressions from 2.6.23 |
| Greg KH | [GIT PATCH] driver core patches against 2.6.24 |
git: | |
| Junio C Hamano | Re: [RFC] Git User's Survey 2008 |
| Junichi Uekawa | Re: [ANNOUNCE] GIT 1.5.4 |
| Marcus Griep | [PATCH] git-svn: Make it scream by minimizing temp files |
| Bill Lear | Meaning of "fatal: protocol error: bad line length character"? |
| Richard Stallman | Real men don't attack straw men |
| GVG GVG | ssh_exchange_identification: Connection closed by remote host |
| farhan ahmed | Re: bash for root? (was: Re: libiconv problem ) |
| Tony Sarendal | bgpd causing black-holes with bgp-only setup |
| Krishna Kumar | [PATCH 9/10 REV5] [IPoIB] Implement batching |
| jamal | Re: [PATCH 2/3][NET_BATCH] net core use batching |
| Andi Kleen | [PATCH] Disable TSO for non standard qdiscs |
| James Chapman | Re: [PATCH][PPPOL2TP]: Fix SMP oops in pppol2tp driver |
