dlan17/linux
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

jbd2: remove useless 'block_error' variable

Browse Source 
The judgement 'if (block_error && success == 0)' is never valid. Just
remove useless 'block_error' variable.

Signed-off-by: Ye Bin <yebin10@huawei.com>
Reviewed-by: Jan Kara <jack@suse.cz>
Reviewed-by: Zhang Yi <yi.zhang@huawei.com>
Link: https://patch.msgid.link/20240930005942.626942-6-yebin@huaweicloud.com
Signed-off-by: Theodore Ts'o <tytso@mit.edu>
This commit is contained in:
Ye Bin
2024-11-12 23:54:14 -05:00
committed by Theodore Ts'o
parent ac626a3d52
commit 0f67827bf4
1 changed files with 2 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files
fs/jbd2/recovery.c
+2 -7
View File
@@ -490,7 +490,7 @@ static __always_inline int jbd2_do_replay(journal_t *journal,
struct buffer_head *bh,
unsigned long *next_log_block,
unsigned int next_commit_ID,
int *success, int *block_error)
int *success)
{
char *tagp;
int flags;
@@ -542,7 +542,6 @@ static __always_inline int jbd2_do_replay(journal_t *journal,
*success = -EFSBADCRC;
pr_err("JBD2: Invalid checksum recovering data block %llu in journal block %lu\n",
blocknr, io_block);
*block_error = 1;
goto skip_write;
}
@@ -596,7 +595,6 @@ static int do_one_pass(journal_t *journal,
unsigned int sequence;
int blocktype;
__u32 crc32_sum = ~0; /* Transactional Checksums */
int block_error = 0;
bool need_check_commit_time = false;
__u64 last_trans_commit_time = 0, commit_time;
@@ -721,8 +719,7 @@ static int do_one_pass(journal_t *journal,
* done here!
*/
err = jbd2_do_replay(journal, info, bh, &next_log_block,
next_commit_ID, &success,
&block_error);
next_commit_ID, &success);
if (err)
goto failed;
@@ -913,8 +910,6 @@ chksum_ok:
success = err;
}
if (block_error && success == 0)
success = -EIO;
return success;
failed: