Merge with /shiny/git/linux-2.6/.git

6 files changed, 117 insertions, 106 deletions

diff --git a/fs/jffs2/build.c b/fs/jffs2/build.c
index 3dd5394921c9..97dc39796e2c 100644
--- a/fs/jffs2/build.c
+++ b/fs/jffs2/build.c
@@ -7,7 +7,7 @@
  *
  * For licensing information, see the file 'LICENCE' in this directory.
  *
- * $Id: build.c,v 1.70 2005/02/28 08:21:05 dedekind Exp $
+ * $Id: build.c,v 1.71 2005/07/12 16:37:08 dedekind Exp $
  *
  */
 
@@ -336,13 +336,6 @@ int jffs2_do_mount_fs(struct jffs2_sb_info *c)
                 c->blocks[i].bad_count = 0;
         }
 
-        init_MUTEX(&c->alloc_sem);
-        init_MUTEX(&c->erase_free_sem);
-        init_waitqueue_head(&c->erase_wait);
-        init_waitqueue_head(&c->inocache_wq);
-        spin_lock_init(&c->erase_completion_lock);
-        spin_lock_init(&c->inocache_lock);
-
         INIT_LIST_HEAD(&c->clean_list);
         INIT_LIST_HEAD(&c->very_dirty_list);
         INIT_LIST_HEAD(&c->dirty_list);
diff --git a/fs/jffs2/erase.c b/fs/jffs2/erase.c
index 6a4c0a3685da..787d84ac2bcd 100644
--- a/fs/jffs2/erase.c
+++ b/fs/jffs2/erase.c
@@ -7,7 +7,7 @@
  *
  * For licensing information, see the file 'LICENCE' in this directory.
  *
- * $Id: erase.c,v 1.76 2005/05/03 15:11:40 dedekind Exp $
+ * $Id: erase.c,v 1.80 2005/07/14 19:46:24 joern Exp $
  *
  */
 
@@ -300,100 +300,86 @@ static void jffs2_free_all_node_refs(struct jffs2_sb_info *c, struct jffs2_erase
         jeb->last_node = NULL;
 }
 
-static void jffs2_mark_erased_block(struct jffs2_sb_info *c, struct jffs2_eraseblock *jeb)
+static int jffs2_block_check_erase(struct jffs2_sb_info *c, struct jffs2_eraseblock *jeb, uint32_t *bad_offset)
 {
-        struct jffs2_raw_node_ref *marker_ref = NULL;
-        unsigned char *ebuf;
+        void *ebuf;
+        uint32_t ofs;
         size_t retlen;
-        int ret;
-        uint32_t bad_offset;
-
-        if ((!jffs2_cleanmarker_oob(c)) && (c->cleanmarker_size > 0)) {
-                marker_ref = jffs2_alloc_raw_node_ref();
-                if (!marker_ref) {
-                        printk(KERN_WARNING "Failed to allocate raw node ref for clean marker\n");
-                        /* Stick it back on the list from whence it came and come back later */
-                        jffs2_erase_pending_trigger(c);
-                        spin_lock(&c->erase_completion_lock);
-                        list_add(&jeb->list, &c->erase_complete_list);
-                        spin_unlock(&c->erase_completion_lock);
-                        return;
-                }
-        }
+        int ret = -EIO;
+        
         ebuf = kmalloc(PAGE_SIZE, GFP_KERNEL);
         if (!ebuf) {
-                printk(KERN_WARNING "Failed to allocate page buffer for verifying erase at 0x%08x. Assuming it worked\n", jeb->offset);
-        } else {
-                uint32_t ofs = jeb->offset;
+                printk(KERN_WARNING "Failed to allocate page buffer for verifying erase at 0x%08x. Refiling\n", jeb->offset);
+                return -EAGAIN;
+        }
 
-                D1(printk(KERN_DEBUG "Verifying erase at 0x%08x\n", jeb->offset));
-                while(ofs < jeb->offset + c->sector_size) {
-                        uint32_t readlen = min((uint32_t)PAGE_SIZE, jeb->offset + c->sector_size - ofs);
-                        int i;
+        D1(printk(KERN_DEBUG "Verifying erase at 0x%08x\n", jeb->offset));
 
-                        bad_offset = ofs;
+        for (ofs = jeb->offset; ofs < jeb->offset + c->sector_size; ) {
+                uint32_t readlen = min((uint32_t)PAGE_SIZE, jeb->offset + c->sector_size - ofs);
+                int i;
 
-                        ret = c->mtd->read(c->mtd, ofs, readlen, &retlen, ebuf);
+                *bad_offset = ofs;
 
-                        if (ret) {
-                                printk(KERN_WARNING "Read of newly-erased block at 0x%08x failed: %d. Putting on bad_list\n", ofs, ret);
-                                goto bad;
-                        }
-                        if (retlen != readlen) {
-                                printk(KERN_WARNING "Short read from newly-erased block at 0x%08x. Wanted %d, got %zd\n", ofs, readlen, retlen);
-                                goto bad;
-                        }
-                        for (i=0; i<readlen; i += sizeof(unsigned long)) {
-                                /* It's OK. We know it's properly aligned */
-                                unsigned long datum = *(unsigned long *)(&ebuf[i]);
-                                if (datum + 1) {
-                                        bad_offset += i;
-                                        printk(KERN_WARNING "Newly-erased block contained word 0x%lx at offset 0x%08x\n", datum, bad_offset);
-                                bad: 
-                                        if ((!jffs2_cleanmarker_oob(c)) && (c->cleanmarker_size > 0))
-                                                jffs2_free_raw_node_ref(marker_ref);
-                                        kfree(ebuf);
-                                bad2:
-                                        spin_lock(&c->erase_completion_lock);
-                                        /* Stick it on a list (any list) so
-                                           erase_failed can take it right off
-                                           again.  Silly, but shouldn't happen
-                                           often. */
-                                        list_add(&jeb->list, &c->erasing_list);
-                                        spin_unlock(&c->erase_completion_lock);
-                                        jffs2_erase_failed(c, jeb, bad_offset);
-                                        return;
-                                }
+                ret = jffs2_flash_read(c, ofs, readlen, &retlen, ebuf);
+                if (ret) {
+                        printk(KERN_WARNING "Read of newly-erased block at 0x%08x failed: %d. Putting on bad_list\n", ofs, ret);
+                        goto fail;
+                }
+                if (retlen != readlen) {
+                        printk(KERN_WARNING "Short read from newly-erased block at 0x%08x. Wanted %d, got %zd\n", ofs, readlen, retlen);
+                        goto fail;
+                }
+                for (i=0; i<readlen; i += sizeof(unsigned long)) {
+                        /* It's OK. We know it's properly aligned */
+                        unsigned long *datum = ebuf + i;
+                        if (*datum + 1) {
+                                *bad_offset += i;
+                                printk(KERN_WARNING "Newly-erased block contained word 0x%lx at offset 0x%08x\n", *datum, *bad_offset);
+                                goto fail;
                         }
-                        ofs += readlen;
-                        cond_resched();
                 }
-                kfree(ebuf);
+                ofs += readlen;
+                cond_resched();
         }
+        ret = 0;
+fail:
+        kfree(ebuf);
+        return ret;
+}
 
-        bad_offset = jeb->offset;
+static void jffs2_mark_erased_block(struct jffs2_sb_info *c, struct jffs2_eraseblock *jeb)
+{
+        struct jffs2_raw_node_ref *marker_ref = NULL;
+        size_t retlen;
+        int ret;
+        uint32_t bad_offset;
+
+        switch (jffs2_block_check_erase(c, jeb, &bad_offset)) {
+        case -EAGAIN:   goto refile;
+        case -EIO:      goto filebad;
+        }
 
         /* Write the erase complete marker */   
         D1(printk(KERN_DEBUG "Writing erased marker to block at 0x%08x\n", jeb->offset));
-        if (jffs2_cleanmarker_oob(c)) {
+        bad_offset = jeb->offset;
 
-                if (jffs2_write_nand_cleanmarker(c, jeb))
-                        goto bad2;
-                        
-                jeb->first_node = jeb->last_node = NULL;
+        /* Cleanmarker in oob area or no cleanmarker at all ? */
+        if (jffs2_cleanmarker_oob(c) || c->cleanmarker_size == 0) {
 
-                jeb->free_size = c->sector_size;
-                jeb->used_size = 0;
-                jeb->dirty_size = 0;
-                jeb->wasted_size = 0;
-        } else if (c->cleanmarker_size == 0) {
-                jeb->first_node = jeb->last_node = NULL;
+                if (jffs2_cleanmarker_oob(c)) {
+                        if (jffs2_write_nand_cleanmarker(c, jeb))
+                                goto filebad;
+                }
 
+                jeb->first_node = jeb->last_node = NULL;
                 jeb->free_size = c->sector_size;
                 jeb->used_size = 0;
                 jeb->dirty_size = 0;
                 jeb->wasted_size = 0;
+
         } else {
+
                 struct kvec vecs[1];
                 struct jffs2_unknown_node marker = {
                         .magic =        cpu_to_je16(JFFS2_MAGIC_BITMASK),
@@ -401,21 +387,28 @@ static void jffs2_mark_erased_block(struct jffs2_sb_info *c, struct jffs2_eraseb
                         .totlen =       cpu_to_je32(c->cleanmarker_size)
                 };
 
+                marker_ref = jffs2_alloc_raw_node_ref();
+                if (!marker_ref) {
+                        printk(KERN_WARNING "Failed to allocate raw node ref for clean marker. Refiling\n");
+                        goto refile;
+                }
+
                 marker.hdr_crc = cpu_to_je32(crc32(0, &marker, sizeof(struct jffs2_unknown_node)-4));
 
                 vecs[0].iov_base = (unsigned char *) &marker;
                 vecs[0].iov_len = sizeof(marker);
                 ret = jffs2_flash_direct_writev(c, vecs, 1, jeb->offset, &retlen);
                 
-                if (ret) {
-                        printk(KERN_WARNING "Write clean marker to block at 0x%08x failed: %d\n",
-                               jeb->offset, ret);
-                        goto bad2;
-                }
-                if (retlen != sizeof(marker)) {
-                        printk(KERN_WARNING "Short write to newly-erased block at 0x%08x: Wanted %zd, got %zd\n",
-                               jeb->offset, sizeof(marker), retlen);
-                        goto bad2;
+                if (ret || retlen != sizeof(marker)) {
+                        if (ret)
+                                printk(KERN_WARNING "Write clean marker to block at 0x%08x failed: %d\n",
+                                       jeb->offset, ret);
+                        else
+                                printk(KERN_WARNING "Short write to newly-erased block at 0x%08x: Wanted %zd, got %zd\n",
+                                       jeb->offset, sizeof(marker), retlen);
+
+                        jffs2_free_raw_node_ref(marker_ref);
+                        goto filebad;
                 }
 
                 marker_ref->next_in_ino = NULL;
@@ -444,5 +437,22 @@ static void jffs2_mark_erased_block(struct jffs2_sb_info *c, struct jffs2_eraseb
         c->nr_free_blocks++;
         spin_unlock(&c->erase_completion_lock);
         wake_up(&c->erase_wait);
-}
+        return;
+
+filebad:
+        spin_lock(&c->erase_completion_lock);
+        /* Stick it on a list (any list) so erase_failed can take it
+           right off again.  Silly, but shouldn't happen often. */
+        list_add(&jeb->list, &c->erasing_list);
+        spin_unlock(&c->erase_completion_lock);
+        jffs2_erase_failed(c, jeb, bad_offset);
+        return;
 
+refile:
+        /* Stick it back on the list from whence it came and come back later */
+        jffs2_erase_pending_trigger(c);
+        spin_lock(&c->erase_completion_lock);
+        list_add(&jeb->list, &c->erase_complete_list);
+        spin_unlock(&c->erase_completion_lock);
+        return;
+}
diff --git a/fs/jffs2/nodelist.c b/fs/jffs2/nodelist.c
index c7bbdeec93a6..4991c348f6ec 100644
--- a/fs/jffs2/nodelist.c
+++ b/fs/jffs2/nodelist.c
@@ -7,7 +7,7 @@
  *
  * For licensing information, see the file 'LICENCE' in this directory.
  *
- * $Id: nodelist.c,v 1.97 2005/07/06 15:18:41 dwmw2 Exp $
+ * $Id: nodelist.c,v 1.98 2005/07/10 15:15:32 dedekind Exp $
  *
  */
 
@@ -55,11 +55,11 @@ void jffs2_add_fd_to_list(struct jffs2_sb_info *c, struct jffs2_full_dirent *new
         });
 }
 
-/* Put a new tmp_dnode_info into the list, keeping the list in 
-   order of increasing version
-*/
-
-static void jffs2_add_tn_to_list(struct jffs2_tmp_dnode_info *tn, struct rb_root *list)
+/* 
+ * Put a new tmp_dnode_info into the temporaty RB-tree, keeping the list in 
+ * order of increasing version.
+ */
+static void jffs2_add_tn_to_tree(struct jffs2_tmp_dnode_info *tn, struct rb_root *list)
 {
         struct rb_node **p = &list->rb_node;
         struct rb_node * parent = NULL;
@@ -420,7 +420,7 @@ int jffs2_get_inode_nodes(struct jffs2_sb_info *c, struct jffs2_inode_info *f,
                         D1(printk(KERN_DEBUG "dnode @%08x: ver %u, offset %04x, dsize %04x\n",
                                   ref_offset(ref), je32_to_cpu(node.i.version),
                                   je32_to_cpu(node.i.offset), je32_to_cpu(node.i.dsize)));
-                        jffs2_add_tn_to_list(tn, &ret_tn);
+                        jffs2_add_tn_to_tree(tn, &ret_tn);
                         break;
 
                 default:
diff --git a/fs/jffs2/os-linux.h b/fs/jffs2/os-linux.h
index 7bf72e012c94..d900c8929b09 100644
--- a/fs/jffs2/os-linux.h
+++ b/fs/jffs2/os-linux.h
@@ -7,7 +7,7 @@
  *
  * For licensing information, see the file 'LICENCE' in this directory.
  *
- * $Id: os-linux.h,v 1.57 2005/07/06 12:13:09 dwmw2 Exp $
+ * $Id: os-linux.h,v 1.58 2005/07/12 02:34:35 tpoynor Exp $
  *
  */
 
@@ -86,6 +86,8 @@ static inline void jffs2_init_inode_info(struct jffs2_inode_info *f)
 #define jffs2_dataflash(c) (0)
 #define jffs2_nor_ecc_flash_setup(c) (0)
 #define jffs2_nor_ecc_flash_cleanup(c) do {} while (0)
+#define jffs2_dataflash_setup(c) (0)
+#define jffs2_dataflash_cleanup(c) do {} while (0)
 
 #else /* NAND and/or ECC'd NOR support present */
 
diff --git a/fs/jffs2/readinode.c b/fs/jffs2/readinode.c
index 081656c1d49e..5b2a83599d73 100644
--- a/fs/jffs2/readinode.c
+++ b/fs/jffs2/readinode.c
@@ -7,7 +7,7 @@
  *
  * For licensing information, see the file 'LICENCE' in this directory.
  *
- * $Id: readinode.c,v 1.120 2005/07/05 21:03:07 dwmw2 Exp $
+ * $Id: readinode.c,v 1.125 2005/07/10 13:13:55 dedekind Exp $
  *
  */
 
@@ -151,6 +151,9 @@ int jffs2_add_full_dnode_to_inode(struct jffs2_sb_info *c, struct jffs2_inode_in
 
         D1(printk(KERN_DEBUG "jffs2_add_full_dnode_to_inode(ino #%u, f %p, fn %p)\n", f->inocache->ino, f, fn));
 
+        if (unlikely(!fn->size))
+                return 0;
+
         newfrag = jffs2_alloc_node_frag();
         if (unlikely(!newfrag))
                 return -ENOMEM;
@@ -158,11 +161,6 @@ int jffs2_add_full_dnode_to_inode(struct jffs2_sb_info *c, struct jffs2_inode_in
         D2(printk(KERN_DEBUG "adding node %04x-%04x @0x%08x on flash, newfrag *%p\n",
                   fn->ofs, fn->ofs+fn->size, ref_offset(fn->raw), newfrag));
         
-        if (unlikely(!fn->size)) {
-                jffs2_free_node_frag(newfrag);
-                return 0;
-        }
-
         newfrag->ofs = fn->ofs;
         newfrag->size = fn->size;
         newfrag->node = fn;
@@ -560,7 +558,6 @@ static int jffs2_do_read_inode_internal(struct jffs2_sb_info *c,
                 }
         next_tn:
                 BUG_ON(rb->rb_left);
-                repl_rb = NULL;
                 if (rb->rb_parent && rb->rb_parent->rb_left == rb) {
                         /* We were then left-hand child of our parent. We need
                            to move our own right-hand child into our place. */
diff --git a/fs/jffs2/super.c b/fs/jffs2/super.c
index 2cf14cf8b35a..aaf9475cfb6a 100644
--- a/fs/jffs2/super.c
+++ b/fs/jffs2/super.c
@@ -7,7 +7,7 @@
  *
  * For licensing information, see the file 'LICENCE' in this directory.
  *
- * $Id: super.c,v 1.106 2005/05/18 11:37:25 dedekind Exp $
+ * $Id: super.c,v 1.107 2005/07/12 16:37:08 dedekind Exp $
  *
  */
 
@@ -140,6 +140,15 @@ static struct super_block *jffs2_get_sb_mtd(struct file_system_type *fs_type,
         D1(printk(KERN_DEBUG "jffs2_get_sb_mtd(): New superblock for device %d (\"%s\")\n",
                   mtd->index, mtd->name));
 
+        /* Initialize JFFS2 superblock locks, the further initialization will be
+         * done later */
+        init_MUTEX(&c->alloc_sem);
+        init_MUTEX(&c->erase_free_sem);
+        init_waitqueue_head(&c->erase_wait);
+        init_waitqueue_head(&c->inocache_wq);
+        spin_lock_init(&c->erase_completion_lock);
+        spin_lock_init(&c->inocache_lock);
+
         sb->s_op = &jffs2_super_operations;
         sb->s_flags = flags | MS_NOATIME;