aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorArtem Bityutskiy <artem.bityutskiy@linux.intel.com>2012-05-16 12:53:46 -0400
committerArtem Bityutskiy <artem.bityutskiy@linux.intel.com>2012-05-16 12:53:46 -0400
commitf70b7e52aa23c9aea5346b9730b402fb55f9079b (patch)
tree20882b6e8345b8836d395248fd7f31fce7d09236
parent1baafd28dc17422f10be9b2a3a75432154e3abc7 (diff)
UBIFS: remove Kconfig debugging option
Have the debugging stuff always compiled-in instead. It simplifies maintanance a lot. Signed-off-by: Artem Bityutskiy <artem.bityutskiy@linux.intel.com>
-rw-r--r--fs/ubifs/Kconfig17
-rw-r--r--fs/ubifs/Makefile4
-rw-r--r--fs/ubifs/commit.c6
-rw-r--r--fs/ubifs/debug.c4
-rw-r--r--fs/ubifs/debug.h153
-rw-r--r--fs/ubifs/dir.c8
-rw-r--r--fs/ubifs/log.c8
-rw-r--r--fs/ubifs/lprops.c6
-rw-r--r--fs/ubifs/lpt.c4
-rw-r--r--fs/ubifs/lpt_commit.c10
-rw-r--r--fs/ubifs/orphan.c10
-rw-r--r--fs/ubifs/sb.c2
-rw-r--r--fs/ubifs/tnc.c4
-rw-r--r--fs/ubifs/tnc_commit.c10
-rw-r--r--fs/ubifs/ubifs.h11
15 files changed, 19 insertions, 238 deletions
diff --git a/fs/ubifs/Kconfig b/fs/ubifs/Kconfig
index e49a23ca117a..ba66d508006a 100644
--- a/fs/ubifs/Kconfig
+++ b/fs/ubifs/Kconfig
@@ -35,20 +35,3 @@ config UBIFS_FS_ZLIB
35 default y 35 default y
36 help 36 help
37 Zlib compresses better than LZO but it is slower. Say 'Y' if unsure. 37 Zlib compresses better than LZO but it is slower. Say 'Y' if unsure.
38
39# Debugging-related stuff
40config UBIFS_FS_DEBUG
41 bool "Enable debugging support"
42 depends on UBIFS_FS
43 select DEBUG_FS
44 select KALLSYMS
45 help
46 This option enables UBIFS debugging support. It makes sure various
47 assertions, self-checks, debugging messages and test modes are compiled
48 in (this all is compiled out otherwise). Assertions are light-weight
49 and this option also enables them. Self-checks, debugging messages and
50 test modes are switched off by default. Thus, it is safe and actually
51 recommended to have debugging support enabled, and it should not slow
52 down UBIFS. You can then further enable / disable individual debugging
53 features using UBIFS module parameters and the corresponding sysfs
54 interfaces.
diff --git a/fs/ubifs/Makefile b/fs/ubifs/Makefile
index 4878614d06f9..2c6f0cb816b4 100644
--- a/fs/ubifs/Makefile
+++ b/fs/ubifs/Makefile
@@ -3,6 +3,4 @@ obj-$(CONFIG_UBIFS_FS) += ubifs.o
3ubifs-y += shrinker.o journal.o file.o dir.o super.o sb.o io.o 3ubifs-y += shrinker.o journal.o file.o dir.o super.o sb.o io.o
4ubifs-y += tnc.o master.o scan.o replay.o log.o commit.o gc.o orphan.o 4ubifs-y += tnc.o master.o scan.o replay.o log.o commit.o gc.o orphan.o
5ubifs-y += budget.o find.o tnc_commit.o compress.o lpt.o lprops.o 5ubifs-y += budget.o find.o tnc_commit.o compress.o lpt.o lprops.o
6ubifs-y += recovery.o ioctl.o lpt_commit.o tnc_misc.o xattr.o 6ubifs-y += recovery.o ioctl.o lpt_commit.o tnc_misc.o xattr.o debug.o
7
8ubifs-$(CONFIG_UBIFS_FS_DEBUG) += debug.o
diff --git a/fs/ubifs/commit.c b/fs/ubifs/commit.c
index 4f795d1c4866..5da20c193634 100644
--- a/fs/ubifs/commit.c
+++ b/fs/ubifs/commit.c
@@ -496,7 +496,9 @@ int ubifs_gc_should_commit(struct ubifs_info *c)
496 return ret; 496 return ret;
497} 497}
498 498
499#ifdef CONFIG_UBIFS_FS_DEBUG 499/*
500 * Everything below is related to debugging.
501 */
500 502
501/** 503/**
502 * struct idx_node - hold index nodes during index tree traversal. 504 * struct idx_node - hold index nodes during index tree traversal.
@@ -734,5 +736,3 @@ out_free:
734 err = -EINVAL; 736 err = -EINVAL;
735 return err; 737 return err;
736} 738}
737
738#endif /* CONFIG_UBIFS_FS_DEBUG */
diff --git a/fs/ubifs/debug.c b/fs/ubifs/debug.c
index 914b4c879938..3eef25343393 100644
--- a/fs/ubifs/debug.c
+++ b/fs/ubifs/debug.c
@@ -34,8 +34,6 @@
34#include <linux/random.h> 34#include <linux/random.h>
35#include "ubifs.h" 35#include "ubifs.h"
36 36
37#ifdef CONFIG_UBIFS_FS_DEBUG
38
39static DEFINE_SPINLOCK(dbg_lock); 37static DEFINE_SPINLOCK(dbg_lock);
40 38
41static const char *get_key_fmt(int fmt) 39static const char *get_key_fmt(int fmt)
@@ -3189,5 +3187,3 @@ void ubifs_debugging_exit(struct ubifs_info *c)
3189{ 3187{
3190 kfree(c->dbg); 3188 kfree(c->dbg);
3191} 3189}
3192
3193#endif /* CONFIG_UBIFS_FS_DEBUG */
diff --git a/fs/ubifs/debug.h b/fs/ubifs/debug.h
index b34aeff07523..7a6a9e279df1 100644
--- a/fs/ubifs/debug.h
+++ b/fs/ubifs/debug.h
@@ -29,8 +29,6 @@ typedef int (*dbg_leaf_callback)(struct ubifs_info *c,
29typedef int (*dbg_znode_callback)(struct ubifs_info *c, 29typedef int (*dbg_znode_callback)(struct ubifs_info *c,
30 struct ubifs_znode *znode, void *priv); 30 struct ubifs_znode *znode, void *priv);
31 31
32#ifdef CONFIG_UBIFS_FS_DEBUG
33
34/* 32/*
35 * The UBIFS debugfs directory name pattern and maximum name length (3 for "ubi" 33 * The UBIFS debugfs directory name pattern and maximum name length (3 for "ubi"
36 * + 1 for "_" and plus 2x2 for 2 UBI numbers and 1 for the trailing zero byte. 34 * + 1 for "_" and plus 2x2 for 2 UBI numbers and 1 for the trailing zero byte.
@@ -317,155 +315,4 @@ void dbg_debugfs_exit(void);
317int dbg_debugfs_init_fs(struct ubifs_info *c); 315int dbg_debugfs_init_fs(struct ubifs_info *c);
318void dbg_debugfs_exit_fs(struct ubifs_info *c); 316void dbg_debugfs_exit_fs(struct ubifs_info *c);
319 317
320#else /* !CONFIG_UBIFS_FS_DEBUG */
321
322/* Use "if (0)" to make compiler check arguments even if debugging is off */
323#define ubifs_assert(expr) do { \
324 if (0) \
325 printk(KERN_CRIT "UBIFS assert failed in %s at %u (pid %d)\n", \
326 __func__, __LINE__, current->pid); \
327} while (0)
328
329#define dbg_err(fmt, ...) do { \
330 if (0) \
331 ubifs_err(fmt, ##__VA_ARGS__); \
332} while (0)
333
334#define ubifs_dbg_msg(fmt, ...) do { \
335 if (0) \
336 printk(KERN_DEBUG fmt "\n", ##__VA_ARGS__); \
337} while (0)
338
339#define ubifs_assert_cmt_locked(c)
340
341#define dbg_msg(fmt, ...) ubifs_dbg_msg(fmt, ##__VA_ARGS__)
342#define dbg_gen(fmt, ...) ubifs_dbg_msg(fmt, ##__VA_ARGS__)
343#define dbg_jnl(fmt, ...) ubifs_dbg_msg(fmt, ##__VA_ARGS__)
344#define dbg_jnlk(key, fmt, ...) ubifs_dbg_msg(fmt, ##__VA_ARGS__)
345#define dbg_tnc(fmt, ...) ubifs_dbg_msg(fmt, ##__VA_ARGS__)
346#define dbg_tnck(key, fmt, ...) ubifs_dbg_msg(fmt, ##__VA_ARGS__)
347#define dbg_lp(fmt, ...) ubifs_dbg_msg(fmt, ##__VA_ARGS__)
348#define dbg_find(fmt, ...) ubifs_dbg_msg(fmt, ##__VA_ARGS__)
349#define dbg_mnt(fmt, ...) ubifs_dbg_msg(fmt, ##__VA_ARGS__)
350#define dbg_mntk(key, fmt, ...) ubifs_dbg_msg(fmt, ##__VA_ARGS__)
351#define dbg_io(fmt, ...) ubifs_dbg_msg(fmt, ##__VA_ARGS__)
352#define dbg_cmt(fmt, ...) ubifs_dbg_msg(fmt, ##__VA_ARGS__)
353#define dbg_budg(fmt, ...) ubifs_dbg_msg(fmt, ##__VA_ARGS__)
354#define dbg_log(fmt, ...) ubifs_dbg_msg(fmt, ##__VA_ARGS__)
355#define dbg_gc(fmt, ...) ubifs_dbg_msg(fmt, ##__VA_ARGS__)
356#define dbg_scan(fmt, ...) ubifs_dbg_msg(fmt, ##__VA_ARGS__)
357#define dbg_rcvry(fmt, ...) ubifs_dbg_msg(fmt, ##__VA_ARGS__)
358
359static inline int ubifs_debugging_init(struct ubifs_info *c) { return 0; }
360static inline void ubifs_debugging_exit(struct ubifs_info *c) { return; }
361static inline const char *dbg_ntype(int type) { return ""; }
362static inline const char *dbg_cstate(int cmt_state) { return ""; }
363static inline const char *dbg_jhead(int jhead) { return ""; }
364static inline const char *
365dbg_get_key_dump(const struct ubifs_info *c,
366 const union ubifs_key *key) { return ""; }
367static inline const char *
368dbg_snprintf_key(const struct ubifs_info *c,
369 const union ubifs_key *key, char *buffer,
370 int len) { return ""; }
371static inline void ubifs_dump_inode(struct ubifs_info *c,
372 const struct inode *inode) { return; }
373static inline void ubifs_dump_node(const struct ubifs_info *c,
374 const void *node) { return; }
375static inline void
376ubifs_dump_budget_req(const struct ubifs_budget_req *req) { return; }
377static inline void
378ubifs_dump_lstats(const struct ubifs_lp_stats *lst) { return; }
379static inline void
380ubifs_dump_budg(struct ubifs_info *c,
381 const struct ubifs_budg_info *bi) { return; }
382static inline void ubifs_dump_lprop(const struct ubifs_info *c,
383 const struct ubifs_lprops *lp){ return; }
384static inline void ubifs_dump_lprops(struct ubifs_info *c) { return; }
385static inline void ubifs_dump_lpt_info(struct ubifs_info *c) { return; }
386static inline void ubifs_dump_leb(const struct ubifs_info *c,
387 int lnum) { return; }
388static inline void
389ubifs_dump_sleb(const struct ubifs_info *c,
390 const struct ubifs_scan_leb *sleb, int offs) { return; }
391static inline void
392ubifs_dump_znode(const struct ubifs_info *c,
393 const struct ubifs_znode *znode) { return; }
394static inline void ubifs_dump_heap(struct ubifs_info *c,
395 struct ubifs_lpt_heap *heap,
396 int cat) { return; }
397static inline void ubifs_dump_pnode(struct ubifs_info *c,
398 struct ubifs_pnode *pnode,
399 struct ubifs_nnode *parent,
400 int iip) { return; }
401static inline void ubifs_dump_tnc(struct ubifs_info *c) { return; }
402static inline void ubifs_dump_index(struct ubifs_info *c) { return; }
403static inline void ubifs_dump_lpt_lebs(const struct ubifs_info *c){ return; }
404
405static inline int dbg_walk_index(struct ubifs_info *c,
406 dbg_leaf_callback leaf_cb,
407 dbg_znode_callback znode_cb,
408 void *priv) { return 0; }
409static inline void dbg_save_space_info(struct ubifs_info *c) { return; }
410static inline int dbg_check_space_info(struct ubifs_info *c) { return 0; }
411static inline int dbg_check_lprops(struct ubifs_info *c) { return 0; }
412static inline int
413dbg_old_index_check_init(struct ubifs_info *c,
414 struct ubifs_zbranch *zroot) { return 0; }
415static inline int
416dbg_check_old_index(struct ubifs_info *c,
417 struct ubifs_zbranch *zroot) { return 0; }
418static inline int dbg_check_cats(struct ubifs_info *c) { return 0; }
419static inline int dbg_check_ltab(struct ubifs_info *c) { return 0; }
420static inline int dbg_chk_lpt_free_spc(struct ubifs_info *c) { return 0; }
421static inline int dbg_chk_lpt_sz(struct ubifs_info *c,
422 int action, int len) { return 0; }
423static inline int
424dbg_check_synced_i_size(const struct ubifs_info *c,
425 struct inode *inode) { return 0; }
426static inline int dbg_check_dir(struct ubifs_info *c,
427 const struct inode *dir) { return 0; }
428static inline int dbg_check_tnc(struct ubifs_info *c, int extra) { return 0; }
429static inline int dbg_check_idx_size(struct ubifs_info *c,
430 long long idx_size) { return 0; }
431static inline int dbg_check_filesystem(struct ubifs_info *c) { return 0; }
432static inline void dbg_check_heap(struct ubifs_info *c,
433 struct ubifs_lpt_heap *heap,
434 int cat, int add_pos) { return; }
435static inline int dbg_check_lpt_nodes(struct ubifs_info *c,
436 struct ubifs_cnode *cnode, int row, int col) { return 0; }
437static inline int dbg_check_inode_size(struct ubifs_info *c,
438 const struct inode *inode,
439 loff_t size) { return 0; }
440static inline int
441dbg_check_data_nodes_order(struct ubifs_info *c,
442 struct list_head *head) { return 0; }
443static inline int
444dbg_check_nondata_nodes_order(struct ubifs_info *c,
445 struct list_head *head) { return 0; }
446
447static inline int dbg_leb_write(struct ubifs_info *c, int lnum,
448 const void *buf, int offset,
449 int len, int dtype) { return 0; }
450static inline int dbg_leb_change(struct ubifs_info *c, int lnum,
451 const void *buf, int len,
452 int dtype) { return 0; }
453static inline int dbg_leb_unmap(struct ubifs_info *c, int lnum) { return 0; }
454static inline int dbg_leb_map(struct ubifs_info *c, int lnum,
455 int dtype) { return 0; }
456
457static inline int dbg_is_chk_gen(const struct ubifs_info *c) { return 0; }
458static inline int dbg_is_chk_index(const struct ubifs_info *c) { return 0; }
459static inline int dbg_is_chk_orph(const struct ubifs_info *c) { return 0; }
460static inline int dbg_is_chk_lprops(const struct ubifs_info *c) { return 0; }
461static inline int dbg_is_chk_fs(const struct ubifs_info *c) { return 0; }
462static inline int dbg_is_tst_rcvry(const struct ubifs_info *c) { return 0; }
463static inline int dbg_is_power_cut(const struct ubifs_info *c) { return 0; }
464
465static inline int dbg_debugfs_init(void) { return 0; }
466static inline void dbg_debugfs_exit(void) { return; }
467static inline int dbg_debugfs_init_fs(struct ubifs_info *c) { return 0; }
468static inline int dbg_debugfs_exit_fs(struct ubifs_info *c) { return 0; }
469
470#endif /* !CONFIG_UBIFS_FS_DEBUG */
471#endif /* !__UBIFS_DEBUG_H__ */ 318#endif /* !__UBIFS_DEBUG_H__ */
diff --git a/fs/ubifs/dir.c b/fs/ubifs/dir.c
index ad6e550acdc0..62a2727f4ecf 100644
--- a/fs/ubifs/dir.c
+++ b/fs/ubifs/dir.c
@@ -170,8 +170,6 @@ struct inode *ubifs_new_inode(struct ubifs_info *c, const struct inode *dir,
170 return inode; 170 return inode;
171} 171}
172 172
173#ifdef CONFIG_UBIFS_FS_DEBUG
174
175static int dbg_check_name(const struct ubifs_info *c, 173static int dbg_check_name(const struct ubifs_info *c,
176 const struct ubifs_dent_node *dent, 174 const struct ubifs_dent_node *dent,
177 const struct qstr *nm) 175 const struct qstr *nm)
@@ -185,12 +183,6 @@ static int dbg_check_name(const struct ubifs_info *c,
185 return 0; 183 return 0;
186} 184}
187 185
188#else
189
190#define dbg_check_name(c, dent, nm) 0
191
192#endif
193
194static struct dentry *ubifs_lookup(struct inode *dir, struct dentry *dentry, 186static struct dentry *ubifs_lookup(struct inode *dir, struct dentry *dentry,
195 struct nameidata *nd) 187 struct nameidata *nd)
196{ 188{
diff --git a/fs/ubifs/log.c b/fs/ubifs/log.c
index f9fd068d1ae0..9967b5a21f41 100644
--- a/fs/ubifs/log.c
+++ b/fs/ubifs/log.c
@@ -29,11 +29,7 @@
29 29
30#include "ubifs.h" 30#include "ubifs.h"
31 31
32#ifdef CONFIG_UBIFS_FS_DEBUG
33static int dbg_check_bud_bytes(struct ubifs_info *c); 32static int dbg_check_bud_bytes(struct ubifs_info *c);
34#else
35#define dbg_check_bud_bytes(c) 0
36#endif
37 33
38/** 34/**
39 * ubifs_search_bud - search bud LEB. 35 * ubifs_search_bud - search bud LEB.
@@ -734,8 +730,6 @@ out_free:
734 return err; 730 return err;
735} 731}
736 732
737#ifdef CONFIG_UBIFS_FS_DEBUG
738
739/** 733/**
740 * dbg_check_bud_bytes - make sure bud bytes calculation are all right. 734 * dbg_check_bud_bytes - make sure bud bytes calculation are all right.
741 * @c: UBIFS file-system description object 735 * @c: UBIFS file-system description object
@@ -767,5 +761,3 @@ static int dbg_check_bud_bytes(struct ubifs_info *c)
767 761
768 return err; 762 return err;
769} 763}
770
771#endif /* CONFIG_UBIFS_FS_DEBUG */
diff --git a/fs/ubifs/lprops.c b/fs/ubifs/lprops.c
index 0cf7a18e174c..86eb8e533249 100644
--- a/fs/ubifs/lprops.c
+++ b/fs/ubifs/lprops.c
@@ -846,7 +846,9 @@ const struct ubifs_lprops *ubifs_fast_find_frdi_idx(struct ubifs_info *c)
846 return lprops; 846 return lprops;
847} 847}
848 848
849#ifdef CONFIG_UBIFS_FS_DEBUG 849/*
850 * Everything below is related to debugging.
851 */
850 852
851/** 853/**
852 * dbg_check_cats - check category heaps and lists. 854 * dbg_check_cats - check category heaps and lists.
@@ -1315,5 +1317,3 @@ int dbg_check_lprops(struct ubifs_info *c)
1315out: 1317out:
1316 return err; 1318 return err;
1317} 1319}
1318
1319#endif /* CONFIG_UBIFS_FS_DEBUG */
diff --git a/fs/ubifs/lpt.c b/fs/ubifs/lpt.c
index b6db389654b0..5d1d97d00966 100644
--- a/fs/ubifs/lpt.c
+++ b/fs/ubifs/lpt.c
@@ -2080,8 +2080,6 @@ out:
2080 return err; 2080 return err;
2081} 2081}
2082 2082
2083#ifdef CONFIG_UBIFS_FS_DEBUG
2084
2085/** 2083/**
2086 * dbg_chk_pnode - check a pnode. 2084 * dbg_chk_pnode - check a pnode.
2087 * @c: the UBIFS file-system description object 2085 * @c: the UBIFS file-system description object
@@ -2274,5 +2272,3 @@ int dbg_check_lpt_nodes(struct ubifs_info *c, struct ubifs_cnode *cnode,
2274 } 2272 }
2275 return 0; 2273 return 0;
2276} 2274}
2277
2278#endif /* CONFIG_UBIFS_FS_DEBUG */
diff --git a/fs/ubifs/lpt_commit.c b/fs/ubifs/lpt_commit.c
index 8294d5af87b3..97822db3afbc 100644
--- a/fs/ubifs/lpt_commit.c
+++ b/fs/ubifs/lpt_commit.c
@@ -30,11 +30,7 @@
30#include <linux/random.h> 30#include <linux/random.h>
31#include "ubifs.h" 31#include "ubifs.h"
32 32
33#ifdef CONFIG_UBIFS_FS_DEBUG
34static int dbg_populate_lsave(struct ubifs_info *c); 33static int dbg_populate_lsave(struct ubifs_info *c);
35#else
36#define dbg_populate_lsave(c) 0
37#endif
38 34
39/** 35/**
40 * first_dirty_cnode - find first dirty cnode. 36 * first_dirty_cnode - find first dirty cnode.
@@ -1497,7 +1493,9 @@ void ubifs_lpt_free(struct ubifs_info *c, int wr_only)
1497 kfree(c->lpt_nod_buf); 1493 kfree(c->lpt_nod_buf);
1498} 1494}
1499 1495
1500#ifdef CONFIG_UBIFS_FS_DEBUG 1496/*
1497 * Everything below is related to debugging.
1498 */
1501 1499
1502/** 1500/**
1503 * dbg_is_all_ff - determine if a buffer contains only 0xFF bytes. 1501 * dbg_is_all_ff - determine if a buffer contains only 0xFF bytes.
@@ -2046,5 +2044,3 @@ static int dbg_populate_lsave(struct ubifs_info *c)
2046 2044
2047 return 1; 2045 return 1;
2048} 2046}
2049
2050#endif /* CONFIG_UBIFS_FS_DEBUG */
diff --git a/fs/ubifs/orphan.c b/fs/ubifs/orphan.c
index 48ac7212e78b..97ad042e4715 100644
--- a/fs/ubifs/orphan.c
+++ b/fs/ubifs/orphan.c
@@ -52,11 +52,7 @@
52 * than the maximum number of orphans allowed. 52 * than the maximum number of orphans allowed.
53 */ 53 */
54 54
55#ifdef CONFIG_UBIFS_FS_DEBUG
56static int dbg_check_orphans(struct ubifs_info *c); 55static int dbg_check_orphans(struct ubifs_info *c);
57#else
58#define dbg_check_orphans(c) 0
59#endif
60 56
61/** 57/**
62 * ubifs_add_orphan - add an orphan. 58 * ubifs_add_orphan - add an orphan.
@@ -725,7 +721,9 @@ int ubifs_mount_orphans(struct ubifs_info *c, int unclean, int read_only)
725 return err; 721 return err;
726} 722}
727 723
728#ifdef CONFIG_UBIFS_FS_DEBUG 724/*
725 * Everything below is related to debugging.
726 */
729 727
730struct check_orphan { 728struct check_orphan {
731 struct rb_node rb; 729 struct rb_node rb;
@@ -968,5 +966,3 @@ out:
968 kfree(ci.node); 966 kfree(ci.node);
969 return err; 967 return err;
970} 968}
971
972#endif /* CONFIG_UBIFS_FS_DEBUG */
diff --git a/fs/ubifs/sb.c b/fs/ubifs/sb.c
index c51f6a4af0f1..a880d0c152e5 100644
--- a/fs/ubifs/sb.c
+++ b/fs/ubifs/sb.c
@@ -130,7 +130,6 @@ static int create_default_filesystem(struct ubifs_info *c)
130 * orphan node. 130 * orphan node.
131 */ 131 */
132 orph_lebs = UBIFS_MIN_ORPH_LEBS; 132 orph_lebs = UBIFS_MIN_ORPH_LEBS;
133#ifdef CONFIG_UBIFS_FS_DEBUG
134 if (c->leb_cnt - min_leb_cnt > 1) 133 if (c->leb_cnt - min_leb_cnt > 1)
135 /* 134 /*
136 * For debugging purposes it is better to have at least 2 135 * For debugging purposes it is better to have at least 2
@@ -138,7 +137,6 @@ static int create_default_filesystem(struct ubifs_info *c)
138 * consolidations and would be stressed more. 137 * consolidations and would be stressed more.
139 */ 138 */
140 orph_lebs += 1; 139 orph_lebs += 1;
141#endif
142 140
143 main_lebs = c->leb_cnt - UBIFS_SB_LEBS - UBIFS_MST_LEBS - log_lebs; 141 main_lebs = c->leb_cnt - UBIFS_SB_LEBS - UBIFS_MST_LEBS - log_lebs;
144 main_lebs -= orph_lebs; 142 main_lebs -= orph_lebs;
diff --git a/fs/ubifs/tnc.c b/fs/ubifs/tnc.c
index 12c25d925d3f..a50b6bd2f4ce 100644
--- a/fs/ubifs/tnc.c
+++ b/fs/ubifs/tnc.c
@@ -3275,8 +3275,6 @@ out_unlock:
3275 return err; 3275 return err;
3276} 3276}
3277 3277
3278#ifdef CONFIG_UBIFS_FS_DEBUG
3279
3280/** 3278/**
3281 * dbg_check_inode_size - check if inode size is correct. 3279 * dbg_check_inode_size - check if inode size is correct.
3282 * @c: UBIFS file-system description object 3280 * @c: UBIFS file-system description object
@@ -3343,5 +3341,3 @@ out_unlock:
3343 mutex_unlock(&c->tnc_mutex); 3341 mutex_unlock(&c->tnc_mutex);
3344 return err; 3342 return err;
3345} 3343}
3346
3347#endif /* CONFIG_UBIFS_FS_DEBUG */
diff --git a/fs/ubifs/tnc_commit.c b/fs/ubifs/tnc_commit.c
index e8cf891185ec..ef5df1e24dca 100644
--- a/fs/ubifs/tnc_commit.c
+++ b/fs/ubifs/tnc_commit.c
@@ -61,11 +61,9 @@ static int make_idx_node(struct ubifs_info *c, struct ubifs_idx_node *idx,
61 } 61 }
62 ubifs_prepare_node(c, idx, len, 0); 62 ubifs_prepare_node(c, idx, len, 0);
63 63
64#ifdef CONFIG_UBIFS_FS_DEBUG
65 znode->lnum = lnum; 64 znode->lnum = lnum;
66 znode->offs = offs; 65 znode->offs = offs;
67 znode->len = len; 66 znode->len = len;
68#endif
69 67
70 err = insert_old_idx_znode(c, znode); 68 err = insert_old_idx_znode(c, znode);
71 69
@@ -456,11 +454,9 @@ static int layout_in_empty_space(struct ubifs_info *c)
456 454
457 offs = buf_offs + used; 455 offs = buf_offs + used;
458 456
459#ifdef CONFIG_UBIFS_FS_DEBUG
460 znode->lnum = lnum; 457 znode->lnum = lnum;
461 znode->offs = offs; 458 znode->offs = offs;
462 znode->len = len; 459 znode->len = len;
463#endif
464 460
465 /* Update the parent */ 461 /* Update the parent */
466 zp = znode->parent; 462 zp = znode->parent;
@@ -536,10 +532,8 @@ static int layout_in_empty_space(struct ubifs_info *c)
536 break; 532 break;
537 } 533 }
538 534
539#ifdef CONFIG_UBIFS_FS_DEBUG
540 c->dbg->new_ihead_lnum = lnum; 535 c->dbg->new_ihead_lnum = lnum;
541 c->dbg->new_ihead_offs = buf_offs; 536 c->dbg->new_ihead_offs = buf_offs;
542#endif
543 537
544 return 0; 538 return 0;
545} 539}
@@ -881,13 +875,11 @@ static int write_index(struct ubifs_info *c)
881 } 875 }
882 offs = buf_offs + used; 876 offs = buf_offs + used;
883 877
884#ifdef CONFIG_UBIFS_FS_DEBUG
885 if (lnum != znode->lnum || offs != znode->offs || 878 if (lnum != znode->lnum || offs != znode->offs ||
886 len != znode->len) { 879 len != znode->len) {
887 ubifs_err("inconsistent znode posn"); 880 ubifs_err("inconsistent znode posn");
888 return -EINVAL; 881 return -EINVAL;
889 } 882 }
890#endif
891 883
892 /* Grab some stuff from znode while we still can */ 884 /* Grab some stuff from znode while we still can */
893 cnext = znode->cnext; 885 cnext = znode->cnext;
@@ -982,13 +974,11 @@ static int write_index(struct ubifs_info *c)
982 break; 974 break;
983 } 975 }
984 976
985#ifdef CONFIG_UBIFS_FS_DEBUG
986 if (lnum != c->dbg->new_ihead_lnum || 977 if (lnum != c->dbg->new_ihead_lnum ||
987 buf_offs != c->dbg->new_ihead_offs) { 978 buf_offs != c->dbg->new_ihead_offs) {
988 ubifs_err("inconsistent ihead"); 979 ubifs_err("inconsistent ihead");
989 return -EINVAL; 980 return -EINVAL;
990 } 981 }
991#endif
992 982
993 c->ihead_lnum = lnum; 983 c->ihead_lnum = lnum;
994 c->ihead_offs = buf_offs; 984 c->ihead_offs = buf_offs;
diff --git a/fs/ubifs/ubifs.h b/fs/ubifs/ubifs.h
index 93d59aceaaef..c2cf509e14db 100644
--- a/fs/ubifs/ubifs.h
+++ b/fs/ubifs/ubifs.h
@@ -762,6 +762,9 @@ struct ubifs_zbranch {
762 * @offs: offset of the corresponding indexing node 762 * @offs: offset of the corresponding indexing node
763 * @len: length of the corresponding indexing node 763 * @len: length of the corresponding indexing node
764 * @zbranch: array of znode branches (@c->fanout elements) 764 * @zbranch: array of znode branches (@c->fanout elements)
765 *
766 * Note! The @lnum, @offs, and @len fields are not really needed - we have them
767 * only for internal consistency check. They could be removed to save some RAM.
765 */ 768 */
766struct ubifs_znode { 769struct ubifs_znode {
767 struct ubifs_znode *parent; 770 struct ubifs_znode *parent;
@@ -772,9 +775,9 @@ struct ubifs_znode {
772 int child_cnt; 775 int child_cnt;
773 int iip; 776 int iip;
774 int alt; 777 int alt;
775#ifdef CONFIG_UBIFS_FS_DEBUG 778 int lnum;
776 int lnum, offs, len; 779 int offs;
777#endif 780 int len;
778 struct ubifs_zbranch zbranch[]; 781 struct ubifs_zbranch zbranch[];
779}; 782};
780 783
@@ -1444,9 +1447,7 @@ struct ubifs_info {
1444 struct rb_root size_tree; 1447 struct rb_root size_tree;
1445 struct ubifs_mount_opts mount_opts; 1448 struct ubifs_mount_opts mount_opts;
1446 1449
1447#ifdef CONFIG_UBIFS_FS_DEBUG
1448 struct ubifs_debug_info *dbg; 1450 struct ubifs_debug_info *dbg;
1449#endif
1450}; 1451};
1451 1452
1452extern struct list_head ubifs_infos; 1453extern struct list_head ubifs_infos;