Commit 2b39e907 authored by Shawn Lin's avatar Shawn Lin Committed by Jaegeuk Kim

f2fs: slightly reorganize read_raw_super_block

read_raw_super_block was introduced to help find the
first valid superblock. Commit da554e48 ("f2fs:
recovering broken superblock during mount") changed the
behaviour to read both of them and check whether need
the recovery flag or not. So the comment before this
function isn't consistent with what it actually does.
Also, the origin code use two tags to round the err
cases, which isn't so readable. So this patch amend
the comment and slightly reorganize it.
Signed-off-by: default avatarShawn Lin <shawn.lin@rock-chips.com>
Signed-off-by: default avatarJaegeuk Kim <jaegeuk@kernel.org>
parent 1515aef0
...@@ -1166,14 +1166,15 @@ static void init_sb_info(struct f2fs_sb_info *sbi) ...@@ -1166,14 +1166,15 @@ static void init_sb_info(struct f2fs_sb_info *sbi)
/* /*
* Read f2fs raw super block. * Read f2fs raw super block.
* Because we have two copies of super block, so read the first one at first, * Because we have two copies of super block, so read both of them
* if the first one is invalid, move to read the second one. * to get the first valid one. If any one of them is broken, we pass
* them recovery flag back to the caller.
*/ */
static int read_raw_super_block(struct super_block *sb, static int read_raw_super_block(struct super_block *sb,
struct f2fs_super_block **raw_super, struct f2fs_super_block **raw_super,
int *valid_super_block, int *recovery) int *valid_super_block, int *recovery)
{ {
int block = 0; int block;
struct buffer_head *bh; struct buffer_head *bh;
struct f2fs_super_block *super, *buf; struct f2fs_super_block *super, *buf;
int err = 0; int err = 0;
...@@ -1181,50 +1182,48 @@ static int read_raw_super_block(struct super_block *sb, ...@@ -1181,50 +1182,48 @@ static int read_raw_super_block(struct super_block *sb,
super = kzalloc(sizeof(struct f2fs_super_block), GFP_KERNEL); super = kzalloc(sizeof(struct f2fs_super_block), GFP_KERNEL);
if (!super) if (!super)
return -ENOMEM; return -ENOMEM;
retry:
bh = sb_bread(sb, block); for (block = 0; block < 2; block++) {
if (!bh) { bh = sb_bread(sb, block);
*recovery = 1; if (!bh) {
f2fs_msg(sb, KERN_ERR, "Unable to read %dth superblock", f2fs_msg(sb, KERN_ERR, "Unable to read %dth superblock",
block + 1); block + 1);
err = -EIO; err = -EIO;
goto next; continue;
} }
buf = (struct f2fs_super_block *)(bh->b_data + F2FS_SUPER_OFFSET); buf = (struct f2fs_super_block *)
(bh->b_data + F2FS_SUPER_OFFSET);
/* sanity checking of raw super */ /* sanity checking of raw super */
if (sanity_check_raw_super(sb, buf)) { if (sanity_check_raw_super(sb, buf)) {
brelse(bh); f2fs_msg(sb, KERN_ERR,
*recovery = 1; "Can't find valid F2FS filesystem in %dth superblock",
f2fs_msg(sb, KERN_ERR, block + 1);
"Can't find valid F2FS filesystem in %dth superblock", err = -EINVAL;
block + 1); brelse(bh);
err = -EINVAL; continue;
goto next; }
}
if (!*raw_super) { if (!*raw_super) {
memcpy(super, buf, sizeof(*super)); memcpy(super, buf, sizeof(*super));
*valid_super_block = block; *valid_super_block = block;
*raw_super = super; *raw_super = super;
}
brelse(bh);
} }
brelse(bh);
next: /* Fail to read any one of the superblocks*/
/* check the validity of the second superblock */ if (err < 0)
if (block == 0) { *recovery = 1;
block++;
goto retry;
}
/* No valid superblock */ /* No valid superblock */
if (!*raw_super) { if (!*raw_super)
kfree(super); kfree(super);
return err; else
} err = 0;
return 0; return err;
} }
static int __f2fs_commit_super(struct f2fs_sb_info *sbi, int block) static int __f2fs_commit_super(struct f2fs_sb_info *sbi, int block)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment