diff options
author | Adrian Bunk <bunk@stusta.de> | 2007-10-17 02:26:41 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@woody.linux-foundation.org> | 2007-10-17 11:42:50 -0400 |
commit | c1206a2c6d87def4af5379e6fef64e928d882ab2 (patch) | |
tree | 388c36ab43b2f9d2cf463871866b0fa97aa4154b /fs/afs | |
parent | a4d63e729edad704af95e5f2a56e8c8eaaa5216b (diff) |
fs/afs/: possible cleanups
This patch contains the following possible cleanups:
- make the following needlessly global functions static:
- rxrpc.c: afs_send_pages()
- vlocation.c: afs_vlocation_queue_for_updates()
- write.c: afs_writepages_region()
- make the following needlessly global variables static:
- mntpt.c: afs_mntpt_expiry_timeout
- proc.c: afs_vlocation_states[]
- server.c: afs_server_timeout
- vlocation.c: afs_vlocation_timeout
- vlocation.c: afs_vlocation_update_timeout
- #if 0 the following unused function:
- cell.c: afs_get_cell_maybe()
- #if 0 the following unused variables:
- callback.c: afs_vnode_update_timeout
- cmservice.c: struct afs_cm_workqueue
Signed-off-by: Adrian Bunk <bunk@stusta.de>
Acked-by: David Howells <dhowells@redhat.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'fs/afs')
-rw-r--r-- | fs/afs/callback.c | 2 | ||||
-rw-r--r-- | fs/afs/cell.c | 2 | ||||
-rw-r--r-- | fs/afs/cmservice.c | 2 | ||||
-rw-r--r-- | fs/afs/internal.h | 1 | ||||
-rw-r--r-- | fs/afs/mntpt.c | 2 | ||||
-rw-r--r-- | fs/afs/proc.c | 2 | ||||
-rw-r--r-- | fs/afs/rxrpc.c | 3 | ||||
-rw-r--r-- | fs/afs/server.c | 2 | ||||
-rw-r--r-- | fs/afs/vlocation.c | 6 | ||||
-rw-r--r-- | fs/afs/write.c | 6 |
10 files changed, 17 insertions, 11 deletions
diff --git a/fs/afs/callback.c b/fs/afs/callback.c index b8243945818d..a78d5b236bb1 100644 --- a/fs/afs/callback.c +++ b/fs/afs/callback.c | |||
@@ -20,7 +20,9 @@ | |||
20 | #include <linux/sched.h> | 20 | #include <linux/sched.h> |
21 | #include "internal.h" | 21 | #include "internal.h" |
22 | 22 | ||
23 | #if 0 | ||
23 | unsigned afs_vnode_update_timeout = 10; | 24 | unsigned afs_vnode_update_timeout = 10; |
25 | #endif /* 0 */ | ||
24 | 26 | ||
25 | #define afs_breakring_space(server) \ | 27 | #define afs_breakring_space(server) \ |
26 | CIRC_SPACE((server)->cb_break_head, (server)->cb_break_tail, \ | 28 | CIRC_SPACE((server)->cb_break_head, (server)->cb_break_tail, \ |
diff --git a/fs/afs/cell.c b/fs/afs/cell.c index 175a567db78c..ccfa89f35259 100644 --- a/fs/afs/cell.c +++ b/fs/afs/cell.c | |||
@@ -265,6 +265,7 @@ struct afs_cell *afs_cell_lookup(const char *name, unsigned namesz) | |||
265 | return cell; | 265 | return cell; |
266 | } | 266 | } |
267 | 267 | ||
268 | #if 0 | ||
268 | /* | 269 | /* |
269 | * try and get a cell record | 270 | * try and get a cell record |
270 | */ | 271 | */ |
@@ -280,6 +281,7 @@ struct afs_cell *afs_get_cell_maybe(struct afs_cell *cell) | |||
280 | write_unlock(&afs_cells_lock); | 281 | write_unlock(&afs_cells_lock); |
281 | return cell; | 282 | return cell; |
282 | } | 283 | } |
284 | #endif /* 0 */ | ||
283 | 285 | ||
284 | /* | 286 | /* |
285 | * destroy a cell record | 287 | * destroy a cell record |
diff --git a/fs/afs/cmservice.c b/fs/afs/cmservice.c index d5b2ad6575bc..47b71c8947f9 100644 --- a/fs/afs/cmservice.c +++ b/fs/afs/cmservice.c | |||
@@ -16,7 +16,9 @@ | |||
16 | #include "internal.h" | 16 | #include "internal.h" |
17 | #include "afs_cm.h" | 17 | #include "afs_cm.h" |
18 | 18 | ||
19 | #if 0 | ||
19 | struct workqueue_struct *afs_cm_workqueue; | 20 | struct workqueue_struct *afs_cm_workqueue; |
21 | #endif /* 0 */ | ||
20 | 22 | ||
21 | static int afs_deliver_cb_init_call_back_state(struct afs_call *, | 23 | static int afs_deliver_cb_init_call_back_state(struct afs_call *, |
22 | struct sk_buff *, bool); | 24 | struct sk_buff *, bool); |
diff --git a/fs/afs/internal.h b/fs/afs/internal.h index 6306438f331f..5ca3625cd39e 100644 --- a/fs/afs/internal.h +++ b/fs/afs/internal.h | |||
@@ -570,7 +570,6 @@ extern int afs_abort_to_error(u32); | |||
570 | */ | 570 | */ |
571 | extern const struct inode_operations afs_mntpt_inode_operations; | 571 | extern const struct inode_operations afs_mntpt_inode_operations; |
572 | extern const struct file_operations afs_mntpt_file_operations; | 572 | extern const struct file_operations afs_mntpt_file_operations; |
573 | extern unsigned long afs_mntpt_expiry_timeout; | ||
574 | 573 | ||
575 | extern int afs_mntpt_check_symlink(struct afs_vnode *, struct key *); | 574 | extern int afs_mntpt_check_symlink(struct afs_vnode *, struct key *); |
576 | extern void afs_mntpt_kill_timer(void); | 575 | extern void afs_mntpt_kill_timer(void); |
diff --git a/fs/afs/mntpt.c b/fs/afs/mntpt.c index 6f8c96fb29eb..5ce43b63c60e 100644 --- a/fs/afs/mntpt.c +++ b/fs/afs/mntpt.c | |||
@@ -42,7 +42,7 @@ const struct inode_operations afs_mntpt_inode_operations = { | |||
42 | static LIST_HEAD(afs_vfsmounts); | 42 | static LIST_HEAD(afs_vfsmounts); |
43 | static DECLARE_DELAYED_WORK(afs_mntpt_expiry_timer, afs_mntpt_expiry_timed_out); | 43 | static DECLARE_DELAYED_WORK(afs_mntpt_expiry_timer, afs_mntpt_expiry_timed_out); |
44 | 44 | ||
45 | unsigned long afs_mntpt_expiry_timeout = 10 * 60; | 45 | static unsigned long afs_mntpt_expiry_timeout = 10 * 60; |
46 | 46 | ||
47 | /* | 47 | /* |
48 | * check a symbolic link to see whether it actually encodes a mountpoint | 48 | * check a symbolic link to see whether it actually encodes a mountpoint |
diff --git a/fs/afs/proc.c b/fs/afs/proc.c index 6edb56683b9a..846c7615ac9e 100644 --- a/fs/afs/proc.c +++ b/fs/afs/proc.c | |||
@@ -513,7 +513,7 @@ static void afs_proc_cell_volumes_stop(struct seq_file *p, void *v) | |||
513 | up_read(&cell->vl_sem); | 513 | up_read(&cell->vl_sem); |
514 | } | 514 | } |
515 | 515 | ||
516 | const char afs_vlocation_states[][4] = { | 516 | static const char afs_vlocation_states[][4] = { |
517 | [AFS_VL_NEW] = "New", | 517 | [AFS_VL_NEW] = "New", |
518 | [AFS_VL_CREATING] = "Crt", | 518 | [AFS_VL_CREATING] = "Crt", |
519 | [AFS_VL_VALID] = "Val", | 519 | [AFS_VL_VALID] = "Val", |
diff --git a/fs/afs/rxrpc.c b/fs/afs/rxrpc.c index 8ccee9ee1d9d..bde3f19c0995 100644 --- a/fs/afs/rxrpc.c +++ b/fs/afs/rxrpc.c | |||
@@ -239,7 +239,8 @@ void afs_flat_call_destructor(struct afs_call *call) | |||
239 | /* | 239 | /* |
240 | * attach the data from a bunch of pages on an inode to a call | 240 | * attach the data from a bunch of pages on an inode to a call |
241 | */ | 241 | */ |
242 | int afs_send_pages(struct afs_call *call, struct msghdr *msg, struct kvec *iov) | 242 | static int afs_send_pages(struct afs_call *call, struct msghdr *msg, |
243 | struct kvec *iov) | ||
243 | { | 244 | { |
244 | struct page *pages[8]; | 245 | struct page *pages[8]; |
245 | unsigned count, n, loop, offset, to; | 246 | unsigned count, n, loop, offset, to; |
diff --git a/fs/afs/server.c b/fs/afs/server.c index 231ae4150279..28f2451419e1 100644 --- a/fs/afs/server.c +++ b/fs/afs/server.c | |||
@@ -13,7 +13,7 @@ | |||
13 | #include <linux/slab.h> | 13 | #include <linux/slab.h> |
14 | #include "internal.h" | 14 | #include "internal.h" |
15 | 15 | ||
16 | unsigned afs_server_timeout = 10; /* server timeout in seconds */ | 16 | static unsigned afs_server_timeout = 10; /* server timeout in seconds */ |
17 | 17 | ||
18 | static void afs_reap_server(struct work_struct *); | 18 | static void afs_reap_server(struct work_struct *); |
19 | 19 | ||
diff --git a/fs/afs/vlocation.c b/fs/afs/vlocation.c index 09e3ad0fc7cc..7b4bbe48112d 100644 --- a/fs/afs/vlocation.c +++ b/fs/afs/vlocation.c | |||
@@ -15,8 +15,8 @@ | |||
15 | #include <linux/sched.h> | 15 | #include <linux/sched.h> |
16 | #include "internal.h" | 16 | #include "internal.h" |
17 | 17 | ||
18 | unsigned afs_vlocation_timeout = 10; /* volume location timeout in seconds */ | 18 | static unsigned afs_vlocation_timeout = 10; /* volume location timeout in seconds */ |
19 | unsigned afs_vlocation_update_timeout = 10 * 60; | 19 | static unsigned afs_vlocation_update_timeout = 10 * 60; |
20 | 20 | ||
21 | static void afs_vlocation_reaper(struct work_struct *); | 21 | static void afs_vlocation_reaper(struct work_struct *); |
22 | static void afs_vlocation_updater(struct work_struct *); | 22 | static void afs_vlocation_updater(struct work_struct *); |
@@ -335,7 +335,7 @@ static int afs_vlocation_fill_in_record(struct afs_vlocation *vl, | |||
335 | /* | 335 | /* |
336 | * queue a vlocation record for updates | 336 | * queue a vlocation record for updates |
337 | */ | 337 | */ |
338 | void afs_vlocation_queue_for_updates(struct afs_vlocation *vl) | 338 | static void afs_vlocation_queue_for_updates(struct afs_vlocation *vl) |
339 | { | 339 | { |
340 | struct afs_vlocation *xvl; | 340 | struct afs_vlocation *xvl; |
341 | 341 | ||
diff --git a/fs/afs/write.c b/fs/afs/write.c index a03b92a0fe1d..bf0955d91f85 100644 --- a/fs/afs/write.c +++ b/fs/afs/write.c | |||
@@ -510,9 +510,9 @@ int afs_writepage(struct page *page, struct writeback_control *wbc) | |||
510 | /* | 510 | /* |
511 | * write a region of pages back to the server | 511 | * write a region of pages back to the server |
512 | */ | 512 | */ |
513 | int afs_writepages_region(struct address_space *mapping, | 513 | static int afs_writepages_region(struct address_space *mapping, |
514 | struct writeback_control *wbc, | 514 | struct writeback_control *wbc, |
515 | pgoff_t index, pgoff_t end, pgoff_t *_next) | 515 | pgoff_t index, pgoff_t end, pgoff_t *_next) |
516 | { | 516 | { |
517 | struct backing_dev_info *bdi = mapping->backing_dev_info; | 517 | struct backing_dev_info *bdi = mapping->backing_dev_info; |
518 | struct afs_writeback *wb; | 518 | struct afs_writeback *wb; |