Whamcloud - gitweb
LU-20 kernel: add descriptions to kernel patches
authorAndreas Dilger <adilger@whamcloud.com>
Mon, 19 Nov 2012 19:30:53 +0000 (12:30 -0700)
committerAndreas Dilger <adilger@whamcloud.com>
Wed, 28 Nov 2012 00:18:43 +0000 (19:18 -0500)
Add some descriptive text to kernel patches so that it is more clear
what the patches are needed for, and whether they can simply be
dropped, or if they should be pushed upstream.

Signed-off-by: Andreas Dilger <adilger@whamcloud.com>
Change-Id: Ie19de1feedc8bedeb7ae890886aab73815500c1e
Reviewed-on: http://review.whamcloud.com/4613
Tested-by: Hudson
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: James Simmons <uja.ornl@gmail.com>
Reviewed-by: Keith Mannthey <kemannthey@gmail.com>
lustre/kernel_patches/patches/bh_lru_size_config.patch
lustre/kernel_patches/patches/export-2.6.32-vanilla.patch

index e84edba..4db0102 100644 (file)
@@ -1,3 +1,20 @@
+Allow increasing the buffer-head per-CPU LRU size to allow efficient
+filesystem operations that access many blocks for each transaction.
+For example, creating a file in a large ext4 directory with quota
+enabled will accesses multiple buffer heads and will overflow the LRU
+at the default 8-block LRU size:
+
+* parent directory inode table block (ctime, nlinks for subdirs)
+* new inode bitmap
+* inode table block
+* 2 quota blocks
+* directory leaf block (not reused, but pollutes one cache entry)
+* 2 levels htree blocks (only one is reused, other pollutes cache)
+* 2 levels indirect/index blocks (only one is reused)
+
+Signed-off-by: Liang Zhen <liang@whamcoud.com>
+Signed-off-by: Andreas Dilger <adilger@whamcloud.com>
+
 --- linux-2.6.32-131.6.1/fs/buffer.c
 +++ linux-2.6.32-131.6.1-pdo/fs/buffer.c
 @@ -1250,8 +1250,7 @@
@@ -19,7 +36,7 @@
 +config BH_LRU_SIZE
 +      int "buffer head per-CPU LRU size"
 +      range 8 64
-+      default "8"
++      default "16"
 +      help
 +        This sets the per-CPU LRU size for buffer heads in memory.
 +        More complex filesystems may be modiyfing multiple blocks
index 0cb7884..95157b3 100644 (file)
@@ -1,7 +1,7 @@
-security_inode_unlink() is used in filter_vfs_unlink()
-to avoid lock ordering problems.  I'm not sure if this
-is still needed with ext4, and it definitely looks to
-be gone with DMU changes.
+security_inode_unlink() was used in filter_vfs_unlink() to avoid
+lock ordering problems.  This is no longer needed for Lustre 2.4
+with the change to osd-ldiskfs, but is kept for compatibility with
+older server code so they can share the same kernel.
 
 Index: linux+rh+chaos/security/security.c
 ===================================================================