* Problem with /proc/sys/vm/lowmem_reserve_ratio
@ 2008-02-19 21:35 Larry Woodman
2008-02-19 23:55 ` Andrew Morton
0 siblings, 1 reply; 5+ messages in thread
From: Larry Woodman @ 2008-02-19 21:35 UTC (permalink / raw)
To: linux-kernel, Andrew Morton
balance_pgdat() calls zone_watermark_ok() three times, the first call
passes a zero(0) in as the 4th argument. This 4th argument is the
classzone_idx which is used as the index into the zone->lowmem_reserve[]
array.
Since setup_per_zone_lowmem_reserve()
always sets the zone->lowmem_reserve[0] = 0(because there is nothing
below the DMA zone), zone_watermark_ok() will not consider the
lowmem_reserve pages when zero is passed as the 4th arg. The
4th argument must be "i" or balance_pgdat wont even get into the main loop
when lowmem_reserve_ratio is lowered.
-------------------------------------------------------------------------
--- linux-2.6.24.noarch/mm/vmscan.c.orig 2008-02-13
11:14:55.000000000 -0500
+++ linux-2.6.24.noarch/mm/vmscan.c 2008-02-13 11:15:02.000000000
-0500
@@ -1375,7 +1375,7 @@ loop_again:
continue;
if (!zone_watermark_ok(zone, order,
zone->pages_high,
- 0, 0)) {
+ i, 0)) {
end_zone = i;
break;
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: Problem with /proc/sys/vm/lowmem_reserve_ratio 2008-02-19 21:35 Problem with /proc/sys/vm/lowmem_reserve_ratio Larry Woodman @ 2008-02-19 23:55 ` Andrew Morton 2008-02-20 12:53 ` Peter Zijlstra 0 siblings, 1 reply; 5+ messages in thread From: Andrew Morton @ 2008-02-19 23:55 UTC (permalink / raw) To: Larry Woodman; +Cc: linux-kernel On Tue, 19 Feb 2008 16:35:49 -0500 Larry Woodman <lwoodman@redhat.com> wrote: > balance_pgdat() calls zone_watermark_ok() three times, the first call > passes a zero(0) in as the 4th argument. This 4th argument is the > classzone_idx which is used as the index into the zone->lowmem_reserve[] > array. > Since setup_per_zone_lowmem_reserve() > always sets the zone->lowmem_reserve[0] = 0(because there is nothing > below the DMA zone), zone_watermark_ok() will not consider the > lowmem_reserve pages when zero is passed as the 4th arg. The > 4th argument must be "i" or balance_pgdat wont even get into the main loop > when lowmem_reserve_ratio is lowered. > > ------------------------------------------------------------------------- > --- linux-2.6.24.noarch/mm/vmscan.c.orig 2008-02-13 > 11:14:55.000000000 -0500 > +++ linux-2.6.24.noarch/mm/vmscan.c 2008-02-13 11:15:02.000000000 > -0500 > @@ -1375,7 +1375,7 @@ loop_again: > continue; > > if (!zone_watermark_ok(zone, order, > zone->pages_high, > > - 0, 0)) { > + i, 0)) { > end_zone = i; > break; Yes, thanks, this is in my things-to-worry-about-when-i-get-home bucket. We should find the changeset which added this and work out if for some reason it was intentional. ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: Problem with /proc/sys/vm/lowmem_reserve_ratio 2008-02-19 23:55 ` Andrew Morton @ 2008-02-20 12:53 ` Peter Zijlstra 2008-02-20 13:33 ` Andrew Morton 0 siblings, 1 reply; 5+ messages in thread From: Peter Zijlstra @ 2008-02-20 12:53 UTC (permalink / raw) To: Andrew Morton; +Cc: Larry Woodman, linux-kernel, Nick Piggin On Tue, 2008-02-19 at 15:55 -0800, Andrew Morton wrote: > On Tue, 19 Feb 2008 16:35:49 -0500 Larry Woodman <lwoodman@redhat.com> wrote: > > > balance_pgdat() calls zone_watermark_ok() three times, the first call > > passes a zero(0) in as the 4th argument. This 4th argument is the > > classzone_idx which is used as the index into the zone->lowmem_reserve[] > > array. > > Since setup_per_zone_lowmem_reserve() > > always sets the zone->lowmem_reserve[0] = 0(because there is nothing > > below the DMA zone), zone_watermark_ok() will not consider the > > lowmem_reserve pages when zero is passed as the 4th arg. The > > 4th argument must be "i" or balance_pgdat wont even get into the main loop > > when lowmem_reserve_ratio is lowered. > > > > ------------------------------------------------------------------------- > > --- linux-2.6.24.noarch/mm/vmscan.c.orig 2008-02-13 > > 11:14:55.000000000 -0500 > > +++ linux-2.6.24.noarch/mm/vmscan.c 2008-02-13 11:15:02.000000000 > > -0500 > > @@ -1375,7 +1375,7 @@ loop_again: > > continue; > > > > if (!zone_watermark_ok(zone, order, > > zone->pages_high, > > > > - 0, 0)) { > > + i, 0)) { > > end_zone = i; > > break; > > Yes, thanks, this is in my things-to-worry-about-when-i-get-home bucket. > We should find the changeset which added this and work out if for some > reason it was intentional. commit e0e1723229b6f96922d10bb932f94d899132b462 Author: nickpiggin <nickpiggin> Date: Tue Jan 4 04:14:42 2005 +0000 [PATCH] mm: teach kswapd about higher order areas Teach kswapd to free memory on behalf of higher order allocators. This could be important for higher order atomic allocations because they otherwise have no means to free the memory themselves. Signed-off-by: Nick Piggin <nickpiggin@yahoo.com.au> Signed-off-by: Andrew Morton <akpm@osdl.org> Signed-off-by: Linus Torvalds <torvalds@osdl.org> BKrev: 41da1832E5flzqtNXq5m70WxihpcMw diff --git a/include/linux/mmzone.h b/include/linux/mmzone.h index 2fd19fa..e048bbc 100644 --- a/include/linux/mmzone.h +++ b/include/linux/mmzone.h @@ -264,8 +264,9 @@ typedef struct pglist_data { range, including holes */ int node_id; struct pglist_data *pgdat_next; - wait_queue_head_t kswapd_wait; + wait_queue_head_t kswapd_wait; struct task_struct *kswapd; + int kswapd_max_order; } pg_data_t; #define node_present_pages(nid) (NODE_DATA(nid)->node_present_pages) @@ -279,7 +280,7 @@ void __get_zone_counts(unsigned long *active, unsigned long *inactive, void get_zone_counts(unsigned long *active, unsigned long *inactive, unsigned long *free); void build_all_zonelists(void); -void wakeup_kswapd(struct zone *zone); +void wakeup_kswapd(struct zone *zone, int order); int zone_watermark_ok(struct zone *z, int order, unsigned long mark, int alloc_type, int can_try_harder, int gfp_high); diff --git a/mm/page_alloc.c b/mm/page_alloc.c index bb11a6d..1f264ba 100644 --- a/mm/page_alloc.c +++ b/mm/page_alloc.c @@ -677,7 +677,7 @@ __alloc_pages(unsigned int gfp_mask, unsigned int order, } for (i = 0; (z = zones[i]) != NULL; i++) - wakeup_kswapd(z); + wakeup_kswapd(z, order); /* * Go through the zonelist again. Let __GFP_HIGH and allocations @@ -1516,6 +1516,7 @@ static void __init free_area_init_core(struct pglist_data *pgdat, pgdat->nr_zones = 0; init_waitqueue_head(&pgdat->kswapd_wait); + pgdat->kswapd_max_order = 0; for (j = 0; j < MAX_NR_ZONES; j++) { struct zone *zone = pgdat->node_zones + j; diff --git a/mm/vmscan.c b/mm/vmscan.c index aa074e5..1062a30 100644 --- a/mm/vmscan.c +++ b/mm/vmscan.c @@ -968,7 +968,7 @@ out: * the page allocator fallback scheme to ensure that aging of pages is balanced * across the zones. */ -static int balance_pgdat(pg_data_t *pgdat, int nr_pages) +static int balance_pgdat(pg_data_t *pgdat, int nr_pages, int order) { int to_free = nr_pages; int all_zones_ok; @@ -1014,7 +1014,8 @@ loop_again: priority != DEF_PRIORITY) continue; - if (zone->free_pages <= zone->pages_high) { + if (!zone_watermark_ok(zone, order, + zone->pages_high, 0, 0, 0)) { end_zone = i; goto scan; } @@ -1049,7 +1050,8 @@ scan: continue; if (nr_pages == 0) { /* Not software suspend */ - if (zone->free_pages <= zone->pages_high) + if (!zone_watermark_ok(zone, order, + zone->pages_high, end_zone, 0, 0)) all_zones_ok = 0; } zone->temp_priority = priority; @@ -1127,6 +1129,7 @@ out: */ static int kswapd(void *p) { + unsigned long order; pg_data_t *pgdat = (pg_data_t*)p; struct task_struct *tsk = current; DEFINE_WAIT(wait); @@ -1155,14 +1158,28 @@ static int kswapd(void *p) */ tsk->flags |= PF_MEMALLOC|PF_KSWAPD; + order = 0; for ( ; ; ) { + unsigned long new_order; if (current->flags & PF_FREEZE) refrigerator(PF_FREEZE); + prepare_to_wait(&pgdat->kswapd_wait, &wait, TASK_INTERRUPTIBLE); - schedule(); + new_order = pgdat->kswapd_max_order; + pgdat->kswapd_max_order = 0; + if (order < new_order) { + /* + * Don't sleep if someone wants a larger 'order' + * allocation + */ + order = new_order; + } else { + schedule(); + order = pgdat->kswapd_max_order; + } finish_wait(&pgdat->kswapd_wait, &wait); - balance_pgdat(pgdat, 0); + balance_pgdat(pgdat, 0, order); } return 0; } @@ -1170,12 +1187,18 @@ static int kswapd(void *p) /* * A zone is low on free memory, so wake its kswapd task to service it. */ -void wakeup_kswapd(struct zone *zone) +void wakeup_kswapd(struct zone *zone, int order) { + pg_data_t *pgdat; + if (zone->present_pages == 0) return; - if (zone->free_pages > zone->pages_low) + + pgdat = zone->zone_pgdat; + if (zone_watermark_ok(zone, order, zone->pages_low, 0, 0, 0)) return; + if (pgdat->kswapd_max_order < order) + pgdat->kswapd_max_order = order; if (!waitqueue_active(&zone->zone_pgdat->kswapd_wait)) return; wake_up_interruptible(&zone->zone_pgdat->kswapd_wait); @@ -1198,7 +1221,7 @@ int shrink_all_memory(int nr_pages) current->reclaim_state = &reclaim_state; for_each_pgdat(pgdat) { int freed; - freed = balance_pgdat(pgdat, nr_to_free); + freed = balance_pgdat(pgdat, nr_to_free, 0); ret += freed; nr_to_free -= freed; if (nr_to_free <= 0) ^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: Problem with /proc/sys/vm/lowmem_reserve_ratio 2008-02-20 12:53 ` Peter Zijlstra @ 2008-02-20 13:33 ` Andrew Morton 0 siblings, 0 replies; 5+ messages in thread From: Andrew Morton @ 2008-02-20 13:33 UTC (permalink / raw) To: Peter Zijlstra; +Cc: Larry Woodman, linux-kernel, Nick Piggin On Wed, 20 Feb 2008 13:53:14 +0100 Peter Zijlstra <a.p.zijlstra@chello.nl> wrote: > > On Tue, 2008-02-19 at 15:55 -0800, Andrew Morton wrote: > > On Tue, 19 Feb 2008 16:35:49 -0500 Larry Woodman <lwoodman@redhat.com> wrote: > > > > > balance_pgdat() calls zone_watermark_ok() three times, the first call > > > passes a zero(0) in as the 4th argument. This 4th argument is the > > > classzone_idx which is used as the index into the zone->lowmem_reserve[] > > > array. > > > Since setup_per_zone_lowmem_reserve() > > > always sets the zone->lowmem_reserve[0] = 0(because there is nothing > > > below the DMA zone), zone_watermark_ok() will not consider the > > > lowmem_reserve pages when zero is passed as the 4th arg. The > > > 4th argument must be "i" or balance_pgdat wont even get into the main loop > > > when lowmem_reserve_ratio is lowered. > > > > > > ------------------------------------------------------------------------- > > > --- linux-2.6.24.noarch/mm/vmscan.c.orig 2008-02-13 > > > 11:14:55.000000000 -0500 > > > +++ linux-2.6.24.noarch/mm/vmscan.c 2008-02-13 11:15:02.000000000 > > > -0500 > > > @@ -1375,7 +1375,7 @@ loop_again: > > > continue; > > > > > > if (!zone_watermark_ok(zone, order, > > > zone->pages_high, > > > > > > - 0, 0)) { > > > + i, 0)) { > > > end_zone = i; > > > break; > > > > Yes, thanks, this is in my things-to-worry-about-when-i-get-home bucket. > > We should find the changeset which added this and work out if for some > > reason it was intentional. > > > commit e0e1723229b6f96922d10bb932f94d899132b462 Thanks. > Author: nickpiggin <nickpiggin> > Date: Tue Jan 4 04:14:42 2005 +0000 > > [PATCH] mm: teach kswapd about higher order areas > > Teach kswapd to free memory on behalf of higher order allocators. This > could be important for higher order atomic allocations because they > otherwise have no means to free the memory themselves. > > Signed-off-by: Nick Piggin <nickpiggin@yahoo.com.au> > Signed-off-by: Andrew Morton <akpm@osdl.org> > Signed-off-by: Linus Torvalds <torvalds@osdl.org> > > BKrev: 41da1832E5flzqtNXq5m70WxihpcMw > > diff --git a/include/linux/mmzone.h b/include/linux/mmzone.h > index 2fd19fa..e048bbc 100644 > --- a/include/linux/mmzone.h > +++ b/include/linux/mmzone.h > @@ -264,8 +264,9 @@ typedef struct pglist_data { > range, including holes */ > int node_id; > struct pglist_data *pgdat_next; > - wait_queue_head_t kswapd_wait; > + wait_queue_head_t kswapd_wait; > struct task_struct *kswapd; > + int kswapd_max_order; > } pg_data_t; > > #define node_present_pages(nid) (NODE_DATA(nid)->node_present_pages) > @@ -279,7 +280,7 @@ void __get_zone_counts(unsigned long *active, unsigned long *inactive, > void get_zone_counts(unsigned long *active, unsigned long *inactive, > unsigned long *free); > void build_all_zonelists(void); > -void wakeup_kswapd(struct zone *zone); > +void wakeup_kswapd(struct zone *zone, int order); > int zone_watermark_ok(struct zone *z, int order, unsigned long mark, > int alloc_type, int can_try_harder, int gfp_high); > > diff --git a/mm/page_alloc.c b/mm/page_alloc.c > index bb11a6d..1f264ba 100644 > --- a/mm/page_alloc.c > +++ b/mm/page_alloc.c > @@ -677,7 +677,7 @@ __alloc_pages(unsigned int gfp_mask, unsigned int order, > } > > for (i = 0; (z = zones[i]) != NULL; i++) > - wakeup_kswapd(z); > + wakeup_kswapd(z, order); > > /* > * Go through the zonelist again. Let __GFP_HIGH and allocations > @@ -1516,6 +1516,7 @@ static void __init free_area_init_core(struct pglist_data *pgdat, > > pgdat->nr_zones = 0; > init_waitqueue_head(&pgdat->kswapd_wait); > + pgdat->kswapd_max_order = 0; > > for (j = 0; j < MAX_NR_ZONES; j++) { > struct zone *zone = pgdat->node_zones + j; > diff --git a/mm/vmscan.c b/mm/vmscan.c > index aa074e5..1062a30 100644 > --- a/mm/vmscan.c > +++ b/mm/vmscan.c > @@ -968,7 +968,7 @@ out: > * the page allocator fallback scheme to ensure that aging of pages is balanced > * across the zones. > */ > -static int balance_pgdat(pg_data_t *pgdat, int nr_pages) > +static int balance_pgdat(pg_data_t *pgdat, int nr_pages, int order) > { > int to_free = nr_pages; > int all_zones_ok; > @@ -1014,7 +1014,8 @@ loop_again: > priority != DEF_PRIORITY) > continue; > > - if (zone->free_pages <= zone->pages_high) { > + if (!zone_watermark_ok(zone, order, > + zone->pages_high, 0, 0, 0)) { > end_zone = i; > goto scan; > } No, it doesn't look like there was a deeper purpose here. Just a thinko? > @@ -1049,7 +1050,8 @@ scan: > continue; > > if (nr_pages == 0) { /* Not software suspend */ > - if (zone->free_pages <= zone->pages_high) > + if (!zone_watermark_ok(zone, order, > + zone->pages_high, end_zone, 0, 0)) > all_zones_ok = 0; > } > zone->temp_priority = priority; ^ permalink raw reply [flat|nested] 5+ messages in thread
* Problem with /proc/sys/vm/lowmem_reserve_ratio
@ 2008-02-13 16:09 Larry Woodman
0 siblings, 0 replies; 5+ messages in thread
From: Larry Woodman @ 2008-02-13 16:09 UTC (permalink / raw)
To: linux-mm
balance_pgdat() calls zone_watermark_ok() three times, the first call
passes a zero(0) in as the 4th argument. This 4th argument is the
classzone_idx which is used as the index into the
zone->lowmem_reserve[] array. Since setup_per_zone_lowmem_reserve()
always sets the zone->lowmem_reserve[0] = 0(because there is nothing
below the DMA zone), zone_watermark_ok() will not consider the
lowmem_reserve pages when zero is passed as the 4th arg. Shouldnt this
4th argument be either "i" or "nr_zones - 1" ???
-------------------------------------------------------------------------
--- linux-2.6.24.noarch/mm/vmscan.c.orig 2008-02-13
11:14:55.000000000 -0500
+++ linux-2.6.24.noarch/mm/vmscan.c 2008-02-13 11:15:02.000000000
-0500
@@ -1375,7 +1375,7 @@ loop_again:
continue;
if (!zone_watermark_ok(zone, order, zone-
>pages_high,
- 0, 0)) {
+ i, 0)) {
end_zone = i;
break;
-------------------------------------------------------------------------
--- linux-2.6.24.noarch/mm/vmscan.c.orig 2008-02-13
11:14:55.000000000 -0500
+++ linux-2.6.24.noarch/mm/vmscan.c 2008-02-13 11:16:35.000000000
-0500
@@ -1375,7 +1375,7 @@ loop_again:
continue;
if (!zone_watermark_ok(zone, order, zone-
>pages_high,
- 0, 0)) {
+ nr_zones - 1, 0)) {
end_zone = i;
break;
}
-------------------------------------------------------------------------
--
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>
^ permalink raw reply [flat|nested] 5+ messages in threadend of thread, other threads:[~2008-02-20 13:34 UTC | newest] Thread overview: 5+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2008-02-19 21:35 Problem with /proc/sys/vm/lowmem_reserve_ratio Larry Woodman 2008-02-19 23:55 ` Andrew Morton 2008-02-20 12:53 ` Peter Zijlstra 2008-02-20 13:33 ` Andrew Morton -- strict thread matches above, loose matches on Subject: below -- 2008-02-13 16:09 Larry Woodman
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.