public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Adrian Bunk <bunk@stusta.de>
To: Andrew Morton <akpm@linux-foundation.org>,
	mark.fasheh@oracle.com, kurt.hackel@oracle.com
Cc: linux-kernel@vger.kernel.org, ocfs2-devel@oss.oracle.com
Subject: [RFC: -mm patch] fs/ocfs2/: make 3 functions static
Date: Tue, 10 Apr 2007 23:08:42 +0200	[thread overview]
Message-ID: <20070410210842.GK28370@stusta.de> (raw)
In-Reply-To: <20070408143559.f5014629.akpm@linux-foundation.org>

On Sun, Apr 08, 2007 at 02:35:59PM -0700, Andrew Morton wrote:
>...
> Changes since 2.6.21-rc5-mm4:
>...
>  git-ocfs2.patch
>...
>  git trees.
>...


This patch makes the following needlessly global functions static:
- aops.c: ocfs2_write_data_page()
- dlmglue.c: ocfs2_dump_meta_lvb_info()
- file.c: ocfs2_set_inode_size()

Signed-off-by: Adrian Bunk <bunk@stusta.de>

---

 fs/ocfs2/aops.c    |    6 ++---
 fs/ocfs2/dlmglue.c |   54 ++++++++++++++++++++++++---------------------
 fs/ocfs2/dlmglue.h |    7 -----
 fs/ocfs2/file.c    |    8 +++---
 fs/ocfs2/file.h    |    5 ----
 5 files changed, 36 insertions(+), 44 deletions(-)

--- linux-2.6.21-rc6-mm1/fs/ocfs2/aops.c.old	2007-04-10 00:38:47.000000000 +0200
+++ linux-2.6.21-rc6-mm1/fs/ocfs2/aops.c	2007-04-10 00:38:55.000000000 +0200
@@ -934,9 +934,9 @@
  * Returns a negative error code or the number of bytes copied into
  * the page.
  */
-int ocfs2_write_data_page(struct inode *inode, handle_t *handle,
-			  u64 *p_blkno, struct page *page,
-			  struct ocfs2_write_ctxt *wc, int new)
+static int ocfs2_write_data_page(struct inode *inode, handle_t *handle,
+				 u64 *p_blkno, struct page *page,
+				 struct ocfs2_write_ctxt *wc, int new)
 {
 	int ret, copied = 0;
 	unsigned int from = 0, to = 0;
--- linux-2.6.21-rc6-mm1/fs/ocfs2/dlmglue.h.old	2007-04-10 00:41:39.000000000 +0200
+++ linux-2.6.21-rc6-mm1/fs/ocfs2/dlmglue.h	2007-04-10 00:47:06.000000000 +0200
@@ -119,11 +119,4 @@
 struct ocfs2_dlm_debug *ocfs2_new_dlm_debug(void);
 void ocfs2_put_dlm_debug(struct ocfs2_dlm_debug *dlm_debug);
 
-/* aids in debugging and tracking lvbs */
-void ocfs2_dump_meta_lvb_info(u64 level,
-			      const char *function,
-			      unsigned int line,
-			      struct ocfs2_lock_res *lockres);
-#define mlog_meta_lvb(__level, __lockres) ocfs2_dump_meta_lvb_info(__level, __PRETTY_FUNCTION__, __LINE__, __lockres)
-
 #endif	/* DLMGLUE_H */
--- linux-2.6.21-rc6-mm1/fs/ocfs2/dlmglue.c.old	2007-04-10 00:42:19.000000000 +0200
+++ linux-2.6.21-rc6-mm1/fs/ocfs2/dlmglue.c	2007-04-10 00:44:23.000000000 +0200
@@ -103,6 +103,35 @@
 static void ocfs2_dentry_post_unlock(struct ocfs2_super *osb,
 				     struct ocfs2_lock_res *lockres);
 
+
+#define mlog_meta_lvb(__level, __lockres) ocfs2_dump_meta_lvb_info(__level, __PRETTY_FUNCTION__, __LINE__, __lockres)
+
+/* This aids in debugging situations where a bad LVB might be involved. */
+static void ocfs2_dump_meta_lvb_info(u64 level,
+				     const char *function,
+				     unsigned int line,
+				     struct ocfs2_lock_res *lockres)
+{
+	struct ocfs2_meta_lvb *lvb = (struct ocfs2_meta_lvb *) lockres->l_lksb.lvb;
+
+	mlog(level, "LVB information for %s (called from %s:%u):\n",
+	     lockres->l_name, function, line);
+	mlog(level, "version: %u, clusters: %u, generation: 0x%x\n",
+	     lvb->lvb_version, be32_to_cpu(lvb->lvb_iclusters),
+	     be32_to_cpu(lvb->lvb_igeneration));
+	mlog(level, "size: %llu, uid %u, gid %u, mode 0x%x\n",
+	     (unsigned long long)be64_to_cpu(lvb->lvb_isize),
+	     be32_to_cpu(lvb->lvb_iuid), be32_to_cpu(lvb->lvb_igid),
+	     be16_to_cpu(lvb->lvb_imode));
+	mlog(level, "nlink %u, atime_packed 0x%llx, ctime_packed 0x%llx, "
+	     "mtime_packed 0x%llx iattr 0x%x\n", be16_to_cpu(lvb->lvb_inlink),
+	     (long long)be64_to_cpu(lvb->lvb_iatime_packed),
+	     (long long)be64_to_cpu(lvb->lvb_ictime_packed),
+	     (long long)be64_to_cpu(lvb->lvb_imtime_packed),
+	     be32_to_cpu(lvb->lvb_iattr));
+}
+
+
 /*
  * OCFS2 Lock Resource Operations
  *
@@ -3078,28 +3107,3 @@
 
 	mlog_exit_void();
 }
-
-/* This aids in debugging situations where a bad LVB might be involved. */
-void ocfs2_dump_meta_lvb_info(u64 level,
-			      const char *function,
-			      unsigned int line,
-			      struct ocfs2_lock_res *lockres)
-{
-	struct ocfs2_meta_lvb *lvb = (struct ocfs2_meta_lvb *) lockres->l_lksb.lvb;
-
-	mlog(level, "LVB information for %s (called from %s:%u):\n",
-	     lockres->l_name, function, line);
-	mlog(level, "version: %u, clusters: %u, generation: 0x%x\n",
-	     lvb->lvb_version, be32_to_cpu(lvb->lvb_iclusters),
-	     be32_to_cpu(lvb->lvb_igeneration));
-	mlog(level, "size: %llu, uid %u, gid %u, mode 0x%x\n",
-	     (unsigned long long)be64_to_cpu(lvb->lvb_isize),
-	     be32_to_cpu(lvb->lvb_iuid), be32_to_cpu(lvb->lvb_igid),
-	     be16_to_cpu(lvb->lvb_imode));
-	mlog(level, "nlink %u, atime_packed 0x%llx, ctime_packed 0x%llx, "
-	     "mtime_packed 0x%llx iattr 0x%x\n", be16_to_cpu(lvb->lvb_inlink),
-	     (long long)be64_to_cpu(lvb->lvb_iatime_packed),
-	     (long long)be64_to_cpu(lvb->lvb_ictime_packed),
-	     (long long)be64_to_cpu(lvb->lvb_imtime_packed),
-	     be32_to_cpu(lvb->lvb_iattr));
-}
--- linux-2.6.21-rc6-mm1/fs/ocfs2/file.h.old	2007-04-10 00:45:00.000000000 +0200
+++ linux-2.6.21-rc6-mm1/fs/ocfs2/file.h	2007-04-10 00:45:07.000000000 +0200
@@ -56,11 +56,6 @@
 int ocfs2_permission(struct inode *inode, int mask,
 		     struct nameidata *nd);
 
-int ocfs2_set_inode_size(handle_t *handle,
-			 struct inode *inode,
-			 struct buffer_head *fe_bh,
-			 u64 new_i_size);
-
 int ocfs2_should_update_atime(struct inode *inode,
 			      struct vfsmount *vfsmnt);
 int ocfs2_update_inode_atime(struct inode *inode,
--- linux-2.6.21-rc6-mm1/fs/ocfs2/file.c.old	2007-04-10 00:45:15.000000000 +0200
+++ linux-2.6.21-rc6-mm1/fs/ocfs2/file.c	2007-04-10 00:45:24.000000000 +0200
@@ -207,10 +207,10 @@
 	return ret;
 }
 
-int ocfs2_set_inode_size(handle_t *handle,
-			 struct inode *inode,
-			 struct buffer_head *fe_bh,
-			 u64 new_i_size)
+static int ocfs2_set_inode_size(handle_t *handle,
+				struct inode *inode,
+				struct buffer_head *fe_bh,
+				u64 new_i_size)
 {
 	int status;
 


  parent reply	other threads:[~2007-04-10 21:08 UTC|newest]

Thread overview: 74+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-04-08 21:35 2.6.21-rc6-mm1 Andrew Morton
2007-04-09 11:13 ` 2.6.21-rc6-mm1 Borislav Petkov
2007-04-09 16:08 ` 2.6.21-rc6-mm1 Rafael J. Wysocki
2007-04-09 16:14   ` 2.6.21-rc6-mm1 Pallipadi, Venkatesh
2007-04-09 17:40     ` 2.6.21-rc6-mm1 Rafael J. Wysocki
2007-04-10 22:20       ` 2.6.21-rc6-mm1 Venki Pallipadi
2007-04-11 19:28         ` 2.6.21-rc6-mm1 Rafael J. Wysocki
2007-04-10 23:59       ` 2.6.21-rc6-mm1 Adam Belay
2007-04-11  0:15         ` 2.6.21-rc6-mm1 Venki Pallipadi
2007-04-11  5:20           ` 2.6.21-rc6-mm1 Stephane Eranian
2007-04-11 10:39             ` 2.6.21-rc6-mm1 Andi Kleen
2007-04-11 13:09               ` 2.6.21-rc6-mm1 Stephane Eranian
2007-04-11 13:19                 ` 2.6.21-rc6-mm1 Andi Kleen
2007-04-11 18:06               ` 2.6.21-rc6-mm1 Andrew Morton
2007-04-09 19:03 ` 2.6.21-rc6-mm1 Torsten Kaiser
2007-04-09 21:42   ` <linux/wait.h> uses definitions from <linux/sched.h> (was Re: 2.6.21-rc6-mm1) Stefan Richter
2007-04-09 22:01     ` Stefan Richter
2007-04-10  0:50 ` 2.6.21-rc6-mm1 Nishanth Aravamudan
2007-04-10  0:56   ` 2.6.21-rc6-mm1 Christoph Lameter
2007-04-10  1:07   ` 2.6.21-rc6-mm1 William Lee Irwin III
2007-04-10 11:28 ` 2.6.21-rc6-mm1 ima "BUG: held lock freed!" Joseph Fannin
2007-04-10 12:24 ` 2.6.21-rc6-mm1 Cornelia Huck
2007-04-10 12:38   ` 2.6.21-rc6-mm1 Ananth N Mavinakayanahalli
2007-04-10 21:08 ` [-mm patch] make struct proc_kpagemap static Adrian Bunk
2007-04-10 21:09   ` Matt Mackall
2007-04-10 21:08 ` [-mm patch] make kernel/module.c:is_exported() static Adrian Bunk
2007-04-10 21:08 ` Adrian Bunk [this message]
2007-04-10 22:21 ` 2.6.21-rc6-mm1 aacraid not finding device Steve Fox
2007-04-13 12:35   ` [PATCH] aacraid: " Salyzyn, Mark
2007-04-11 19:42 ` 2.6.21-rc6-mm1 USB related boot hang Helge Hafting
2007-04-11 20:43   ` Andrew Morton
2007-04-11 23:07     ` Helge Hafting
2007-04-11 23:25       ` Andrew Morton
2007-04-12  7:50       ` Jiri Kosina
2007-04-12  8:02         ` Jiri Kosina
2007-04-12 11:42           ` Helge Hafting
2007-04-12 16:47             ` Andrew Morton
2007-04-12 18:56               ` Helge Hafting
2007-04-12 15:31           ` Jiri Kosina
2007-04-12 16:01             ` Corey Minyard
2007-04-12 16:55             ` Andrew Morton
2007-04-12 17:25               ` Greg KH
2007-04-12 17:49               ` Jiri Kosina
2007-04-12 17:58                 ` Greg KH
2007-04-12 18:17                 ` Jiri Kosina
2007-04-12 21:06                   ` Corey Minyard
2007-04-12 20:19             ` Helge Hafting
2007-04-12 19:22         ` Helge Hafting
2007-04-12 18:32       ` Jiri Kosina
2007-04-12 20:25         ` Helge Hafting
2007-04-12 23:16           ` Jiri Kosina
2007-04-25  9:54             ` Helge Hafting
2007-04-25 11:28               ` Jiri Kosina
2007-04-25 12:45                 ` Helge Hafting
2007-04-26 18:38                 ` Helge Hafting
2007-04-26 22:28                 ` 2.6.21-rc6-mm1 USB related boot hang - bisection result Helge Hafting
2007-04-26 22:39                   ` Jiri Kosina
2007-04-26 23:13                     ` Andrew Morton
2007-04-27 21:04                     ` Helge Hafting
2007-04-27 22:41                       ` Andrew Morton
2007-04-13 23:45 ` CPU_IDLE prevents resuming from STR [was: Re: 2.6.21-rc6-mm1] Mattia Dongili
2007-04-16  2:40   ` Shaohua Li
2007-04-17  2:50     ` Joshua Wise
2007-04-17  2:50       ` Shaohua Li
2007-04-17  6:47       ` Shaohua Li
2007-04-18 23:00         ` Joshua Wise
2007-04-19  1:05           ` Shaohua Li
2007-04-24  8:10 ` 2.6.21-rc6-mm1 J.A. Magallón
2007-04-24 11:58   ` 2.6.21-rc6-mm1 Andrew Morton
2007-04-24 13:43     ` 2.6.21-rc6-mm1 J.A. Magallón
2007-04-24 17:22       ` 2.6.21-rc6-mm1 Andrew Morton
2007-04-25 20:50         ` 2.6.21-rc6-mm1 J.A. Magallón
2007-04-25 21:39           ` start_udev and devpts [Re: 2.6.21-rc6-mm1] J.A. Magallón
2007-04-25 22:26             ` J.A. Magallón

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20070410210842.GK28370@stusta.de \
    --to=bunk@stusta.de \
    --cc=akpm@linux-foundation.org \
    --cc=kurt.hackel@oracle.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mark.fasheh@oracle.com \
    --cc=ocfs2-devel@oss.oracle.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox