From mboxrd@z Thu Jan 1 00:00:00 1970 Date: Wed, 15 Jan 2003 23:47:04 -0800 From: "Martin J. Bligh" Subject: [PATCH] make vm_enough_memory more efficient Message-ID: <66360000.1042703224@titus> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7bit Content-Disposition: inline Sender: owner-linux-mm@kvack.org Return-Path: To: Andrew Morton Cc: linux-mm mailing list , linux-kernel List-ID: vm_enough_memory seems to call si_meminfo just to get the total RAM, which seems far too expensive. This replaces the comment saying "this is crap" with some code that's less crap. Not heavily tested (compiles and boots), but seems pretty obvious. M. diff -urpN -X /home/fletch/.diff.exclude virgin/mm/mmap.c vm_enough_memory/mm/mmap.c --- virgin/mm/mmap.c Mon Jan 13 21:09:28 2003 +++ vm_enough_memory/mm/mmap.c Wed Jan 15 23:41:39 2003 @@ -72,7 +72,6 @@ inline void vm_unacct_memory(long pages) int vm_enough_memory(long pages) { unsigned long free, allowed; - struct sysinfo i; atomic_add(pages, &vm_committed_space); @@ -113,12 +112,7 @@ int vm_enough_memory(long pages) return 0; } - /* - * FIXME: need to add arch hooks to get the bits we need - * without this higher overhead crap - */ - si_meminfo(&i); - allowed = i.totalram * sysctl_overcommit_ratio / 100; + allowed = totalram_pages * sysctl_overcommit_ratio / 100; allowed += total_swap_pages; if (atomic_read(&vm_committed_space) < allowed) -- 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/