diff options
| author | Dan Carpenter <[email protected]> | 2024-10-11 19:43:44 +0000 |
|---|---|---|
| committer | Theodore Ts'o <[email protected]> | 2024-11-13 04:54:15 +0000 |
| commit | 27349b4d2ed072cabedd5115f0542b3b7b538aa8 (patch) | |
| tree | 099a0ab2cefd48ca11b531329ebc42c55f490220 /fs/ext4/fast_commit.c | |
| parent | ext4: use string choices helpers (diff) | |
| download | kernel-27349b4d2ed072cabedd5115f0542b3b7b538aa8.tar.gz kernel-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 <[email protected]>
Link: https://patch.msgid.link/[email protected]
Signed-off-by: Theodore Ts'o <[email protected]>
Diffstat (limited to 'fs/ext4/fast_commit.c')
| -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; } |