[PATCH] total_free_shortage() using zone_free_shortage() (fwd)

[Marcelo Tosatti]

Linus, 

Could you please apply this patch ? 

Without it total_free_shortage() will use zone->pages_min to calculcate
the sum of the shortages, while zone_free_shortage() is correctly using
zone->pages_high: Boom.

---------- Forwarded message ----------
Date: Mon, 6 Aug 2001 20:19:56 -0300 (BRT)
From: Marcelo Tosatti <[e-mail]>
To: Linus Torvalds <[e-mail]>
Cc: lkml <[e-mail]>, Rik van Riel <[e-mail]>
Subject: [PATCH] total_free_shortage() using zone_free_shortage()

Linus, 

The following patch changes total_free_shortage() to use
zone_free_shortage() to calculate the sum of perzone free shortages.

This way we isolate the calculation variables in zone_free_shortage(). 

Against 2.4.8-pre4. Please apply. 

diff -Nur linux.orig/mm/vmscan.c linux/mm/vmscan.c
--- linux.orig/mm/vmscan.c	Mon Aug  6 21:29:11 2001
+++ linux/mm/vmscan.c	Mon Aug  6 21:37:53 2001
@@ -807,12 +807,8 @@
 		int i;
 		for(i = 0; i < MAX_NR_ZONES; i++) {
 			zone_t *zone = pgdat->node_zones+ i;
-			if (zone->size && (zone->inactive_clean_pages +
-					zone->free_pages < zone->pages_min)) {
-				sum += zone->pages_min;
-				sum -= zone->free_pages;
-				sum -= zone->inactive_clean_pages;
-			}
+
+			sum += zone_free_shortage(zone);
 		}
 		pgdat = pgdat->node_next;
 	} while (pgdat);