GFS2: decouple quota allocations from block allocations

This patch separates the code pertaining to allocations into two
parts: quota-related information and block reservations.
This patch also moves all the block reservation structure allocations to
function gfs2_inplace_reserve to simplify the code, and moves
the frees to function gfs2_inplace_release.

Signed-off-by: Bob Peterson <rpeterso@redhat.com>
Signed-off-by: Steven Whitehouse <swhiteho@redhat.com>

1 files changed, 28 insertions, 30 deletions

diff --git a/fs/gfs2/inode.c b/fs/gfs2/inode.c
index 3ab192bac7d3..ab8c429880a5 100644
--- a/fs/gfs2/inode.c
+++ b/fs/gfs2/inode.c
@@ -391,11 +391,11 @@ static int alloc_dinode(struct gfs2_inode *dip, u64 *no_addr, u64 *generation)
         int error;
         int dblocks = 0;
 
-        if (gfs2_alloc_get(dip) == NULL)
-                return -ENOMEM;
+        error = gfs2_rindex_update(sdp);
+        if (error)
+                fs_warn(sdp, "rindex update returns %d\n", error);
 
-        dip->i_alloc->al_requested = RES_DINODE;
-        error = gfs2_inplace_reserve(dip);
+        error = gfs2_inplace_reserve(dip, RES_DINODE);
         if (error)
                 goto out;
 
@@ -410,7 +410,6 @@ static int alloc_dinode(struct gfs2_inode *dip, u64 *no_addr, u64 *generation)
 out_ipreserv:
         gfs2_inplace_release(dip);
 out:
-        gfs2_alloc_put(dip);
         return error;
 }
 
@@ -526,7 +525,7 @@ static int make_dinode(struct gfs2_inode *dip, struct gfs2_glock *gl,
         int error;
 
         munge_mode_uid_gid(dip, &mode, &uid, &gid);
-        if (!gfs2_alloc_get(dip))
+        if (!gfs2_qadata_get(dip))
                 return -ENOMEM;
 
         error = gfs2_quota_lock(dip, uid, gid);
@@ -548,7 +547,7 @@ static int make_dinode(struct gfs2_inode *dip, struct gfs2_glock *gl,
 out_quota:
         gfs2_quota_unlock(dip);
 out:
-        gfs2_alloc_put(dip);
+        gfs2_qadata_put(dip);
         return error;
 }
 
@@ -556,13 +555,13 @@ static int link_dinode(struct gfs2_inode *dip, const struct qstr *name,
                        struct gfs2_inode *ip)
 {
         struct gfs2_sbd *sdp = GFS2_SB(&dip->i_inode);
-        struct gfs2_alloc *al;
+        struct gfs2_qadata *qa;
         int alloc_required;
         struct buffer_head *dibh;
         int error;
 
-        al = gfs2_alloc_get(dip);
-        if (!al)
+        qa = gfs2_qadata_get(dip);
+        if (!qa)
                 return -ENOMEM;
 
         error = gfs2_quota_lock(dip, NO_QUOTA_CHANGE, NO_QUOTA_CHANGE);
@@ -577,9 +576,7 @@ static int link_dinode(struct gfs2_inode *dip, const struct qstr *name,
                 if (error)
                         goto fail_quota_locks;
 
-                al->al_requested = sdp->sd_max_dirres;
-
-                error = gfs2_inplace_reserve(dip);
+                error = gfs2_inplace_reserve(dip, sdp->sd_max_dirres);
                 if (error)
                         goto fail_quota_locks;
 
@@ -620,7 +617,7 @@ fail_quota_locks:
         gfs2_quota_unlock(dip);
 
 fail:
-        gfs2_alloc_put(dip);
+        gfs2_qadata_put(dip);
         return error;
 }
 
@@ -729,9 +726,12 @@ static int gfs2_create_inode(struct inode *dir, struct dentry *dentry,
                 brelse(bh);
 
         gfs2_trans_end(sdp);
-        gfs2_inplace_release(dip);
+        /* Check if we reserved space in the rgrp. Function link_dinode may
+           not, depending on whether alloc is required. */
+        if (dip->i_res)
+                gfs2_inplace_release(dip);
         gfs2_quota_unlock(dip);
-        gfs2_alloc_put(dip);
+        gfs2_qadata_put(dip);
         mark_inode_dirty(inode);
         gfs2_glock_dq_uninit_m(2, ghs);
         d_instantiate(dentry, inode);
@@ -876,8 +876,9 @@ static int gfs2_link(struct dentry *old_dentry, struct inode *dir,
         error = 0;
 
         if (alloc_required) {
-                struct gfs2_alloc *al = gfs2_alloc_get(dip);
-                if (!al) {
+                struct gfs2_qadata *qa = gfs2_qadata_get(dip);
+
+                if (!qa) {
                         error = -ENOMEM;
                         goto out_gunlock;
                 }
@@ -886,9 +887,7 @@ static int gfs2_link(struct dentry *old_dentry, struct inode *dir,
                 if (error)
                         goto out_alloc;
 
-                al->al_requested = sdp->sd_max_dirres;
-
-                error = gfs2_inplace_reserve(dip);
+                error = gfs2_inplace_reserve(dip, sdp->sd_max_dirres);
                 if (error)
                         goto out_gunlock_q;
 
@@ -931,7 +930,7 @@ out_gunlock_q:
                 gfs2_quota_unlock(dip);
 out_alloc:
         if (alloc_required)
-                gfs2_alloc_put(dip);
+                gfs2_qadata_put(dip);
 out_gunlock:
         gfs2_glock_dq(ghs + 1);
 out_child:
@@ -1354,8 +1353,9 @@ static int gfs2_rename(struct inode *odir, struct dentry *odentry,
         error = 0;
 
         if (alloc_required) {
-                struct gfs2_alloc *al = gfs2_alloc_get(ndip);
-                if (!al) {
+                struct gfs2_qadata *qa = gfs2_qadata_get(ndip);
+
+                if (!qa) {
                         error = -ENOMEM;
                         goto out_gunlock;
                 }
@@ -1364,9 +1364,7 @@ static int gfs2_rename(struct inode *odir, struct dentry *odentry,
                 if (error)
                         goto out_alloc;
 
-                al->al_requested = sdp->sd_max_dirres;
-
-                error = gfs2_inplace_reserve(ndip);
+                error = gfs2_inplace_reserve(ndip, sdp->sd_max_dirres);
                 if (error)
                         goto out_gunlock_q;
 
@@ -1427,7 +1425,7 @@ out_gunlock_q:
                 gfs2_quota_unlock(ndip);
 out_alloc:
         if (alloc_required)
-                gfs2_alloc_put(ndip);
+                gfs2_qadata_put(ndip);
 out_gunlock:
         while (x--) {
                 gfs2_glock_dq(ghs + x);
@@ -1588,7 +1586,7 @@ static int setattr_chown(struct inode *inode, struct iattr *attr)
         if (!(attr->ia_valid & ATTR_GID) || ogid == ngid)
                 ogid = ngid = NO_QUOTA_CHANGE;
 
-        if (!gfs2_alloc_get(ip))
+        if (!gfs2_qadata_get(ip))
                 return -ENOMEM;
 
         error = gfs2_quota_lock(ip, nuid, ngid);
@@ -1620,7 +1618,7 @@ out_end_trans:
 out_gunlock_q:
         gfs2_quota_unlock(ip);
 out_alloc:
-        gfs2_alloc_put(ip);
+        gfs2_qadata_put(ip);
         return error;
 }