summaryrefslogtreecommitdiffstats
path: root/fs/anon_inodes.c
diff options
context:
space:
mode:
authorEric Sandeen <sandeen@redhat.com>2008-02-10 01:13:33 -0500
committerTheodore Ts'o <tytso@mit.edu>2008-02-10 01:13:33 -0500
commit256bdb497c6f562462f1e89fc8e1409f61ef40cb (patch)
treee8c16516bba080b4e40eccebb5a3ea5fb25cf5fd /fs/anon_inodes.c
parentc4e35e07af162ea4d642b1c6ffacbb63c3ed1804 (diff)
downloadlinux-256bdb497c6f562462f1e89fc8e1409f61ef40cb.tar.gz
linux-256bdb497c6f562462f1e89fc8e1409f61ef40cb.tar.bz2
linux-256bdb497c6f562462f1e89fc8e1409f61ef40cb.zip
ext4: allocate struct ext4_allocation_context from a kmem cache
struct ext4_allocation_context is rather large, and this bloats the stack of many functions which use it. Allocating it from a named slab cache will alleviate this. For example, with this change (on top of the noinline patch sent earlier): -ext4_mb_new_blocks 200 +ext4_mb_new_blocks 40 -ext4_mb_free_blocks 344 +ext4_mb_free_blocks 168 -ext4_mb_release_inode_pa 216 +ext4_mb_release_inode_pa 40 -ext4_mb_release_group_pa 192 +ext4_mb_release_group_pa 24 Most of these stack-allocated structs are actually used only for mballoc history; and in those cases often a smaller struct would do. So changing that may be another way around it, at least for those functions, if preferred. For now, in those cases where the ac is only for history, an allocation failure simply skips the history recording, and does not cause any other failures. Signed-off-by: Eric Sandeen <sandeen@redhat.com> Signed-off-by: Mingming Cao <cmm@us.ibm.com> Signed-off-by: "Theodore Ts'o" <tytso@mit.edu>
Diffstat (limited to 'fs/anon_inodes.c')
0 files changed, 0 insertions, 0 deletions