[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date: Mon, 13 Oct 2014 10:28:03 -0700
From: "Darrick J. Wong" <darrick.wong@...cle.com>
To: Dmitry Monakhov <dmonakhov@...nvz.org>
Cc: linux-ext4@...r.kernel.org
Subject: Re: [PATCH 1/2] ext4: Replace open coded mdata csum feature to
helper function v2
On Mon, Oct 13, 2014 at 09:02:36PM +0400, Dmitry Monakhov wrote:
> "Darrick J. Wong" <darrick.wong@...cle.com> writes:
>
> > On Mon, Oct 13, 2014 at 01:06:06AM +0400, Dmitry Monakhov wrote:
> >> Besides the fact that this replacement improves code readability
> >> it also protects from errors caused direct EXT4_S(sb)->s_es manipulation
> >> which may result attempt to use uninitialized csum machinery.
> >>
> >> #Testcase_BEGIN
> >> IMG=/dev/ram0
> >> MNT=/mnt
> >> mkfs.ext4 $IMG
> >> mount $IMG $MNT
> >> #Enable feature directly on disk, on mounted fs
> >> tune2fs -O metadata_csum $IMG
> >> # Provoke metadata update, likey result in OOPS
> >> touch $MNT/test
> >> umount $MNT
> >> #Testcase_END
> >>
> >> # Replacement script
> >> @@
> >> expression E;
> >> @@
> >> - EXT4_HAS_RO_COMPAT_FEATURE(E, EXT4_FEATURE_RO_COMPAT_METADATA_CSUM)
> >> + ext4_has_metadata_csum(E)
> >>
> >> Changes v1->v2
> >> - fix early ext_fill_super stage
> >>
> >> https://bugzilla.kernel.org/show_bug.cgi?id=82201
> >>
> >> Signed-off-by: Dmitry Monakhov <dmonakhov@...nvz.org>
> >> ---
> >> fs/ext4/bitmap.c | 12 ++++--------
> >> fs/ext4/ext4.h | 8 ++++++++
> >> fs/ext4/extents.c | 6 ++----
> >> fs/ext4/ialloc.c | 3 +--
> >> fs/ext4/inline.c | 3 +--
> >> fs/ext4/inode.c | 9 +++------
> >> fs/ext4/ioctl.c | 3 +--
> >> fs/ext4/mmp.c | 6 ++----
> >> fs/ext4/namei.c | 39 +++++++++++++--------------------------
> >> fs/ext4/resize.c | 3 +--
> >> fs/ext4/super.c | 15 +++++----------
> >> fs/ext4/xattr.c | 6 ++----
> >> 12 files changed, 43 insertions(+), 70 deletions(-)
> >>
> >> diff --git a/fs/ext4/bitmap.c b/fs/ext4/bitmap.c
> >> index 3285aa5..b610779 100644
> >> --- a/fs/ext4/bitmap.c
> >> +++ b/fs/ext4/bitmap.c
> >> @@ -24,8 +24,7 @@ int ext4_inode_bitmap_csum_verify(struct super_block *sb, ext4_group_t group,
> >> __u32 provided, calculated;
> >> struct ext4_sb_info *sbi = EXT4_SB(sb);
> >>
> >> - if (!EXT4_HAS_RO_COMPAT_FEATURE(sb,
> >> - EXT4_FEATURE_RO_COMPAT_METADATA_CSUM))
> >> + if (!ext4_has_metadata_csum(sb))
> >> return 1;
> >>
> >> provided = le16_to_cpu(gdp->bg_inode_bitmap_csum_lo);
> >> @@ -46,8 +45,7 @@ void ext4_inode_bitmap_csum_set(struct super_block *sb, ext4_group_t group,
> >> __u32 csum;
> >> struct ext4_sb_info *sbi = EXT4_SB(sb);
> >>
> >> - if (!EXT4_HAS_RO_COMPAT_FEATURE(sb,
> >> - EXT4_FEATURE_RO_COMPAT_METADATA_CSUM))
> >> + if (!ext4_has_metadata_csum(sb))
> >> return;
> >>
> >> csum = ext4_chksum(sbi, sbi->s_csum_seed, (__u8 *)bh->b_data, sz);
> >> @@ -65,8 +63,7 @@ int ext4_block_bitmap_csum_verify(struct super_block *sb, ext4_group_t group,
> >> struct ext4_sb_info *sbi = EXT4_SB(sb);
> >> int sz = EXT4_CLUSTERS_PER_GROUP(sb) / 8;
> >>
> >> - if (!EXT4_HAS_RO_COMPAT_FEATURE(sb,
> >> - EXT4_FEATURE_RO_COMPAT_METADATA_CSUM))
> >> + if (!ext4_has_metadata_csum(sb))
> >> return 1;
> >>
> >> provided = le16_to_cpu(gdp->bg_block_bitmap_csum_lo);
> >> @@ -91,8 +88,7 @@ void ext4_block_bitmap_csum_set(struct super_block *sb, ext4_group_t group,
> >> __u32 csum;
> >> struct ext4_sb_info *sbi = EXT4_SB(sb);
> >>
> >> - if (!EXT4_HAS_RO_COMPAT_FEATURE(sb,
> >> - EXT4_FEATURE_RO_COMPAT_METADATA_CSUM))
> >> + if (!ext4_has_metadata_csum(sb))
> >> return;
> >>
> >> csum = ext4_chksum(sbi, sbi->s_csum_seed, (__u8 *)bh->b_data, sz);
> >> diff --git a/fs/ext4/ext4.h b/fs/ext4/ext4.h
> >> index 012e89b..1483d9c 100644
> >> --- a/fs/ext4/ext4.h
> >> +++ b/fs/ext4/ext4.h
> >> @@ -2337,6 +2337,14 @@ static inline int ext4_has_group_desc_csum(struct super_block *sb)
> >> EXT4_FEATURE_RO_COMPAT_METADATA_CSUM);
> >
> > Please change the raw metadata_csum flag check in ext4_has_group_desc_csum(),
> > otherwise we'll still suffer from this bug.
> No, this one is safe because ext4_group_desc_csum() will check
> ext4_has_metadata_csum() internally. Other stuff should be safe.
> So you'll see csum corruption messages which is expected and correct.
Hang on a second...
Let's say that we mount a FS with ^uninit_bg,^metadata_csum (i.e. no
checksumming at all). s_chksum_driver == NULL.
Then, the metadata_csum flag somehow gets turned on by accident. This is the
crash scenario that this patch is trying to prevent.
Take a look at ext4_group_desc_csum_verify/set():
int ext4_group_desc_csum_verify(struct super_block *sb, __u32 block_group,
struct ext4_group_desc *gdp)
{
if (ext4_has_group_desc_csum(sb) &&
(gdp->bg_checksum != ext4_group_desc_csum(EXT4_SB(sb),
block_group, gdp)))
return 0;
return 1;
}
void ext4_group_desc_csum_set(struct super_block *sb, __u32 block_group,
struct ext4_group_desc *gdp)
{
if (!ext4_has_group_desc_csum(sb))
return;
gdp->bg_checksum = ext4_group_desc_csum(EXT4_SB(sb), block_group, gdp);
}
Now, the metadata_csum feature flag is set, so the if test fails and we call
ext4_group_desc_csum(). That function notices that s_chksum_driver == NULL and
returns the crc16 of the group descriptor, causing _verify() to return 0
(failure) and _set() to stuff an incorrect value into bg_checksum.
Oh well, Ted already took your patch; I guess I'll just send him the fix myself.
> >
> >> }
> >>
> >> +static inline int ext4_has_metadata_csum(struct super_block *sb)
> >> +{
> >> + WARN_ON_ONCE(EXT4_HAS_RO_COMPAT_FEATURE(sb,
> >> + EXT4_FEATURE_RO_COMPAT_METADATA_CSUM) &&
> >> + !EXT4_SB(sb)->s_chksum_driver);
Now that I think more about this -- does this warrant an ext4_error()? If we
hit this point there's something seriously wrong.
--D
> >
> > I was about to protest about not checking the flag and s_chksum_driver...
> >
> > Then I realized that tune2fs can set and clear the metadata_csum flag on a
> > mounted FS. Oh crap.
> >
> > So maybe it would be a good idea to check both the driver pointer and the
> > feature flag just to make sure the user really wants checksumming, though in
> > the meantime I have a e2fsprogs bug to fix.
> >
> > --D
> >
> >> +
> >> + return (EXT4_SB(sb)->s_chksum_driver != NULL);
> >> +}
> >> static inline ext4_fsblk_t ext4_blocks_count(struct ext4_super_block *es)
> >> {
> >> return ((ext4_fsblk_t)le32_to_cpu(es->s_blocks_count_hi) << 32) |
> >> diff --git a/fs/ext4/extents.c b/fs/ext4/extents.c
> >> index c3ed9af..37043d0 100644
> >> --- a/fs/ext4/extents.c
> >> +++ b/fs/ext4/extents.c
> >> @@ -73,8 +73,7 @@ static int ext4_extent_block_csum_verify(struct inode *inode,
> >> {
> >> struct ext4_extent_tail *et;
> >>
> >> - if (!EXT4_HAS_RO_COMPAT_FEATURE(inode->i_sb,
> >> - EXT4_FEATURE_RO_COMPAT_METADATA_CSUM))
> >> + if (!ext4_has_metadata_csum(inode->i_sb))
> >> return 1;
> >>
> >> et = find_ext4_extent_tail(eh);
> >> @@ -88,8 +87,7 @@ static void ext4_extent_block_csum_set(struct inode *inode,
> >> {
> >> struct ext4_extent_tail *et;
> >>
> >> - if (!EXT4_HAS_RO_COMPAT_FEATURE(inode->i_sb,
> >> - EXT4_FEATURE_RO_COMPAT_METADATA_CSUM))
> >> + if (!ext4_has_metadata_csum(inode->i_sb))
> >> return;
> >>
> >> et = find_ext4_extent_tail(eh);
> >> diff --git a/fs/ext4/ialloc.c b/fs/ext4/ialloc.c
> >> index 5b87fc3..8012a5d 100644
> >> --- a/fs/ext4/ialloc.c
> >> +++ b/fs/ext4/ialloc.c
> >> @@ -1011,8 +1011,7 @@ got:
> >> spin_unlock(&sbi->s_next_gen_lock);
> >>
> >> /* Precompute checksum seed for inode metadata */
> >> - if (EXT4_HAS_RO_COMPAT_FEATURE(sb,
> >> - EXT4_FEATURE_RO_COMPAT_METADATA_CSUM)) {
> >> + if (ext4_has_metadata_csum(sb)) {
> >> __u32 csum;
> >> __le32 inum = cpu_to_le32(inode->i_ino);
> >> __le32 gen = cpu_to_le32(inode->i_generation);
> >> diff --git a/fs/ext4/inline.c b/fs/ext4/inline.c
> >> index 378aadf..3ea6269 100644
> >> --- a/fs/ext4/inline.c
> >> +++ b/fs/ext4/inline.c
> >> @@ -1128,8 +1128,7 @@ static int ext4_finish_convert_inline_dir(handle_t *handle,
> >> memcpy((void *)de, buf + EXT4_INLINE_DOTDOT_SIZE,
> >> inline_size - EXT4_INLINE_DOTDOT_SIZE);
> >>
> >> - if (EXT4_HAS_RO_COMPAT_FEATURE(inode->i_sb,
> >> - EXT4_FEATURE_RO_COMPAT_METADATA_CSUM))
> >> + if (ext4_has_metadata_csum(inode->i_sb))
> >> csum_size = sizeof(struct ext4_dir_entry_tail);
> >>
> >> inode->i_size = inode->i_sb->s_blocksize;
> >> diff --git a/fs/ext4/inode.c b/fs/ext4/inode.c
> >> index 0dd9150..e9777f9 100644
> >> --- a/fs/ext4/inode.c
> >> +++ b/fs/ext4/inode.c
> >> @@ -83,8 +83,7 @@ static int ext4_inode_csum_verify(struct inode *inode, struct ext4_inode *raw,
> >>
> >> if (EXT4_SB(inode->i_sb)->s_es->s_creator_os !=
> >> cpu_to_le32(EXT4_OS_LINUX) ||
> >> - !EXT4_HAS_RO_COMPAT_FEATURE(inode->i_sb,
> >> - EXT4_FEATURE_RO_COMPAT_METADATA_CSUM))
> >> + !ext4_has_metadata_csum(inode->i_sb))
> >> return 1;
> >>
> >> provided = le16_to_cpu(raw->i_checksum_lo);
> >> @@ -105,8 +104,7 @@ static void ext4_inode_csum_set(struct inode *inode, struct ext4_inode *raw,
> >>
> >> if (EXT4_SB(inode->i_sb)->s_es->s_creator_os !=
> >> cpu_to_le32(EXT4_OS_LINUX) ||
> >> - !EXT4_HAS_RO_COMPAT_FEATURE(inode->i_sb,
> >> - EXT4_FEATURE_RO_COMPAT_METADATA_CSUM))
> >> + !ext4_has_metadata_csum(inode->i_sb))
> >> return;
> >>
> >> csum = ext4_inode_csum(inode, raw, ei);
> >> @@ -3928,8 +3926,7 @@ struct inode *ext4_iget(struct super_block *sb, unsigned long ino)
> >> ei->i_extra_isize = 0;
> >>
> >> /* Precompute checksum seed for inode metadata */
> >> - if (EXT4_HAS_RO_COMPAT_FEATURE(sb,
> >> - EXT4_FEATURE_RO_COMPAT_METADATA_CSUM)) {
> >> + if (ext4_has_metadata_csum(sb)) {
> >> struct ext4_sb_info *sbi = EXT4_SB(inode->i_sb);
> >> __u32 csum;
> >> __le32 inum = cpu_to_le32(inode->i_ino);
> >> diff --git a/fs/ext4/ioctl.c b/fs/ext4/ioctl.c
> >> index 3d5de16..bfda18a 100644
> >> --- a/fs/ext4/ioctl.c
> >> +++ b/fs/ext4/ioctl.c
> >> @@ -331,8 +331,7 @@ flags_out:
> >> if (!inode_owner_or_capable(inode))
> >> return -EPERM;
> >>
> >> - if (EXT4_HAS_RO_COMPAT_FEATURE(inode->i_sb,
> >> - EXT4_FEATURE_RO_COMPAT_METADATA_CSUM)) {
> >> + if (ext4_has_metadata_csum(inode->i_sb)) {
> >> ext4_warning(sb, "Setting inode version is not "
> >> "supported with metadata_csum enabled.");
> >> return -ENOTTY;
> >> diff --git a/fs/ext4/mmp.c b/fs/ext4/mmp.c
> >> index 32bce84..8313ca3 100644
> >> --- a/fs/ext4/mmp.c
> >> +++ b/fs/ext4/mmp.c
> >> @@ -20,8 +20,7 @@ static __le32 ext4_mmp_csum(struct super_block *sb, struct mmp_struct *mmp)
> >>
> >> static int ext4_mmp_csum_verify(struct super_block *sb, struct mmp_struct *mmp)
> >> {
> >> - if (!EXT4_HAS_RO_COMPAT_FEATURE(sb,
> >> - EXT4_FEATURE_RO_COMPAT_METADATA_CSUM))
> >> + if (!ext4_has_metadata_csum(sb))
> >> return 1;
> >>
> >> return mmp->mmp_checksum == ext4_mmp_csum(sb, mmp);
> >> @@ -29,8 +28,7 @@ static int ext4_mmp_csum_verify(struct super_block *sb, struct mmp_struct *mmp)
> >>
> >> static void ext4_mmp_csum_set(struct super_block *sb, struct mmp_struct *mmp)
> >> {
> >> - if (!EXT4_HAS_RO_COMPAT_FEATURE(sb,
> >> - EXT4_FEATURE_RO_COMPAT_METADATA_CSUM))
> >> + if (!ext4_has_metadata_csum(sb))
> >> return;
> >>
> >> mmp->mmp_checksum = ext4_mmp_csum(sb, mmp);
> >> diff --git a/fs/ext4/namei.c b/fs/ext4/namei.c
> >> index 7037ecf..61756f9 100644
> >> --- a/fs/ext4/namei.c
> >> +++ b/fs/ext4/namei.c
> >> @@ -124,8 +124,7 @@ static struct buffer_head *__ext4_read_dirblock(struct inode *inode,
> >> "directory leaf block found instead of index block");
> >> return ERR_PTR(-EIO);
> >> }
> >> - if (!EXT4_HAS_RO_COMPAT_FEATURE(inode->i_sb,
> >> - EXT4_FEATURE_RO_COMPAT_METADATA_CSUM) ||
> >> + if (!ext4_has_metadata_csum(inode->i_sb) ||
> >> buffer_verified(bh))
> >> return bh;
> >>
> >> @@ -338,8 +337,7 @@ int ext4_dirent_csum_verify(struct inode *inode, struct ext4_dir_entry *dirent)
> >> {
> >> struct ext4_dir_entry_tail *t;
> >>
> >> - if (!EXT4_HAS_RO_COMPAT_FEATURE(inode->i_sb,
> >> - EXT4_FEATURE_RO_COMPAT_METADATA_CSUM))
> >> + if (!ext4_has_metadata_csum(inode->i_sb))
> >> return 1;
> >>
> >> t = get_dirent_tail(inode, dirent);
> >> @@ -360,8 +358,7 @@ static void ext4_dirent_csum_set(struct inode *inode,
> >> {
> >> struct ext4_dir_entry_tail *t;
> >>
> >> - if (!EXT4_HAS_RO_COMPAT_FEATURE(inode->i_sb,
> >> - EXT4_FEATURE_RO_COMPAT_METADATA_CSUM))
> >> + if (!ext4_has_metadata_csum(inode->i_sb))
> >> return;
> >>
> >> t = get_dirent_tail(inode, dirent);
> >> @@ -436,8 +433,7 @@ static int ext4_dx_csum_verify(struct inode *inode,
> >> struct dx_tail *t;
> >> int count_offset, limit, count;
> >>
> >> - if (!EXT4_HAS_RO_COMPAT_FEATURE(inode->i_sb,
> >> - EXT4_FEATURE_RO_COMPAT_METADATA_CSUM))
> >> + if (!ext4_has_metadata_csum(inode->i_sb))
> >> return 1;
> >>
> >> c = get_dx_countlimit(inode, dirent, &count_offset);
> >> @@ -466,8 +462,7 @@ static void ext4_dx_csum_set(struct inode *inode, struct ext4_dir_entry *dirent)
> >> struct dx_tail *t;
> >> int count_offset, limit, count;
> >>
> >> - if (!EXT4_HAS_RO_COMPAT_FEATURE(inode->i_sb,
> >> - EXT4_FEATURE_RO_COMPAT_METADATA_CSUM))
> >> + if (!ext4_has_metadata_csum(inode->i_sb))
> >> return;
> >>
> >> c = get_dx_countlimit(inode, dirent, &count_offset);
> >> @@ -555,8 +550,7 @@ static inline unsigned dx_root_limit(struct inode *dir, unsigned infosize)
> >> unsigned entry_space = dir->i_sb->s_blocksize - EXT4_DIR_REC_LEN(1) -
> >> EXT4_DIR_REC_LEN(2) - infosize;
> >>
> >> - if (EXT4_HAS_RO_COMPAT_FEATURE(dir->i_sb,
> >> - EXT4_FEATURE_RO_COMPAT_METADATA_CSUM))
> >> + if (ext4_has_metadata_csum(dir->i_sb))
> >> entry_space -= sizeof(struct dx_tail);
> >> return entry_space / sizeof(struct dx_entry);
> >> }
> >> @@ -565,8 +559,7 @@ static inline unsigned dx_node_limit(struct inode *dir)
> >> {
> >> unsigned entry_space = dir->i_sb->s_blocksize - EXT4_DIR_REC_LEN(0);
> >>
> >> - if (EXT4_HAS_RO_COMPAT_FEATURE(dir->i_sb,
> >> - EXT4_FEATURE_RO_COMPAT_METADATA_CSUM))
> >> + if (ext4_has_metadata_csum(dir->i_sb))
> >> entry_space -= sizeof(struct dx_tail);
> >> return entry_space / sizeof(struct dx_entry);
> >> }
> >> @@ -1524,8 +1517,7 @@ static struct ext4_dir_entry_2 *do_split(handle_t *handle, struct inode *dir,
> >> int csum_size = 0;
> >> int err = 0, i;
> >>
> >> - if (EXT4_HAS_RO_COMPAT_FEATURE(dir->i_sb,
> >> - EXT4_FEATURE_RO_COMPAT_METADATA_CSUM))
> >> + if (ext4_has_metadata_csum(dir->i_sb))
> >> csum_size = sizeof(struct ext4_dir_entry_tail);
> >>
> >> bh2 = ext4_append(handle, dir, &newblock);
> >> @@ -1691,8 +1683,7 @@ static int add_dirent_to_buf(handle_t *handle, struct dentry *dentry,
> >> int csum_size = 0;
> >> int err;
> >>
> >> - if (EXT4_HAS_RO_COMPAT_FEATURE(inode->i_sb,
> >> - EXT4_FEATURE_RO_COMPAT_METADATA_CSUM))
> >> + if (ext4_has_metadata_csum(inode->i_sb))
> >> csum_size = sizeof(struct ext4_dir_entry_tail);
> >>
> >> if (!de) {
> >> @@ -1759,8 +1750,7 @@ static int make_indexed_dir(handle_t *handle, struct dentry *dentry,
> >> struct fake_dirent *fde;
> >> int csum_size = 0;
> >>
> >> - if (EXT4_HAS_RO_COMPAT_FEATURE(inode->i_sb,
> >> - EXT4_FEATURE_RO_COMPAT_METADATA_CSUM))
> >> + if (ext4_has_metadata_csum(inode->i_sb))
> >> csum_size = sizeof(struct ext4_dir_entry_tail);
> >>
> >> blocksize = dir->i_sb->s_blocksize;
> >> @@ -1877,8 +1867,7 @@ static int ext4_add_entry(handle_t *handle, struct dentry *dentry,
> >> ext4_lblk_t block, blocks;
> >> int csum_size = 0;
> >>
> >> - if (EXT4_HAS_RO_COMPAT_FEATURE(inode->i_sb,
> >> - EXT4_FEATURE_RO_COMPAT_METADATA_CSUM))
> >> + if (ext4_has_metadata_csum(inode->i_sb))
> >> csum_size = sizeof(struct ext4_dir_entry_tail);
> >>
> >> sb = dir->i_sb;
> >> @@ -2142,8 +2131,7 @@ static int ext4_delete_entry(handle_t *handle,
> >> return err;
> >> }
> >>
> >> - if (EXT4_HAS_RO_COMPAT_FEATURE(dir->i_sb,
> >> - EXT4_FEATURE_RO_COMPAT_METADATA_CSUM))
> >> + if (ext4_has_metadata_csum(dir->i_sb))
> >> csum_size = sizeof(struct ext4_dir_entry_tail);
> >>
> >> BUFFER_TRACE(bh, "get_write_access");
> >> @@ -2362,8 +2350,7 @@ static int ext4_init_new_dir(handle_t *handle, struct inode *dir,
> >> int csum_size = 0;
> >> int err;
> >>
> >> - if (EXT4_HAS_RO_COMPAT_FEATURE(dir->i_sb,
> >> - EXT4_FEATURE_RO_COMPAT_METADATA_CSUM))
> >> + if (ext4_has_metadata_csum(dir->i_sb))
> >> csum_size = sizeof(struct ext4_dir_entry_tail);
> >>
> >> if (ext4_test_inode_state(inode, EXT4_STATE_MAY_INLINE_DATA)) {
> >> diff --git a/fs/ext4/resize.c b/fs/ext4/resize.c
> >> index bb0e80f..d5afb0a 100644
> >> --- a/fs/ext4/resize.c
> >> +++ b/fs/ext4/resize.c
> >> @@ -1210,8 +1210,7 @@ static int ext4_set_bitmap_checksums(struct super_block *sb,
> >> {
> >> struct buffer_head *bh;
> >>
> >> - if (!EXT4_HAS_RO_COMPAT_FEATURE(sb,
> >> - EXT4_FEATURE_RO_COMPAT_METADATA_CSUM))
> >> + if (!ext4_has_metadata_csum(sb))
> >> return 0;
> >>
> >> bh = ext4_get_bitmap(sb, group_data->inode_bitmap);
> >> diff --git a/fs/ext4/super.c b/fs/ext4/super.c
> >> index a0811cc..5afe42d 100644
> >> --- a/fs/ext4/super.c
> >> +++ b/fs/ext4/super.c
> >> @@ -140,8 +140,7 @@ static __le32 ext4_superblock_csum(struct super_block *sb,
> >> static int ext4_superblock_csum_verify(struct super_block *sb,
> >> struct ext4_super_block *es)
> >> {
> >> - if (!EXT4_HAS_RO_COMPAT_FEATURE(sb,
> >> - EXT4_FEATURE_RO_COMPAT_METADATA_CSUM))
> >> + if (!ext4_has_metadata_csum(sb))
> >> return 1;
> >>
> >> return es->s_checksum == ext4_superblock_csum(sb, es);
> >> @@ -151,8 +150,7 @@ void ext4_superblock_csum_set(struct super_block *sb)
> >> {
> >> struct ext4_super_block *es = EXT4_SB(sb)->s_es;
> >>
> >> - if (!EXT4_HAS_RO_COMPAT_FEATURE(sb,
> >> - EXT4_FEATURE_RO_COMPAT_METADATA_CSUM))
> >> + if (!ext4_has_metadata_csum(sb))
> >> return;
> >>
> >> es->s_checksum = ext4_superblock_csum(sb, es);
> >> @@ -1989,8 +1987,7 @@ static __le16 ext4_group_desc_csum(struct ext4_sb_info *sbi, __u32 block_group,
> >> __u16 crc = 0;
> >> __le32 le_group = cpu_to_le32(block_group);
> >>
> >> - if ((sbi->s_es->s_feature_ro_compat &
> >> - cpu_to_le32(EXT4_FEATURE_RO_COMPAT_METADATA_CSUM))) {
> >> + if (ext4_has_metadata_csum(sbi->s_sb)) {
> >> /* Use new metadata_csum algorithm */
> >> __le16 save_csum;
> >> __u32 csum32;
> >> @@ -3199,8 +3196,7 @@ static int set_journal_csum_feature_set(struct super_block *sb)
> >> int compat, incompat;
> >> struct ext4_sb_info *sbi = EXT4_SB(sb);
> >>
> >> - if (EXT4_HAS_RO_COMPAT_FEATURE(sb,
> >> - EXT4_FEATURE_RO_COMPAT_METADATA_CSUM)) {
> >> + if (ext4_has_metadata_csum(sb)) {
> >> /* journal checksum v3 */
> >> compat = 0;
> >> incompat = JBD2_FEATURE_INCOMPAT_CSUM_V3;
> >> @@ -3508,8 +3504,7 @@ static int ext4_fill_super(struct super_block *sb, void *data, int silent)
> >> }
> >>
> >> /* Precompute checksum seed for all metadata */
> >> - if (EXT4_HAS_RO_COMPAT_FEATURE(sb,
> >> - EXT4_FEATURE_RO_COMPAT_METADATA_CSUM))
> >> + if (ext4_has_metadata_csum(sb))
> >> sbi->s_csum_seed = ext4_chksum(sbi, ~0, es->s_uuid,
> >> sizeof(es->s_uuid));
> >>
> >> diff --git a/fs/ext4/xattr.c b/fs/ext4/xattr.c
> >> index 42823ab..1e09fc7 100644
> >> --- a/fs/ext4/xattr.c
> >> +++ b/fs/ext4/xattr.c
> >> @@ -142,8 +142,7 @@ static int ext4_xattr_block_csum_verify(struct inode *inode,
> >> sector_t block_nr,
> >> struct ext4_xattr_header *hdr)
> >> {
> >> - if (EXT4_HAS_RO_COMPAT_FEATURE(inode->i_sb,
> >> - EXT4_FEATURE_RO_COMPAT_METADATA_CSUM) &&
> >> + if (ext4_has_metadata_csum(inode->i_sb) &&
> >> (hdr->h_checksum != ext4_xattr_block_csum(inode, block_nr, hdr)))
> >> return 0;
> >> return 1;
> >> @@ -153,8 +152,7 @@ static void ext4_xattr_block_csum_set(struct inode *inode,
> >> sector_t block_nr,
> >> struct ext4_xattr_header *hdr)
> >> {
> >> - if (!EXT4_HAS_RO_COMPAT_FEATURE(inode->i_sb,
> >> - EXT4_FEATURE_RO_COMPAT_METADATA_CSUM))
> >> + if (!ext4_has_metadata_csum(inode->i_sb))
> >> return;
> >>
> >> hdr->h_checksum = ext4_xattr_block_csum(inode, block_nr, hdr);
> >> --
> >> 1.7.1
> >>
> >> --
> >> To unsubscribe from this list: send the line "unsubscribe linux-ext4" in
> >> the body of a message to majordomo@...r.kernel.org
> >> More majordomo info at http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe linux-ext4" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists