This rewrites nilfs_checkpoint_is_mounted() function so that it
decides whether a checkpoint is mounted by whether the corresponding
root object is found in checkpoint tree.

Signed-off-by: Ryusuke Konishi <[email protected]>
---
 fs/nilfs2/the_nilfs.c |   28 +++++++++++-----------------
 1 files changed, 11 insertions(+), 17 deletions(-)

diff --git a/fs/nilfs2/the_nilfs.c b/fs/nilfs2/the_nilfs.c
index da4aa43..3bb1579 100644
--- a/fs/nilfs2/the_nilfs.c
+++ b/fs/nilfs2/the_nilfs.c
@@ -954,26 +954,20 @@ struct nilfs_sb_info *nilfs_find_sbinfo(struct the_nilfs 
*nilfs,
 int nilfs_checkpoint_is_mounted(struct the_nilfs *nilfs, __u64 cno,
                                int snapshot_mount)
 {
-       struct nilfs_sb_info *sbi;
-       int ret = 0;
+       struct nilfs_root *root;
+       int ret;
 
-       down_read(&nilfs->ns_super_sem);
-       if (cno == 0 || cno > nilfs->ns_cno)
-               goto out_unlock;
+       if (cno < 0 || cno > nilfs->ns_cno)
+               return false;
 
-       list_for_each_entry(sbi, &nilfs->ns_supers, s_list) {
-               if (sbi->s_snapshot_cno == cno &&
-                   (!snapshot_mount || nilfs_test_opt(sbi, SNAPSHOT))) {
-                                       /* exclude read-only mounts */
-                       ret++;
-                       break;
-               }
-       }
-       /* for protecting recent checkpoints */
        if (cno >= nilfs_last_cno(nilfs))
-               ret++;
+               return true;    /* protect recent checkpoints */
 
- out_unlock:
-       up_read(&nilfs->ns_super_sem);
+       ret = false;
+       root = nilfs_lookup_root(nilfs, cno);
+       if (root) {
+               ret = true;
+               nilfs_put_root(root);
+       }
        return ret;
 }
-- 
1.6.6.2

--
To unsubscribe from this list: send the line "unsubscribe linux-nilfs" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to