diff options
author | Daniel Rosenberg <drosen@google.com> | 2014-08-19 00:13:10 -0700 |
---|---|---|
committer | Daniel Rosenberg <drosen@google.com> | 2014-08-19 00:16:46 -0700 |
commit | b63632828014d3bdac0d689591532fe78bc81bc7 (patch) | |
tree | 345f6605d05f5a331defb41f3506bc02c6ec07ec /f2fs_utils | |
parent | aff4a27b96454d6868a7a22974f560147a13188f (diff) | |
download | extras-b63632828014d3bdac0d689591532fe78bc81bc7.tar.gz |
f2fs_sparseblock: Fixed handling of journaled sit entries
Bug: 15749466
Change-Id: I3f7f13698e88efd7ffeb00c1ba3a070527a3f6a0
Signed-off-by: Daniel Rosenberg <drosen@google.com>
Diffstat (limited to 'f2fs_utils')
-rw-r--r-- | f2fs_utils/f2fs_sparseblock.c | 29 |
1 files changed, 15 insertions, 14 deletions
diff --git a/f2fs_utils/f2fs_sparseblock.c b/f2fs_utils/f2fs_sparseblock.c index 46d1f72a..2bcd4476 100644 --- a/f2fs_utils/f2fs_sparseblock.c +++ b/f2fs_utils/f2fs_sparseblock.c @@ -23,6 +23,10 @@ #member, le64_to_cpu((ptr)->member), le64_to_cpu((ptr)->member) ); \ } while (0); +#define segno_in_journal(sum, i) (sum->sit_j.entries[i].segno) + +#define sit_in_journal(sum, i) (sum->sit_j.entries[i].se) + static void dbg_print_raw_sb_info(struct f2fs_super_block *sb) { SLOGD("\n"); @@ -130,6 +134,12 @@ static void dbg_print_info_struct(struct f2fs_info *info) SLOGD("blocks_per_sit: %"PRIu64, info->blocks_per_sit); SLOGD("sit_blocks loc: %p", info->sit_blocks); SLOGD("sit_sums loc: %p", info->sit_sums); + SLOGD("sit_sums num: %d", le16_to_cpu(info->sit_sums->n_sits)); + unsigned int i; + for(i = 0; i < (le16_to_cpu(info->sit_sums->n_sits)); i++) { + SLOGD("entry %d in journal entries is for segment %d",i, le32_to_cpu(segno_in_journal(info->sit_sums, i))); + } + SLOGD("cp_blkaddr: %"PRIu64, info->cp_blkaddr); SLOGD("cp_valid_cp_blkaddr: %"PRIu64, info->cp_valid_cp_blkaddr); SLOGD("sit_blkaddr: %"PRIu64, info->sit_blkaddr); @@ -358,7 +368,7 @@ static int get_sit_summary(int fd, struct f2fs_info *info, struct f2fs_checkpoin if (read_structure_blk(fd, blk_addr, buffer, 1)) return -1; - memcpy(info->sit_sums, buffer, sizeof(struct f2fs_summary_block)); + memcpy(info->sit_sums, buffer, sizeof(struct f2fs_summary_block)); } return 0; } @@ -469,14 +479,10 @@ int f2fs_test_bit(unsigned int nr, const char *p) return (mask & *addr) != 0; } -#define segno_in_journal(sum, i) (sum->sit_j.entries[i].segno) - -#define sit_in_journal(sum, i) (sum->sit_j.entries[i].se) - int run_on_used_blocks(u64 startblock, struct f2fs_info *info, int (*func)(u64 pos, void *data), void *data) { struct f2fs_sit_block sit_block_cache; - struct f2fs_sit_entry * sit_entry, *sit_entry1, *sit_entry2; - u64 sit_block_num_cur = 0, segnum=0, block_offset; + struct f2fs_sit_entry * sit_entry; + u64 sit_block_num_cur = 0, segnum = 0, block_offset; u64 block; unsigned int used, found, started = 0, i; @@ -492,8 +498,8 @@ int run_on_used_blocks(u64 startblock, struct f2fs_info *info, int (*func)(u64 p segnum = (block - info->main_blkaddr)/info->blocks_per_segment; /* check the SIT entries in the journal */ - found=0; - for(i = 0; i<SIT_JOURNAL_ENTRIES; i++) { + found = 0; + for(i = 0; i < le16_to_cpu(info->sit_sums->n_sits); i++) { if (le32_to_cpu(segno_in_journal(info->sit_sums, i)) == segnum) { sit_entry = &sit_in_journal(info->sit_sums, i); found = 1; @@ -509,11 +515,6 @@ int run_on_used_blocks(u64 startblock, struct f2fs_info *info, int (*func)(u64 p block_offset = (block - info->main_blkaddr) % info->blocks_per_segment; - if (block_offset == 0 && GET_SIT_VBLOCKS(sit_entry) == 0) { - block += info->blocks_per_segment; - continue; - } - used = f2fs_test_bit(block_offset, (char *)sit_entry->valid_map); if(used) if (func(block, data)) |