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;
next prev 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