[patch-2.3.99-pre7-1] cleanup fs/inode.c fs/dcache.c

From: Tigran Aivazian (tigran@veritas.com)
Date: Sun Apr 30 2000 - 15:06:15 EST


Hi Linus,

I changed the way hashtable size is reported in inode_init() and
dcache_init() to be consistent with buffer_init() and page_cache_init().

Also, when allocation fails, it is wiser to dump the parameters and then
panic instead of just panic - this is what
buffer_init()/page_cache_init() do and so I changed
inode_init()/dcache_init() to do likewise.

Please consider the patch against 2.3.99-pre7-1.

Regards,
Tigran

--- fs/dcache.c.0 Sun Apr 30 20:59:17 2000
+++ fs/dcache.c Sun Apr 30 20:59:37 2000
@@ -1098,10 +1098,11 @@
                         __get_free_pages(GFP_ATOMIC, order);
         } while (dentry_hashtable == NULL && --order >= 0);
 
+ printk("VFS: dcache hash table entries: %d (order: %ld, %ld bytes)\n",
+ nr_hash, order, (PAGE_SIZE << order));
+
         if (!dentry_hashtable)
                 panic("Failed to allocate dcache hash table\n");
-
- printk("VFS: DCACHE hash table configured to %d entries\n", nr_hash);
 
         d = dentry_hashtable;
         i = nr_hash;
--- fs/inode.c.0 Sun Apr 30 20:57:42 2000
+++ fs/inode.c Sun Apr 30 20:58:52 2000
@@ -876,10 +876,11 @@
                         __get_free_pages(GFP_ATOMIC, order);
         } while (inode_hashtable == NULL && --order >= 0);
 
+ printk("VFS: inode hash table entries: %d (order: %ld, %ld bytes)\n",
+ nr_hash, order, (PAGE_SIZE << order));
+
         if (!inode_hashtable)
                 panic("Failed to allocate inode hash table\n");
-
- printk("VFS: INODE hash table configured to %d entries\n", nr_hash);
 
         head = inode_hashtable;
         i = nr_hash;

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



This archive was generated by hypermail 2b29 : Sun Apr 30 2000 - 21:00:18 EST