Commit 901195ed authored by Greg Kroah-Hartman's avatar Greg Kroah-Hartman

Kobject: change GFS2 to use kobject_init_and_add

Stop using kobject_register, as this way we can control the sending of
the uevent properly, after everything is properly initialized.

Cc: Steven Whitehouse <swhiteho@redhat.com>
Cc: Kay Sievers <kay.sievers@vrfy.org>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@suse.de>
parent 43b98c4a
...@@ -168,23 +168,6 @@ static struct kobj_type dlm_ktype = { ...@@ -168,23 +168,6 @@ static struct kobj_type dlm_ktype = {
static struct kset *dlm_kset; static struct kset *dlm_kset;
static int kobject_setup(struct dlm_ls *ls)
{
char lsname[DLM_LOCKSPACE_LEN];
int error;
memset(lsname, 0, DLM_LOCKSPACE_LEN);
snprintf(lsname, DLM_LOCKSPACE_LEN, "%s", ls->ls_name);
error = kobject_set_name(&ls->ls_kobj, "%s", lsname);
if (error)
return error;
ls->ls_kobj.kset = dlm_kset;
ls->ls_kobj.ktype = &dlm_ktype;
return 0;
}
static int do_uevent(struct dlm_ls *ls, int in) static int do_uevent(struct dlm_ls *ls, int in)
{ {
int error; int error;
...@@ -545,13 +528,12 @@ static int new_lockspace(char *name, int namelen, void **lockspace, ...@@ -545,13 +528,12 @@ static int new_lockspace(char *name, int namelen, void **lockspace,
goto out_delist; goto out_delist;
} }
error = kobject_setup(ls); ls->ls_kobj.kset = dlm_kset;
if (error) error = kobject_init_and_add(&ls->ls_kobj, &dlm_ktype, NULL,
goto out_stop; "%s", ls->ls_name);
error = kobject_register(&ls->ls_kobj);
if (error) if (error)
goto out_stop; goto out_stop;
kobject_uevent(&ls->ls_kobj, KOBJ_ADD);
/* let kobject handle freeing of ls if there's an error */ /* let kobject handle freeing of ls if there's an error */
do_unreg = 1; do_unreg = 1;
......
...@@ -195,19 +195,12 @@ int gdlm_kobject_setup(struct gdlm_ls *ls, struct kobject *fskobj) ...@@ -195,19 +195,12 @@ int gdlm_kobject_setup(struct gdlm_ls *ls, struct kobject *fskobj)
{ {
int error; int error;
error = kobject_set_name(&ls->kobj, "%s", "lock_module");
if (error) {
log_error("can't set kobj name %d", error);
return error;
}
ls->kobj.kset = gdlm_kset; ls->kobj.kset = gdlm_kset;
ls->kobj.ktype = &gdlm_ktype; error = kobject_init_and_add(&ls->kobj, &gdlm_ktype, fskobj,
ls->kobj.parent = fskobj; "lock_module");
error = kobject_register(&ls->kobj);
if (error) if (error)
log_error("can't register kobj %d", error); log_error("can't register kobj %d", error);
kobject_uevent(&ls->kobj, KOBJ_ADD);
return error; return error;
} }
......
...@@ -494,13 +494,8 @@ int gfs2_sys_fs_add(struct gfs2_sbd *sdp) ...@@ -494,13 +494,8 @@ int gfs2_sys_fs_add(struct gfs2_sbd *sdp)
int error; int error;
sdp->sd_kobj.kset = gfs2_kset; sdp->sd_kobj.kset = gfs2_kset;
sdp->sd_kobj.ktype = &gfs2_ktype; error = kobject_init_and_add(&sdp->sd_kobj, &gfs2_ktype, NULL,
"%s", sdp->sd_table_name);
error = kobject_set_name(&sdp->sd_kobj, "%s", sdp->sd_table_name);
if (error)
goto fail;
error = kobject_register(&sdp->sd_kobj);
if (error) if (error)
goto fail; goto fail;
...@@ -520,6 +515,7 @@ int gfs2_sys_fs_add(struct gfs2_sbd *sdp) ...@@ -520,6 +515,7 @@ int gfs2_sys_fs_add(struct gfs2_sbd *sdp)
if (error) if (error)
goto fail_args; goto fail_args;
kobject_uevent(&sdp->sd_kobj, KOBJ_ADD);
return 0; return 0;
fail_args: fail_args:
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment