diff options
author | Shijie Luo <luoshijie1@huawei.com> | 2020-01-23 02:00:54 -0500 |
---|---|---|
committer | Theodore Ts'o <tytso@mit.edu> | 2020-01-25 02:25:56 -0500 |
commit | 17c51d836c19ecc58714fc671a914a47b6ae4db7 (patch) | |
tree | 662426c502305bbff381304452f9c28cd725e557 /fs/jbd2 | |
parent | 8d6ce136790268fba2fc66cb8d6fa2161d4b2385 (diff) | |
download | lwn-17c51d836c19ecc58714fc671a914a47b6ae4db7.tar.gz lwn-17c51d836c19ecc58714fc671a914a47b6ae4db7.zip |
jbd2: remove pointless assertion in __journal_remove_journal_head
Only when jh->b_jcount = 0 in jbd2_journal_put_journal_head, we are allowed
to call __journal_remove_journal_head. This assertion is meaningless,
just remove it.
Signed-off-by: Shijie Luo <luoshijie1@huawei.com>
Reviewed-by: Jan Kara <jack@suse.cz>
Link: https://lore.kernel.org/r/20200123070054.50585-1-luoshijie1@huawei.com
Signed-off-by: Theodore Ts'o <tytso@mit.edu>
Diffstat (limited to 'fs/jbd2')
-rw-r--r-- | fs/jbd2/journal.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/fs/jbd2/journal.c b/fs/jbd2/journal.c index 069b22eba795..1c61491fd86f 100644 --- a/fs/jbd2/journal.c +++ b/fs/jbd2/journal.c @@ -2560,7 +2560,6 @@ static void __journal_remove_journal_head(struct buffer_head *bh) { struct journal_head *jh = bh2jh(bh); - J_ASSERT_JH(jh, jh->b_jcount >= 0); J_ASSERT_JH(jh, jh->b_transaction == NULL); J_ASSERT_JH(jh, jh->b_next_transaction == NULL); J_ASSERT_JH(jh, jh->b_cp_transaction == NULL); |