]> pilppa.com Git - linux-2.6-omap-h63xx.git/commitdiff
UBIFS: fix assertions
authorArtem Bityutskiy <Artem.Bityutskiy@nokia.com>
Mon, 26 Jan 2009 14:12:20 +0000 (16:12 +0200)
committerArtem Bityutskiy <Artem.Bityutskiy@nokia.com>
Mon, 26 Jan 2009 16:22:47 +0000 (18:22 +0200)
I introduce wrong assertions in one of the previous commits, this
patch fixes them.

Also, initialize debugfs after the debugging check. This is a little
nicer because we want the FS data to be accessible to external users
after everything has been initialized.

Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>
fs/ubifs/super.c

index fd7fc7f3b7a64fddbe0281b2423aa27996a31c61..dbfc88714716b6dcd8d98499a46d2f138366a1fc 100644 (file)
@@ -1320,20 +1320,21 @@ static int mount_ubifs(struct ubifs_info *c)
                else {
                        c->need_recovery = 0;
                        ubifs_msg("recovery completed");
+                       /* GC LEB has to be empty and taken at this point */
+                       ubifs_assert(c->lst.taken_empty_lebs == 1);
                }
-       }
+       } else
+               ubifs_assert(c->lst.taken_empty_lebs == 1);
 
-       err = dbg_debugfs_init_fs(c);
+       err = dbg_check_filesystem(c);
        if (err)
                goto out_infos;
 
-       err = dbg_check_filesystem(c);
+       err = dbg_debugfs_init_fs(c);
        if (err)
                goto out_infos;
 
        c->always_chk_crc = 0;
-       /* GC LEB has to be empty and taken at this point */
-       ubifs_assert(c->lst.taken_empty_lebs == 1);
 
        ubifs_msg("mounted UBI device %d, volume %d, name \"%s\"",
                  c->vi.ubi_num, c->vi.vol_id, c->vi.name);
@@ -1663,7 +1664,7 @@ static void ubifs_remount_ro(struct ubifs_info *c)
        int i, err;
 
        ubifs_assert(!c->need_recovery);
-       ubifs_assert(!c->ro_media);
+       ubifs_assert(!(c->vfs_sb->s_flags & MS_RDONLY));
 
        commit_on_unmount(c);
        mutex_lock(&c->umount_mutex);