diff options
author | Jan Kara <jack@suse.cz> | 2012-03-13 22:27:44 -0400 |
---|---|---|
committer | Theodore Ts'o <tytso@mit.edu> | 2012-03-13 22:27:44 -0400 |
commit | c254c9ec14d5c418c8f36ea7573edae2470a1dc1 (patch) | |
tree | f7f7fd284408a2c986fbd78093e22c6a8850367f /fs/jbd2 | |
parent | 5bebccf90127859085f6a020f08ff7f648e285a0 (diff) |
jbd2: remove always true condition in __journal_try_to_free_buffer()
The check b_jlist == BJ_None in __journal_try_to_free_buffer() is
always true (__jbd2_journal_temp_unlink_buffer() also checks this in
an assertion) so just remove it.
Signed-off-by: Jan Kara <jack@suse.cz>
Signed-off-by: "Theodore Ts'o" <tytso@mit.edu>
Diffstat (limited to 'fs/jbd2')
-rw-r--r-- | fs/jbd2/transaction.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/fs/jbd2/transaction.c b/fs/jbd2/transaction.c index 3eb326a54bf1..fd052a88e9ec 100644 --- a/fs/jbd2/transaction.c +++ b/fs/jbd2/transaction.c | |||
@@ -1676,10 +1676,8 @@ __journal_try_to_free_buffer(journal_t *journal, struct buffer_head *bh) | |||
1676 | spin_lock(&journal->j_list_lock); | 1676 | spin_lock(&journal->j_list_lock); |
1677 | if (jh->b_cp_transaction != NULL && jh->b_transaction == NULL) { | 1677 | if (jh->b_cp_transaction != NULL && jh->b_transaction == NULL) { |
1678 | /* written-back checkpointed metadata buffer */ | 1678 | /* written-back checkpointed metadata buffer */ |
1679 | if (jh->b_jlist == BJ_None) { | 1679 | JBUFFER_TRACE(jh, "remove from checkpoint list"); |
1680 | JBUFFER_TRACE(jh, "remove from checkpoint list"); | 1680 | __jbd2_journal_remove_checkpoint(jh); |
1681 | __jbd2_journal_remove_checkpoint(jh); | ||
1682 | } | ||
1683 | } | 1681 | } |
1684 | spin_unlock(&journal->j_list_lock); | 1682 | spin_unlock(&journal->j_list_lock); |
1685 | out: | 1683 | out: |