diff options
| author | Eric Dumazet <eric.dumazet@gmail.com> | 2011-07-28 06:55:13 +0200 |
|---|---|---|
| committer | Al Viro <viro@zeniv.linux.org.uk> | 2011-08-01 01:41:17 -0400 |
| commit | c4ae0c65455c1bb30d1b71c6dd9a1a62aadde8ef (patch) | |
| tree | 75d45726ff5f09955054a665e368b919faf4c105 /fs/inode.c | |
| parent | f2ee7abf4c40c8e6bffced923a7c01ea2d1f6c97 (diff) | |
| download | lwn-c4ae0c65455c1bb30d1b71c6dd9a1a62aadde8ef.tar.gz lwn-c4ae0c65455c1bb30d1b71c6dd9a1a62aadde8ef.zip | |
vfs: avoid call to inode_lru_list_del() if possible
inode_lru_list_del() is expensive because of per superblock lru locking,
while some inodes are not in lru list.
Adding a check in iput_final() can speedup pipe/sockets workloads on
SMP.
Signed-off-by: Eric Dumazet <eric.dumazet@gmail.com>
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Diffstat (limited to 'fs/inode.c')
| -rw-r--r-- | fs/inode.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/fs/inode.c b/fs/inode.c index e445be2a18f9..5aab80dc008c 100644 --- a/fs/inode.c +++ b/fs/inode.c @@ -1330,7 +1330,8 @@ static void iput_final(struct inode *inode) } inode->i_state |= I_FREEING; - inode_lru_list_del(inode); + if (!list_empty(&inode->i_lru)) + inode_lru_list_del(inode); spin_unlock(&inode->i_lock); evict(inode); |