[JFFS2] Debug code clean up - step 3
authorArtem B. Bityutskiy <dedekind@infradead.org>
Sun, 24 Jul 2005 15:14:17 +0000 (16:14 +0100)
committerThomas Gleixner <tglx@mtd.linutronix.de>
Sun, 6 Nov 2005 16:06:49 +0000 (17:06 +0100)
Various simplifiactions. printk format corrections.
Convert more code to use the new debug functions.

Signed-off-by: Artem B. Bityutskiy <dedekind@infradead.org>
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
14 files changed:
fs/jffs2/build.c
fs/jffs2/debug.c
fs/jffs2/debug.h
fs/jffs2/erase.c
fs/jffs2/fs.c
fs/jffs2/gc.c
fs/jffs2/nodelist.c
fs/jffs2/nodelist.h
fs/jffs2/nodemgmt.c
fs/jffs2/read.c
fs/jffs2/readinode.c
fs/jffs2/scan.c
fs/jffs2/wbuf.c
fs/jffs2/write.c

index b87d4010d36f3b5f99fd63646c3f1bac9bb3a746..f08984a64a5a0073d19e3c41be2620dc8281ef6d 100644 (file)
@@ -7,7 +7,7 @@
  *
  * For licensing information, see the file 'LICENCE' in this directory.
  *
- * $Id: build.c,v 1.74 2005/07/17 12:01:42 dedekind Exp $
+ * $Id: build.c,v 1.75 2005/07/22 10:32:07 dedekind Exp $
  *
  */
 
@@ -104,7 +104,7 @@ static int jffs2_build_filesystem(struct jffs2_sb_info *c)
                goto exit;
 
        D1(printk(KERN_DEBUG "Scanned flash completely\n"));
-       D2(jffs2_dbg_dump_block_lists(c));
+       jffs2_dbg_dump_block_lists_nolock(c);
 
        c->flags |= JFFS2_SB_FLAG_BUILDING;
        /* Now scan the directory tree, increasing nlink according to every dirent found. */
@@ -168,7 +168,7 @@ static int jffs2_build_filesystem(struct jffs2_sb_info *c)
        c->flags &= ~JFFS2_SB_FLAG_BUILDING;
        
        D1(printk(KERN_DEBUG "Pass 3 complete\n"));
-       jffs2_dbg_dump_block_lists(c);
+       jffs2_dbg_dump_block_lists_nolock(c);
 
        /* Rotate the lists by some number to ensure wear levelling */
        jffs2_rotate_lists(c);
index 9da524ca4e66ea813503961b2b0da80bd00dede5..fb88eb19711b7338f10b3dbd53ead5c2c20c2973 100644 (file)
@@ -7,18 +7,30 @@
  *
  * For licensing information, see the file 'LICENCE' in this directory.
  *
- * $Id: debug.c,v 1.1 2005/07/17 06:56:20 dedekind Exp $
+ * $Id: debug.c,v 1.7 2005/07/24 15:14:14 dedekind Exp $
  *
  */
 #include <linux/kernel.h>
 #include <linux/pagemap.h>
+#include <linux/crc32.h>
+#include <linux/jffs2.h>
 #include "nodelist.h"
 #include "debug.h"
 
 #ifdef JFFS2_DBG_PARANOIA_CHECKS
-
+/*
+ * Check the fragtree.
+ */
 void
-jffs2_dbg_fragtree_paranoia_check(struct jffs2_inode_info *f)
+__jffs2_dbg_fragtree_paranoia_check(struct jffs2_inode_info *f)
+{
+       down(&f->sem);
+       __jffs2_dbg_fragtree_paranoia_check_nolock(f);
+       up(&f->sem);
+}
+       
+void
+__jffs2_dbg_fragtree_paranoia_check_nolock(struct jffs2_inode_info *f)
 {
        struct jffs2_node_frag *frag;
        int bitched = 0;
@@ -31,7 +43,7 @@ jffs2_dbg_fragtree_paranoia_check(struct jffs2_inode_info *f)
 
                if (ref_flags(fn->raw) == REF_PRISTINE) {
                        if (fn->frags > 1) {
-                               printk(KERN_ERR "REF_PRISTINE node at 0x%08x had %d frags. Tell dwmw2\n",
+                               JFFS2_ERROR("REF_PRISTINE node at 0x%08x had %d frags. Tell dwmw2.\n",
                                                ref_offset(fn->raw), fn->frags);
                                bitched = 1;
                        }
@@ -43,15 +55,15 @@ jffs2_dbg_fragtree_paranoia_check(struct jffs2_inode_info *f)
                           to tell a hole node. */
                        if (frag->ofs & (PAGE_CACHE_SIZE-1) && frag_prev(frag)
                                        && frag_prev(frag)->size < PAGE_CACHE_SIZE && frag_prev(frag)->node) {
-                               printk(KERN_ERR "REF_PRISTINE node at 0x%08x had a previous non-hole frag "
-                                               "in the same page. Tell dwmw2\n", ref_offset(fn->raw));
+                               JFFS2_ERROR("REF_PRISTINE node at 0x%08x had a previous non-hole frag "
+                                               "in the same page. Tell dwmw2.\n", ref_offset(fn->raw));
                                bitched = 1;
                        }
 
                        if ((frag->ofs+frag->size) & (PAGE_CACHE_SIZE-1) && frag_next(frag)
                                        && frag_next(frag)->size < PAGE_CACHE_SIZE && frag_next(frag)->node) {
-                               printk(KERN_ERR "REF_PRISTINE node at 0x%08x (%08x-%08x) had a following "
-                                               "non-hole frag in the same page. Tell dwmw2\n",
+                               JFFS2_ERROR("REF_PRISTINE node at 0x%08x (%08x-%08x) had a following "
+                                               "non-hole frag in the same page. Tell dwmw2.\n",
                                               ref_offset(fn->raw), frag->ofs, frag->ofs+frag->size);
                                bitched = 1;
                        }
@@ -59,8 +71,8 @@ jffs2_dbg_fragtree_paranoia_check(struct jffs2_inode_info *f)
        }
 
        if (bitched) {
-               printk(KERN_ERR "Fragtree is corrupted. Fragtree dump:\n");
-               jffs2_dbg_dump_fragtree(f);
+               JFFS2_ERROR("fragtree is corrupted.\n");
+               __jffs2_dbg_dump_fragtree_nolock(f);
                BUG();
        }
 }
@@ -69,7 +81,8 @@ jffs2_dbg_fragtree_paranoia_check(struct jffs2_inode_info *f)
  * Check if the flash contains all 0xFF before we start writing.
  */
 void
-jffs2_dbg_prewrite_paranoia_check(struct jffs2_sb_info *c, uint32_t ofs, int len)
+__jffs2_dbg_prewrite_paranoia_check(struct jffs2_sb_info *c,
+                                   uint32_t ofs, int len)
 {
        size_t retlen;
        int ret, i;
@@ -81,8 +94,8 @@ jffs2_dbg_prewrite_paranoia_check(struct jffs2_sb_info *c, uint32_t ofs, int len
 
        ret = jffs2_flash_read(c, ofs, len, &retlen, buf);
        if (ret || (retlen != len)) {
-               printk(KERN_WARNING "read %d bytes failed or short in %s(). ret %d, retlen %zd\n",
-                               len, __FUNCTION__, ret, retlen);
+               JFFS2_WARNING("read %d bytes failed or short. ret %d, retlen %zd.\n",
+                               len, ret, retlen);
                kfree(buf);
                return;
        }
@@ -93,9 +106,9 @@ jffs2_dbg_prewrite_paranoia_check(struct jffs2_sb_info *c, uint32_t ofs, int len
                        ret = 1;
 
        if (ret) {
-               printk(KERN_ERR "ARGH. About to write node to %#08x on flash, but there are data "
-                               "already there. The first corrupted byte is at %#08x.\n", ofs, ofs + i);
-               jffs2_dbg_dump_buffer(buf, len, ofs);
+               JFFS2_ERROR("argh, about to write node to %#08x on flash, but there are data "
+                       "already there. The first corrupted byte is at %#08x offset.\n", ofs, ofs + i);
+               __jffs2_dbg_dump_buffer(buf, len, ofs);
                kfree(buf);
                BUG();
        }
@@ -107,7 +120,17 @@ jffs2_dbg_prewrite_paranoia_check(struct jffs2_sb_info *c, uint32_t ofs, int len
  * Check the space accounting and node_ref list correctness for the JFFS2 erasable block 'jeb'.
  */
 void
-jffs2_dbg_acct_paranoia_check(struct jffs2_sb_info *c, struct jffs2_eraseblock *jeb)
+__jffs2_dbg_acct_paranoia_check(struct jffs2_sb_info *c,
+                               struct jffs2_eraseblock *jeb)
+{
+       spin_lock(&c->erase_completion_lock);
+       __jffs2_dbg_acct_paranoia_check_nolock(c, jeb);
+       spin_unlock(&c->erase_completion_lock);
+}
+       
+void
+__jffs2_dbg_acct_paranoia_check_nolock(struct jffs2_sb_info *c,
+                                      struct jffs2_eraseblock *jeb)
 {
        uint32_t my_used_size = 0;
        uint32_t my_unchecked_size = 0;
@@ -119,11 +142,9 @@ jffs2_dbg_acct_paranoia_check(struct jffs2_sb_info *c, struct jffs2_eraseblock *
 
                if (ref2->flash_offset < jeb->offset ||
                                ref2->flash_offset > jeb->offset + c->sector_size) {
-                       printk(KERN_ERR "node_ref %#08x shouldn't be in block at %#08x!\n",
+                       JFFS2_ERROR("node_ref %#08x shouldn't be in block at %#08x.\n",
                                ref_offset(ref2), jeb->offset);
-                       jffs2_dbg_dump_node_refs(c, jeb);
-                       jffs2_dbg_dump_block_lists(c);
-                       BUG();
+                       goto error;
 
                }
                if (ref_flags(ref2) == REF_UNCHECKED)
@@ -134,69 +155,82 @@ jffs2_dbg_acct_paranoia_check(struct jffs2_sb_info *c, struct jffs2_eraseblock *
                        my_dirty_size += totlen;
 
                if ((!ref2->next_phys) != (ref2 == jeb->last_node)) {
-                       printk(KERN_ERR "node_ref for node at %#08x (mem %p) has next_phys at %#08x (mem %p), "
-                                       "last_node is at %#08x (mem %p)\n",
-                                       ref_offset(ref2), ref2, ref_offset(ref2->next_phys), ref2->next_phys,
-                                       ref_offset(jeb->last_node), jeb->last_node);
-                       jffs2_dbg_dump_node_refs(c, jeb);
-                       jffs2_dbg_dump_block_lists(c);
-                       BUG();
+                       JFFS2_ERROR("node_ref for node at %#08x (mem %p) has next_phys at %#08x (mem %p), "
+                               "last_node is at %#08x (mem %p).\n",
+                               ref_offset(ref2), ref2, ref_offset(ref2->next_phys), ref2->next_phys,
+                               ref_offset(jeb->last_node), jeb->last_node);
+                       goto error;
                }
                ref2 = ref2->next_phys;
        }
 
        if (my_used_size != jeb->used_size) {
-               printk(KERN_ERR "Calculated used size %#08x != stored used size %#08x\n",
-                               my_used_size, jeb->used_size);
-               jffs2_dbg_dump_node_refs(c, jeb);
-               jffs2_dbg_dump_block_lists(c);
-               BUG();
+               JFFS2_ERROR("Calculated used size %#08x != stored used size %#08x.\n",
+                       my_used_size, jeb->used_size);
+               goto error;
        }
 
        if (my_unchecked_size != jeb->unchecked_size) {
-               printk(KERN_ERR "Calculated unchecked size %#08x != stored unchecked size %#08x\n",
-                               my_unchecked_size, jeb->unchecked_size);
-               jffs2_dbg_dump_node_refs(c, jeb);
-               jffs2_dbg_dump_block_lists(c);
-               BUG();
+               JFFS2_ERROR("Calculated unchecked size %#08x != stored unchecked size %#08x.\n",
+                       my_unchecked_size, jeb->unchecked_size);
+               goto error;
        }
 
+#if 0
+       /* This should work when we implement ref->__totlen elemination */
        if (my_dirty_size != jeb->dirty_size + jeb->wasted_size) {
-               printk(KERN_ERR "Calculated dirty+wasted size %#08x != stored dirty + wasted size %#08x\n",
+               JFFS2_ERROR("Calculated dirty+wasted size %#08x != stored dirty + wasted size %#08x\n",
                        my_dirty_size, jeb->dirty_size + jeb->wasted_size);
-               jffs2_dbg_dump_node_refs(c, jeb);
-               jffs2_dbg_dump_block_lists(c);
-               BUG();
+               goto error;
        }
 
        if (jeb->free_size == 0
                && my_used_size + my_unchecked_size + my_dirty_size != c->sector_size) {
-               printk(KERN_ERR "The sum of all nodes in block (%#x) != size of block (%#x)\n",
+               JFFS2_ERROR("The sum of all nodes in block (%#x) != size of block (%#x)\n",
                        my_used_size + my_unchecked_size + my_dirty_size,
                        c->sector_size);
-               jffs2_dbg_dump_node_refs(c, jeb);
-               jffs2_dbg_dump_block_lists(c);
-               BUG();
+               goto error;
        }
+#endif
+
+       return;
+
+error:
+       __jffs2_dbg_dump_node_refs_nolock(c, jeb);
+       __jffs2_dbg_dump_jeb_nolock(jeb);
+       __jffs2_dbg_dump_block_lists_nolock(c);
+       BUG();
+       
 }
-#endif /* JFFS2_PARANOIA_CHECKS */
+#endif /* JFFS2_DBG_PARANOIA_CHECKS */
 
-#if defined(JFFS2_PARANOIA_CHECKS) || (CONFIG_JFFS2_FS_DEBUG > 0)
+#if defined(JFFS2_DBG_DUMPS) || defined(JFFS2_DBG_PARANOIA_CHECKS)
 /*
  * Dump the node_refs of the 'jeb' JFFS2 eraseblock.
  */
 void
-jffs2_dbg_dump_node_refs(struct jffs2_sb_info *c, struct jffs2_eraseblock *jeb)
+__jffs2_dbg_dump_node_refs(struct jffs2_sb_info *c,
+                          struct jffs2_eraseblock *jeb)
+{
+       spin_lock(&c->erase_completion_lock);
+       __jffs2_dbg_dump_node_refs_nolock(c, jeb);
+       spin_unlock(&c->erase_completion_lock);
+}
+
+void
+__jffs2_dbg_dump_node_refs_nolock(struct jffs2_sb_info *c,
+                                 struct jffs2_eraseblock *jeb)
 {
        struct jffs2_raw_node_ref *ref;
        int i = 0;
 
+       JFFS2_DEBUG("Dump node_refs of the eraseblock %#08x\n", jeb->offset);
        if (!jeb->first_node) {
-               printk(KERN_DEBUG "no nodes in block %#08x\n", jeb->offset);
+               JFFS2_DEBUG("no nodes in the eraseblock %#08x\n", jeb->offset);
                return;
        }
 
-       printk(KERN_DEBUG);
+       printk(JFFS2_DBG_LVL);
        for (ref = jeb->first_node; ; ref = ref->next_phys) {
                printk("%#08x(%#x)", ref_offset(ref), ref->__totlen);
                if (ref->next_phys)
@@ -205,46 +239,83 @@ jffs2_dbg_dump_node_refs(struct jffs2_sb_info *c, struct jffs2_eraseblock *jeb)
                        break;
                if (++i == 4) {
                        i = 0;
-                       printk("\n" KERN_DEBUG);
+                       printk("\n" JFFS2_DBG_LVL);
                }
        }
        printk("\n");
 }
 
+/*
+ * Dump an eraseblock's space accounting.
+ */
+void
+__jffs2_dbg_dump_jeb(struct jffs2_sb_info *c, struct jffs2_eraseblock *jeb)
+{
+       spin_lock(&c->erase_completion_lock);
+       __jffs2_dbg_dump_jeb_nolock(jeb);
+       spin_unlock(&c->erase_completion_lock);
+}
+
 void
-jffs2_dbg_dump_block_lists(struct jffs2_sb_info *c)
+__jffs2_dbg_dump_jeb_nolock(struct jffs2_eraseblock *jeb)
 {
-       printk(KERN_DEBUG "flash_size: %#08x\n",        c->flash_size);
-       printk(KERN_DEBUG "used_size: %#08x\n",         c->used_size);
-       printk(KERN_DEBUG "dirty_size: %#08x\n",        c->dirty_size);
-       printk(KERN_DEBUG "wasted_size: %#08x\n",       c->wasted_size);
-       printk(KERN_DEBUG "unchecked_size: %#08x\n",    c->unchecked_size);
-       printk(KERN_DEBUG "free_size: %#08x\n",         c->free_size);
-       printk(KERN_DEBUG "erasing_size: %#08x\n",      c->erasing_size);
-       printk(KERN_DEBUG "bad_size: %#08x\n",          c->bad_size);
-       printk(KERN_DEBUG "sector_size: %#08x\n",       c->sector_size);
-       printk(KERN_DEBUG "jffs2_reserved_blocks size: %#08x\n",
+       if (!jeb)
+               return;
+
+       JFFS2_DEBUG("dump space accounting for the eraseblock at %#08x:\n",
+                       jeb->offset);
+
+       printk(JFFS2_DBG_LVL "used_size: %#08x\n",      jeb->used_size);
+       printk(JFFS2_DBG_LVL "dirty_size: %#08x\n",     jeb->dirty_size);
+       printk(JFFS2_DBG_LVL "wasted_size: %#08x\n",    jeb->wasted_size);
+       printk(JFFS2_DBG_LVL "unchecked_size: %#08x\n", jeb->unchecked_size);
+       printk(JFFS2_DBG_LVL "free_size: %#08x\n",      jeb->free_size);
+}
+
+void
+__jffs2_dbg_dump_block_lists(struct jffs2_sb_info *c)
+{
+       spin_lock(&c->erase_completion_lock);
+       __jffs2_dbg_dump_block_lists_nolock(c);
+       spin_unlock(&c->erase_completion_lock);
+}
+
+void
+__jffs2_dbg_dump_block_lists_nolock(struct jffs2_sb_info *c)
+{
+       JFFS2_DEBUG("dump JFFS2 blocks lists:\n");
+       
+       printk(JFFS2_DBG_LVL "flash_size: %#08x\n",     c->flash_size);
+       printk(JFFS2_DBG_LVL "used_size: %#08x\n",      c->used_size);
+       printk(JFFS2_DBG_LVL "dirty_size: %#08x\n",     c->dirty_size);
+       printk(JFFS2_DBG_LVL "wasted_size: %#08x\n",    c->wasted_size);
+       printk(JFFS2_DBG_LVL "unchecked_size: %#08x\n", c->unchecked_size);
+       printk(JFFS2_DBG_LVL "free_size: %#08x\n",      c->free_size);
+       printk(JFFS2_DBG_LVL "erasing_size: %#08x\n",   c->erasing_size);
+       printk(JFFS2_DBG_LVL "bad_size: %#08x\n",       c->bad_size);
+       printk(JFFS2_DBG_LVL "sector_size: %#08x\n",    c->sector_size);
+       printk(JFFS2_DBG_LVL "jffs2_reserved_blocks size: %#08x\n",
                                c->sector_size * c->resv_blocks_write);
 
        if (c->nextblock)
-               printk(KERN_DEBUG "nextblock: %#08x (used %#08x, dirty %#08x, wasted %#08x, "
-                               "unchecked %#08x, free %#08x)\n",
-                               c->nextblock->offset, c->nextblock->used_size,
-                               c->nextblock->dirty_size, c->nextblock->wasted_size,
-                               c->nextblock->unchecked_size, c->nextblock->free_size);
+               printk(JFFS2_DBG_LVL "nextblock: %#08x (used %#08x, dirty %#08x, wasted %#08x, "
+                       "unchecked %#08x, free %#08x)\n",
+                       c->nextblock->offset, c->nextblock->used_size,
+                       c->nextblock->dirty_size, c->nextblock->wasted_size,
+                       c->nextblock->unchecked_size, c->nextblock->free_size);
        else
-               printk(KERN_DEBUG "nextblock: NULL\n");
+               printk(JFFS2_DBG_LVL "nextblock: NULL\n");
 
        if (c->gcblock)
-               printk(KERN_DEBUG "gcblock: %#08x (used %#08x, dirty %#08x, wasted %#08x, "
-                               "unchecked %#08x, free %#08x)\n",
-                               c->gcblock->offset, c->gcblock->used_size, c->gcblock->dirty_size,
-                               c->gcblock->wasted_size, c->gcblock->unchecked_size, c->gcblock->free_size);
+               printk(JFFS2_DBG_LVL "gcblock: %#08x (used %#08x, dirty %#08x, wasted %#08x, "
+                       "unchecked %#08x, free %#08x)\n",
+                       c->gcblock->offset, c->gcblock->used_size, c->gcblock->dirty_size,
+                       c->gcblock->wasted_size, c->gcblock->unchecked_size, c->gcblock->free_size);
        else
-               printk(KERN_DEBUG "gcblock: NULL\n");
+               printk(JFFS2_DBG_LVL "gcblock: NULL\n");
 
        if (list_empty(&c->clean_list)) {
-               printk(KERN_DEBUG "clean_list: empty\n");
+               printk(JFFS2_DBG_LVL "clean_list: empty\n");
        } else {
                struct list_head *this;
                int numblocks = 0;
@@ -255,19 +326,19 @@ jffs2_dbg_dump_block_lists(struct jffs2_sb_info *c)
                        numblocks ++;
                        dirty += jeb->wasted_size;
                        if (!(jeb->used_size == 0 && jeb->dirty_size == 0 && jeb->wasted_size == 0)) {
-                               printk(KERN_DEBUG "clean_list: %#08x (used %#08x, dirty %#08x, wasted %#08x, "
-                                               "unchecked %#08x, free %#08x)\n",
-                                               jeb->offset, jeb->used_size, jeb->dirty_size, jeb->wasted_size,
-                                               jeb->unchecked_size, jeb->free_size);
+                               printk(JFFS2_DBG_LVL "clean_list: %#08x (used %#08x, dirty %#08x, wasted %#08x, "
+                                       "unchecked %#08x, free %#08x)\n",
+                                       jeb->offset, jeb->used_size, jeb->dirty_size, jeb->wasted_size,
+                                       jeb->unchecked_size, jeb->free_size);
                        }
                }
 
-               printk (KERN_DEBUG "Contains %d blocks with total wasted size %u, average wasted size: %u\n",
-                               numblocks, dirty, dirty / numblocks);
+               printk (JFFS2_DBG_LVL "Contains %d blocks with total wasted size %u, average wasted size: %u\n",
+                       numblocks, dirty, dirty / numblocks);
        }
 
        if (list_empty(&c->very_dirty_list)) {
-               printk(KERN_DEBUG "very_dirty_list: empty\n");
+               printk(JFFS2_DBG_LVL "very_dirty_list: empty\n");
        } else {
                struct list_head *this;
                int numblocks = 0;
@@ -279,19 +350,19 @@ jffs2_dbg_dump_block_lists(struct jffs2_sb_info *c)
                        numblocks ++;
                        dirty += jeb->dirty_size;
                        if (!(jeb->used_size == 0 && jeb->dirty_size == 0 && jeb->wasted_size == 0)) {
-                               printk(KERN_DEBUG "very_dirty_list: %#08x (used %#08x, dirty %#08x, wasted %#08x, "
-                                               "unchecked %#08x, free %#08x)\n",
-                                               jeb->offset, jeb->used_size, jeb->dirty_size, jeb->wasted_size,
-                                               jeb->unchecked_size, jeb->free_size);
+                               printk(JFFS2_DBG_LVL "very_dirty_list: %#08x (used %#08x, dirty %#08x, wasted %#08x, "
+                                       "unchecked %#08x, free %#08x)\n",
+                                       jeb->offset, jeb->used_size, jeb->dirty_size, jeb->wasted_size,
+                                       jeb->unchecked_size, jeb->free_size);
                        }
                }
 
-               printk (KERN_DEBUG "Contains %d blocks with total dirty size %u, average dirty size: %u\n",
-                               numblocks, dirty, dirty / numblocks);
+               printk (JFFS2_DBG_LVL "Contains %d blocks with total dirty size %u, average dirty size: %u\n",
+                       numblocks, dirty, dirty / numblocks);
        }
 
        if (list_empty(&c->dirty_list)) {
-               printk(KERN_DEBUG "dirty_list: empty\n");
+               printk(JFFS2_DBG_LVL "dirty_list: empty\n");
        } else {
                struct list_head *this;
                int numblocks = 0;
@@ -303,19 +374,19 @@ jffs2_dbg_dump_block_lists(struct jffs2_sb_info *c)
                        numblocks ++;
                        dirty += jeb->dirty_size;
                        if (!(jeb->used_size == 0 && jeb->dirty_size == 0 && jeb->wasted_size == 0)) {
-                               printk(KERN_DEBUG "dirty_list: %#08x (used %#08x, dirty %#08x, wasted %#08x, "
-                                               "unchecked %#08x, free %#08x)\n",
-                                               jeb->offset, jeb->used_size, jeb->dirty_size, jeb->wasted_size,
-                                               jeb->unchecked_size, jeb->free_size);
+                               printk(JFFS2_DBG_LVL "dirty_list: %#08x (used %#08x, dirty %#08x, wasted %#08x, "
+                                       "unchecked %#08x, free %#08x)\n",
+                                       jeb->offset, jeb->used_size, jeb->dirty_size, jeb->wasted_size,
+                                       jeb->unchecked_size, jeb->free_size);
                        }
                }
 
-               printk (KERN_DEBUG "Contains %d blocks with total dirty size %u, average dirty size: %u\n",
+               printk (JFFS2_DBG_LVL "contains %d blocks with total dirty size %u, average dirty size: %u\n",
                        numblocks, dirty, dirty / numblocks);
        }
 
        if (list_empty(&c->erasable_list)) {
-               printk(KERN_DEBUG "erasable_list: empty\n");
+               printk(JFFS2_DBG_LVL "erasable_list: empty\n");
        } else {
                struct list_head *this;
 
@@ -323,16 +394,16 @@ jffs2_dbg_dump_block_lists(struct jffs2_sb_info *c)
                        struct jffs2_eraseblock *jeb = list_entry(this, struct jffs2_eraseblock, list);
 
                        if (!(jeb->used_size == 0 && jeb->dirty_size == 0 && jeb->wasted_size == 0)) {
-                               printk(KERN_DEBUG "erasable_list: %#08x (used %#08x, dirty %#08x, wasted %#08x, "
-                                               "unchecked %#08x, free %#08x)\n",
-                                               jeb->offset, jeb->used_size, jeb->dirty_size, jeb->wasted_size,
-                                               jeb->unchecked_size, jeb->free_size);
+                               printk(JFFS2_DBG_LVL "erasable_list: %#08x (used %#08x, dirty %#08x, wasted %#08x, "
+                                       "unchecked %#08x, free %#08x)\n",
+                                       jeb->offset, jeb->used_size, jeb->dirty_size, jeb->wasted_size,
+                                       jeb->unchecked_size, jeb->free_size);
                        }
                }
        }
 
        if (list_empty(&c->erasing_list)) {
-               printk(KERN_DEBUG "erasing_list: empty\n");
+               printk(JFFS2_DBG_LVL "erasing_list: empty\n");
        } else {
                struct list_head *this;
 
@@ -340,16 +411,16 @@ jffs2_dbg_dump_block_lists(struct jffs2_sb_info *c)
                        struct jffs2_eraseblock *jeb = list_entry(this, struct jffs2_eraseblock, list);
 
                        if (!(jeb->used_size == 0 && jeb->dirty_size == 0 && jeb->wasted_size == 0)) {
-                               printk(KERN_DEBUG "erasing_list: %#08x (used %#08x, dirty %#08x, wasted %#08x, "
-                                               "unchecked %#08x, free %#08x)\n",
-                                               jeb->offset, jeb->used_size, jeb->dirty_size, jeb->wasted_size,
-                                               jeb->unchecked_size, jeb->free_size);
+                               printk(JFFS2_DBG_LVL "erasing_list: %#08x (used %#08x, dirty %#08x, wasted %#08x, "
+                                       "unchecked %#08x, free %#08x)\n",
+                                       jeb->offset, jeb->used_size, jeb->dirty_size, jeb->wasted_size,
+                                       jeb->unchecked_size, jeb->free_size);
                        }
                }
        }
 
        if (list_empty(&c->erase_pending_list)) {
-               printk(KERN_DEBUG "erase_pending_list: empty\n");
+               printk(JFFS2_DBG_LVL "erase_pending_list: empty\n");
        } else {
                struct list_head *this;
 
@@ -357,16 +428,16 @@ jffs2_dbg_dump_block_lists(struct jffs2_sb_info *c)
                        struct jffs2_eraseblock *jeb = list_entry(this, struct jffs2_eraseblock, list);
 
                        if (!(jeb->used_size == 0 && jeb->dirty_size == 0 && jeb->wasted_size == 0)) {
-                               printk(KERN_DEBUG "erase_pending_list: %#08x (used %#08x, dirty %#08x, wasted %#08x, "
-                                               "unchecked %#08x, free %#08x)\n",
-                                               jeb->offset, jeb->used_size, jeb->dirty_size, jeb->wasted_size,
-                                               jeb->unchecked_size, jeb->free_size);
+                               printk(JFFS2_DBG_LVL "erase_pending_list: %#08x (used %#08x, dirty %#08x, wasted %#08x, "
+                                       "unchecked %#08x, free %#08x)\n",
+                                       jeb->offset, jeb->used_size, jeb->dirty_size, jeb->wasted_size,
+                                       jeb->unchecked_size, jeb->free_size);
                        }
                }
        }
 
        if (list_empty(&c->erasable_pending_wbuf_list)) {
-               printk(KERN_DEBUG "erasable_pending_wbuf_list: empty\n");
+               printk(JFFS2_DBG_LVL "erasable_pending_wbuf_list: empty\n");
        } else {
                struct list_head *this;
 
@@ -374,16 +445,16 @@ jffs2_dbg_dump_block_lists(struct jffs2_sb_info *c)
                        struct jffs2_eraseblock *jeb = list_entry(this, struct jffs2_eraseblock, list);
 
                        if (!(jeb->used_size == 0 && jeb->dirty_size == 0 && jeb->wasted_size == 0)) {
-                               printk(KERN_DEBUG "erasable_pending_wbuf_list: %#08x (used %#08x, dirty %#08x, "
-                                               "wasted %#08x, unchecked %#08x, free %#08x)\n",
-                                               jeb->offset, jeb->used_size, jeb->dirty_size, jeb->wasted_size,
-                                               jeb->unchecked_size, jeb->free_size);
+                               printk(JFFS2_DBG_LVL "erasable_pending_wbuf_list: %#08x (used %#08x, dirty %#08x, "
+                                       "wasted %#08x, unchecked %#08x, free %#08x)\n",
+                                       jeb->offset, jeb->used_size, jeb->dirty_size, jeb->wasted_size,
+                                       jeb->unchecked_size, jeb->free_size);
                        }
                }
        }
 
        if (list_empty(&c->free_list)) {
-               printk(KERN_DEBUG "free_list: empty\n");
+               printk(JFFS2_DBG_LVL "free_list: empty\n");
        } else {
                struct list_head *this;
 
@@ -391,16 +462,16 @@ jffs2_dbg_dump_block_lists(struct jffs2_sb_info *c)
                        struct jffs2_eraseblock *jeb = list_entry(this, struct jffs2_eraseblock, list);
 
                        if (!(jeb->used_size == 0 && jeb->dirty_size == 0 && jeb->wasted_size == 0)) {
-                               printk(KERN_DEBUG "free_list: %#08x (used %#08x, dirty %#08x, wasted %#08x, "
-                                               "unchecked %#08x, free %#08x)\n",
-                                               jeb->offset, jeb->used_size, jeb->dirty_size, jeb->wasted_size,
-                                               jeb->unchecked_size, jeb->free_size);
+                               printk(JFFS2_DBG_LVL "free_list: %#08x (used %#08x, dirty %#08x, wasted %#08x, "
+                                       "unchecked %#08x, free %#08x)\n",
+                                       jeb->offset, jeb->used_size, jeb->dirty_size, jeb->wasted_size,
+                                       jeb->unchecked_size, jeb->free_size);
                        }
                }
        }
 
        if (list_empty(&c->bad_list)) {
-               printk(KERN_DEBUG "bad_list: empty\n");
+               printk(JFFS2_DBG_LVL "bad_list: empty\n");
        } else {
                struct list_head *this;
 
@@ -408,16 +479,16 @@ jffs2_dbg_dump_block_lists(struct jffs2_sb_info *c)
                        struct jffs2_eraseblock *jeb = list_entry(this, struct jffs2_eraseblock, list);
 
                        if (!(jeb->used_size == 0 && jeb->dirty_size == 0 && jeb->wasted_size == 0)) {
-                               printk(KERN_DEBUG "bad_list: %#08x (used %#08x, dirty %#08x, wasted %#08x, "
-                                               "unchecked %#08x, free %#08x)\n",
-                                               jeb->offset, jeb->used_size, jeb->dirty_size, jeb->wasted_size,
-                                               jeb->unchecked_size, jeb->free_size);
+                               printk(JFFS2_DBG_LVL "bad_list: %#08x (used %#08x, dirty %#08x, wasted %#08x, "
+                                       "unchecked %#08x, free %#08x)\n",
+                                       jeb->offset, jeb->used_size, jeb->dirty_size, jeb->wasted_size,
+                                       jeb->unchecked_size, jeb->free_size);
                        }
                }
        }
 
        if (list_empty(&c->bad_used_list)) {
-               printk(KERN_DEBUG "bad_used_list: empty\n");
+               printk(JFFS2_DBG_LVL "bad_used_list: empty\n");
        } else {
                struct list_head *this;
 
@@ -425,34 +496,42 @@ jffs2_dbg_dump_block_lists(struct jffs2_sb_info *c)
                        struct jffs2_eraseblock *jeb = list_entry(this, struct jffs2_eraseblock, list);
 
                        if (!(jeb->used_size == 0 && jeb->dirty_size == 0 && jeb->wasted_size == 0)) {
-                               printk(KERN_DEBUG "bad_used_list: %#08x (used %#08x, dirty %#08x, wasted %#08x, "
-                                               "unchecked %#08x, free %#08x)\n",
-                                               jeb->offset, jeb->used_size, jeb->dirty_size, jeb->wasted_size,
-                                               jeb->unchecked_size, jeb->free_size);
+                               printk(JFFS2_DBG_LVL "bad_used_list: %#08x (used %#08x, dirty %#08x, wasted %#08x, "
+                                       "unchecked %#08x, free %#08x)\n",
+                                       jeb->offset, jeb->used_size, jeb->dirty_size, jeb->wasted_size,
+                                       jeb->unchecked_size, jeb->free_size);
                        }
                }
        }
 }
 
 void
-jffs2_dbg_dump_fragtree(struct jffs2_inode_info *f)
+__jffs2_dbg_dump_fragtree(struct jffs2_inode_info *f)
+{
+       down(&f->sem);
+       jffs2_dbg_dump_fragtree_nolock(f);
+       up(&f->sem);
+}
+
+void
+__jffs2_dbg_dump_fragtree_nolock(struct jffs2_inode_info *f)
 {
        struct jffs2_node_frag *this = frag_first(&f->fragtree);
        uint32_t lastofs = 0;
        int buggy = 0;
 
-       printk(KERN_DEBUG "inode is ino #%u\n", f->inocache->ino);
+       JFFS2_DEBUG("dump fragtree of ino #%u\n", f->inocache->ino);
        while(this) {
                if (this->node)
-                       printk(KERN_DEBUG "frag %#04x-%#04x: %#08x(%d) on flash (*%p), left (%p), "
-                                       "right (%p), parent (%p)\n",
-                                       this->ofs, this->ofs+this->size, ref_offset(this->node->raw),
-                                       ref_flags(this->node->raw), this, frag_left(this), frag_right(this),
-                                       frag_parent(this));
+                       printk(JFFS2_DBG_LVL "frag %#04x-%#04x: %#08x(%d) on flash (*%p), left (%p), "
+                               "right (%p), parent (%p)\n",
+                               this->ofs, this->ofs+this->size, ref_offset(this->node->raw),
+                               ref_flags(this->node->raw), this, frag_left(this), frag_right(this),
+                               frag_parent(this));
                else
-                       printk(KERN_DEBUG "frag %#04x-%#04x: hole (*%p). left (%p), right (%p), parent (%p)\n",
-                                       this->ofs, this->ofs+this->size, this, frag_left(this),
-                                       frag_right(this), frag_parent(this));
+                       printk(JFFS2_DBG_LVL "frag %#04x-%#04x: hole (*%p). left (%p), right (%p), parent (%p)\n",
+                               this->ofs, this->ofs+this->size, this, frag_left(this),
+                               frag_right(this), frag_parent(this));
                if (this->ofs != lastofs)
                        buggy = 1;
                lastofs = this->ofs + this->size;
@@ -460,36 +539,171 @@ jffs2_dbg_dump_fragtree(struct jffs2_inode_info *f)
        }
 
        if (f->metadata)
-               printk(KERN_DEBUG "metadata at 0x%08x\n", ref_offset(f->metadata->raw));
+               printk(JFFS2_DBG_LVL "metadata at 0x%08x\n", ref_offset(f->metadata->raw));
 
        if (buggy) {
-               printk(KERN_ERR "Error! %s(): Frag tree got a hole in it\n", __FUNCTION__);
+               JFFS2_ERROR("frag tree got a hole in it.\n");
                BUG();
        }
 }
 
-#define JFFS3_BUFDUMP_BYTES_PER_LINE   8
+#define JFFS2_BUFDUMP_BYTES_PER_LINE   32
 void
-jffs2_dbg_dump_buffer(char *buf, int len, uint32_t offs)
+__jffs2_dbg_dump_buffer(unsigned char *buf, int len, uint32_t offs)
 {
-       int i = 0;
-       int skip = offs & ~(JFFS3_BUFDUMP_BYTES_PER_LINE - 1);
+       int skip;
+       int i;
+       
+       JFFS2_DEBUG("dump from offset %#08x to offset %#08x (%x bytes).\n",
+               offs, offs + len, len);
+       i = skip = offs % JFFS2_BUFDUMP_BYTES_PER_LINE;
+       offs = offs & ~(JFFS2_BUFDUMP_BYTES_PER_LINE - 1);
+       
+       if (skip != 0)
+               printk(JFFS2_DBG_LVL "%#08x: ", offs);
+       
+       while (skip--)
+               printk("   ");
 
        while (i < len) {
-               int j = 0;
-
-               printk(KERN_DEBUG "0x#x: \n");
-               while (skip) {
-                       printk("   ");
-                       skip -= 1;
+               if ((i % JFFS2_BUFDUMP_BYTES_PER_LINE) == 0 && i != len -1) {
+                       if (i != 0)
+                               printk("\n");
+                       offs += JFFS2_BUFDUMP_BYTES_PER_LINE;
+                       printk(JFFS2_DBG_LVL "%0#8x: ", offs);
                }
 
-               while (j < JFFS3_BUFDUMP_BYTES_PER_LINE) {
-                       if (i + j < len)
-                               printk(" %#02x", buf[i + j++]);
+               printk("%02x ", buf[i]);
+               
+               i += 1;
+       }
+
+       printk("\n");
+}
+
+/*
+ * Dump a JFFS2 node.
+ */
+void
+__jffs2_dbg_dump_node(struct jffs2_sb_info *c, uint32_t ofs)
+{
+       union jffs2_node_union node;
+       int len = sizeof(union jffs2_node_union);
+       size_t retlen;
+       uint32_t crc;
+       int ret;
+       
+       JFFS2_DEBUG("dump node at offset %#08x.\n", ofs);
+
+       ret = jffs2_flash_read(c, ofs, len, &retlen, (unsigned char *)&node);
+       if (ret || (retlen != len)) {
+               JFFS2_ERROR("read %d bytes failed or short. ret %d, retlen %zd.\n",
+                       len, ret, retlen);
+               return;
+       }
+
+       printk(JFFS2_DBG_LVL "magic:\t%#04x\n",
+               je16_to_cpu(node.u.magic));
+       printk(JFFS2_DBG_LVL "nodetype:\t%#04x\n",
+               je16_to_cpu(node.u.nodetype));
+       printk(JFFS2_DBG_LVL "totlen:\t%#08x\n",
+               je32_to_cpu(node.u.totlen));
+       printk(JFFS2_DBG_LVL "hdr_crc:\t%#08x\n",
+               je32_to_cpu(node.u.hdr_crc));
+       
+       crc = crc32(0, &node.u, sizeof(node.u) - 4);
+       if (crc != je32_to_cpu(node.u.hdr_crc)) {
+               JFFS2_ERROR("wrong common header CRC.\n");
+               return;
+       }
+       
+       if (je16_to_cpu(node.u.magic) != JFFS2_MAGIC_BITMASK &&
+               je16_to_cpu(node.u.magic) != JFFS2_OLD_MAGIC_BITMASK)
+       {
+               JFFS2_ERROR("wrong node magic: %#04x instead of %#04x.\n",
+                       je16_to_cpu(node.u.magic), JFFS2_MAGIC_BITMASK);
+               return;
+       }
+
+       switch(je16_to_cpu(node.u.nodetype)) {
+
+       case JFFS2_NODETYPE_INODE:
+
+               printk(JFFS2_DBG_LVL "the node is inode node\n");
+               printk(JFFS2_DBG_LVL "ino:\t%#08x\n",
+                               je32_to_cpu(node.i.ino));
+               printk(JFFS2_DBG_LVL "version:\t%#08x\n",
+                               je32_to_cpu(node.i.version));
+               printk(JFFS2_DBG_LVL "mode:\t%#08x\n",
+                               node.i.mode.m);
+               printk(JFFS2_DBG_LVL "uid:\t%#04x\n",
+                               je16_to_cpu(node.i.uid));
+               printk(JFFS2_DBG_LVL "gid:\t%#04x\n",
+                               je16_to_cpu(node.i.gid));
+               printk(JFFS2_DBG_LVL "isize:\t%#08x\n",
+                               je32_to_cpu(node.i.isize));
+               printk(JFFS2_DBG_LVL "atime:\t%#08x\n",
+                               je32_to_cpu(node.i.atime));
+               printk(JFFS2_DBG_LVL "mtime:\t%#08x\n",
+                               je32_to_cpu(node.i.mtime));
+               printk(JFFS2_DBG_LVL "ctime:\t%#08x\n",
+                               je32_to_cpu(node.i.ctime));
+               printk(JFFS2_DBG_LVL "offset:\t%#08x\n",
+                               je32_to_cpu(node.i.offset));
+               printk(JFFS2_DBG_LVL "csize:\t%#08x\n",
+                               je32_to_cpu(node.i.csize));
+               printk(JFFS2_DBG_LVL "dsize:\t%#08x\n",
+                               je32_to_cpu(node.i.dsize));
+               printk(JFFS2_DBG_LVL "compr:\t%#02x\n",
+                               node.i.compr);
+               printk(JFFS2_DBG_LVL "usercompr:\t%#02x\n",
+                               node.i.usercompr);
+               printk(JFFS2_DBG_LVL "flags:\t%#04x\n",
+                               je16_to_cpu(node.i.flags));
+               printk(JFFS2_DBG_LVL "data_crc:\t%#08x\n",
+                               je32_to_cpu(node.i.data_crc));
+               printk(JFFS2_DBG_LVL "node_crc:\t%#08x\n",
+                               je32_to_cpu(node.i.node_crc));
+               crc = crc32(0, &node.i, sizeof(node.i) - 8); 
+               if (crc != je32_to_cpu(node.i.node_crc)) {
+                       JFFS2_ERROR("wrong node header CRC.\n");
+                       return;
+               }
+               break;
+
+       case JFFS2_NODETYPE_DIRENT:
+
+               printk(JFFS2_DBG_LVL "the node is dirent node\n");
+               printk(JFFS2_DBG_LVL "pino:\t%#08x\n",
+                               je32_to_cpu(node.d.pino));
+               printk(JFFS2_DBG_LVL "version:\t%#08x\n",
+                               je32_to_cpu(node.d.version));
+               printk(JFFS2_DBG_LVL "ino:\t%#08x\n",
+                               je32_to_cpu(node.d.ino));
+               printk(JFFS2_DBG_LVL "mctime:\t%#08x\n",
+                               je32_to_cpu(node.d.mctime));
+               printk(JFFS2_DBG_LVL "nsize:\t%#02x\n",
+                               node.d.nsize);
+               printk(JFFS2_DBG_LVL "type:\t%#02x\n",
+                               node.d.type);
+               printk(JFFS2_DBG_LVL "node_crc:\t%#08x\n",
+                               je32_to_cpu(node.d.node_crc));
+               printk(JFFS2_DBG_LVL "name_crc:\t%#08x\n",
+                               je32_to_cpu(node.d.name_crc));
+               
+               node.d.name[node.d.nsize] = '\0';
+               printk(JFFS2_DBG_LVL "name:\t\"%s\"\n", node.d.name);
+
+               crc = crc32(0, &node.d, sizeof(node.d) - 8); 
+               if (crc != je32_to_cpu(node.d.node_crc)) {
+                       JFFS2_ERROR("wrong node header CRC.\n");
+                       return;
                }
+               break;
 
-               i += JFFS3_BUFDUMP_BYTES_PER_LINE;
+       default:
+               printk(JFFS2_DBG_LVL "node type is unknown\n");
+               break;
        }
 }
-#endif /* JFFS2_PARANOIA_CHECKS || CONFIG_JFFS2_FS_DEBUG > 0 */
+#endif /* JFFS2_DBG_DUMPS || JFFS2_DBG_PARANOIA_CHECKS */
index 7d14f7b01397852063060567b38fd1e4fe91187a..51ff099a6502cead2fe553e07bac2fa8a07f9990 100644 (file)
@@ -7,7 +7,7 @@
  *
  * For licensing information, see the file 'LICENCE' in this directory.
  *
- * $Id: debug.h,v 1.1 2005/07/17 06:56:20 dedekind Exp $
+ * $Id: debug.h,v 1.5 2005/07/24 15:14:14 dedekind Exp $
  *
  */
 #ifndef _JFFS2_DEBUG_H_
 
 #include <linux/config.h>
 
+/* ------------------------------------------------ */
+/* TODO: remove */
+#undef CONFIG_JFFS2_FS_DEBUG
+#define CONFIG_JFFS2_FS_DEBUG 0
+//#define JFFS2_DBG_PARANOIA_CHECKS
+//#define JFFS2_DBG_DUMPS
+#define JFFS2_DBG_READINODE_MESSAGES
+//#define JFFS2_DBG_FRAGTREE_MESSAGES
+//#define JFFS2_DBG_FRAGTREE2_MESSAGES
+#undef KERN_DEBUG
+#undef KERN_WARNING
+#undef KERN_NOTICE
+#undef KERN_ERR
+#define KERN_DEBUG   KERN_CRIT
+#define KERN_WARNING KERN_CRIT
+#define KERN_NOTICE  KERN_CRIT
+#define KERN_ERR     KERN_CRIT
+/* ------------------------------------------------ */
+
 #ifndef CONFIG_JFFS2_FS_DEBUG
 #define CONFIG_JFFS2_FS_DEBUG 1
 #endif
 
-#if CONFIG_JFFS2_FS_DEBUG > 0
+#if CONFIG_JFFS2_FS_DEBUG == 1
+/* Enable "paranoia" checks and dumps */
 #define JFFS2_DBG_PARANOIA_CHECKS
+#define JFFS2_DBG_DUMPS
+#define JFFS2_DBG_READINODE_MESSAGES
+#define JFFS2_DBG_FRAGTREE_MESSAGES
+#define JFFS2_DBG_DENTLIST_MESSAGES
+#define JFFS2_DBG_NODEREF_MESSAGES
+#define JFFS2_DBG_INOCACHE_MESSAGES
+#endif
+
+#if CONFIG_JFFS2_FS_DEBUG == 2
+#define JFFS2_DBG_FRAGTREE2_MESSAGES
+#endif
+
+/* Enable JFFS2 sanity checks by default */
+#define JFFS2_DBG_SANITY_CHECKS
+
+/* 
+ * Dx() are mainly used for debugging messages, they must go away and be
+ * superseded by nicer JFFS2_DBG_XXX() macros...
+ */
+#if CONFIG_JFFS2_FS_DEBUG > 0
 #define D1(x) x
 #else
 #define D1(x)
 #define D2(x)
 #endif
 
-/* Enable JFFS2 sanity checks */
-#define JFFS2_DBG_SANITY_CHECKS
+/* The prefixes of JFFS2 messages */
+#define JFFS2_DBG_MSG_PREFIX "[JFFS2 DBG]"
+#define JFFS2_ERR_MSG_PREFIX "JFFS2 error: "
+#define JFFS2_WARN_MSG_PREFIX "JFFS2 warning: "
+#define JFFS2_NOTICE_MSG_PREFIX "JFFS2 notice: "
 
-#if CONFIG_JFFS2_FS_DEBUG > 0
-void
-jffs2_dbg_dump_block_lists(struct jffs2_sb_info *c);
+#define JFFS2_ERR_LVL          KERN_ERR
+#define JFFS2_WARN_LVL         KERN_WARNING
+#define JFFS2_NOTICE_LVL       KERN_NOTICE
+#define JFFS2_DBG_LVL          KERN_DEBUG
 
-void
-jffs2_dbg_dump_node_refs(struct jffs2_sb_info *c, struct jffs2_eraseblock *jeb);
+/* JFFS2 message macros */
+#define JFFS2_ERROR(fmt, ...)                                          \
+       do {                                                            \
+               printk(JFFS2_ERR_LVL JFFS2_ERR_MSG_PREFIX " %s: "       \
+                               fmt, __FUNCTION__, ##__VA_ARGS__);      \
+       } while(0)
 
-void
-jffs2_dbg_dump_fragtree(struct jffs2_inode_info *f);
+#define JFFS2_WARNING(fmt, ...)                                                \
+       do {                                                            \
+               printk(JFFS2_WARN_LVL JFFS2_WARN_MSG_PREFIX " %s: "     \
+                               fmt, __FUNCTION__, ##__VA_ARGS__);      \
+       } while(0)
+                       
+#define JFFS2_NOTICE(fmt, ...)                                         \
+       do {                                                            \
+               printk(JFFS2_NOTICE_LVL JFFS2_NOTICE_MSG_PREFIX " %s: " \
+                               fmt, __FUNCTION__, ##__VA_ARGS__);      \
+       } while(0)
 
-void
-jffs2_dbg_dump_buffer(char *buf, int len, uint32_t offs);
+#define JFFS2_DEBUG(fmt, ...)                                          \
+       do {                                                            \
+               printk(JFFS2_DBG_LVL JFFS2_DBG_MSG_PREFIX " %s: "       \
+                               fmt, __FUNCTION__, ##__VA_ARGS__);      \
+       } while(0)
+
+/* 
+ * We split our debugging messages on several parts, depending on the JFFS2
+ * subsystem the message belongs to.
+ */
+/* Read inode debugging messages */
+#ifdef JFFS2_DBG_READINODE_MESSAGES
+#define JFFS2_DBG_READINODE(fmt, ...)  JFFS2_DEBUG(fmt, ##__VA_ARGS__)
+#else
+#define JFFS2_DBG_READINODE(fmt, ...)
 #endif
 
-#ifdef JFFS2_DBG_PARANOIA_CHECKS
-void
-jffs2_dbg_fragtree_paranoia_check(struct jffs2_inode_info *f);
+/* Fragtree build debugging messages */
+#ifdef JFFS2_DBG_FRAGTREE_MESSAGES
+#define JFFS2_DBG_FRAGTREE(fmt, ...)   JFFS2_DEBUG(fmt, ##__VA_ARGS__)
+#else
+#define JFFS2_DBG_FRAGTREE(fmt, ...)
+#endif
+
+/* Directory entry list manilulation debugging messages */
+#ifdef JFFS2_DBG_DENTLIST_MESSAGES
+#define JFFS2_DBG_DENTLIST(fmt, ...)   JFFS2_DEBUG(fmt, ##__VA_ARGS__)
+#else
+#define JFFS2_DBG_DENTLIST(fmt, ...)
+#endif
+#ifdef JFFS2_DBG_FRAGTREE2_MESSAGES
+#define JFFS2_DBG_FRAGTREE2(fmt, ...)  JFFS2_DEBUG(fmt, ##__VA_ARGS__)
+#else
+#define JFFS2_DBG_FRAGTREE2(fmt, ...)
+#endif
+
+/* Plays with node_refs */
+#ifdef JFFS2_DBG_NODEREF_MESSAGES
+#define JFFS2_DBG_NODEREF(fmt, ...)    JFFS2_DEBUG(fmt, ##__VA_ARGS__)
+#else
+#define JFFS2_DBG_NODEREF(fmt, ...)
+#endif
 
+/* Plays with the list of inodes (JFFS2 inocache) */
+#ifdef JFFS2_DBG_INOCACHE_MESSAGES
+#define JFFS2_DBG_INOCACHE(fmt, ...)   JFFS2_DEBUG(fmt, ##__VA_ARGS__)
+#else
+#define JFFS2_DBG_INOCACHE(fmt, ...)
+#endif
+
+/* "Paranoia" checks */
+void
+__jffs2_dbg_fragtree_paranoia_check(struct jffs2_inode_info *f);
 void
-jffs2_dbg_acct_paranoia_check(struct jffs2_sb_info *c,
-                             struct jffs2_eraseblock *jeb);
+__jffs2_dbg_fragtree_paranoia_check_nolock(struct jffs2_inode_info *f);
+void
+__jffs2_dbg_acct_paranoia_check(struct jffs2_sb_info *c,
+                               struct jffs2_eraseblock *jeb);
+void
+__jffs2_dbg_acct_paranoia_check_nolock(struct jffs2_sb_info *c,
+                                      struct jffs2_eraseblock *jeb);
+void
+__jffs2_dbg_prewrite_paranoia_check(struct jffs2_sb_info *c,
+                                   uint32_t ofs, int len);
 
+/* "Dump" functions */
+void
+__jffs2_dbg_dump_jeb(struct jffs2_sb_info *c, struct jffs2_eraseblock *jeb);
+void
+__jffs2_dbg_dump_jeb_nolock(struct jffs2_eraseblock *jeb);
+void
+__jffs2_dbg_dump_block_lists(struct jffs2_sb_info *c);
+void
+__jffs2_dbg_dump_block_lists_nolock(struct jffs2_sb_info *c);
+void
+__jffs2_dbg_dump_node_refs(struct jffs2_sb_info *c,
+                          struct jffs2_eraseblock *jeb);
+void
+__jffs2_dbg_dump_node_refs_nolock(struct jffs2_sb_info *c,
+                                 struct jffs2_eraseblock *jeb);
+void
+__jffs2_dbg_dump_fragtree(struct jffs2_inode_info *f);
+void
+__jffs2_dbg_dump_fragtree_nolock(struct jffs2_inode_info *f);
 void
-jffs2_dbg_prewrite_paranoia_check(struct jffs2_sb_info *c,
-                                 uint32_t ofs, int len);
+__jffs2_dbg_dump_buffer(unsigned char *buf, int len, uint32_t offs);
+void
+__jffs2_dbg_dump_node(struct jffs2_sb_info *c, uint32_t ofs);
+
+#ifdef JFFS2_DBG_PARANOIA_CHECKS
+#define jffs2_dbg_fragtree_paranoia_check(f)                   \
+       __jffs2_dbg_fragtree_paranoia_check(f)
+#define jffs2_dbg_fragtree_paranoia_check_nolock(f)            \
+       __jffs2_dbg_fragtree_paranoia_check_nolock(f)
+#define jffs2_dbg_acct_paranoia_check(c, jeb)                  \
+       __jffs2_dbg_acct_paranoia_check(c,jeb)
+#define jffs2_dbg_acct_paranoia_check_nolock(c, jeb)           \
+       __jffs2_dbg_acct_paranoia_check_nolock(c,jeb)
+#define jffs2_dbg_prewrite_paranoia_check(c, ofs, len)         \
+       __jffs2_dbg_prewrite_paranoia_check(c, ofs, len)
 #else
 #define jffs2_dbg_fragtree_paranoia_check(f)
+#define jffs2_dbg_fragtree_paranoia_check_nolock(f)
 #define jffs2_dbg_acct_paranoia_check(c, jeb)
+#define jffs2_dbg_acct_paranoia_check_nolock(c, jeb)
 #define jffs2_dbg_prewrite_paranoia_check(c, ofs, len)
 #endif /* !JFFS2_PARANOIA_CHECKS */
 
+#ifdef JFFS2_DBG_DUMPS
+#define jffs2_dbg_dump_jeb(c, jeb)                             \
+       __jffs2_dbg_dump_jeb(c, jeb);
+#define jffs2_dbg_dump_jeb_nolock(jeb)                         \
+       __jffs2_dbg_dump_jeb_nolock(jeb);
+#define jffs2_dbg_dump_block_lists(c)                          \
+       __jffs2_dbg_dump_block_lists(c)
+#define jffs2_dbg_dump_block_lists_nolock(c)                   \
+       __jffs2_dbg_dump_block_lists_nolock(c)
+#define jffs2_dbg_dump_fragtree(f)                             \
+       __jffs2_dbg_dump_fragtree(f);
+#define jffs2_dbg_dump_fragtree_nolock(f)                      \
+       __jffs2_dbg_dump_fragtree_nolock(f);
+#define jffs2_dbg_dump_buffer(buf, len, offs)                  \
+       __jffs2_dbg_dump_buffer(*buf, len, offs);
+#define jffs2_dbg_dump_node(c, ofs)                            \
+       __jffs2_dbg_dump_node(c, ofs);
+#else
+#define jffs2_dbg_dump_jeb(c, jeb)
+#define jffs2_dbg_dump_jeb_nolock(jeb)
+#define jffs2_dbg_dump_block_lists(c)
+#define jffs2_dbg_dump_block_lists_nolock(c)
+#define jffs2_dbg_dump_fragtree(f)
+#define jffs2_dbg_dump_fragtree_nolock(f)
+#define jffs2_dbg_dump_buffer(buf, len, offs)
+#define jffs2_dbg_dump_node(c, ofs)
+#endif /* !JFFS2_DBG_DUMPS */
+
+/*
+ * Sanity checks are supposed to be light-weight and enabled by default.
+ */
 #ifdef JFFS2_DBG_SANITY_CHECKS
 /*
  * Check the space accounting of the file system and of
- * the JFFS3 erasable block 'jeb'.
+ * the JFFS2 erasable block 'jeb'.
  */
 static inline void
-jffs2_dbg_acct_sanity_check(struct jffs2_sb_info *c,
-                           struct jffs2_eraseblock *jeb)
+jffs2_dbg_acct_sanity_check_nolock(struct jffs2_sb_info *c,
+                                  struct jffs2_eraseblock *jeb)
 {
        if (unlikely(jeb && jeb->used_size + jeb->dirty_size +
                        jeb->free_size + jeb->wasted_size +
                        jeb->unchecked_size != c->sector_size)) {
-               printk(KERN_ERR "Eeep. Space accounting for block at 0x%08x is screwed\n", jeb->offset);
-               printk(KERN_ERR "free %#08x + dirty %#08x + used %#08x + wasted %#08x + unchecked "
-                               "%#08x != total %#08x\n", jeb->free_size, jeb->dirty_size, jeb->used_size,
-                               jeb->wasted_size, jeb->unchecked_size, c->sector_size);
+               JFFS2_ERROR("eeep, space accounting for block at 0x%08x is screwed.\n", jeb->offset);
+               JFFS2_ERROR("free %#08x + dirty %#08x + used %#08x + wasted %#08x + unchecked "
+                       "%#08x != total %#08x.\n", jeb->free_size, jeb->dirty_size, jeb->used_size,
+                       jeb->wasted_size, jeb->unchecked_size, c->sector_size);
                BUG();
        }
 
        if (unlikely(c->used_size + c->dirty_size + c->free_size + c->erasing_size + c->bad_size
                                + c->wasted_size + c->unchecked_size != c->flash_size)) {
-               printk(KERN_ERR "Eeep. Space accounting superblock info is screwed\n");
-               printk(KERN_ERR "free %#08x + dirty %#08x + used %#08x + erasing %#08x + bad %#08x + "
-                               "wasted %#08x + unchecked %#08x != total %#08x\n",
-                               c->free_size, c->dirty_size, c->used_size, c->erasing_size, c->bad_size,
-                               c->wasted_size, c->unchecked_size, c->flash_size);
+               JFFS2_ERROR("eeep, space accounting superblock info is screwed.\n");
+               JFFS2_ERROR("free %#08x + dirty %#08x + used %#08x + erasing %#08x + bad %#08x + "
+                       "wasted %#08x + unchecked %#08x != total %#08x.\n",
+                       c->free_size, c->dirty_size, c->used_size, c->erasing_size, c->bad_size,
+                       c->wasted_size, c->unchecked_size, c->flash_size);
                BUG();
        }
 }
-#else
+
 static inline void
 jffs2_dbg_acct_sanity_check(struct jffs2_sb_info *c,
-                           struct jffs2_eraseblock *jeb);
+                           struct jffs2_eraseblock *jeb)
+{
+       spin_lock(&c->erase_completion_lock);
+       jffs2_dbg_acct_sanity_check_nolock(c, jeb);
+       spin_unlock(&c->erase_completion_lock);
+}
+#else
+#define jffs2_dbg_acct_sanity_check(c, jeb)
+#define jffs2_dbg_acct_sanity_check_nolock(c, jeb)
 #endif /* !JFFS2_DBG_SANITY_CHECKS */
 
 #endif /* _JFFS2_DEBUG_H_ */
index af0c7d431883b77918751f9f9e4aa9cf8f3376cb..a8a0908142d95e5e4a39fcccc1c9b1c925e21f98 100644 (file)
@@ -7,7 +7,7 @@
  *
  * For licensing information, see the file 'LICENCE' in this directory.
  *
- * $Id: erase.c,v 1.81 2005/07/17 06:56:20 dedekind Exp $
+ * $Id: erase.c,v 1.83 2005/07/22 10:32:08 dedekind Exp $
  *
  */
 
@@ -48,7 +48,8 @@ static void jffs2_erase_block(struct jffs2_sb_info *c,
 #else /* Linux */
        struct erase_info *instr;
 
-       D1(printk(KERN_DEBUG "jffs2_erase_block(): erase block %#x (range %#x-%#x)\n", jeb->offset, jeb->offset, jeb->offset + c->sector_size));
+       D1(printk(KERN_DEBUG "jffs2_erase_block(): erase block %#08x (range %#08x-%#08x)\n",
+                               jeb->offset, jeb->offset, jeb->offset + c->sector_size));
        instr = kmalloc(sizeof(struct erase_info) + sizeof(struct erase_priv_struct), GFP_KERNEL);
        if (!instr) {
                printk(KERN_WARNING "kmalloc for struct erase_info in jffs2_erase_block failed. Refiling block for later\n");
@@ -429,8 +430,8 @@ static void jffs2_mark_erased_block(struct jffs2_sb_info *c, struct jffs2_eraseb
        c->free_size += jeb->free_size;
        c->used_size += jeb->used_size;
 
-       jffs2_dbg_acct_sanity_check(c,jeb);
-       jffs2_dbg_acct_paranoia_check(c, jeb);
+       jffs2_dbg_acct_sanity_check_nolock(c,jeb);
+       jffs2_dbg_acct_paranoia_check_nolock(c, jeb);
 
        list_add_tail(&jeb->list, &c->free_list);
        c->nr_erasing_blocks--;
index a6661e8be25a06cf92d7e673faf4fc0d7922b0a7..34731614b57a670a1e9378b50dcbbfec79adec9a 100644 (file)
@@ -7,7 +7,7 @@
  *
  * For licensing information, see the file 'LICENCE' in this directory.
  *
- * $Id: fs.c,v 1.59 2005/07/18 11:21:19 dedekind Exp $
+ * $Id: fs.c,v 1.60 2005/07/22 10:32:08 dedekind Exp $
  *
  */
 
@@ -194,19 +194,15 @@ int jffs2_statfs(struct super_block *sb, struct kstatfs *buf)
        buf->f_namelen = JFFS2_MAX_NAME_LEN;
 
        spin_lock(&c->erase_completion_lock);
-
        avail = c->dirty_size + c->free_size;
        if (avail > c->sector_size * c->resv_blocks_write)
                avail -= c->sector_size * c->resv_blocks_write;
        else
                avail = 0;
+       spin_unlock(&c->erase_completion_lock);
 
        buf->f_bavail = buf->f_bfree = avail >> PAGE_SHIFT;
 
-       jffs2_dbg_dump_block_lists(c);
-
-       spin_unlock(&c->erase_completion_lock);
-
        return 0;
 }
 
index 6b5da0a1d319d6059ab1860ab7cc13e3f557dd40..362cfeed7327816d6063a8cf4879d9df63a3f7cd 100644 (file)
@@ -7,7 +7,7 @@
  *
  * For licensing information, see the file 'LICENCE' in this directory.
  *
- * $Id: gc.c,v 1.150 2005/07/17 12:01:43 dedekind Exp $
+ * $Id: gc.c,v 1.152 2005/07/24 15:14:14 dedekind Exp $
  *
  */
 
@@ -111,7 +111,6 @@ again:
                ret->wasted_size = 0;
        }
 
-       jffs2_dbg_dump_block_lists(c);
        return ret;
 }
 
@@ -142,7 +141,7 @@ int jffs2_garbage_collect_pass(struct jffs2_sb_info *c)
                if (c->checked_ino > c->highest_ino) {
                        printk(KERN_CRIT "Checked all inodes but still 0x%x bytes of unchecked space?\n",
                               c->unchecked_size);
-                       jffs2_dbg_dump_block_lists(c);
+                       jffs2_dbg_dump_block_lists_nolock(c);
                        spin_unlock(&c->erase_completion_lock);
                        BUG();
                }
@@ -485,7 +484,8 @@ static int jffs2_garbage_collect_live(struct jffs2_sb_info *c,  struct jffs2_era
                if (ref_obsolete(raw)) {
                        printk(KERN_WARNING "But it's obsolete so we don't mind too much\n");
                } else {
-                       ret = -EIO;
+                       jffs2_dbg_dump_node(c, ref_offset(raw));
+                       BUG();
                }
        }
  upnout:
index a04111530f4df0b44e039d68fdfd56a4664f335f..9d08d33881864ef40bbf207b2cf2f602b554e268 100644 (file)
@@ -7,7 +7,7 @@
  *
  * For licensing information, see the file 'LICENCE' in this directory.
  *
- * $Id: nodelist.c,v 1.99 2005/07/15 10:13:54 dedekind Exp $
+ * $Id: nodelist.c,v 1.100 2005/07/22 10:32:08 dedekind Exp $
  *
  */
 
@@ -203,7 +203,7 @@ read_direntry(struct jffs2_sb_info *c,
                        return -EIO;
                        
                if (unlikely(err)) {
-                       printk(KERN_WARNING "Read remainder of name in jffs2_get_inode_nodes(): error %d\n", err);
+                       printk(KERN_WARNING "Read remainder of name: error %d\n", err);
                        jffs2_free_full_dirent(fd);
                        return -EIO;
                }
index 6802e0968d3e822d7bc463b62482be6172354d19..dde9b86e43b587684c8edbc9aef331ba4be16a6d 100644 (file)
@@ -7,7 +7,7 @@
  *
  * For licensing information, see the file 'LICENCE' in this directory.
  *
- * $Id: nodelist.h,v 1.132 2005/07/17 06:56:21 dedekind Exp $
+ * $Id: nodelist.h,v 1.133 2005/07/22 10:32:08 dedekind Exp $
  *
  */
 
@@ -236,7 +236,7 @@ static inline uint32_t ref_totlen(struct jffs2_sb_info *c,
                       ret, ref->__totlen);
                if (!jeb)
                        jeb = &c->blocks[ref->flash_offset / c->sector_size];
-               jffs2_dbg_dump_node_refs(c, jeb);
+               jffs2_dbg_dump_node_refs_nolock(c, jeb);
                BUG();
        }
 #endif
index 424be1e1ca92548a2ae082e1afb76ed862367a23..fe7e70a4055bc41a5b19e6415d3dc2457ede2813 100644 (file)
@@ -7,7 +7,7 @@
  *
  * For licensing information, see the file 'LICENCE' in this directory.
  *
- * $Id: nodemgmt.c,v 1.123 2005/07/17 06:56:21 dedekind Exp $
+ * $Id: nodemgmt.c,v 1.124 2005/07/20 15:32:28 dedekind Exp $
  *
  */
 
@@ -349,8 +349,8 @@ int jffs2_add_physical_node_ref(struct jffs2_sb_info *c, struct jffs2_raw_node_r
                list_add_tail(&jeb->list, &c->clean_list);
                c->nextblock = NULL;
        }
-       jffs2_dbg_acct_sanity_check(c,jeb);
-       jffs2_dbg_acct_paranoia_check(c, jeb);
+       jffs2_dbg_acct_sanity_check_nolock(c,jeb);
+       jffs2_dbg_acct_paranoia_check_nolock(c, jeb);
 
        spin_unlock(&c->erase_completion_lock);
 
@@ -430,7 +430,7 @@ void jffs2_mark_node_obsolete(struct jffs2_sb_info *c, struct jffs2_raw_node_ref
                               ref_totlen(c, jeb, ref), blocknr, ref->flash_offset, jeb->used_size);
                        BUG();
                })
-               D1(printk(KERN_DEBUG "Obsoleting node at 0x%08x of len %x: ", ref_offset(ref), ref_totlen(c, jeb, ref)));
+               D1(printk(KERN_DEBUG "Obsoleting node at 0x%08x of len %#x: ", ref_offset(ref), ref_totlen(c, jeb, ref)));
                jeb->used_size -= ref_totlen(c, jeb, ref);
                c->used_size -= ref_totlen(c, jeb, ref);
        }
@@ -466,9 +466,8 @@ void jffs2_mark_node_obsolete(struct jffs2_sb_info *c, struct jffs2_raw_node_ref
        }
        ref->flash_offset = ref_offset(ref) | REF_OBSOLETE;
        
-       jffs2_dbg_acct_sanity_check(c, jeb);
-
-       jffs2_dbg_acct_paranoia_check(c, jeb);
+       jffs2_dbg_acct_sanity_check_nolock(c, jeb);
+       jffs2_dbg_acct_paranoia_check_nolock(c, jeb);
 
        if (c->flags & JFFS2_SB_FLAG_SCANNING) {
                /* Flash scanning is in progress. Don't muck about with the block
index 9706534786ec0e0117a834de61b502e593c56601..e38e6c5f143a202af72f6fad63c1a635f700355b 100644 (file)
@@ -7,7 +7,7 @@
  *
  * For licensing information, see the file 'LICENCE' in this directory.
  *
- * $Id: read.c,v 1.40 2005/07/17 06:56:21 dedekind Exp $
+ * $Id: read.c,v 1.41 2005/07/22 10:32:08 dedekind Exp $
  *
  */
 
@@ -174,7 +174,6 @@ int jffs2_read_inode_range(struct jffs2_sb_info *c, struct jffs2_inode_info *f,
                        if (frag) {
                                D1(printk(KERN_NOTICE "Eep. Hole in ino #%u fraglist. frag->ofs = 0x%08x, offset = 0x%08x\n", f->inocache->ino, frag->ofs, offset));
                                holesize = min(holesize, frag->ofs - offset);
-                               D2(jffs2_dbg_dump_fragtree(f));
                        }
                        D1(printk(KERN_DEBUG "Filling non-frag hole from %d-%d\n", offset, offset+holesize));
                        memset(buf, 0, holesize);
index 02a20d7b4d5f3a25503725448b10303d637c2015..02b02c1146bfd763b2f41dab48adf1da6469645c 100644 (file)
@@ -7,7 +7,7 @@
  *
  * For licensing information, see the file 'LICENCE' in this directory.
  *
- * $Id: readinode.c,v 1.128 2005/07/17 12:01:43 dedekind Exp $
+ * $Id: readinode.c,v 1.129 2005/07/22 10:32:08 dedekind Exp $
  *
  */
 
@@ -92,8 +92,8 @@ int jffs2_add_full_dnode_to_inode(struct jffs2_sb_info *c, struct jffs2_inode_in
                                mark_ref_normal(next->node->raw);
                }
        }
-       jffs2_dbg_fragtree_paranoia_check(f);
-       jffs2_dbg_dump_fragtree(f);
+       jffs2_dbg_fragtree_paranoia_check_nolock(f);
+       jffs2_dbg_dump_fragtree_nolock(f);
        return 0;
 }
 
@@ -480,7 +480,7 @@ static int jffs2_do_read_inode_internal(struct jffs2_sb_info *c,
 
                jffs2_free_tmp_dnode_info(tn);
        }
-       jffs2_dbg_fragtree_paranoia_check(f);
+       jffs2_dbg_fragtree_paranoia_check_nolock(f);
 
        if (!fn) {
                /* No data nodes for this inode. */
index a2a51b7c2fd949aa7f2d897d2d99f0c53b801aaa..fcd6314cf179e6acf415ac39edc233e6c7acd670 100644 (file)
@@ -7,7 +7,7 @@
  *
  * For licensing information, see the file 'LICENCE' in this directory.
  *
- * $Id: scan.c,v 1.120 2005/07/17 06:56:21 dedekind Exp $
+ * $Id: scan.c,v 1.121 2005/07/20 15:32:28 dedekind Exp $
  *
  */
 #include <linux/kernel.h>
@@ -130,7 +130,7 @@ int jffs2_scan_medium(struct jffs2_sb_info *c)
                if (ret < 0)
                        goto out;
 
-               jffs2_dbg_acct_paranoia_check(c, jeb);
+               jffs2_dbg_acct_paranoia_check_nolock(c, jeb);
 
                /* Now decide which list to put it on */
                switch(ret) {
@@ -370,7 +370,7 @@ static int jffs2_scan_eraseblock (struct jffs2_sb_info *c, struct jffs2_eraseblo
 scan_more:     
        while(ofs < jeb->offset + c->sector_size) {
 
-               jffs2_dbg_acct_paranoia_check(c, jeb);
+               jffs2_dbg_acct_paranoia_check_nolock(c, jeb);
 
                cond_resched();
 
index 5a820f4ea35876f6d921efc1305f196dc4982daa..251ac3d86f997f7a014ac0b40088c7515bf91f03 100644 (file)
@@ -9,7 +9,7 @@
  *
  * For licensing information, see the file 'LICENCE' in this directory.
  *
- * $Id: wbuf.c,v 1.94 2005/07/17 12:01:43 dedekind Exp $
+ * $Id: wbuf.c,v 1.96 2005/07/22 10:32:08 dedekind Exp $
  *
  */
 
@@ -139,7 +139,6 @@ static void jffs2_block_refile(struct jffs2_sb_info *c, struct jffs2_eraseblock
 {
        D1(printk("About to refile bad block at %08x\n", jeb->offset));
 
-       jffs2_dbg_dump_block_lists(c);
        /* File the existing block on the bad_used_list.... */
        if (c->nextblock == jeb)
                c->nextblock = NULL;
@@ -156,7 +155,6 @@ static void jffs2_block_refile(struct jffs2_sb_info *c, struct jffs2_eraseblock
                c->nr_erasing_blocks++;
                jffs2_erase_pending_trigger(c);
        }
-       jffs2_dbg_dump_block_lists(c);
 
        /* Adjust its size counts accordingly */
        c->wasted_size += jeb->free_size;
@@ -164,8 +162,9 @@ static void jffs2_block_refile(struct jffs2_sb_info *c, struct jffs2_eraseblock
        jeb->wasted_size += jeb->free_size;
        jeb->free_size = 0;
 
-       jffs2_dbg_acct_sanity_check(c,jeb);
-       jffs2_dbg_acct_paranoia_check(c, jeb);
+       jffs2_dbg_dump_block_lists_nolock(c);
+       jffs2_dbg_acct_sanity_check_nolock(c,jeb);
+       jffs2_dbg_acct_paranoia_check_nolock(c, jeb);
 }
 
 /* Recover from failure to write wbuf. Recover the nodes up to the
@@ -392,11 +391,11 @@ static void jffs2_wbuf_recover(struct jffs2_sb_info *c)
        else
                jeb->last_node = container_of(first_raw, struct jffs2_raw_node_ref, next_phys);
 
-       jffs2_dbg_acct_sanity_check(c,jeb);
-        jffs2_dbg_acct_paranoia_check(c, jeb);
+       jffs2_dbg_acct_sanity_check_nolock(c, jeb);
+        jffs2_dbg_acct_paranoia_check_nolock(c, jeb);
 
-       jffs2_dbg_acct_sanity_check(c,new_jeb);
-        jffs2_dbg_acct_paranoia_check(c, new_jeb);
+       jffs2_dbg_acct_sanity_check_nolock(c, new_jeb);
+        jffs2_dbg_acct_paranoia_check_nolock(c, new_jeb);
 
        spin_unlock(&c->erase_completion_lock);
 
index b6a53e5aaa003f65a64a4fa44708f353f14d6e65..4c418e69cfc6380dd8c5b05784527837200b7846 100644 (file)
@@ -7,7 +7,7 @@
  *
  * For licensing information, see the file 'LICENCE' in this directory.
  *
- * $Id: write.c,v 1.93 2005/07/17 06:56:21 dedekind Exp $
+ * $Id: write.c,v 1.94 2005/07/20 15:50:51 dedekind Exp $
  *
  */
 
@@ -223,8 +223,6 @@ struct jffs2_full_dirent *jffs2_write_dirent(struct jffs2_sb_info *c, struct jff
                  je32_to_cpu(rd->pino), name, name, je32_to_cpu(rd->ino),
                  je32_to_cpu(rd->name_crc)));
 
-       jffs2_dbg_prewrite_paranoia_check(c, flash_ofs, vecs[0].iov_len + vecs[1].iov_len);
-
        D1(if(je32_to_cpu(rd->hdr_crc) != crc32(0, rd, sizeof(struct jffs2_unknown_node)-4)) {
                printk(KERN_CRIT "Eep. CRC not correct in jffs2_write_dirent()\n");
                BUG();
@@ -236,6 +234,8 @@ struct jffs2_full_dirent *jffs2_write_dirent(struct jffs2_sb_info *c, struct jff
        vecs[1].iov_base = (unsigned char *)name;
        vecs[1].iov_len = namelen;
        
+       jffs2_dbg_prewrite_paranoia_check(c, flash_ofs, vecs[0].iov_len + vecs[1].iov_len);
+
        raw = jffs2_alloc_raw_node_ref();
 
        if (!raw)