lkml.org 
[lkml]   [2005]   [Dec]   [17]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
Date
From
Subject[-mm patch] fs/ocfs2/: small cleanups
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>

---

BTW: Could you add a MAINTAINERS entry for ocfs2?

fs/ocfs2/cluster/sys.c | 4 ++--
fs/ocfs2/dlm/dlmmaster.c | 4 +---
fs/ocfs2/dlm/dlmthread.c | 3 ---
3 files changed, 3 insertions(+), 8 deletions(-)

--- linux-2.6.15-rc5-mm3-full/fs/ocfs2/cluster/sys.c.old 2005-12-17 20:07:58.000000000 +0100
+++ linux-2.6.15-rc5-mm3-full/fs/ocfs2/cluster/sys.c 2005-12-17 20:09:46.000000000 +0100
@@ -50,7 +50,7 @@
return snprintf(buf, PAGE_SIZE, "%u\n", O2NM_API_VERSION);
}

-O2CB_ATTR(interface_revision, S_IFREG | S_IRUGO, o2cb_interface_revision_show, NULL);
+static O2CB_ATTR(interface_revision, S_IFREG | S_IRUGO, o2cb_interface_revision_show, NULL);

static struct attribute *o2cb_attrs[] = {
&o2cb_attr_interface_revision.attr,
@@ -73,7 +73,7 @@
};

/* gives us o2cb_subsys */
-decl_subsys(o2cb, NULL, NULL);
+static decl_subsys(o2cb, NULL, NULL);

static ssize_t
o2cb_show(struct kobject * kobj, struct attribute * attr, char * buffer)
--- linux-2.6.15-rc5-mm3-full/fs/ocfs2/dlm/dlmmaster.c.old 2005-12-17 20:10:22.000000000 +0100
+++ linux-2.6.15-rc5-mm3-full/fs/ocfs2/dlm/dlmmaster.c 2005-12-17 20:11:00.000000000 +0100
@@ -48,6 +48,7 @@
#include "dlmapi.h"
#include "dlmcommon.h"
#include "dlmdebug.h"
+#include "dlmdomain.h"

#define MLOG_MASK_PREFIX (ML_DLM|ML_DLM_MASTER)
#include "cluster/masklog.h"
@@ -173,9 +174,6 @@
spin_unlock(&dlm->master_lock);
}

-extern spinlock_t dlm_domain_lock;
-extern struct list_head dlm_domains;
-
int dlm_dump_all_mles(const char __user *data, unsigned int len)
{
struct list_head *iter;
--- linux-2.6.15-rc5-mm3-full/fs/ocfs2/dlm/dlmthread.c.old 2005-12-17 20:11:11.000000000 +0100
+++ linux-2.6.15-rc5-mm3-full/fs/ocfs2/dlm/dlmthread.c 2005-12-17 20:11:31.000000000 +0100
@@ -52,9 +52,6 @@
#define MLOG_MASK_PREFIX (ML_DLM|ML_DLM_THREAD)
#include "cluster/masklog.h"

-extern spinlock_t dlm_domain_lock;
-extern struct list_head dlm_domains;
-
static int dlm_thread(void *data);

static void dlm_flush_asts(struct dlm_ctxt *dlm);
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2005-12-17 22:33    [W:0.031 / U:4.692 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site