From mboxrd@z Thu Jan 1 00:00:00 1970 Date: Tue, 7 Aug 2001 23:06:13 -0300 (BRT) From: Marcelo Tosatti Subject: [PATCH] total_free_shortage() using zone_free_shortage() (fwd) Message-ID: MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: owner-linux-mm@kvack.org Return-Path: To: Linus Torvalds , lkml , linux-mm@kvack.org List-ID: 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 To: Linus Torvalds Cc: lkml , Rik van Riel 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); -- 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/