nilfs2: add update functions of virtual block address to dat

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>

diff --git a/fs/nilfs2/bmap.c b/fs/nilfs2/bmap.c
index 99d58a028b94c3c9698661bac527e14ed2812a72..13e95a907ece993a543ca9d728bc014511de6f91 100644 (file)
--- a/fs/nilfs2/bmap.c
+++ b/fs/nilfs2/bmap.c
@@ -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;

diff --git a/fs/nilfs2/dat.c b/fs/nilfs2/dat.c
index 1cfcc1a0fdf1c3885f6a5475b2ef494af07d889e..1ff8e15bd36b52dad460dd1f28e7797916b87165 100644 (file)
--- a/fs/nilfs2/dat.c
+++ b/fs/nilfs2/dat.c
@@ -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

diff --git a/fs/nilfs2/dat.h b/fs/nilfs2/dat.h
index 91dc3372a39a834e3a9ca05ca2547c45b9117d19..406070d3ff4922b97dc62e36c28f47b47d671d74 100644 (file)
--- a/fs/nilfs2/dat.h
+++ b/fs/nilfs2/dat.h
@@ -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);