NFS: Open-code the nfs_kset kset_create_and_add()
authorBenjamin Coddington <bcodding@redhat.com>
Thu, 15 Jun 2023 18:07:24 +0000 (14:07 -0400)
committerTrond Myklebust <trond.myklebust@hammerspace.com>
Mon, 19 Jun 2023 18:59:07 +0000 (14:59 -0400)
In preparation to make objects below /sys/fs/nfs namespace aware, we need
to define our own kobj_type for the nfs kset so that we can add the
.child_ns_type member in a following patch.  No functional change here, only
the unrolling of kset_create_and_add().

Signed-off-by: Benjamin Coddington <bcodding@redhat.com>
Signed-off-by: Trond Myklebust <trond.myklebust@hammerspace.com>
fs/nfs/sysfs.c

index 8f89aeca6272b8aa1b978784bc7687b694b45ef7..9adb8ac08d9aadce63edc94e8547c53deaa90313 100644 (file)
@@ -25,12 +25,23 @@ static void nfs_netns_object_release(struct kobject *kobj)
        kfree(kobj);
 }
 
+static void nfs_kset_release(struct kobject *kobj)
+{
+       struct kset *kset = container_of(kobj, struct kset, kobj);
+       kfree(kset);
+}
+
 static const struct kobj_ns_type_operations *nfs_netns_object_child_ns_type(
                const struct kobject *kobj)
 {
        return &net_ns_type_operations;
 }
 
+static struct kobj_type nfs_kset_type = {
+       .release = nfs_kset_release,
+       .sysfs_ops = &kobj_sysfs_ops,
+};
+
 static struct kobj_type nfs_netns_object_type = {
        .release = nfs_netns_object_release,
        .sysfs_ops = &kobj_sysfs_ops,
@@ -55,13 +66,32 @@ static struct kobject *nfs_netns_object_alloc(const char *name,
 
 int nfs_sysfs_init(void)
 {
-       nfs_kset = kset_create_and_add("nfs", NULL, fs_kobj);
+       int ret;
+
+       nfs_kset = kzalloc(sizeof(*nfs_kset), GFP_KERNEL);
        if (!nfs_kset)
                return -ENOMEM;
+
+       ret = kobject_set_name(&nfs_kset->kobj, "nfs");
+       if (ret) {
+               kfree(nfs_kset);
+               return ret;
+       }
+
+       nfs_kset->kobj.parent = fs_kobj;
+       nfs_kset->kobj.ktype = &nfs_kset_type;
+       nfs_kset->kobj.kset = NULL;
+
+       ret = kset_register(nfs_kset);
+       if (ret) {
+               kfree(nfs_kset);
+               return ret;
+       }
+
        nfs_net_kobj = nfs_netns_object_alloc("net", nfs_kset, NULL);
        if (!nfs_net_kobj) {
                kset_unregister(nfs_kset);
-               nfs_kset = NULL;
+               kfree(nfs_kset);
                return -ENOMEM;
        }
        return 0;