The reduced member_sysfs.c is no longer related to lockspace members.
Move what's left into lockspace.c which is the only file that uses the
remaining functions.

Signed-off-by: David Teigland <[EMAIL PROTECTED]>

---

 Makefile       |    1 
 lockspace.c    |  155 +++++++++++++++++++++++++++++++++++++++++++++++++++--
 lockspace.h    |    1 
 main.c         |   14 +---
 member.c       |    2 
 member_sysfs.c |  165 ---------------------------------------------------------
 member_sysfs.h |   22 -------
 7 files changed, 156 insertions(+), 204 deletions(-)

diff -urpN a/drivers/dlm/Makefile b/drivers/dlm/Makefile
--- a/drivers/dlm/Makefile      2005-08-18 13:26:02.648375344 +0800
+++ b/drivers/dlm/Makefile      2005-08-18 13:26:25.736865360 +0800
@@ -9,7 +9,6 @@ dlm-y :=                        ast.o \
                                lowcomms.o \
                                main.o \
                                member.o \
-                               member_sysfs.o \
                                memory.o \
                                midcomms.o \
                                rcom.o \
diff -urpN a/drivers/dlm/lockspace.c b/drivers/dlm/lockspace.c
--- a/drivers/dlm/lockspace.c   2005-08-18 13:26:02.651374888 +0800
+++ b/drivers/dlm/lockspace.c   2005-08-18 13:26:25.737865208 +0800
@@ -14,7 +14,6 @@
 #include "dlm_internal.h"
 #include "lockspace.h"
 #include "member.h"
-#include "member_sysfs.h"
 #include "recoverd.h"
 #include "ast.h"
 #include "dir.h"
@@ -38,13 +37,159 @@ static spinlock_t          lslist_lock;
 static struct task_struct *    scand_task;
 
 
+static ssize_t dlm_control_store(struct dlm_ls *ls, const char *buf, size_t 
len)
+{
+       ssize_t ret = len;
+       int n = simple_strtol(buf, NULL, 0);
+
+       switch (n) {
+       case 0:
+               dlm_ls_stop(ls);
+               break;
+       case 1:
+               dlm_ls_start(ls);
+               break;
+       default:
+               ret = -EINVAL;
+       }
+       return ret;
+}
+
+static ssize_t dlm_event_store(struct dlm_ls *ls, const char *buf, size_t len)
+{
+       ls->ls_uevent_result = simple_strtol(buf, NULL, 0);
+       set_bit(LSFL_UEVENT_WAIT, &ls->ls_flags);
+       wake_up(&ls->ls_uevent_wait);
+       return len;
+}
+
+static ssize_t dlm_id_show(struct dlm_ls *ls, char *buf)
+{
+       return sprintf(buf, "%u\n", ls->ls_global_id);
+}
+
+static ssize_t dlm_id_store(struct dlm_ls *ls, const char *buf, size_t len)
+{
+       ls->ls_global_id = simple_strtoul(buf, NULL, 0);
+       return len;
+}
+
+struct dlm_attr {
+       struct attribute attr;
+       ssize_t (*show)(struct dlm_ls *, char *);
+       ssize_t (*store)(struct dlm_ls *, const char *, size_t);
+};
+
+static struct dlm_attr dlm_attr_control = {
+       .attr  = {.name = "control", .mode = S_IWUSR},
+       .store = dlm_control_store
+};
+
+static struct dlm_attr dlm_attr_event = {
+       .attr  = {.name = "event_done", .mode = S_IWUSR},
+       .store = dlm_event_store
+};
+
+static struct dlm_attr dlm_attr_id = {
+       .attr  = {.name = "id", .mode = S_IRUGO | S_IWUSR},
+       .show  = dlm_id_show,
+       .store = dlm_id_store
+};
+
+static struct attribute *dlm_attrs[] = {
+       &dlm_attr_control.attr,
+       &dlm_attr_event.attr,
+       &dlm_attr_id.attr,
+       NULL,
+};
+
+static ssize_t dlm_attr_show(struct kobject *kobj, struct attribute *attr,
+                            char *buf)
+{
+       struct dlm_ls *ls  = container_of(kobj, struct dlm_ls, ls_kobj);
+       struct dlm_attr *a = container_of(attr, struct dlm_attr, attr);
+       return a->show ? a->show(ls, buf) : 0;
+}
+
+static ssize_t dlm_attr_store(struct kobject *kobj, struct attribute *attr,
+                             const char *buf, size_t len)
+{
+       struct dlm_ls *ls  = container_of(kobj, struct dlm_ls, ls_kobj);
+       struct dlm_attr *a = container_of(attr, struct dlm_attr, attr);
+       return a->store ? a->store(ls, buf, len) : len;
+}
+
+static struct sysfs_ops dlm_attr_ops = {
+       .show  = dlm_attr_show,
+       .store = dlm_attr_store,
+};
+
+static struct kobj_type dlm_ktype = {
+       .default_attrs = dlm_attrs,
+       .sysfs_ops     = &dlm_attr_ops,
+};
+
+static struct kset dlm_kset = {
+       .subsys = &kernel_subsys,
+       .kobj   = {.name = "dlm",},
+       .ktype  = &dlm_ktype,
+};
+
+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)
+{
+       int error;
+
+       if (in)
+               kobject_uevent(&ls->ls_kobj, KOBJ_ONLINE, NULL);
+       else
+               kobject_uevent(&ls->ls_kobj, KOBJ_OFFLINE, NULL);
+
+       error = wait_event_interruptible(ls->ls_uevent_wait,
+                       test_and_clear_bit(LSFL_UEVENT_WAIT, &ls->ls_flags));
+       if (error)
+               goto out;
+
+       error = ls->ls_uevent_result;
+ out:
+       return error;
+}
+
+
 int dlm_lockspace_init(void)
 {
+       int error;
+
        ls_count = 0;
        init_MUTEX(&ls_lock);
        INIT_LIST_HEAD(&lslist);
        spin_lock_init(&lslist_lock);
-       return 0;
+
+       error = kset_register(&dlm_kset);
+       if (error)
+               printk("dlm_lockspace_init: cannot register kset %d\n", error);
+       return error;
+}
+
+void dlm_lockspace_exit(void)
+{
+       kset_unregister(&dlm_kset);
 }
 
 static int dlm_scand(void *data)
@@ -310,7 +455,7 @@ static int new_lockspace(char *name, int
 
        dlm_create_debug_file(ls);
 
-       error = dlm_kobject_setup(ls);
+       error = kobject_setup(ls);
        if (error)
                goto out_del;
 
@@ -318,7 +463,7 @@ static int new_lockspace(char *name, int
        if (error)
                goto out_del;
 
-       error = dlm_uevent(ls, 1);
+       error = do_uevent(ls, 1);
        if (error)
                goto out_unreg;
 
@@ -409,7 +554,7 @@ static int release_lockspace(struct dlm_
                return -EBUSY;
 
        if (force < 3)
-               dlm_uevent(ls, 0);
+               do_uevent(ls, 0);
 
        dlm_recoverd_stop(ls);
 
diff -urpN a/drivers/dlm/lockspace.h b/drivers/dlm/lockspace.h
--- a/drivers/dlm/lockspace.h   2005-08-17 17:19:22.000000000 +0800
+++ b/drivers/dlm/lockspace.h   2005-08-18 13:26:25.737865208 +0800
@@ -15,6 +15,7 @@
 #define __LOCKSPACE_DOT_H__
 
 int dlm_lockspace_init(void);
+void dlm_lockspace_exit(void);
 struct dlm_ls *dlm_find_lockspace_global(uint32_t id);
 struct dlm_ls *dlm_find_lockspace_local(void *id);
 struct dlm_ls *dlm_find_lockspace_name(char *name, int namelen);
diff -urpN a/drivers/dlm/main.c b/drivers/dlm/main.c
--- a/drivers/dlm/main.c        2005-08-18 13:26:02.653374584 +0800
+++ b/drivers/dlm/main.c        2005-08-18 13:26:25.738865056 +0800
@@ -13,9 +13,7 @@
 
 #include "dlm_internal.h"
 #include "lockspace.h"
-#include "member_sysfs.h"
 #include "lock.h"
-#include "device.h"
 #include "memory.h"
 #include "lowcomms.h"
 #include "config.h"
@@ -40,13 +38,9 @@ static int __init init_dlm(void)
        if (error)
                goto out_mem;
 
-       error = dlm_member_sysfs_init();
-       if (error)
-               goto out_mem;
-
        error = dlm_config_init();
        if (error)
-               goto out_member;
+               goto out_lockspace;
 
        error = dlm_register_debugfs();
        if (error)
@@ -64,8 +58,8 @@ static int __init init_dlm(void)
        dlm_unregister_debugfs();
  out_config:
        dlm_config_exit();
- out_member:
-       dlm_member_sysfs_exit();
+ out_lockspace:
+       dlm_lockspace_exit();
  out_mem:
        dlm_memory_exit();
  out:
@@ -75,9 +69,9 @@ static int __init init_dlm(void)
 static void __exit exit_dlm(void)
 {
        dlm_lowcomms_exit();
-       dlm_member_sysfs_exit();
        dlm_config_exit();
        dlm_memory_exit();
+       dlm_lockspace_exit();
        dlm_unregister_debugfs();
 }
 
diff -urpN a/drivers/dlm/member.c b/drivers/dlm/member.c
--- a/drivers/dlm/member.c      2005-08-18 13:26:02.654374432 +0800
+++ b/drivers/dlm/member.c      2005-08-18 13:26:25.738865056 +0800
@@ -221,7 +221,7 @@ int dlm_recover_members(struct dlm_ls *l
 }
 
 /*
- * Following called from member_sysfs.c
+ * Following called from lockspace.c
  */
 
 int dlm_ls_stop(struct dlm_ls *ls)
diff -urpN a/drivers/dlm/member_sysfs.c b/drivers/dlm/member_sysfs.c
--- a/drivers/dlm/member_sysfs.c        2005-08-18 13:26:02.655374280 +0800
+++ b/drivers/dlm/member_sysfs.c        1970-01-01 07:30:00.000000000 +0730
@@ -1,165 +0,0 @@
-/******************************************************************************
-*******************************************************************************
-**
-**  Copyright (C) 2005 Red Hat, Inc.  All rights reserved.
-**
-**  This copyrighted material is made available to anyone wishing to use,
-**  modify, copy, or redistribute it subject to the terms and conditions
-**  of the GNU General Public License v.2.
-**
-*******************************************************************************
-******************************************************************************/
-
-#include "dlm_internal.h"
-#include "member.h"
-
-
-static ssize_t dlm_control_store(struct dlm_ls *ls, const char *buf, size_t 
len)
-{
-       ssize_t ret = len;
-       int n = simple_strtol(buf, NULL, 0);
-
-       switch (n) {
-       case 0:
-               dlm_ls_stop(ls);
-               break;
-       case 1:
-               dlm_ls_start(ls);
-               break;
-       default:
-               ret = -EINVAL;
-       }
-       return ret;
-}
-
-static ssize_t dlm_event_store(struct dlm_ls *ls, const char *buf, size_t len)
-{
-       ls->ls_uevent_result = simple_strtol(buf, NULL, 0);
-       set_bit(LSFL_UEVENT_WAIT, &ls->ls_flags);
-       wake_up(&ls->ls_uevent_wait);
-       return len;
-}
-
-static ssize_t dlm_id_show(struct dlm_ls *ls, char *buf)
-{
-       return sprintf(buf, "%u\n", ls->ls_global_id);
-}
-
-static ssize_t dlm_id_store(struct dlm_ls *ls, const char *buf, size_t len)
-{
-       ls->ls_global_id = simple_strtoul(buf, NULL, 0);
-       return len;
-}
-
-struct dlm_attr {
-       struct attribute attr;
-       ssize_t (*show)(struct dlm_ls *, char *);
-       ssize_t (*store)(struct dlm_ls *, const char *, size_t);
-};
-
-static struct dlm_attr dlm_attr_control = {
-       .attr  = {.name = "control", .mode = S_IWUSR},
-       .store = dlm_control_store
-};
-
-static struct dlm_attr dlm_attr_event = {
-       .attr  = {.name = "event_done", .mode = S_IWUSR},
-       .store = dlm_event_store
-};
-
-static struct dlm_attr dlm_attr_id = {
-       .attr  = {.name = "id", .mode = S_IRUGO | S_IWUSR},
-       .show  = dlm_id_show,
-       .store = dlm_id_store
-};
-
-static struct attribute *dlm_attrs[] = {
-       &dlm_attr_control.attr,
-       &dlm_attr_event.attr,
-       &dlm_attr_id.attr,
-       NULL,
-};
-
-static ssize_t dlm_attr_show(struct kobject *kobj, struct attribute *attr,
-                            char *buf)
-{
-       struct dlm_ls *ls  = container_of(kobj, struct dlm_ls, ls_kobj);
-       struct dlm_attr *a = container_of(attr, struct dlm_attr, attr);
-       return a->show ? a->show(ls, buf) : 0;
-}
-
-static ssize_t dlm_attr_store(struct kobject *kobj, struct attribute *attr,
-                             const char *buf, size_t len)
-{
-       struct dlm_ls *ls  = container_of(kobj, struct dlm_ls, ls_kobj);
-       struct dlm_attr *a = container_of(attr, struct dlm_attr, attr);
-       return a->store ? a->store(ls, buf, len) : len;
-}
-
-static struct sysfs_ops dlm_attr_ops = {
-       .show  = dlm_attr_show,
-       .store = dlm_attr_store,
-};
-
-static struct kobj_type dlm_ktype = {
-       .default_attrs = dlm_attrs,
-       .sysfs_ops     = &dlm_attr_ops,
-};
-
-static struct kset dlm_kset = {
-       .subsys = &kernel_subsys,
-       .kobj   = {.name = "dlm",},
-       .ktype  = &dlm_ktype,
-};
-
-int dlm_member_sysfs_init(void)
-{
-       int error;
-
-       error = kset_register(&dlm_kset);
-       if (error)
-               printk("dlm_lockspace_init: cannot register kset %d\n", error);
-       return error;
-}
-
-void dlm_member_sysfs_exit(void)
-{
-       kset_unregister(&dlm_kset);
-}
-
-int dlm_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;
-}
-
-int dlm_uevent(struct dlm_ls *ls, int in)
-{
-       int error;
-
-       if (in)
-               kobject_uevent(&ls->ls_kobj, KOBJ_ONLINE, NULL);
-       else
-               kobject_uevent(&ls->ls_kobj, KOBJ_OFFLINE, NULL);
-
-       error = wait_event_interruptible(ls->ls_uevent_wait,
-                       test_and_clear_bit(LSFL_UEVENT_WAIT, &ls->ls_flags));
-       if (error)
-               goto out;
-
-       error = ls->ls_uevent_result;
- out:
-       return error;
-}
-
diff -urpN a/drivers/dlm/member_sysfs.h b/drivers/dlm/member_sysfs.h
--- a/drivers/dlm/member_sysfs.h        2005-08-17 17:19:22.000000000 +0800
+++ b/drivers/dlm/member_sysfs.h        1970-01-01 07:30:00.000000000 +0730
@@ -1,22 +0,0 @@
-/******************************************************************************
-*******************************************************************************
-**
-**  Copyright (C) 2005 Red Hat, Inc.  All rights reserved.
-**
-**  This copyrighted material is made available to anyone wishing to use,
-**  modify, copy, or redistribute it subject to the terms and conditions
-**  of the GNU General Public License v.2.
-**
-*******************************************************************************
-******************************************************************************/
-
-#ifndef __MEMBER_SYSFS_DOT_H__
-#define __MEMBER_SYSFS_DOT_H__
-
-int dlm_member_sysfs_init(void);
-void dlm_member_sysfs_exit(void);
-int dlm_kobject_setup(struct dlm_ls *ls);
-int dlm_uevent(struct dlm_ls *ls, int in);
-
-#endif                          /* __MEMBER_SYSFS_DOT_H__ */
-
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to