From nobody Fri Dec 19 22:02:48 2025 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (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 9805413DBB6; Sun, 24 Mar 2024 23:09:46 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1711321786; cv=none; b=RH6OMEU0tQiC84q0b5ID4gH9KuPptzRnodMCGju6Ve2flj07kEOF7Rzf8SeBJf47bol4UIxY6rQbrzJdSIWYq5/pwY1F/pRzwwpHjNV4QYxV9wmSA1RLuJfj/4E+OE4AvobmtC9xj1uL7P/UhS8+VVfeSY3dOMttknsFwfpNEMA= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1711321786; c=relaxed/simple; bh=/OP39yM5r4QOS1vn0CDVDoDYwQdaIcTs0I+c23KntS0=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=NntyxIMJd9lG5FuBy7U4tQjN99SF5WfSvHOJ6Yfx3Gj3yKxRqj2AaLW6r3tbzTT4/trGtT2zEXWEfo1EpbLEBp+nNxuilGzrkPTqkyAjQd6Y0OtMd4SJBySgHahFpXd7rhmIYe7c7i70dkCIZ0KpG6eK9psqhCq1NPa5A8nkvkc= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=qg7IcNUF; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="qg7IcNUF" Received: by smtp.kernel.org (Postfix) with ESMTPSA id D7345C433C7; Sun, 24 Mar 2024 23:09:45 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1711321786; bh=/OP39yM5r4QOS1vn0CDVDoDYwQdaIcTs0I+c23KntS0=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=qg7IcNUFeGErp46TH2ziWqIqojs1XS0zfniqZx3YlOO+sN46RcM1pmnzHJ6VMrOtK +JX+a347W+OBW+/66sNWGdX0DMPL7JuetQK5kUCaN9KfpTmsFeDS08CwHCu6uVvB8L +2upowzXYk7MjtpRnijK4lcvUEEZPEoMaNhXLAvcqVwIFxj7xLLw4+cmPZXexdG+GF 2bjHuoJjcej/fQwJlmednywQf8SkO6wSRMRREpqo99NaK2AO2vVkVi7welqhANtBwW oMFiUZUy72lTWBgoDVrw5hz+2ICKPvWBCZmgbYrDR0eETeIfq14a9kn38/2JMu0k6O cNMtCVzE/0NeQ== From: Sasha Levin To: linux-kernel@vger.kernel.org, stable@vger.kernel.org Cc: Chao Yu , Jaegeuk Kim , Sasha Levin Subject: [PATCH 6.6 514/638] f2fs: introduce get_dnode_addr() to clean up codes Date: Sun, 24 Mar 2024 18:59:11 -0400 Message-ID: <20240324230116.1348576-515-sashal@kernel.org> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20240324230116.1348576-1-sashal@kernel.org> References: <20240324230116.1348576-1-sashal@kernel.org> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-stable: review X-Patchwork-Hint: Ignore Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" From: Chao Yu [ Upstream commit 2020cd48e41cb8470bb1ca0835033d13d3178425 ] Just cleanup, no logic changes. Signed-off-by: Chao Yu Signed-off-by: Jaegeuk Kim Stable-dep-of: 54607494875e ("f2fs: compress: fix to avoid inconsistence be= wteen i_blocks and dnode") Signed-off-by: Sasha Levin --- fs/f2fs/data.c | 11 ++--------- fs/f2fs/f2fs.h | 18 +++++++++++++++--- fs/f2fs/file.c | 8 +------- fs/f2fs/inode.c | 32 ++++++++++++++------------------ 4 files changed, 32 insertions(+), 37 deletions(-) diff --git a/fs/f2fs/data.c b/fs/f2fs/data.c index 7bfccb4570a2b..ec5088f814cab 100644 --- a/fs/f2fs/data.c +++ b/fs/f2fs/data.c @@ -1182,16 +1182,9 @@ static int f2fs_submit_page_read(struct inode *inode= , struct page *page, =20 static void __set_data_blkaddr(struct dnode_of_data *dn) { - struct f2fs_node *rn =3D F2FS_NODE(dn->node_page); - __le32 *addr_array; - int base =3D 0; + __le32 *addr =3D get_dnode_addr(dn->inode, dn->node_page); =20 - if (IS_INODE(dn->node_page) && f2fs_has_extra_attr(dn->inode)) - base =3D get_extra_isize(dn->inode); - - /* Get physical address of data block */ - addr_array =3D blkaddr_in_node(rn); - addr_array[base + dn->ofs_in_node] =3D cpu_to_le32(dn->data_blkaddr); + addr[dn->ofs_in_node] =3D cpu_to_le32(dn->data_blkaddr); } =20 /* diff --git a/fs/f2fs/f2fs.h b/fs/f2fs/f2fs.h index d64114de07919..1155790892b13 100644 --- a/fs/f2fs/f2fs.h +++ b/fs/f2fs/f2fs.h @@ -3270,12 +3270,13 @@ static inline bool f2fs_is_cow_file(struct inode *i= node) return is_inode_flag_set(inode, FI_COW_FILE); } =20 +static inline __le32 *get_dnode_addr(struct inode *inode, + struct page *node_page); static inline void *inline_data_addr(struct inode *inode, struct page *pag= e) { - struct f2fs_inode *ri =3D F2FS_INODE(page); - int extra_size =3D get_extra_isize(inode); + __le32 *addr =3D get_dnode_addr(inode, page); =20 - return (void *)&(ri->i_addr[extra_size + DEF_INLINE_RESERVED_SIZE]); + return (void *)(addr + DEF_INLINE_RESERVED_SIZE); } =20 static inline int f2fs_has_inline_dentry(struct inode *inode) @@ -3418,6 +3419,17 @@ static inline int get_inline_xattr_addrs(struct inod= e *inode) return F2FS_I(inode)->i_inline_xattr_size; } =20 +static inline __le32 *get_dnode_addr(struct inode *inode, + struct page *node_page) +{ + int base =3D 0; + + if (IS_INODE(node_page) && f2fs_has_extra_attr(inode)) + base =3D get_extra_isize(inode); + + return blkaddr_in_node(F2FS_NODE(node_page)) + base; +} + #define f2fs_get_inode_mode(i) \ ((is_inode_flag_set(i, FI_ACL_MODE)) ? \ (F2FS_I(i)->i_acl_mode) : ((i)->i_mode)) diff --git a/fs/f2fs/file.c b/fs/f2fs/file.c index c5ccdd3e58666..9cfb0bce52c4b 100644 --- a/fs/f2fs/file.c +++ b/fs/f2fs/file.c @@ -557,20 +557,14 @@ static int f2fs_file_open(struct inode *inode, struct= file *filp) void f2fs_truncate_data_blocks_range(struct dnode_of_data *dn, int count) { struct f2fs_sb_info *sbi =3D F2FS_I_SB(dn->inode); - struct f2fs_node *raw_node; int nr_free =3D 0, ofs =3D dn->ofs_in_node, len =3D count; __le32 *addr; - int base =3D 0; bool compressed_cluster =3D false; int cluster_index =3D 0, valid_blocks =3D 0; int cluster_size =3D F2FS_I(dn->inode)->i_cluster_size; bool released =3D !atomic_read(&F2FS_I(dn->inode)->i_compr_blocks); =20 - if (IS_INODE(dn->node_page) && f2fs_has_extra_attr(dn->inode)) - base =3D get_extra_isize(dn->inode); - - raw_node =3D F2FS_NODE(dn->node_page); - addr =3D blkaddr_in_node(raw_node) + base + ofs; + addr =3D get_dnode_addr(dn->inode, dn->node_page) + ofs; =20 /* Assumption: truncation starts with cluster */ for (; count > 0; count--, addr++, dn->ofs_in_node++, cluster_index++) { diff --git a/fs/f2fs/inode.c b/fs/f2fs/inode.c index 8d8e90152ca87..dd61449f05b85 100644 --- a/fs/f2fs/inode.c +++ b/fs/f2fs/inode.c @@ -61,35 +61,31 @@ void f2fs_set_inode_flags(struct inode *inode) S_ENCRYPTED|S_VERITY|S_CASEFOLD); } =20 -static void __get_inode_rdev(struct inode *inode, struct f2fs_inode *ri) +static void __get_inode_rdev(struct inode *inode, struct page *node_page) { - int extra_size =3D get_extra_isize(inode); + __le32 *addr =3D get_dnode_addr(inode, node_page); =20 if (S_ISCHR(inode->i_mode) || S_ISBLK(inode->i_mode) || S_ISFIFO(inode->i_mode) || S_ISSOCK(inode->i_mode)) { - if (ri->i_addr[extra_size]) - inode->i_rdev =3D old_decode_dev( - le32_to_cpu(ri->i_addr[extra_size])); + if (addr[0]) + inode->i_rdev =3D old_decode_dev(le32_to_cpu(addr[0])); else - inode->i_rdev =3D new_decode_dev( - le32_to_cpu(ri->i_addr[extra_size + 1])); + inode->i_rdev =3D new_decode_dev(le32_to_cpu(addr[1])); } } =20 -static void __set_inode_rdev(struct inode *inode, struct f2fs_inode *ri) +static void __set_inode_rdev(struct inode *inode, struct page *node_page) { - int extra_size =3D get_extra_isize(inode); + __le32 *addr =3D get_dnode_addr(inode, node_page); =20 if (S_ISCHR(inode->i_mode) || S_ISBLK(inode->i_mode)) { if (old_valid_dev(inode->i_rdev)) { - ri->i_addr[extra_size] =3D - cpu_to_le32(old_encode_dev(inode->i_rdev)); - ri->i_addr[extra_size + 1] =3D 0; + addr[0] =3D cpu_to_le32(old_encode_dev(inode->i_rdev)); + addr[1] =3D 0; } else { - ri->i_addr[extra_size] =3D 0; - ri->i_addr[extra_size + 1] =3D - cpu_to_le32(new_encode_dev(inode->i_rdev)); - ri->i_addr[extra_size + 2] =3D 0; + addr[0] =3D 0; + addr[1] =3D cpu_to_le32(new_encode_dev(inode->i_rdev)); + addr[2] =3D 0; } } } @@ -463,7 +459,7 @@ static int do_read_inode(struct inode *inode) } =20 /* get rdev by using inline_info */ - __get_inode_rdev(inode, ri); + __get_inode_rdev(inode, node_page); =20 if (!f2fs_need_inode_block_update(sbi, inode->i_ino)) fi->last_disk_size =3D inode->i_size; @@ -736,7 +732,7 @@ void f2fs_update_inode(struct inode *inode, struct page= *node_page) } } =20 - __set_inode_rdev(inode, ri); + __set_inode_rdev(inode, node_page); =20 /* deleted inode */ if (inode->i_nlink =3D=3D 0) --=20 2.43.0