[PATCH] erofs: remove useless src in erofs_xattr_copy_to_buffer()
Hongbo Li
lihongbo22 at huawei.com
Tue Dec 30 00:09:04 AEDT 2025
On 2025/12/29 18:05, Ferry Meng wrote:
> Use it->kaddr directly.
>
> Signed-off-by: Ferry Meng <mengferry at linux.alibaba.com>
Reviewed-by: Hongbo Li <lihongbo22 at huawei.com>
Thanks,
Hongbo
> ---
> fs/erofs/xattr.c | 4 +---
> 1 file changed, 1 insertion(+), 3 deletions(-)
>
> diff --git a/fs/erofs/xattr.c b/fs/erofs/xattr.c
> index 396536d9a862..c5c481b3f32d 100644
> --- a/fs/erofs/xattr.c
> +++ b/fs/erofs/xattr.c
> @@ -182,17 +182,15 @@ static int erofs_xattr_copy_to_buffer(struct erofs_xattr_iter *it,
> {
> unsigned int slice, processed;
> struct super_block *sb = it->sb;
> - void *src;
>
> for (processed = 0; processed < len; processed += slice) {
> it->kaddr = erofs_bread(&it->buf, it->pos, true);
> if (IS_ERR(it->kaddr))
> return PTR_ERR(it->kaddr);
>
> - src = it->kaddr;
> slice = min_t(unsigned int, sb->s_blocksize -
> erofs_blkoff(sb, it->pos), len - processed);
> - memcpy(it->buffer + it->buffer_ofs, src, slice);
> + memcpy(it->buffer + it->buffer_ofs, it->kaddr, slice);
> it->buffer_ofs += slice;
> it->pos += slice;
> }
More information about the Linux-erofs
mailing list