| Message ID | 20250102035321.391540-2-sunjunchao2870@gmail.com (mailing list archive) |
|---|---|
| State | New |
| Headers | show |
| Series | [1/2] ocfs2: Add a sanity check for corrupted file system. | expand |
On 2025/1/2 11:53, Julian Sun wrote: > Correct the value of l_next_free_rec to l_count during the online > check, as done in the check_el() function in ocfs2_tools. > > Signed-off-by: Julian Sun <sunjunchao2870@gmail.com> > --- > fs/ocfs2/inode.c | 24 ++++++++++++++++++++++++ > 1 file changed, 24 insertions(+) > > diff --git a/fs/ocfs2/inode.c b/fs/ocfs2/inode.c > index 2cc5c99fe941..1fc9544dc1fa 100644 > --- a/fs/ocfs2/inode.c > +++ b/fs/ocfs2/inode.c > @@ -200,6 +200,20 @@ struct inode *ocfs2_iget(struct ocfs2_super *osb, u64 blkno, unsigned flags, > return inode; > } > > +static int has_extents(struct ocfs2_dinode *di) Better to rename it to 'ocfs2_dinode_has_extents'. > +{ > + /* inodes flagged with other stuff in id2 */ > + if (di->i_flags & (OCFS2_SUPER_BLOCK_FL | OCFS2_LOCAL_ALLOC_FL | > + OCFS2_CHAIN_FL | OCFS2_DEALLOC_FL)) > + return 0; > + /* i_flags doesn't indicate when id2 is a fast symlink */ > + if (S_ISLNK(di->i_mode) && di->i_size && di->i_clusters == 0) > + return 0; > + if (di->i_dyn_features & OCFS2_INLINE_DATA_FL) > + return 0; > + > + return 1; > +} > > /* > * here's how inodes get read from disk: > @@ -1528,6 +1542,16 @@ static int ocfs2_filecheck_repair_inode_block(struct super_block *sb, > return -OCFS2_FILECHECK_ERR_VALIDFLAG; > } > > + if (has_extents(di) && le16_to_cpu(di->id2.i_list.l_next_free_rec) > > + le16_to_cpu(di->id2.i_list.l_count)) { > + di->id2.i_list.l_next_free_rec = di->id2.i_list.l_count; > + changed = 1; > + mlog(ML_ERROR, > + "Filecheck: reset dinode #%llu: l_next_free_rec to %u\n", > + (unsigned long long)bh->b_blocknr, > + le16_to_cpu(di->id2.i_list.l_next_free_rec)); > + } > + I'd like to move this check behind fs_generation since they are commoner check. Thanks, Joseph > if (le64_to_cpu(di->i_blkno) != bh->b_blocknr) { > di->i_blkno = cpu_to_le64(bh->b_blocknr); > changed = 1;
On Thu, 2025-01-02 at 14:54 +0800, Joseph Qi wrote: > > > On 2025/1/2 11:53, Julian Sun wrote: > > Correct the value of l_next_free_rec to l_count during the online > > check, as done in the check_el() function in ocfs2_tools. > > > > Signed-off-by: Julian Sun <sunjunchao2870@gmail.com> > > --- > > fs/ocfs2/inode.c | 24 ++++++++++++++++++++++++ > > 1 file changed, 24 insertions(+) > > > > diff --git a/fs/ocfs2/inode.c b/fs/ocfs2/inode.c > > index 2cc5c99fe941..1fc9544dc1fa 100644 > > --- a/fs/ocfs2/inode.c > > +++ b/fs/ocfs2/inode.c > > @@ -200,6 +200,20 @@ struct inode *ocfs2_iget(struct ocfs2_super *osb, u64 blkno, unsigned flags, > > return inode; > > } > > > > +static int has_extents(struct ocfs2_dinode *di) > > Better to rename it to 'ocfs2_dinode_has_extents'. Reasonable. > > > +{ > > + /* inodes flagged with other stuff in id2 */ > > + if (di->i_flags & (OCFS2_SUPER_BLOCK_FL | OCFS2_LOCAL_ALLOC_FL | > > + OCFS2_CHAIN_FL | OCFS2_DEALLOC_FL)) > > + return 0; > > + /* i_flags doesn't indicate when id2 is a fast symlink */ > > + if (S_ISLNK(di->i_mode) && di->i_size && di->i_clusters == 0) > > + return 0; > > + if (di->i_dyn_features & OCFS2_INLINE_DATA_FL) > > + return 0; > > + > > + return 1; > > +} > > > > /* > > * here's how inodes get read from disk: > > @@ -1528,6 +1542,16 @@ static int ocfs2_filecheck_repair_inode_block(struct super_block *sb, > > return -OCFS2_FILECHECK_ERR_VALIDFLAG; > > } > > > > + if (has_extents(di) && le16_to_cpu(di->id2.i_list.l_next_free_rec) > > > + le16_to_cpu(di->id2.i_list.l_count)) { > > + di->id2.i_list.l_next_free_rec = di->id2.i_list.l_count; > > + changed = 1; > > + mlog(ML_ERROR, > > + "Filecheck: reset dinode #%llu: l_next_free_rec to %u\n", > > + (unsigned long long)bh->b_blocknr, > > + le16_to_cpu(di->id2.i_list.l_next_free_rec)); > > + } > > + > > I'd like to move this check behind fs_generation since they are commoner check. Sure, will fix it in next version. > > Thanks, > Joseph > > > if (le64_to_cpu(di->i_blkno) != bh->b_blocknr) { > > di->i_blkno = cpu_to_le64(bh->b_blocknr); > > changed = 1; >
diff --git a/fs/ocfs2/inode.c b/fs/ocfs2/inode.c index 2cc5c99fe941..1fc9544dc1fa 100644 --- a/fs/ocfs2/inode.c +++ b/fs/ocfs2/inode.c @@ -200,6 +200,20 @@ struct inode *ocfs2_iget(struct ocfs2_super *osb, u64 blkno, unsigned flags, return inode; } +static int has_extents(struct ocfs2_dinode *di) +{ + /* inodes flagged with other stuff in id2 */ + if (di->i_flags & (OCFS2_SUPER_BLOCK_FL | OCFS2_LOCAL_ALLOC_FL | + OCFS2_CHAIN_FL | OCFS2_DEALLOC_FL)) + return 0; + /* i_flags doesn't indicate when id2 is a fast symlink */ + if (S_ISLNK(di->i_mode) && di->i_size && di->i_clusters == 0) + return 0; + if (di->i_dyn_features & OCFS2_INLINE_DATA_FL) + return 0; + + return 1; +} /* * here's how inodes get read from disk: @@ -1528,6 +1542,16 @@ static int ocfs2_filecheck_repair_inode_block(struct super_block *sb, return -OCFS2_FILECHECK_ERR_VALIDFLAG; } + if (has_extents(di) && le16_to_cpu(di->id2.i_list.l_next_free_rec) > + le16_to_cpu(di->id2.i_list.l_count)) { + di->id2.i_list.l_next_free_rec = di->id2.i_list.l_count; + changed = 1; + mlog(ML_ERROR, + "Filecheck: reset dinode #%llu: l_next_free_rec to %u\n", + (unsigned long long)bh->b_blocknr, + le16_to_cpu(di->id2.i_list.l_next_free_rec)); + } + if (le64_to_cpu(di->i_blkno) != bh->b_blocknr) { di->i_blkno = cpu_to_le64(bh->b_blocknr); changed = 1;
Correct the value of l_next_free_rec to l_count during the online check, as done in the check_el() function in ocfs2_tools. Signed-off-by: Julian Sun <sunjunchao2870@gmail.com> --- fs/ocfs2/inode.c | 24 ++++++++++++++++++++++++ 1 file changed, 24 insertions(+)