[PATCH v2] ubifs: use default_groups in kobj_type

Greg Kroah-Hartman posted 1 patch 4 years, 5 months ago
fs/ubifs/sysfs.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
[PATCH v2] ubifs: use default_groups in kobj_type
Posted by Greg Kroah-Hartman 4 years, 5 months ago
There are currently 2 ways to create a set of sysfs files for a
kobj_type, through the default_attrs field, and the default_groups
field.  Move the ubifs sysfs code to use default_groups field which has
been the preferred way since aa30f47cf666 ("kobject: Add support for
default attribute groups to kobj_type") so that we can soon get rid of
the obsolete default_attrs field.

Cc: Richard Weinberger <richard@nod.at>
Cc: Stefan Schaeckeler <schaecsn@gmx.net>
Cc: linux-mtd@lists.infradead.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
---
v2: fix up the changelog text to properly refer to the ubifs code
    instead of some other subsystem as pointed out by Michael Walle

 fs/ubifs/sysfs.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/fs/ubifs/sysfs.c b/fs/ubifs/sysfs.c
index 7acc5a74e5fa..06ad8fa1fcfb 100644
--- a/fs/ubifs/sysfs.c
+++ b/fs/ubifs/sysfs.c
@@ -42,6 +42,7 @@ static struct attribute *ubifs_attrs[] = {
 	ATTR_LIST(errors_crc),
 	NULL,
 };
+ATTRIBUTE_GROUPS(ubifs);
 
 static ssize_t ubifs_attr_show(struct kobject *kobj,
 			       struct attribute *attr, char *buf)
@@ -74,7 +75,7 @@ static const struct sysfs_ops ubifs_attr_ops = {
 };
 
 static struct kobj_type ubifs_sb_ktype = {
-	.default_attrs	= ubifs_attrs,
+	.default_groups	= ubifs_groups,
 	.sysfs_ops	= &ubifs_attr_ops,
 	.release	= ubifs_sb_release,
 };
-- 
2.34.1

Re: [PATCH v2] ubifs: use default_groups in kobj_type
Posted by Richard Weinberger 4 years, 5 months ago
----- Ursprüngliche Mail -----
> Von: "Greg Kroah-Hartman" <gregkh@linuxfoundation.org>
> An: "linux-kernel" <linux-kernel@vger.kernel.org>
> CC: "Greg Kroah-Hartman" <gregkh@linuxfoundation.org>, "richard" <richard@nod.at>, "schaecsn" <schaecsn@gmx.net>,
> "linux-mtd" <linux-mtd@lists.infradead.org>
> Gesendet: Freitag, 14. Januar 2022 11:48:20
> Betreff: [PATCH v2] ubifs: use default_groups in kobj_type

> There are currently 2 ways to create a set of sysfs files for a
> kobj_type, through the default_attrs field, and the default_groups
> field.  Move the ubifs sysfs code to use default_groups field which has
> been the preferred way since aa30f47cf666 ("kobject: Add support for
> default attribute groups to kobj_type") so that we can soon get rid of
> the obsolete default_attrs field.
> 
> Cc: Richard Weinberger <richard@nod.at>
> Cc: Stefan Schaeckeler <schaecsn@gmx.net>
> Cc: linux-mtd@lists.infradead.org
> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
> ---
> v2: fix up the changelog text to properly refer to the ubifs code
>    instead of some other subsystem as pointed out by Michael Walle

Acked-by: Richard Weinberger <richard@nod.at>

Thanks,
//richard