Let's create /sys/fs/f2fs/<disk>/feature_list/ to meet sysfs rule.

Note that there are three feature list entries:
1) /sys/fs/f2fs/features
  : shows runtime features supported by in-kernel f2fs along with Kconfig
    - ref. F2FS_FEATURE_RO_ATTR()

2) /sys/fs/f2fs/$s_id/features <deprecated>
  : shows on-disk features enabled by mkfs.f2fs, used for old kernels

3) /sys/fs/f2fs/$s_id/feature_list
  : shows on-disk features enabled by mkfs.f2fs per instance.
    this list covers old feature list provided by 2) and beyond
    - ref. F2FS_SB_FEATURE_RO_ATTR()

Signed-off-by: Jaegeuk Kim <[email protected]>
---
 Documentation/ABI/testing/sysfs-fs-f2fs |  29 +++++-
 fs/f2fs/f2fs.h                          |   5 +
 fs/f2fs/sysfs.c                         | 118 ++++++++++++++++++++++++
 3 files changed, 151 insertions(+), 1 deletion(-)

diff --git a/Documentation/ABI/testing/sysfs-fs-f2fs 
b/Documentation/ABI/testing/sysfs-fs-f2fs
index 5088281e312e..86d5e0e12a07 100644
--- a/Documentation/ABI/testing/sysfs-fs-f2fs
+++ b/Documentation/ABI/testing/sysfs-fs-f2fs
@@ -203,7 +203,34 @@ Description:       Shows total written kbytes issued to 
disk.
 What:          /sys/fs/f2fs/<disk>/features
 Date:          July 2017
 Contact:       "Jaegeuk Kim" <[email protected]>
-Description:   Shows all enabled features in current device.
+Description:   <deprecated: should use /sys/fs/f2fs/<disk>/feature_list/
+               Shows all enabled features in current device.
+               Supported features:
+               encryption, blkzoned, extra_attr, projquota, inode_checksum,
+               flexible_inline_xattr, quota_ino, inode_crtime, lost_found,
+               verity, sb_checksum, casefold, readonly, compression,
+               encrypted_casefold, pin_file.
+
+What:          /sys/fs/f2fs/<disk>/feature_list/
+Date:          June 2021
+Contact:       "Jaegeuk Kim" <[email protected]>
+Description:   Expand /sys/fs/f2fs/<disk>/features to meet sysfs rule.
+               Supported on-disk features:
+               encryption, block_zoned (aka blkzoned), extra_attr,
+               project_quota (aka projquota), inode_checksum,
+               flexible_inline_xattr, quota_ino, inode_crtime, lost_found,
+               verity, sb_checksum, casefold, readonly, compression.
+               Note that, encrypted_casefold and pin_file were moved into
+               /sys/fs/f2fs/features/.
+
+What:          /sys/fs/f2fs/feature/
+Date:          July 2017
+Contact:       "Jaegeuk Kim" <[email protected]>
+Description:   Shows all enabled kernel features.
+               Supported features:
+               in addition to all in /sys/fs/f2fs/<disk>/feature_list/,
+               test_dummy_encryption_v2, encrypted_casefold, pin_file,
+               atomic_write
 
 What:          /sys/fs/f2fs/<disk>/inject_rate
 Date:          May 2016
diff --git a/fs/f2fs/f2fs.h b/fs/f2fs/f2fs.h
index 8903c43091f8..c8c29effd844 100644
--- a/fs/f2fs/f2fs.h
+++ b/fs/f2fs/f2fs.h
@@ -172,6 +172,8 @@ struct f2fs_mount_info {
 
 #define __F2FS_HAS_FEATURE(raw_super, mask)                            \
        ((raw_super->feature & cpu_to_le32(mask)) != 0)
+#define F2FS_MATCH_FEATURE(sbi, mask)                                  \
+       ((sbi->raw_super->feature & cpu_to_le32(mask)) == cpu_to_le32(mask))
 #define F2FS_HAS_FEATURE(sbi, mask)    __F2FS_HAS_FEATURE(sbi->raw_super, mask)
 #define F2FS_SET_FEATURE(sbi, mask)                                    \
        (sbi->raw_super->feature |= cpu_to_le32(mask))
@@ -1665,6 +1667,9 @@ struct f2fs_sb_info {
        struct kobject s_stat_kobj;             /* /sys/fs/f2fs/<devname>/stat 
*/
        struct completion s_stat_kobj_unregister;
 
+       struct kobject s_feature_list_kobj;             /* 
/sys/fs/f2fs/<devname>/feature_list */
+       struct completion s_feature_list_kobj_unregister;
+
        /* For shrinker support */
        struct list_head s_list;
        int s_ndevs;                            /* number of devices */
diff --git a/fs/f2fs/sysfs.c b/fs/f2fs/sysfs.c
index 2a76c959a7b4..f1d8b151cc36 100644
--- a/fs/f2fs/sysfs.c
+++ b/fs/f2fs/sysfs.c
@@ -569,6 +569,20 @@ static void f2fs_sb_release(struct kobject *kobj)
        complete(&sbi->s_kobj_unregister);
 }
 
+/*
+ * Note that there are three feature list entries:
+ * 1) /sys/fs/f2fs/features
+ *   : shows runtime features supported by in-kernel f2fs along with Kconfig
+ *     - ref. F2FS_FEATURE_RO_ATTR()
+ *
+ * 2) /sys/fs/f2fs/$s_id/features <deprecated>
+ *   : shows on-disk features enabled by mkfs.f2fs, used for old kernels
+ *
+ * 3) /sys/fs/f2fs/$s_id/feature_list
+ *   : shows on-disk features enabled by mkfs.f2fs per instance.
+ *     this list covers old feature list provided by 2) and beyond
+ *     - ref. F2FS_SB_FEATURE_RO_ATTR()
+ */
 enum feat_id {
        FEAT_CRYPTO = 0,
        FEAT_BLKZONED,
@@ -587,6 +601,7 @@ enum feat_id {
        FEAT_RO,
        FEAT_TEST_DUMMY_ENCRYPTION_V2,
        FEAT_ENCRYPTED_CASEFOLD,
+       FEAT_PIN_FILE,
 };
 
 static ssize_t f2fs_feature_show(struct f2fs_attr *a,
@@ -610,6 +625,7 @@ static ssize_t f2fs_feature_show(struct f2fs_attr *a,
        case FEAT_RO:
        case FEAT_TEST_DUMMY_ENCRYPTION_V2:
        case FEAT_ENCRYPTED_CASEFOLD:
+       case FEAT_PIN_FILE:
                return sprintf(buf, "supported\n");
        }
        return 0;
@@ -743,6 +759,7 @@ F2FS_RW_ATTR(F2FS_SBI, f2fs_sb_info, compr_written_block, 
compr_written_block);
 F2FS_RW_ATTR(F2FS_SBI, f2fs_sb_info, compr_saved_block, compr_saved_block);
 F2FS_RW_ATTR(F2FS_SBI, f2fs_sb_info, compr_new_inode, compr_new_inode);
 #endif
+F2FS_FEATURE_RO_ATTR(pin_file, FEAT_PIN_FILE);
 
 /* For ATGC */
 F2FS_RW_ATTR(ATGC_INFO, atgc_management, atgc_candidate_ratio, 
candidate_ratio);
@@ -856,6 +873,7 @@ static struct attribute *f2fs_feat_attrs[] = {
 #ifdef CONFIG_F2FS_FS_COMPRESSION
        ATTR_LIST(compression),
 #endif
+       ATTR_LIST(pin_file),
        NULL,
 };
 ATTRIBUTE_GROUPS(f2fs_feat);
@@ -867,6 +885,64 @@ static struct attribute *f2fs_stat_attrs[] = {
 };
 ATTRIBUTE_GROUPS(f2fs_stat);
 
+static ssize_t f2fs_sb_feature_show(struct f2fs_attr *a,
+               struct f2fs_sb_info *sbi, char *buf)
+{
+       if (F2FS_MATCH_FEATURE(sbi, a->id))
+               return sprintf(buf, "supported\n");
+       return sprintf(buf, "unsupported\n");
+}
+
+#define F2FS_SB_FEATURE_RO_ATTR(_name, _feat)                  \
+static struct f2fs_attr f2fs_attr_sb_##_name = {               \
+       .attr = {.name = __stringify(_name), .mode = 0444 },    \
+       .show   = f2fs_sb_feature_show,                         \
+       .id     = F2FS_FEATURE_##_feat,                         \
+}
+
+#define F2FS_SB_FEATURE_RO_ATTR2(_name, _feat1, _feat2)                \
+static struct f2fs_attr f2fs_attr_sb_##_name = {               \
+       .attr = {.name = __stringify(_name), .mode = 0444 },    \
+       .show   = f2fs_sb_feature_show,                         \
+       .id     = F2FS_FEATURE_##_feat1 | F2FS_FEATURE_##_feat2,\
+}
+
+F2FS_SB_FEATURE_RO_ATTR(encryption, ENCRYPT);
+F2FS_SB_FEATURE_RO_ATTR(block_zoned, BLKZONED);
+F2FS_SB_FEATURE_RO_ATTR(extra_attr, EXTRA_ATTR);
+F2FS_SB_FEATURE_RO_ATTR(project_quota, PRJQUOTA);
+F2FS_SB_FEATURE_RO_ATTR(inode_checksum, INODE_CHKSUM);
+F2FS_SB_FEATURE_RO_ATTR(flexible_inline_xattr, FLEXIBLE_INLINE_XATTR);
+F2FS_SB_FEATURE_RO_ATTR(quota_ino, QUOTA_INO);
+F2FS_SB_FEATURE_RO_ATTR(inode_crtime, INODE_CRTIME);
+F2FS_SB_FEATURE_RO_ATTR(lost_found, LOST_FOUND);
+F2FS_SB_FEATURE_RO_ATTR(verity, VERITY);
+F2FS_SB_FEATURE_RO_ATTR(sb_checksum, SB_CHKSUM);
+F2FS_SB_FEATURE_RO_ATTR(casefold, CASEFOLD);
+F2FS_SB_FEATURE_RO_ATTR(compression, COMPRESSION);
+F2FS_SB_FEATURE_RO_ATTR(readonly, RO);
+F2FS_SB_FEATURE_RO_ATTR2(encrypted_casefold, ENCRYPT, CASEFOLD);
+
+static struct attribute *f2fs_sb_feat_attrs[] = {
+       ATTR_LIST(sb_encryption),
+       ATTR_LIST(sb_block_zoned),
+       ATTR_LIST(sb_extra_attr),
+       ATTR_LIST(sb_project_quota),
+       ATTR_LIST(sb_inode_checksum),
+       ATTR_LIST(sb_flexible_inline_xattr),
+       ATTR_LIST(sb_quota_ino),
+       ATTR_LIST(sb_inode_crtime),
+       ATTR_LIST(sb_lost_found),
+       ATTR_LIST(sb_verity),
+       ATTR_LIST(sb_sb_checksum),
+       ATTR_LIST(sb_casefold),
+       ATTR_LIST(sb_compression),
+       ATTR_LIST(sb_readonly),
+       ATTR_LIST(sb_encrypted_casefold),
+       NULL,
+};
+ATTRIBUTE_GROUPS(f2fs_sb_feat);
+
 static const struct sysfs_ops f2fs_attr_ops = {
        .show   = f2fs_attr_show,
        .store  = f2fs_attr_store,
@@ -933,6 +1009,34 @@ static struct kobj_type f2fs_stat_ktype = {
        .release        = f2fs_stat_kobj_release,
 };
 
+static ssize_t f2fs_sb_feat_attr_show(struct kobject *kobj,
+                               struct attribute *attr, char *buf)
+{
+       struct f2fs_sb_info *sbi = container_of(kobj, struct f2fs_sb_info,
+                                                       s_feature_list_kobj);
+       struct f2fs_attr *a = container_of(attr, struct f2fs_attr, attr);
+
+       return a->show ? a->show(a, sbi, buf) : 0;
+}
+
+static void f2fs_feature_list_kobj_release(struct kobject *kobj)
+{
+       struct f2fs_sb_info *sbi = container_of(kobj, struct f2fs_sb_info,
+                                                       s_feature_list_kobj);
+       complete(&sbi->s_feature_list_kobj_unregister);
+}
+
+static const struct sysfs_ops f2fs_feature_list_attr_ops = {
+       .show   = f2fs_sb_feat_attr_show,
+};
+
+static struct kobj_type f2fs_feature_list_ktype = {
+       .default_groups = f2fs_sb_feat_groups,
+       .sysfs_ops      = &f2fs_feature_list_attr_ops,
+       .release        = f2fs_feature_list_kobj_release,
+};
+
+
 static int __maybe_unused segment_info_seq_show(struct seq_file *seq,
                                                void *offset)
 {
@@ -1149,6 +1253,14 @@ int f2fs_register_sysfs(struct f2fs_sb_info *sbi)
        if (err)
                goto put_stat_kobj;
 
+       sbi->s_feature_list_kobj.kset = &f2fs_kset;
+       init_completion(&sbi->s_feature_list_kobj_unregister);
+       err = kobject_init_and_add(&sbi->s_feature_list_kobj,
+                                       &f2fs_feature_list_ktype,
+                                       &sbi->s_kobj, "feature_list");
+       if (err)
+               goto put_feature_list_kobj;
+
        if (f2fs_proc_root)
                sbi->s_proc = proc_mkdir(sb->s_id, f2fs_proc_root);
 
@@ -1163,6 +1275,9 @@ int f2fs_register_sysfs(struct f2fs_sb_info *sbi)
                                victim_bits_seq_show, sb);
        }
        return 0;
+put_feature_list_kobj:
+       kobject_put(&sbi->s_feature_list_kobj);
+       wait_for_completion(&sbi->s_feature_list_kobj_unregister);
 put_stat_kobj:
        kobject_put(&sbi->s_stat_kobj);
        wait_for_completion(&sbi->s_stat_kobj_unregister);
@@ -1185,6 +1300,9 @@ void f2fs_unregister_sysfs(struct f2fs_sb_info *sbi)
        kobject_del(&sbi->s_stat_kobj);
        kobject_put(&sbi->s_stat_kobj);
        wait_for_completion(&sbi->s_stat_kobj_unregister);
+       kobject_del(&sbi->s_feature_list_kobj);
+       kobject_put(&sbi->s_feature_list_kobj);
+       wait_for_completion(&sbi->s_feature_list_kobj_unregister);
 
        kobject_del(&sbi->s_kobj);
        kobject_put(&sbi->s_kobj);
-- 
2.32.0.rc1.229.g3e70b5a671-goog



_______________________________________________
Linux-f2fs-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel

Reply via email to