From nobody Tue Dec 2 02:05:58 2025 Received: from dggsgout11.his.huawei.com (dggsgout11.his.huawei.com [45.249.212.51]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 790B133BBDB; Fri, 21 Nov 2025 09:16:51 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=45.249.212.51 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1763716615; cv=none; b=ETw2tkJCl4emvNSWKaY+Ro62X81aMzcfjGEr//k1dfh1JHqT4MRsEqz2I4dd+EYVeAMJaeIxZR8RvdRNZVLZSbFGSErV+FjVTKrPi3DDP+EKl2e3TurxK0eKzTHAEwvKccHZ0Kl0Oa0yKCFbE+NwwGL/7gt5bOnaEMd3mBKShXI= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1763716615; c=relaxed/simple; bh=YrykeLcgX4JHiuiCkTkDHn+PsZ01hXsMjuKDqOGcAYA=; h=From:To:Cc:Subject:Date:Message-Id:In-Reply-To:References: MIME-Version; b=lDwuHfzlsAN6LJ5YRhiDioBvWA/yAQQiZp6hk+hf0mh8uIB1XMja95ugldMcjyv5cTh0WJRjIWcBInpXvNTN8kAng4zwefg1CF1oqEiCpVAAoudrWKU7O07trLpjL/RFdKAO7oZbYjFDASk2gsS8ldhMEbXN+VAYsk0/q7ZA2dc= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=huaweicloud.com; spf=none smtp.mailfrom=huaweicloud.com; arc=none smtp.client-ip=45.249.212.51 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=huaweicloud.com Authentication-Results: smtp.subspace.kernel.org; spf=none smtp.mailfrom=huaweicloud.com Received: from mail.maildlp.com (unknown [172.19.163.235]) by dggsgout11.his.huawei.com (SkyGuard) with ESMTPS id 4dCV280X4HzYQvH3; Fri, 21 Nov 2025 17:16:00 +0800 (CST) Received: from mail02.huawei.com (unknown [10.116.40.75]) by mail.maildlp.com (Postfix) with ESMTP id 526F21A0BC1; Fri, 21 Nov 2025 17:16:43 +0800 (CST) Received: from huaweicloud.com (unknown [10.50.87.129]) by APP2 (Coremail) with SMTP id Syh0CgBHpXv4LSBpaLMDBg--.2072S13; Fri, 21 Nov 2025 17:16:43 +0800 (CST) From: libaokun@huaweicloud.com To: linux-ext4@vger.kernel.org Cc: tytso@mit.edu, adilger.kernel@dilger.ca, jack@suse.cz, linux-kernel@vger.kernel.org, kernel@pankajraghav.com, mcgrof@kernel.org, ebiggers@kernel.org, willy@infradead.org, yi.zhang@huawei.com, yangerkun@huawei.com, chengzhihao1@huawei.com, libaokun1@huawei.com, libaokun@huaweicloud.com Subject: [PATCH v4 09/24] ext4: add EXT4_LBLK_TO_B macro for logical block to bytes conversion Date: Fri, 21 Nov 2025 17:06:39 +0800 Message-Id: <20251121090654.631996-10-libaokun@huaweicloud.com> X-Mailer: git-send-email 2.39.2 In-Reply-To: <20251121090654.631996-1-libaokun@huaweicloud.com> References: <20251121090654.631996-1-libaokun@huaweicloud.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-CM-TRANSID: Syh0CgBHpXv4LSBpaLMDBg--.2072S13 X-Coremail-Antispam: 1UD129KBjvJXoW3Ar4rKryxZw43ZF43ZFWDArb_yoWxCw43pF Z09F18GF409Fyj9r48KrWDZr12g3W7K3yUXFWru34Ygas7tr1FqF1ktF1fZa45trWxZ3ZI vF45K347Ww43GrDanT9S1TB71UUUUU7qnTZGkaVYY2UrUUUUjbIjqfuFe4nvWSU5nxnvy2 9KBjDU0xBIdaVrnRJUUUQa14x267AKxVWrJVCq3wAFc2x0x2IEx4CE42xK8VAvwI8IcIk0 rVWrJVCq3wAFIxvE14AKwVWUJVWUGwA2048vs2IY020E87I2jVAFwI0_JF0E3s1l82xGYI kIc2x26xkF7I0E14v26ryj6s0DM28lY4IEw2IIxxk0rwA2F7IY1VAKz4vEj48ve4kI8wA2 z4x0Y4vE2Ix0cI8IcVAFwI0_tr0E3s1l84ACjcxK6xIIjxv20xvEc7CjxVAFwI0_Gr1j6F 4UJwA2z4x0Y4vEx4A2jsIE14v26rxl6s0DM28EF7xvwVC2z280aVCY1x0267AKxVW0oVCq 3wAS0I0E0xvYzxvE52x082IY62kv0487Mc02F40EFcxC0VAKzVAqx4xG6I80ewAv7VC0I7 IYx2IY67AKxVWUXVWUAwAv7VC2z280aVAFwI0_Gr1j6F4UJwAm72CE4IkC6x0Yz7v_Jr0_ Gr1lF7xvr2IYc2Ij64vIr41lF7I21c0EjII2zVCS5cI20VAGYxC7M4IIrI8v6xkF7I0E8c xan2IY04v7M4kE6xkIj40Ew7xC0wCY1x0262kKe7AKxVWUtVW8ZwCF04k20xvY0x0EwIxG rwCFx2IqxVCFs4IE7xkEbVWUJVW8JwC20s026c02F40E14v26r1j6r18MI8I3I0E7480Y4 vE14v26r106r1rMI8E67AF67kF1VAFwI0_Jw0_GFylIxkGc2Ij64vIr41lIxAIcVC0I7IY x2IY67AKxVWUCVW8JwCI42IY6xIIjxv20xvEc7CjxVAFwI0_Gr1j6F4UJwCI42IY6xAIw2 0EY4v20xvaj40_Jr0_JF4lIxAIcVC2z280aVAFwI0_Jr0_Gr1lIxAIcVC2z280aVCY1x02 67AKxVW8Jr0_Cr1UYxBIdaVFxhVjvjDU0xZFpf9x0JUdsqAUUUUU= X-CM-SenderInfo: 5olet0hnxqqx5xdzvxpfor3voofrz/1tbiAgATBWkf34sfFQAEs4 Content-Type: text/plain; charset="utf-8" From: Baokun Li No functional changes. Signed-off-by: Baokun Li Reviewed-by: Zhang Yi Reviewed-by: Jan Kara --- fs/ext4/ext4.h | 1 + fs/ext4/extents.c | 2 +- fs/ext4/inode.c | 20 +++++++++----------- fs/ext4/namei.c | 8 +++----- fs/ext4/verity.c | 2 +- 5 files changed, 15 insertions(+), 18 deletions(-) diff --git a/fs/ext4/ext4.h b/fs/ext4/ext4.h index 6fe8cc3bf9a5..c00ce6db69f0 100644 --- a/fs/ext4/ext4.h +++ b/fs/ext4/ext4.h @@ -368,6 +368,7 @@ struct ext4_io_submit { blkbits)) #define EXT4_B_TO_LBLK(inode, offset) \ (round_up((offset), i_blocksize(inode)) >> (inode)->i_blkbits) +#define EXT4_LBLK_TO_B(inode, lblk) ((loff_t)(lblk) << (inode)->i_blkbits) =20 /* Translate a block number to a cluster number */ #define EXT4_B2C(sbi, blk) ((blk) >> (sbi)->s_cluster_bits) diff --git a/fs/ext4/extents.c b/fs/ext4/extents.c index c7d219e6c6d8..13c3cfeb13bc 100644 --- a/fs/ext4/extents.c +++ b/fs/ext4/extents.c @@ -4562,7 +4562,7 @@ static int ext4_alloc_file_blocks(struct file *file, = ext4_lblk_t offset, * allow a full retry cycle for any remaining allocations */ retries =3D 0; - epos =3D (loff_t)(map.m_lblk + ret) << blkbits; + epos =3D EXT4_LBLK_TO_B(inode, map.m_lblk + ret); inode_set_ctime_current(inode); if (new_size) { if (epos > new_size) diff --git a/fs/ext4/inode.c b/fs/ext4/inode.c index e8da685ede49..65a9c04bc691 100644 --- a/fs/ext4/inode.c +++ b/fs/ext4/inode.c @@ -831,9 +831,8 @@ int ext4_map_blocks(handle_t *handle, struct inode *ino= de, !(flags & EXT4_GET_BLOCKS_ZERO) && !ext4_is_quota_file(inode) && ext4_should_order_data(inode)) { - loff_t start_byte =3D - (loff_t)map->m_lblk << inode->i_blkbits; - loff_t length =3D (loff_t)map->m_len << inode->i_blkbits; + loff_t start_byte =3D EXT4_LBLK_TO_B(inode, map->m_lblk); + loff_t length =3D EXT4_LBLK_TO_B(inode, map->m_len); =20 if (flags & EXT4_GET_BLOCKS_IO_SUBMIT) ret =3D ext4_jbd2_inode_add_wait(handle, inode, @@ -2233,7 +2232,6 @@ static int mpage_process_folio(struct mpage_da_data *= mpd, struct folio *folio, ext4_lblk_t lblk =3D *m_lblk; ext4_fsblk_t pblock =3D *m_pblk; int err =3D 0; - int blkbits =3D mpd->inode->i_blkbits; ssize_t io_end_size =3D 0; struct ext4_io_end_vec *io_end_vec =3D ext4_last_io_end_vec(io_end); =20 @@ -2259,7 +2257,8 @@ static int mpage_process_folio(struct mpage_da_data *= mpd, struct folio *folio, err =3D PTR_ERR(io_end_vec); goto out; } - io_end_vec->offset =3D (loff_t)mpd->map.m_lblk << blkbits; + io_end_vec->offset =3D EXT4_LBLK_TO_B(mpd->inode, + mpd->map.m_lblk); } *map_bh =3D true; goto out; @@ -2269,7 +2268,7 @@ static int mpage_process_folio(struct mpage_da_data *= mpd, struct folio *folio, bh->b_blocknr =3D pblock++; } clear_buffer_unwritten(bh); - io_end_size +=3D (1 << blkbits); + io_end_size +=3D i_blocksize(mpd->inode); } while (lblk++, (bh =3D bh->b_this_page) !=3D head); =20 io_end_vec->size +=3D io_end_size; @@ -2471,7 +2470,7 @@ static int mpage_map_and_submit_extent(handle_t *hand= le, io_end_vec =3D ext4_alloc_io_end_vec(io_end); if (IS_ERR(io_end_vec)) return PTR_ERR(io_end_vec); - io_end_vec->offset =3D ((loff_t)map->m_lblk) << inode->i_blkbits; + io_end_vec->offset =3D EXT4_LBLK_TO_B(inode, map->m_lblk); do { err =3D mpage_map_one_extent(handle, mpd); if (err < 0) { @@ -3511,8 +3510,8 @@ static void ext4_set_iomap(struct inode *inode, struc= t iomap *iomap, iomap->dax_dev =3D EXT4_SB(inode->i_sb)->s_daxdev; else iomap->bdev =3D inode->i_sb->s_bdev; - iomap->offset =3D (u64) map->m_lblk << blkbits; - iomap->length =3D (u64) map->m_len << blkbits; + iomap->offset =3D EXT4_LBLK_TO_B(inode, map->m_lblk); + iomap->length =3D EXT4_LBLK_TO_B(inode, map->m_len); =20 if ((map->m_flags & EXT4_MAP_MAPPED) && !ext4_test_inode_flag(inode, EXT4_INODE_EXTENTS)) @@ -3686,7 +3685,6 @@ static int ext4_iomap_alloc(struct inode *inode, stru= ct ext4_map_blocks *map, unsigned int flags) { handle_t *handle; - u8 blkbits =3D inode->i_blkbits; int ret, dio_credits, m_flags =3D 0, retries =3D 0; bool force_commit =3D false; =20 @@ -3745,7 +3743,7 @@ static int ext4_iomap_alloc(struct inode *inode, stru= ct ext4_map_blocks *map, * i_disksize out to i_size. This could be beyond where direct I/O is * happening and thus expose allocated blocks to direct I/O reads. */ - else if (((loff_t)map->m_lblk << blkbits) >=3D i_size_read(inode)) + else if (EXT4_LBLK_TO_B(inode, map->m_lblk) >=3D i_size_read(inode)) m_flags =3D EXT4_GET_BLOCKS_CREATE; else if (ext4_test_inode_flag(inode, EXT4_INODE_EXTENTS)) m_flags =3D EXT4_GET_BLOCKS_IO_CREATE_EXT; diff --git a/fs/ext4/namei.c b/fs/ext4/namei.c index 045616033515..c4b5e252af0e 100644 --- a/fs/ext4/namei.c +++ b/fs/ext4/namei.c @@ -1076,7 +1076,7 @@ static int htree_dirblock_to_tree(struct file *dir_fi= le, for (; de < top; de =3D ext4_next_entry(de, dir->i_sb->s_blocksize)) { if (ext4_check_dir_entry(dir, NULL, de, bh, bh->b_data, bh->b_size, - (block<i_sb)) + EXT4_LBLK_TO_B(dir, block) + ((char *)de - bh->b_data))) { /* silently ignore the rest of the block */ break; @@ -1630,7 +1630,7 @@ static struct buffer_head *__ext4_find_entry(struct i= node *dir, } set_buffer_verified(bh); i =3D search_dirblock(bh, dir, fname, - block << EXT4_BLOCK_SIZE_BITS(sb), res_dir); + EXT4_LBLK_TO_B(dir, block), res_dir); if (i =3D=3D 1) { EXT4_I(dir)->i_dir_start_lookup =3D block; ret =3D bh; @@ -1710,7 +1710,6 @@ static struct buffer_head * ext4_dx_find_entry(struct= inode *dir, struct ext4_filename *fname, struct ext4_dir_entry_2 **res_dir) { - struct super_block * sb =3D dir->i_sb; struct dx_frame frames[EXT4_HTREE_LEVEL], *frame; struct buffer_head *bh; ext4_lblk_t block; @@ -1729,8 +1728,7 @@ static struct buffer_head * ext4_dx_find_entry(struct= inode *dir, goto errout; =20 retval =3D search_dirblock(bh, dir, fname, - block << EXT4_BLOCK_SIZE_BITS(sb), - res_dir); + EXT4_LBLK_TO_B(dir, block), res_dir); if (retval =3D=3D 1) goto success; brelse(bh); diff --git a/fs/ext4/verity.c b/fs/ext4/verity.c index b0acb0c50313..415d9c4d8a32 100644 --- a/fs/ext4/verity.c +++ b/fs/ext4/verity.c @@ -302,7 +302,7 @@ static int ext4_get_verity_descriptor_location(struct i= node *inode, =20 end_lblk =3D le32_to_cpu(last_extent->ee_block) + ext4_ext_get_actual_len(last_extent); - desc_size_pos =3D (u64)end_lblk << inode->i_blkbits; + desc_size_pos =3D EXT4_LBLK_TO_B(inode, end_lblk); ext4_free_ext_path(path); =20 if (desc_size_pos < sizeof(desc_size_disk)) --=20 2.46.1