diff options
author | Jaegeuk Kim <jaegeuk@kernel.org> | 2015-11-22 10:53:07 +0800 |
---|---|---|
committer | Greg Wallace <greg@gregtwallace.com> | 2016-01-19 22:02:18 -0500 |
commit | 04d6106998b934a87ca9c51fffd233e2e9bb968d (patch) | |
tree | b3c58a4ee9fddcfb2c5a4259dc5ef95e650eed49 | |
parent | f9c29e73210212e7c041934ca0f97d80764d0554 (diff) | |
download | android_external_f2fs-tools-04d6106998b934a87ca9c51fffd233e2e9bb968d.tar.gz android_external_f2fs-tools-04d6106998b934a87ca9c51fffd233e2e9bb968d.tar.bz2 android_external_f2fs-tools-04d6106998b934a87ca9c51fffd233e2e9bb968d.zip |
fsck.f2fs: support a readonly filesystem
If f2fs is mounted as ro, we can do fsck.f2fs.
Change-Id: If20a055242824fd94110bcde12988ddc019a4011
Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>
-rw-r--r-- | fsck/fsck.c | 15 | ||||
-rw-r--r-- | fsck/main.c | 15 | ||||
-rw-r--r-- | include/f2fs_fs.h | 1 | ||||
-rw-r--r-- | lib/libf2fs.c | 17 | ||||
-rw-r--r-- | mkfs/f2fs_format_main.c | 4 |
5 files changed, 35 insertions, 17 deletions
diff --git a/fsck/fsck.c b/fsck/fsck.c index d3b5dd4..510c39d 100644 --- a/fsck/fsck.c +++ b/fsck/fsck.c @@ -174,7 +174,7 @@ static int is_valid_ssa_node_blk(struct f2fs_sb_info *sbi, u32 nid, need_fix = 1; } } - if (need_fix) { + if (need_fix && !config.ro) { u64 ssa_blk; int ret2; @@ -289,7 +289,7 @@ static int is_valid_ssa_data_blk(struct f2fs_sb_info *sbi, u32 blk_addr, need_fix = 1; } } - if (need_fix) { + if (need_fix && !config.ro) { u64 ssa_blk; int ret2; @@ -719,7 +719,7 @@ skip_blkcnt_fix: nid, i_links); } } - if (need_fix) { + if (need_fix && !config.ro) { /* drop extent information to avoid potential wrong access */ node_blk->i.i_ext.len = 0; ret = dev_write_block(node_blk, ni->blk_addr); @@ -750,7 +750,7 @@ int fsck_chk_dnode_blk(struct f2fs_sb_info *sbi, struct f2fs_inode *inode, FIX_MSG("[0x%x] dn.addr[%d] = 0", nid, idx); } } - if (need_fix) { + if (need_fix && !config.ro) { ret = dev_write_block(node_blk, ni->blk_addr); ASSERT(ret >= 0); } @@ -1069,7 +1069,7 @@ int fsck_chk_dentry_blk(struct f2fs_sb_info *sbi, u32 blk_addr, de_blk->dentry, de_blk->filename, NR_DENTRY_IN_BLOCK, last_blk, encrypted); - if (dentries < 0) { + if (dentries < 0 && !config.ro) { ret = dev_write_block(de_blk, blk_addr); ASSERT(ret >= 0); DBG(1, "[%3d] Dentry Block [0x%x] Fixed hash_codes\n\n", @@ -1172,7 +1172,8 @@ void fsck_chk_orphan_node(struct f2fs_sb_info *sbi) else if (ret) ASSERT_MSG("[0x%x] wrong orphan inode", ino); } - if (config.fix_on && entry_count != new_entry_count) { + if (!config.ro && config.fix_on && + entry_count != new_entry_count) { new_blk->entry_count = cpu_to_le32(new_entry_count); ret = dev_write_block(new_blk, start_blk + i); ASSERT(ret >= 0); @@ -1475,7 +1476,7 @@ int fsck_verify(struct f2fs_sb_info *sbi) } /* fix global metadata */ - if (force || (config.bug_on && config.fix_on)) { + if (force || (config.bug_on && config.fix_on && !config.ro)) { fix_hard_links(sbi); fix_nat_entries(sbi); rewrite_sit_area_bitmap(sbi); diff --git a/fsck/main.c b/fsck/main.c index 844ee8a..3db4a44 100644 --- a/fsck/main.c +++ b/fsck/main.c @@ -197,8 +197,17 @@ int main(int argc, char **argv) f2fs_parse_options(argc, argv); - if (f2fs_dev_is_umounted(&config) < 0) - return -1; + if (f2fs_dev_is_umounted(&config) < 0) { + if (!config.ro) { + MSG(0, "\tError: Not available on mounted device!\n"); + return -1; + } + + /* allow ro-mounted partition */ + MSG(0, "Info: Check FS only due to RO\n"); + config.fix_on = 0; + config.auto_fix = 0; + } /* Get device */ if (f2fs_get_device_info(&config) < 0) @@ -228,7 +237,7 @@ fsck_again: f2fs_do_umount(sbi); out: if (config.func == FSCK && config.bug_on) { - if (config.fix_on == 0 && config.auto_fix == 0) { + if (!config.ro && config.fix_on == 0 && config.auto_fix == 0) { char ans[255] = {0}; retry: printf("Do you want to fix this partition? [Y/N] "); diff --git a/include/f2fs_fs.h b/include/f2fs_fs.h index 4164af3..0efcb0b 100644 --- a/include/f2fs_fs.h +++ b/include/f2fs_fs.h @@ -275,6 +275,7 @@ struct f2fs_configuration { int fix_on; int bug_on; int auto_fix; + int ro; __le32 feature; /* defined features */ } __attribute__((packed)); diff --git a/lib/libf2fs.c b/lib/libf2fs.c index c42620b..f0a8861 100644 --- a/lib/libf2fs.c +++ b/lib/libf2fs.c @@ -365,9 +365,11 @@ void f2fs_init_configuration(struct f2fs_configuration *c) c->device_name = NULL; c->trim = 1; c->bytes_reserved = 0; + c->ro = 0; } -static int is_mounted(const char *mpt, const char *device) +static int is_mounted(struct f2fs_configuration *c, + const char *mpt, const char *device) { #ifdef __linux__ FILE *file = NULL; @@ -378,8 +380,11 @@ static int is_mounted(const char *mpt, const char *device) return 0; while ((mnt = getmntent(file)) != NULL) { - if (!strcmp(device, mnt->mnt_fsname)) + if (!strcmp(device, mnt->mnt_fsname)) { + if (hasmntopt(mnt, MNTOPT_RO)) + config.ro = 1; break; + } } endmntent(file); return mnt ? 1 : 0; @@ -395,9 +400,9 @@ int f2fs_dev_is_umounted(struct f2fs_configuration *c) int ret = 0; #ifdef __linux__ - ret = is_mounted(MOUNTED, c->device_name); + ret = is_mounted(c, MOUNTED, c->device_name); if (ret) { - MSG(0, "\tError: Not available on mounted device!\n"); + MSG(0, "Info: Mounted device!\n"); return -1; } #endif @@ -406,9 +411,9 @@ int f2fs_dev_is_umounted(struct f2fs_configuration *c) * if failed due to /etc/mtab file not present * try with /proc/mounts. */ - ret = is_mounted("/proc/mounts", c->device_name); + ret = is_mounted(c, "/proc/mounts", c->device_name); if (ret) { - MSG(0, "\tError: Not available on mounted device!\n"); + MSG(0, "Info: Mounted device!\n"); return -1; } diff --git a/mkfs/f2fs_format_main.c b/mkfs/f2fs_format_main.c index 5d10639..c1c0f8d 100644 --- a/mkfs/f2fs_format_main.c +++ b/mkfs/f2fs_format_main.c @@ -149,8 +149,10 @@ int main(int argc, char *argv[]) f2fs_show_info(); - if (f2fs_dev_is_umounted(&config) < 0) + if (f2fs_dev_is_umounted(&config) < 0) { + MSG(0, "\tError: Not available on mounted device!\n"); return -1; + } if (f2fs_get_device_info(&config) < 0) return -1; |