Re: [PATCH -next 5/6] ext4: rename xattr_find_entry() and __xattr_check_inode()

From: Jan Kara
Date: Tue Dec 06 2022 - 07:07:50 EST


On Tue 06-12-22 09:58:05, Ye Bin wrote:
> From: Ye Bin <yebin10@xxxxxxxxxx>
>
> xattr_find_entry() and __xattr_check_inode() is in EXT4 xattr module. so
> add 'ext4' prefix to unify name style.
>
> Signed-off-by: Ye Bin <yebin10@xxxxxxxxxx>

Looks nice. Just one nit below:

> @@ -1862,7 +1862,7 @@ ext4_xattr_block_find(struct inode *inode, struct ext4_xattr_info *i,
> bs->s.first = BFIRST(bs->bh);
> bs->s.end = bs->bh->b_data + bs->bh->b_size;
> bs->s.here = bs->s.first;
> - error = xattr_find_entry(inode, &bs->s.here, bs->s.end,
> + error = ext4_xattr_find_entry(inode, &bs->s.here, bs->s.end,
> i->name_index, i->name, 1);
> if (error && error != -ENODATA)
> return error;
> @@ -2222,11 +2222,11 @@ int ext4_xattr_ibody_find(struct inode *inode, struct ext4_xattr_info *i,
> is->s.here = is->s.first;
> is->s.end = (void *)raw_inode + EXT4_SB(inode->i_sb)->s_inode_size;
> if (ext4_test_inode_state(inode, EXT4_STATE_XATTR)) {
> - error = xattr_check_inode(inode, header, is->s.end);
> + error = ext4_xattr_check_inode(inode, header, is->s.end);
> if (error)
> return error;
> /* Find the named attribute. */
> - error = xattr_find_entry(inode, &is->s.here, is->s.end,
> + error = ext4_xattr_find_entry(inode, &is->s.here, is->s.end,
> i->name_index, i->name, 0);
> if (error && error != -ENODATA)
> return error;

The indentation of arguments in the above should be updated as well to look
like:

error = ext4_xattr_find_entry(inode, &is->s.here, is->s.end,
i->name_index, i->name, 0);

Honza
--
Jan Kara <jack@xxxxxxxx>
SUSE Labs, CR