diff options
author | Theodore Ts'o <tytso@mit.edu> | 2012-11-28 13:03:30 -0500 |
---|---|---|
committer | Theodore Ts'o <tytso@mit.edu> | 2012-11-28 13:03:30 -0500 |
commit | 4a092d737955301da22b9d5e07f5036da821a932 (patch) | |
tree | e0532dda14a20822c61448a395cc60b6803b0f46 /fs/ext4/ext4_extents.h | |
parent | 766f44d46a726cb59f52a75c5c87425a10c4bade (diff) |
ext4: rationalize ext4_extents.h inclusion
Previously, ext4_extents.h was being included at the end of ext4.h,
which was bad for a number of reasons: (a) it was not being included
in the expected place, and (b) it caused the header to be included
multiple times. There were #ifdef's to prevent this from causing any
problems, but it still was unnecessary.
By moving the function declarations that were in ext4_extents.h to
ext4.h, which is standard practice for where the function declarations
for the rest of ext4.h can be found, we can remove ext4_extents.h from
being included in ext4.h at all, and then we can only include
ext4_extents.h where it is needed in ext4's source files.
It should be possible to move a few more things into ext4.h, and
further reduce the number of source files that need to #include
ext4_extents.h, but that's a cleanup for another day.
Reported-by: Sachin Kamat <sachin.kamat@linaro.org>
Reported-by: Wei Yongjun <weiyj.lk@gmail.com>
Signed-off-by: "Theodore Ts'o" <tytso@mit.edu>
Diffstat (limited to 'fs/ext4/ext4_extents.h')
-rw-r--r-- | fs/ext4/ext4_extents.h | 25 |
1 files changed, 0 insertions, 25 deletions
diff --git a/fs/ext4/ext4_extents.h b/fs/ext4/ext4_extents.h index 173b6c54532..487fda12bc0 100644 --- a/fs/ext4/ext4_extents.h +++ b/fs/ext4/ext4_extents.h | |||
@@ -43,16 +43,6 @@ | |||
43 | #define CHECK_BINSEARCH__ | 43 | #define CHECK_BINSEARCH__ |
44 | 44 | ||
45 | /* | 45 | /* |
46 | * Turn on EXT_DEBUG to get lots of info about extents operations. | ||
47 | */ | ||
48 | #define EXT_DEBUG__ | ||
49 | #ifdef EXT_DEBUG | ||
50 | #define ext_debug(fmt, ...) printk(fmt, ##__VA_ARGS__) | ||
51 | #else | ||
52 | #define ext_debug(fmt, ...) no_printk(fmt, ##__VA_ARGS__) | ||
53 | #endif | ||
54 | |||
55 | /* | ||
56 | * If EXT_STATS is defined then stats numbers are collected. | 46 | * If EXT_STATS is defined then stats numbers are collected. |
57 | * These number will be displayed at umount time. | 47 | * These number will be displayed at umount time. |
58 | */ | 48 | */ |
@@ -286,20 +276,5 @@ static inline void ext4_idx_store_pblock(struct ext4_extent_idx *ix, | |||
286 | 0xffff); | 276 | 0xffff); |
287 | } | 277 | } |
288 | 278 | ||
289 | extern int ext4_ext_calc_metadata_amount(struct inode *inode, | ||
290 | ext4_lblk_t lblocks); | ||
291 | extern int ext4_extent_tree_init(handle_t *, struct inode *); | ||
292 | extern int ext4_ext_calc_credits_for_single_extent(struct inode *inode, | ||
293 | int num, | ||
294 | struct ext4_ext_path *path); | ||
295 | extern int ext4_can_extents_be_merged(struct inode *inode, | ||
296 | struct ext4_extent *ex1, | ||
297 | struct ext4_extent *ex2); | ||
298 | extern int ext4_ext_insert_extent(handle_t *, struct inode *, struct ext4_ext_path *, struct ext4_extent *, int); | ||
299 | extern struct ext4_ext_path *ext4_ext_find_extent(struct inode *, ext4_lblk_t, | ||
300 | struct ext4_ext_path *); | ||
301 | extern void ext4_ext_drop_refs(struct ext4_ext_path *); | ||
302 | extern int ext4_ext_check_inode(struct inode *inode); | ||
303 | extern int ext4_find_delalloc_cluster(struct inode *inode, ext4_lblk_t lblk); | ||
304 | #endif /* _EXT4_EXTENTS */ | 279 | #endif /* _EXT4_EXTENTS */ |
305 | 280 | ||