[-mm patch] fs/reiser4/: possible cleanups
From: Adrian Bunk
Date: Fri Jun 23 2006 - 06:54:16 EST
This patch contains the following possible cleanups:
- make needlessly global code static
- proper prototype for the following function:
- plugin/file/file.c: init_uf_coord()
- "extern inline" -> "static inline"
- #if 0/comment out the following unused functions:
- plugin/file/cryptcompress.c: jnode_of_cluster()
- plugin/plugin.c: force_plugin()
- plugin/plugin_set.c: plugin_set_compression()
- remove the following unused variable:
- plugin/file/file.c: xversion
Signed-off-by: Adrian Bunk <bunk@xxxxxxxxx>
---
fs/reiser4/block_alloc.c | 10 +++++-----
fs/reiser4/flush_queue.c | 2 +-
fs/reiser4/jnode.c | 7 ++++---
fs/reiser4/jnode.h | 4 ----
fs/reiser4/plugin/cluster.h | 1 -
fs/reiser4/plugin/file/cryptcompress.c | 6 ++++--
fs/reiser4/plugin/file/cryptcompress.h | 2 --
fs/reiser4/plugin/file/file.c | 4 +---
fs/reiser4/plugin/file/file.h | 4 ++--
fs/reiser4/plugin/item/extent_file_ops.c | 2 --
fs/reiser4/plugin/plugin.c | 2 ++
fs/reiser4/plugin/plugin.h | 1 -
fs/reiser4/plugin/plugin_set.c | 2 +-
fs/reiser4/plugin/plugin_set.h | 1 -
fs/reiser4/super.h | 1 -
fs/reiser4/super_ops.c | 4 ++--
fs/reiser4/txnmgr.h | 1 -
17 files changed, 22 insertions(+), 32 deletions(-)
--- linux-2.6.17-mm1-full/fs/reiser4/block_alloc.c.old 2006-06-22 16:00:18.000000000 +0200
+++ linux-2.6.17-mm1-full/fs/reiser4/block_alloc.c 2006-06-22 16:02:35.000000000 +0200
@@ -554,8 +554,8 @@
/* adjust sb block counters, if real (on-disk) block allocation immediately
follows grabbing of free disk space. */
-void grabbed2used(reiser4_context *ctx, reiser4_super_info_data *sbinfo,
- __u64 count)
+static void grabbed2used(reiser4_context *ctx, reiser4_super_info_data *sbinfo,
+ __u64 count)
{
sub_from_ctx_grabbed(ctx, count);
@@ -570,8 +570,8 @@
}
/* adjust sb block counters when @count unallocated blocks get mapped to disk */
-void fake_allocated2used(reiser4_super_info_data *sbinfo, __u64 count,
- reiser4_ba_flags_t flags)
+static void fake_allocated2used(reiser4_super_info_data *sbinfo, __u64 count,
+ reiser4_ba_flags_t flags)
{
spin_lock_reiser4_super(sbinfo);
@@ -583,7 +583,7 @@
spin_unlock_reiser4_super(sbinfo);
}
-void flush_reserved2used(txn_atom * atom, __u64 count)
+static void flush_reserved2used(txn_atom * atom, __u64 count)
{
reiser4_super_info_data *sbinfo;
--- linux-2.6.17-mm1-full/fs/reiser4/txnmgr.h.old 2006-06-22 16:02:59.000000000 +0200
+++ linux-2.6.17-mm1-full/fs/reiser4/txnmgr.h 2006-06-22 16:03:04.000000000 +0200
@@ -661,7 +661,6 @@
extern void fq_put(flush_queue_t *);
extern void fuse_fq(txn_atom * to, txn_atom * from);
extern void queue_jnode(flush_queue_t *, jnode *);
-extern void mark_jnode_queued(flush_queue_t *, jnode *);
extern int write_fq(flush_queue_t *, long *, int);
extern int current_atom_finish_all_fq(void);
--- linux-2.6.17-mm1-full/fs/reiser4/flush_queue.c.old 2006-06-22 16:03:13.000000000 +0200
+++ linux-2.6.17-mm1-full/fs/reiser4/flush_queue.c 2006-06-22 16:03:20.000000000 +0200
@@ -191,7 +191,7 @@
}
/* */
-void mark_jnode_queued(flush_queue_t * fq, jnode * node)
+static void mark_jnode_queued(flush_queue_t * fq, jnode * node)
{
JF_SET(node, JNODE_FLUSH_QUEUED);
count_enqueued_node(fq);
--- linux-2.6.17-mm1-full/fs/reiser4/jnode.h.old 2006-06-22 16:04:24.000000000 +0200
+++ linux-2.6.17-mm1-full/fs/reiser4/jnode.h 2006-06-22 16:05:25.000000000 +0200
@@ -349,12 +349,8 @@
extern jnode *jnode_by_page(struct page *pg) NONNULL;
extern jnode *jnode_of_page(struct page *pg) NONNULL;
void jnode_attach_page(jnode * node, struct page *pg);
-jnode *find_get_jnode(reiser4_tree * tree,
- struct address_space *mapping, oid_t oid,
- unsigned long index);
void unhash_unformatted_jnode(jnode *);
-struct page *jnode_get_page_locked(jnode *, gfp_t gfp_flags);
extern jnode *page_next_jnode(jnode * node) NONNULL;
extern void jnode_init(jnode * node, reiser4_tree * tree, jnode_type) NONNULL;
extern void jnode_make_dirty(jnode * node) NONNULL;
--- linux-2.6.17-mm1-full/fs/reiser4/jnode.c.old 2006-06-22 16:04:40.000000000 +0200
+++ linux-2.6.17-mm1-full/fs/reiser4/jnode.c 2006-06-22 16:05:34.000000000 +0200
@@ -537,8 +537,9 @@
* allocate new jnode, insert it, and also insert into radix tree for the
* given inode/mapping.
*/
-jnode *find_get_jnode(reiser4_tree * tree, struct address_space *mapping,
- oid_t oid, unsigned long index)
+static jnode *find_get_jnode(reiser4_tree * tree,
+ struct address_space *mapping,
+ oid_t oid, unsigned long index)
{
jnode *result;
jnode *shadow;
@@ -786,7 +787,7 @@
/* Lock a page attached to jnode, create and attach page to jnode if it had no
* one. */
-struct page *jnode_get_page_locked(jnode * node, gfp_t gfp_flags)
+static struct page *jnode_get_page_locked(jnode * node, gfp_t gfp_flags)
{
struct page *page;
--- linux-2.6.17-mm1-full/fs/reiser4/plugin/file/cryptcompress.h.old 2006-06-22 16:06:08.000000000 +0200
+++ linux-2.6.17-mm1-full/fs/reiser4/plugin/file/cryptcompress.h 2006-06-22 16:07:20.000000000 +0200
@@ -459,7 +459,6 @@
void save_file_hint(struct file *, const hint_t *);
void hint_init_zero(hint_t *);
int crc_inode_ok(struct inode *inode);
-int jnode_of_cluster(const jnode * node, struct page * page);
extern int ctail_read_disk_cluster (reiser4_cluster_t *, struct inode *, int);
extern int do_readpage_ctail(struct inode *, reiser4_cluster_t *,
struct page * page);
@@ -472,7 +471,6 @@
int (*can_inherit)(struct inode * child,
struct inode * parent));
void attach_crypto_stat(struct inode * inode, crypto_stat_t * info);
-void detach_crypto_stat(struct inode * inode);
void change_crypto_stat(struct inode * inode, crypto_stat_t * new);
crypto_stat_t * alloc_crypto_stat (struct inode * inode);
--- linux-2.6.17-mm1-full/fs/reiser4/plugin/cluster.h.old 2006-06-22 16:06:47.000000000 +0200
+++ linux-2.6.17-mm1-full/fs/reiser4/plugin/cluster.h 2006-06-22 16:06:53.000000000 +0200
@@ -240,7 +240,6 @@
int inflate_cluster(reiser4_cluster_t *, struct inode *);
int find_cluster(reiser4_cluster_t *, struct inode *, int read, int write);
-void forget_cluster_pages(struct page **page, int nrpages);
int flush_cluster_pages(reiser4_cluster_t *, jnode *, struct inode *);
int deflate_cluster(reiser4_cluster_t *, struct inode *);
void truncate_page_cluster(struct inode *inode, cloff_t start);
--- linux-2.6.17-mm1-full/fs/reiser4/plugin/file/cryptcompress.c.old 2006-06-22 16:06:19.000000000 +0200
+++ linux-2.6.17-mm1-full/fs/reiser4/plugin/file/cryptcompress.c 2006-06-22 16:07:39.000000000 +0200
@@ -380,7 +380,7 @@
}
#endif /* REISER4_DEBUG */
-void detach_crypto_stat(struct inode * inode)
+static void detach_crypto_stat(struct inode * inode)
{
assert("edward-1385", inode != NULL);
assert("edward-1386", host_allows_crypto_stat(inode));
@@ -1574,6 +1574,7 @@
understand that pages of cryptcompress files are not
flushable.
*/
+#if 0
int jnode_of_cluster(const jnode * node, struct page * page)
{
assert("edward-1339", node != NULL);
@@ -1597,6 +1598,7 @@
}
return 0;
}
+#endif /* 0 */
/* put cluster pages */
void release_cluster_pages(reiser4_cluster_t * clust)
@@ -1751,7 +1753,7 @@
jput(node);
}
-void forget_cluster_pages(struct page **pages, int nr)
+static void forget_cluster_pages(struct page **pages, int nr)
{
int i;
for (i = 0; i < nr; i++) {
--- linux-2.6.17-mm1-full/fs/reiser4/plugin/file/file.h.old 2006-06-22 16:07:55.000000000 +0200
+++ linux-2.6.17-mm1-full/fs/reiser4/plugin/file/file.h 2006-06-22 16:18:08.000000000 +0200
@@ -114,7 +114,6 @@
const reiser4_key *, znode_lock_mode,
struct inode *);
-void validate_extended_coord(uf_coord_t *, loff_t offset);
int load_file_hint(struct file *, hint_t *);
void save_file_hint(struct file *, const hint_t *);
@@ -235,8 +234,9 @@
int find_or_create_extent(struct page *);
int equal_to_ldk(znode *, const reiser4_key *);
+void init_uf_coord(uf_coord_t *uf_coord, lock_handle *lh);
-extern inline int cbk_errored(int cbk_result)
+static inline int cbk_errored(int cbk_result)
{
return (cbk_result != CBK_COORD_NOTFOUND
&& cbk_result != CBK_COORD_FOUND);
--- linux-2.6.17-mm1-full/fs/reiser4/plugin/item/extent_file_ops.c.old 2006-06-22 16:49:24.000000000 +0200
+++ linux-2.6.17-mm1-full/fs/reiser4/plugin/item/extent_file_ops.c 2006-06-22 16:49:51.000000000 +0200
@@ -734,8 +734,6 @@
return count;
}
-void init_uf_coord(uf_coord_t *uf_coord, lock_handle *lh);
-
/**
* update_extent
* @file:
--- linux-2.6.17-mm1-full/fs/reiser4/plugin/file/file.c.old 2006-06-22 16:08:12.000000000 +0200
+++ linux-2.6.17-mm1-full/fs/reiser4/plugin/file/file.c 2006-06-22 16:08:37.000000000 +0200
@@ -103,7 +103,7 @@
uf_coord->valid = 0;
}
-void validate_extended_coord(uf_coord_t *uf_coord, loff_t offset)
+static void validate_extended_coord(uf_coord_t *uf_coord, loff_t offset)
{
assert("vs-1333", uf_coord->valid == 0);
@@ -760,8 +760,6 @@
hint->ext_coord.lh, lock_mode, ZNODE_LOCK_LOPRI);
}
-int xversion;
-
/**
* find_or_create_extent -
* @page:
--- linux-2.6.17-mm1-full/fs/reiser4/plugin/plugin.h.old 2006-06-22 16:09:55.000000000 +0200
+++ linux-2.6.17-mm1-full/fs/reiser4/plugin/plugin.h 2006-06-22 16:10:05.000000000 +0200
@@ -886,7 +886,6 @@
int grab_plugin(struct inode *self, struct inode *ancestor, pset_member memb);
int grab_plugin_from(struct inode *self, pset_member memb,
reiser4_plugin * plug);
-int force_plugin(struct inode *self, pset_member memb, reiser4_plugin * plug);
/* defined in fs/reiser4/plugin/object.c */
extern file_plugin file_plugins[LAST_FILE_PLUGIN_ID];
--- linux-2.6.17-mm1-full/fs/reiser4/plugin/plugin.c.old 2006-06-22 16:10:12.000000000 +0200
+++ linux-2.6.17-mm1-full/fs/reiser4/plugin/plugin.c 2006-06-22 16:11:03.000000000 +0200
@@ -348,6 +348,7 @@
return result;
}
+#if 0
int force_plugin(struct inode *self, pset_member memb, reiser4_plugin * plug)
{
reiser4_inode *info;
@@ -362,6 +363,7 @@
update_plugin_mask(info, memb);
return result;
}
+#endif /* 0 */
reiser4_plugin_type_data plugins[REISER4_PLUGIN_TYPES] = {
/* C90 initializers */
--- linux-2.6.17-mm1-full/fs/reiser4/plugin/plugin_set.h.old 2006-06-22 16:11:33.000000000 +0200
+++ linux-2.6.17-mm1-full/fs/reiser4/plugin/plugin_set.h 2006-06-22 16:11:42.000000000 +0200
@@ -57,7 +57,6 @@
extern int plugin_set_hash(plugin_set ** set, hash_plugin * plug);
extern int plugin_set_fibration(plugin_set ** set, fibration_plugin * plug);
extern int plugin_set_sd(plugin_set ** set, item_plugin * plug);
-extern int plugin_set_compression(plugin_set ** set, compression_plugin * plug);
extern int plugin_set_cluster(plugin_set ** set, cluster_plugin * plug);
extern int init_plugin_set(void);
--- linux-2.6.17-mm1-full/fs/reiser4/plugin/plugin_set.c.old 2006-06-22 16:11:54.000000000 +0200
+++ linux-2.6.17-mm1-full/fs/reiser4/plugin/plugin_set.c 2006-06-22 16:12:25.000000000 +0200
@@ -322,7 +322,7 @@
DEFINE_PLUGIN_SET(item_plugin, sd)
/* DEFINE_PLUGIN_SET(cipher_plugin, cipher) */
/* DEFINE_PLUGIN_SET(digest_plugin, digest) */
- DEFINE_PLUGIN_SET(compression_plugin, compression)
+ /* DEFINE_PLUGIN_SET(compression_plugin, compression) */
/* DEFINE_PLUGIN_SET(compression_mode_plugin, compression_mode) */
DEFINE_PLUGIN_SET(cluster_plugin, cluster)
/* DEFINE_PLUGIN_SET(regular_plugin, regular_entry) */
--- linux-2.6.17-mm1-full/fs/reiser4/super.h.old 2006-06-22 16:12:41.000000000 +0200
+++ linux-2.6.17-mm1-full/fs/reiser4/super.h 2006-06-22 16:12:46.000000000 +0200
@@ -452,7 +452,6 @@
extern struct super_operations reiser4_super_operations;
extern struct export_operations reiser4_export_operations;
extern struct dentry_operations reiser4_dentry_operations;
-extern struct dentry *reiser4_debugfs_root;
/* __REISER4_SUPER_H__ */
#endif
--- linux-2.6.17-mm1-full/fs/reiser4/super_ops.c.old 2006-06-22 16:12:55.000000000 +0200
+++ linux-2.6.17-mm1-full/fs/reiser4/super_ops.c 2006-06-22 16:13:16.000000000 +0200
@@ -16,6 +16,8 @@
/* slab cache for inodes */
static kmem_cache_t *inode_cache;
+static struct dentry *reiser4_debugfs_root = NULL;
+
/**
* init_once - constructor for reiser4 inodes
* @obj: inode to be initialized
@@ -593,8 +595,6 @@
*cachep = NULL;
}
-struct dentry *reiser4_debugfs_root = NULL;
-
/**
* init_reiser4 - reiser4 initialization entry point
*
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/