nilfs2: add update functions of virtual block address to dat
authorRyusuke Konishi <konishi.ryusuke@lab.ntt.co.jp>
Sat, 15 Aug 2009 08:22:13 +0000 (17:22 +0900)
committerRyusuke Konishi <konishi.ryusuke@lab.ntt.co.jp>
Mon, 14 Sep 2009 09:27:15 +0000 (18:27 +0900)
This is a preparation for the successive cleanup ("nilfs2: allow btree
to directly call dat operations").

This adds functions bundling a few operations to change an entry of
virtual block address on the dat file.

Signed-off-by: Ryusuke Konishi <konishi.ryusuke@lab.ntt.co.jp>
fs/nilfs2/bmap.c
fs/nilfs2/dat.c
fs/nilfs2/dat.h

index 99d58a028b94c3c9698661bac527e14ed2812a72..13e95a907ece993a543ca9d728bc014511de6f91 100644 (file)
@@ -533,38 +533,25 @@ int nilfs_bmap_prepare_update_v(struct nilfs_bmap *bmap,
                                union nilfs_bmap_ptr_req *oldreq,
                                union nilfs_bmap_ptr_req *newreq)
 {
-       struct inode *dat = nilfs_bmap_get_dat(bmap);
-       int ret;
-
-       ret = nilfs_dat_prepare_end(dat, &oldreq->bpr_req);
-       if (ret < 0)
-               return ret;
-       ret = nilfs_dat_prepare_alloc(dat, &newreq->bpr_req);
-       if (ret < 0)
-               nilfs_dat_abort_end(dat, &oldreq->bpr_req);
-
-       return ret;
+       return nilfs_dat_prepare_update(nilfs_bmap_get_dat(bmap),
+                                       &oldreq->bpr_req, &newreq->bpr_req);
 }
 
 void nilfs_bmap_commit_update_v(struct nilfs_bmap *bmap,
                                union nilfs_bmap_ptr_req *oldreq,
                                union nilfs_bmap_ptr_req *newreq)
 {
-       struct inode *dat = nilfs_bmap_get_dat(bmap);
-
-       nilfs_dat_commit_end(dat, &oldreq->bpr_req,
-                            bmap->b_ptr_type == NILFS_BMAP_PTR_VS);
-       nilfs_dat_commit_alloc(dat, &newreq->bpr_req);
+       nilfs_dat_commit_update(nilfs_bmap_get_dat(bmap),
+                               &oldreq->bpr_req, &newreq->bpr_req,
+                               bmap->b_ptr_type == NILFS_BMAP_PTR_VS);
 }
 
 void nilfs_bmap_abort_update_v(struct nilfs_bmap *bmap,
                               union nilfs_bmap_ptr_req *oldreq,
                               union nilfs_bmap_ptr_req *newreq)
 {
-       struct inode *dat = nilfs_bmap_get_dat(bmap);
-
-       nilfs_dat_abort_end(dat, &oldreq->bpr_req);
-       nilfs_dat_abort_alloc(dat, &newreq->bpr_req);
+       nilfs_dat_abort_update(nilfs_bmap_get_dat(bmap),
+                              &oldreq->bpr_req, &newreq->bpr_req);
 }
 
 static struct lock_class_key nilfs_bmap_dat_lock_key;
index 1cfcc1a0fdf1c3885f6a5475b2ef494af07d889e..1ff8e15bd36b52dad460dd1f28e7797916b87165 100644 (file)
@@ -211,6 +211,37 @@ void nilfs_dat_abort_end(struct inode *dat, struct nilfs_palloc_req *req)
        nilfs_dat_abort_entry(dat, req);
 }
 
+int nilfs_dat_prepare_update(struct inode *dat,
+                            struct nilfs_palloc_req *oldreq,
+                            struct nilfs_palloc_req *newreq)
+{
+       int ret;
+
+       ret = nilfs_dat_prepare_end(dat, oldreq);
+       if (!ret) {
+               ret = nilfs_dat_prepare_alloc(dat, newreq);
+               if (ret < 0)
+                       nilfs_dat_abort_end(dat, oldreq);
+       }
+       return ret;
+}
+
+void nilfs_dat_commit_update(struct inode *dat,
+                            struct nilfs_palloc_req *oldreq,
+                            struct nilfs_palloc_req *newreq, int dead)
+{
+       nilfs_dat_commit_end(dat, oldreq, dead);
+       nilfs_dat_commit_alloc(dat, newreq);
+}
+
+void nilfs_dat_abort_update(struct inode *dat,
+                           struct nilfs_palloc_req *oldreq,
+                           struct nilfs_palloc_req *newreq)
+{
+       nilfs_dat_abort_end(dat, oldreq);
+       nilfs_dat_abort_alloc(dat, newreq);
+}
+
 /**
  * nilfs_dat_mark_dirty -
  * @dat: DAT file inode
index 91dc3372a39a834e3a9ca05ca2547c45b9117d19..406070d3ff4922b97dc62e36c28f47b47d671d74 100644 (file)
@@ -41,6 +41,12 @@ void nilfs_dat_commit_start(struct inode *, struct nilfs_palloc_req *,
 int nilfs_dat_prepare_end(struct inode *, struct nilfs_palloc_req *);
 void nilfs_dat_commit_end(struct inode *, struct nilfs_palloc_req *, int);
 void nilfs_dat_abort_end(struct inode *, struct nilfs_palloc_req *);
+int nilfs_dat_prepare_update(struct inode *, struct nilfs_palloc_req *,
+                            struct nilfs_palloc_req *);
+void nilfs_dat_commit_update(struct inode *, struct nilfs_palloc_req *,
+                            struct nilfs_palloc_req *, int);
+void nilfs_dat_abort_update(struct inode *, struct nilfs_palloc_req *,
+                           struct nilfs_palloc_req *);
 
 int nilfs_dat_mark_dirty(struct inode *, __u64);
 int nilfs_dat_freev(struct inode *, __u64 *, size_t);