Skip to contentSkip to conversation

cachefs: possible cleanups

PATCH In review Adrian Bunk 2 participants 2 messages Jan 4, 2005

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 cachefs: possible cleanups

The patch below contains the following possible cleanups:
- journal.c: make cachefs_ondisc_ujnl_marks static
- misc.c: #if 0 the unused global function __cachefs_page_get_private

diffstat output:

 fs/cachefs/journal.c |    2 +-
 fs/cachefs/misc.c    |    2 ++
 2 files changed, 3 insertions(+), 1 deletion(-)
fs/cachefs/journal.c renamed from fs/cachefs/journal.c.old +1 -1
@@ -19,7 +19,7 @@
 
 #define UJNL_WRAP(X) ((X) & (CACHEFS_ONDISC_UJNL_NUMENTS - 1))
 
-const char *cachefs_ondisc_ujnl_marks[] = {
+static const char *cachefs_ondisc_ujnl_marks[] = {
 	"Null     ",
 	"Batch    ",
 	"Ack      ",
fs/cachefs/misc.c renamed from fs/cachefs/misc.c.old +2 -0
@@ -30,6 +30,7 @@
  * get a page caching token from for a page, allocating it and attaching it to
  * the page's private pointer if it doesn't exist
  */
+#if 0
 struct fscache_page * __cachefs_page_get_private(struct page *page,
 						 unsigned gfp_flags)
 {
@@ -51,6 +52,7 @@
 } /* end __cachefs_page_get_private() */
 
 EXPORT_SYMBOL(__cachefs_page_get_private);
+#endif  /*  0  */
 
 /*****************************************************************************/
 /*
#1 lore raw

hoi :)

Quoted 2 lines · Adrian Bunk

The patch below contains the following possible cleanups:
- misc.c: #if 0 the unused global function __cachefs_page_get_private

it looks like this function should be removed, it is a duplicate of __fscache_page_get_private in fs/fscache/page.c (from before the fscache split)

Signature · 1 line
Martin Waitz