diff options
author | Adrian Bunk <bunk@stusta.de> | 2005-12-19 14:16:07 -0500 |
---|---|---|
committer | Joel Becker <joel.becker@oracle.com> | 2006-01-03 14:45:55 -0500 |
commit | 82353b594c784deabb8d9764b477e65c2b3726f9 (patch) | |
tree | 9182f0ac8aeca6ab49953da7c346a74d5853bd94 | |
parent | b4e40a51881931bfcbc78a585e875bb2784d6d10 (diff) |
[PATCH] This patch contains the following cleanups:
- cluster/sys.c: make needlessly global code static
- dlm/: "extern" declarations for variables belong into header files
(and in this case, they are already in dlmdomain.h)
Signed-off-by: Adrian Bunk <bunk@stusta.de>
Signed-off-by: Mark Fasheh <mark.fasheh@oracle.com>
-rw-r--r-- | fs/ocfs2/cluster/sys.c | 4 | ||||
-rw-r--r-- | fs/ocfs2/dlm/dlmmaster.c | 4 | ||||
-rw-r--r-- | fs/ocfs2/dlm/dlmthread.c | 3 |
3 files changed, 3 insertions, 8 deletions
diff --git a/fs/ocfs2/cluster/sys.c b/fs/ocfs2/cluster/sys.c index f1e99461bb0d..1d9f6acafa2e 100644 --- a/fs/ocfs2/cluster/sys.c +++ b/fs/ocfs2/cluster/sys.c | |||
@@ -50,7 +50,7 @@ static ssize_t o2cb_interface_revision_show(char *buf) | |||
50 | return snprintf(buf, PAGE_SIZE, "%u\n", O2NM_API_VERSION); | 50 | return snprintf(buf, PAGE_SIZE, "%u\n", O2NM_API_VERSION); |
51 | } | 51 | } |
52 | 52 | ||
53 | O2CB_ATTR(interface_revision, S_IFREG | S_IRUGO, o2cb_interface_revision_show, NULL); | 53 | static O2CB_ATTR(interface_revision, S_IFREG | S_IRUGO, o2cb_interface_revision_show, NULL); |
54 | 54 | ||
55 | static struct attribute *o2cb_attrs[] = { | 55 | static struct attribute *o2cb_attrs[] = { |
56 | &o2cb_attr_interface_revision.attr, | 56 | &o2cb_attr_interface_revision.attr, |
@@ -73,7 +73,7 @@ static struct kobj_type o2cb_subsys_type = { | |||
73 | }; | 73 | }; |
74 | 74 | ||
75 | /* gives us o2cb_subsys */ | 75 | /* gives us o2cb_subsys */ |
76 | decl_subsys(o2cb, NULL, NULL); | 76 | static decl_subsys(o2cb, NULL, NULL); |
77 | 77 | ||
78 | static ssize_t | 78 | static ssize_t |
79 | o2cb_show(struct kobject * kobj, struct attribute * attr, char * buffer) | 79 | o2cb_show(struct kobject * kobj, struct attribute * attr, char * buffer) |
diff --git a/fs/ocfs2/dlm/dlmmaster.c b/fs/ocfs2/dlm/dlmmaster.c index 047279546b4f..27e984f7e4cd 100644 --- a/fs/ocfs2/dlm/dlmmaster.c +++ b/fs/ocfs2/dlm/dlmmaster.c | |||
@@ -48,6 +48,7 @@ | |||
48 | #include "dlmapi.h" | 48 | #include "dlmapi.h" |
49 | #include "dlmcommon.h" | 49 | #include "dlmcommon.h" |
50 | #include "dlmdebug.h" | 50 | #include "dlmdebug.h" |
51 | #include "dlmdomain.h" | ||
51 | 52 | ||
52 | #define MLOG_MASK_PREFIX (ML_DLM|ML_DLM_MASTER) | 53 | #define MLOG_MASK_PREFIX (ML_DLM|ML_DLM_MASTER) |
53 | #include "cluster/masklog.h" | 54 | #include "cluster/masklog.h" |
@@ -178,9 +179,6 @@ static void dlm_dump_mles(struct dlm_ctxt *dlm) | |||
178 | spin_unlock(&dlm->master_lock); | 179 | spin_unlock(&dlm->master_lock); |
179 | } | 180 | } |
180 | 181 | ||
181 | extern spinlock_t dlm_domain_lock; | ||
182 | extern struct list_head dlm_domains; | ||
183 | |||
184 | int dlm_dump_all_mles(const char __user *data, unsigned int len) | 182 | int dlm_dump_all_mles(const char __user *data, unsigned int len) |
185 | { | 183 | { |
186 | struct list_head *iter; | 184 | struct list_head *iter; |
diff --git a/fs/ocfs2/dlm/dlmthread.c b/fs/ocfs2/dlm/dlmthread.c index 92cd5cd66db8..5be9d14f12cb 100644 --- a/fs/ocfs2/dlm/dlmthread.c +++ b/fs/ocfs2/dlm/dlmthread.c | |||
@@ -52,9 +52,6 @@ | |||
52 | #define MLOG_MASK_PREFIX (ML_DLM|ML_DLM_THREAD) | 52 | #define MLOG_MASK_PREFIX (ML_DLM|ML_DLM_THREAD) |
53 | #include "cluster/masklog.h" | 53 | #include "cluster/masklog.h" |
54 | 54 | ||
55 | extern spinlock_t dlm_domain_lock; | ||
56 | extern struct list_head dlm_domains; | ||
57 | |||
58 | static int dlm_thread(void *data); | 55 | static int dlm_thread(void *data); |
59 | 56 | ||
60 | static void dlm_flush_asts(struct dlm_ctxt *dlm); | 57 | static void dlm_flush_asts(struct dlm_ctxt *dlm); |