1. Patchew
  2. linux
  3. View series

[PATCH 6.1.y] erofs: get rid of z_erofs_fill_inode()

driz2t@qq.com posted 1 patch 1 week, 1 day ago
There is a newer version of this series
fs/erofs/inode.c     |  12 ++++++++----
fs/erofs/internal.h  |   2 --
fs/erofs/zmap.c      |  18 ------------------
scripts/extract-cert | Bin 0 -> 14608 bytes
4 files changed, 8 insertions(+), 24 deletions(-)
create mode 100755 scripts/extract-cert
[PATCH 6.1.y] erofs: get rid of z_erofs_fill_inode()
Posted by driz2t@qq.com 1 week, 1 day ago 
This is a backport of upstream commit 4fdadd5b0f0c723c812842454f8cca1619f2e731.
(erofs: get rid of z_erofs_fill_inode())

Reported-by: syzbot+016d861797fd718491a8@syzkaller.appspotmail.com
Tested-by: syzbot+016d861797fd718491a8@syzkaller.appspotmail.com
Signed-off-by: Changjian Liu <driz2t@qq.com>
---
 fs/erofs/inode.c     |  12 ++++++++----
 fs/erofs/internal.h  |   2 --
 fs/erofs/zmap.c      |  18 ------------------
 scripts/extract-cert | Bin 0 -> 14608 bytes
 4 files changed, 8 insertions(+), 24 deletions(-)
 create mode 100755 scripts/extract-cert

diff --git a/fs/erofs/inode.c b/fs/erofs/inode.c
index 3cbef6318b7b..484572504b4d 100644
--- a/fs/erofs/inode.c
+++ b/fs/erofs/inode.c
@@ -280,11 +280,15 @@ static int erofs_fill_inode(struct inode *inode)
      }
 
      if (erofs_inode_is_data_compressed(vi->datalayout)) {
+#ifdef CONFIG_EROFS_FS_ZIP
            if (!erofs_is_fscache_mode(inode->i_sb) &&
-               inode->i_sb->s_blocksize_bits == PAGE_SHIFT)
-                 err = z_erofs_fill_inode(inode);
-           else
-                 err = -EOPNOTSUPP;
+               inode->i_sb->s_blocksize_bits == PAGE_SHIFT) {
+                 inode->i_mapping->a_ops = &z_erofs_aops;
+                 err = 0;
+                 goto out_unlock;
+           }
+#endif
+           err = -EOPNOTSUPP;
            goto out_unlock;
      }
      inode->i_mapping->a_ops = &erofs_raw_access_aops;
diff --git a/fs/erofs/internal.h b/fs/erofs/internal.h
index 126970932805..1a4d08a93339 100644
--- a/fs/erofs/internal.h
+++ b/fs/erofs/internal.h
@@ -423,12 +423,10 @@ enum {
 extern const struct iomap_ops z_erofs_iomap_report_ops;
 
 #ifdef CONFIG_EROFS_FS_ZIP
-int z_erofs_fill_inode(struct inode *inode);
 int z_erofs_map_blocks_iter(struct inode *inode,
                      struct erofs_map_blocks *map,
                      int flags);
 #else
-static inline int z_erofs_fill_inode(struct inode *inode) { return -EOPNOTSUPP; }
 static inline int z_erofs_map_blocks_iter(struct inode *inode,
                                struct erofs_map_blocks *map,
                                int flags)
diff --git a/fs/erofs/zmap.c b/fs/erofs/zmap.c
index d2d7fe826091..ff84533da0c4 100644
--- a/fs/erofs/zmap.c
+++ b/fs/erofs/zmap.c
@@ -7,24 +7,6 @@
 #include <asm/unaligned.h>
 #include <trace/events/erofs.h>
 
-int z_erofs_fill_inode(struct inode *inode)
-{
-     struct erofs_inode *const vi = EROFS_I(inode);
-     struct erofs_sb_info *sbi = EROFS_SB(inode->i_sb);
-
-     if (!erofs_sb_has_big_pcluster(sbi) &&
-         !erofs_sb_has_ztailpacking(sbi) && !erofs_sb_has_fragments(sbi) &&
-         vi->datalayout == EROFS_INODE_COMPRESSED_FULL) {
-           vi->z_advise = 0;
-           vi->z_algorithmtype[0] = 0;
-           vi->z_algorithmtype[1] = 0;
-           vi->z_logical_clusterbits = inode->i_sb->s_blocksize_bits;
-           set_bit(EROFS_I_Z_INITED_BIT, &vi->flags);
-     }
-     inode->i_mapping->a_ops = &z_erofs_aops;
-     return 0;
-}
-
 struct z_erofs_maprecorder {
      struct inode *inode;
      struct erofs_map_blocks *map;
-- 
2.43.0
Re: [PATCH 6.1.y] erofs: get rid of z_erofs_fill_inode()
Posted by Gao Xiang 1 week, 1 day ago 

On 2026/3/25 14:57, driz2t@qq.com wrote:
> This is a backport of upstream commit 4fdadd5b0f0c723c812842454f8cca1619f2e731.
> (erofs: get rid of z_erofs_fill_inode())
> 
> Reported-by: syzbot+016d861797fd718491a8@syzkaller.appspotmail.com
> Tested-by: syzbot+016d861797fd718491a8@syzkaller.appspotmail.com
> Signed-off-by: Changjian Liu <driz2t@qq.com>

Please follow the stable patch style.


> ---
>   fs/erofs/inode.c     |  12 ++++++++----
>   fs/erofs/internal.h  |   2 --
>   fs/erofs/zmap.c      |  18 ------------------
>   scripts/extract-cert | Bin 0 -> 14608 bytes

scripts/extract-cert | Bin 0 -> 14608 bytes
What's that.

Thanks,
Gao Xiang

>   4 files changed, 8 insertions(+), 24 deletions(-)
>   create mode 100755 scripts/extract-cert
> 
> diff --git a/fs/erofs/inode.c b/fs/erofs/inode.c
> index 3cbef6318b7b..484572504b4d 100644
> --- a/fs/erofs/inode.c
> +++ b/fs/erofs/inode.c
> @@ -280,11 +280,15 @@ static int erofs_fill_inode(struct inode *inode)
>        }
>   
>        if (erofs_inode_is_data_compressed(vi->datalayout)) {
> +#ifdef CONFIG_EROFS_FS_ZIP
>              if (!erofs_is_fscache_mode(inode->i_sb) &&
> -               inode->i_sb->s_blocksize_bits == PAGE_SHIFT)
> -                 err = z_erofs_fill_inode(inode);
> -           else
> -                 err = -EOPNOTSUPP;
> +               inode->i_sb->s_blocksize_bits == PAGE_SHIFT) {
> +                 inode->i_mapping->a_ops = &z_erofs_aops;
> +                 err = 0;
> +                 goto out_unlock;
> +           }
> +#endif
> +           err = -EOPNOTSUPP;
>              goto out_unlock;
>        }
>        inode->i_mapping->a_ops = &erofs_raw_access_aops;
> diff --git a/fs/erofs/internal.h b/fs/erofs/internal.h
> index 126970932805..1a4d08a93339 100644
> --- a/fs/erofs/internal.h
> +++ b/fs/erofs/internal.h
> @@ -423,12 +423,10 @@ enum {
>   extern const struct iomap_ops z_erofs_iomap_report_ops;
>   
>   #ifdef CONFIG_EROFS_FS_ZIP
> -int z_erofs_fill_inode(struct inode *inode);
>   int z_erofs_map_blocks_iter(struct inode *inode,
>                        struct erofs_map_blocks *map,
>                        int flags);
>   #else
> -static inline int z_erofs_fill_inode(struct inode *inode) { return -EOPNOTSUPP; }
>   static inline int z_erofs_map_blocks_iter(struct inode *inode,
>                                  struct erofs_map_blocks *map,
>                                  int flags)
> diff --git a/fs/erofs/zmap.c b/fs/erofs/zmap.c
> index d2d7fe826091..ff84533da0c4 100644
> --- a/fs/erofs/zmap.c
> +++ b/fs/erofs/zmap.c
> @@ -7,24 +7,6 @@
>   #include <asm/unaligned.h>
>   #include <trace/events/erofs.h>
>   
> -int z_erofs_fill_inode(struct inode *inode)
> -{
> -     struct erofs_inode *const vi = EROFS_I(inode);
> -     struct erofs_sb_info *sbi = EROFS_SB(inode->i_sb);
> -
> -     if (!erofs_sb_has_big_pcluster(sbi) &&
> -         !erofs_sb_has_ztailpacking(sbi) && !erofs_sb_has_fragments(sbi) &&
> -         vi->datalayout == EROFS_INODE_COMPRESSED_FULL) {
> -           vi->z_advise = 0;
> -           vi->z_algorithmtype[0] = 0;
> -           vi->z_algorithmtype[1] = 0;
> -           vi->z_logical_clusterbits = inode->i_sb->s_blocksize_bits;
> -           set_bit(EROFS_I_Z_INITED_BIT, &vi->flags);
> -     }
> -     inode->i_mapping->a_ops = &z_erofs_aops;
> -     return 0;
> -}
> -
>   struct z_erofs_maprecorder {
>        struct inode *inode;
>        struct erofs_map_blocks *map;
> -- 
> 2.43.0

Patchew 2.1-devel-ea86937

© 2016 - 2026 Red Hat, Inc.