diff options
| author | Dan Carpenter <dan.carpenter@linaro.org> | 2024-10-11 22:43:44 +0300 |
|---|---|---|
| committer | Theodore Ts'o <tytso@mit.edu> | 2024-11-12 23:54:15 -0500 |
| commit | 27349b4d2ed072cabedd5115f0542b3b7b538aa8 (patch) | |
| tree | 099a0ab2cefd48ca11b531329ebc42c55f490220 /fs/ext4 | |
| parent | 867b73909ae0fb6a44552f22ad589c0511c9f2a4 (diff) | |
| download | lwn-27349b4d2ed072cabedd5115f0542b3b7b538aa8.tar.gz lwn-27349b4d2ed072cabedd5115f0542b3b7b538aa8.zip | |
ext4: cleanup variable name in ext4_fc_del()
The variables "&EXT4_SB(inode->i_sb)->s_fc_lock" and "&sbi->s_fc_lock"
are the same lock. This function uses a mix of both, which is a bit
unsightly and confuses Smatch.
Signed-off-by: Dan Carpenter <dan.carpenter@linaro.org>
Link: https://patch.msgid.link/96008557-8ff4-44cc-b5e3-ce242212f1a3@stanley.mountain
Signed-off-by: Theodore Ts'o <tytso@mit.edu>
Diffstat (limited to 'fs/ext4')
| -rw-r--r-- | fs/ext4/fast_commit.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/fs/ext4/fast_commit.c b/fs/ext4/fast_commit.c index 3dee94a38a68..26c4fc37edcf 100644 --- a/fs/ext4/fast_commit.c +++ b/fs/ext4/fast_commit.c @@ -291,9 +291,9 @@ void ext4_fc_del(struct inode *inode) return; restart: - spin_lock(&EXT4_SB(inode->i_sb)->s_fc_lock); + spin_lock(&sbi->s_fc_lock); if (list_empty(&ei->i_fc_list) && list_empty(&ei->i_fc_dilist)) { - spin_unlock(&EXT4_SB(inode->i_sb)->s_fc_lock); + spin_unlock(&sbi->s_fc_lock); return; } |