Size max files based on memory size

Message ID 20020204055346.A18191@averell
State New, archived
Headers show
Series
  • Size max files based on memory size
Related show

Commit Message

Andi Kleen Feb. 4, 2002, 4:53 a.m. UTC
Hi,

The default files_max settings of 8192 is rather low for many workloads. 
This patch changes it to be sized based on the available memory.  It assumes
file + inode + dentry are roughly 1K and will use upto 10% of the available
memory for files_max. 

Also removes two obsolete prototypes in the include file. 

Patch for 2.5.3.  Please consider applying.

Thanks,
-Andi


-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Patch

--- linux-2.5.3-work/fs/dcache.c-FILESSZ	Tue Jan 15 17:53:31 2002
+++ linux-2.5.3-work/fs/dcache.c	Mon Feb  4 05:43:51 2002
@@ -1283,6 +1283,7 @@ 
 
 	dcache_init(mempages);
 	inode_init(mempages);
+	files_init(mempages); 
 	mnt_init(mempages);
 	bdev_cache_init();
 	cdev_cache_init();
--- linux-2.5.3-work/fs/file_table.c-FILESSZ	Mon Sep 17 22:16:30 2001
+++ linux-2.5.3-work/fs/file_table.c	Mon Feb  4 05:43:43 2002
@@ -186,3 +186,17 @@ 
 	file_list_unlock();
 	return 0;
 }
+
+void __init files_init(unsigned long mempages)
+{ 
+	int n; 
+	/* One file with associated inode and dcache is very roughly 1K. 
+	 * Per default don't use more than 10% of our memory for files. 
+	 */ 
+
+	n = (mempages * (PAGE_SIZE / 1024)) / 10;
+	files_stat.max_files = n; 
+	if (files_stat.max_files < NR_FILE)
+		files_stat.max_files = NR_FILE;
+} 
+
--- linux-2.5.3-work/include/linux/fs.h-FILESSZ	Wed Jan 30 22:43:34 2002
+++ linux-2.5.3-work/include/linux/fs.h	Mon Feb  4 05:43:43 2002
@@ -208,6 +208,7 @@ 
 extern void buffer_init(unsigned long);
 extern void inode_init(unsigned long);
 extern void mnt_init(unsigned long);
+extern void files_init(unsigned long);
 
 /* bh state bits */
 enum bh_state_bits {
@@ -1475,8 +1476,6 @@ 
 	}
 	return 0;
 }
-unsigned long generate_cluster(kdev_t, int b[], int);
-unsigned long generate_cluster_swab32(kdev_t, int b[], int);
 extern kdev_t ROOT_DEV;
 extern char root_device_name[];