xfs: calculate XFS_TRANS_QM_QUOTAOFF_END space log reservation at mount time
authorJeff Liu <jeff.liu@oracle.com>
Mon, 28 Jan 2013 13:27:21 +0000 (21:27 +0800)
committerBen Myers <bpm@sgi.com>
Fri, 1 Feb 2013 20:45:50 +0000 (14:45 -0600)
Convert the calculation for end of quotaoff log space reservation
from runtime to mount time.

Signed-off-by: Jie Liu <jeff.liu@oracle.com>
CC: Dave Chinner <david@fromorbit.com>
Reviewed-by: Mark Tinguely <tinguely@sgi.com>
Signed-off-by: Ben Myers <bpm@sgi.com>
fs/xfs/xfs_mount.h
fs/xfs/xfs_qm_syscalls.c
fs/xfs/xfs_trans.c
fs/xfs/xfs_trans.h

index 4f5e148ffd1c3e7240e3526ccc9dfdb31527998e..b46009dc7b130fd44f10ad3e0d76c4865eb95807 100644 (file)
@@ -44,6 +44,7 @@ typedef struct xfs_trans_reservations {
        uint    tr_qm_setqlim;  /* adjust quota limits */
        uint    tr_qm_dqalloc;  /* allocate quota on disk */
        uint    tr_qm_quotaoff; /* turn quota off */
+       uint    tr_qm_equotaoff;/* end of turn quota off */
 } xfs_trans_reservations_t;
 
 #ifndef __KERNEL__
index 4605f8914cb4eca8659c588254266a59ee81325d..cf9a34051e070def83dbf53db44e84f451807ba3 100644 (file)
@@ -639,8 +639,9 @@ xfs_qm_log_quotaoff_end(
 
        tp = xfs_trans_alloc(mp, XFS_TRANS_QM_QUOTAOFF_END);
 
-       if ((error = xfs_trans_reserve(tp, 0, sizeof(xfs_qoff_logitem_t) * 2,
-                                     0, 0, XFS_DEFAULT_LOG_COUNT))) {
+       error = xfs_trans_reserve(tp, 0, XFS_QM_QUOTAOFF_END_LOG_RES(mp),
+                                 0, 0, XFS_DEFAULT_LOG_COUNT);
+       if (error) {
                xfs_trans_cancel(tp, 0);
                return (error);
        }
index a557c82e58f0ca8cfc90c5abd0f3928242f523c0..84643fa5e46580fbf793c9084b901074d6f58f3c 100644 (file)
@@ -594,6 +594,17 @@ xfs_calc_qm_quotaoff_reservation(
                xfs_calc_buf_res(1, mp->m_sb.sb_sectsize);
 }
 
+/*
+ * End of turning off quotas.
+ *    the xfs_qoff_logitem_t: sizeof(struct xfs_qoff_logitem) * 2
+ */
+STATIC uint
+xfs_calc_qm_quotaoff_end_reservation(
+       struct xfs_mount        *mp)
+{
+       return sizeof(struct xfs_qoff_logitem) * 2;
+}
+
 /*
  * Initialize the precomputed transaction reservation values
  * in the mount structure.
@@ -629,6 +640,7 @@ xfs_trans_init(
        resp->tr_qm_setqlim = xfs_calc_qm_setqlim_reservation(mp);
        resp->tr_qm_dqalloc = xfs_calc_qm_dqalloc_reservation(mp);
        resp->tr_qm_quotaoff = xfs_calc_qm_quotaoff_reservation(mp);
+       resp->tr_qm_equotaoff = xfs_calc_qm_quotaoff_end_reservation(mp);
 }
 
 /*
index 1e103dad17d16134fa7393fa851a0a05e30400f8..57339e404cf6e33ce5dbeca834919d63a51c7f40 100644 (file)
@@ -266,6 +266,7 @@ struct xfs_log_item_desc {
 #define XFS_QM_SETQLIM_LOG_RES(mp)     ((mp)->m_reservations.tr_qm_setqlim)
 #define XFS_QM_DQALLOC_LOG_RES(mp)     ((mp)->m_reservations.tr_qm_dqalloc)
 #define XFS_QM_QUOTAOFF_LOG_RES(mp)    ((mp)->m_reservations.tr_qm_quotaoff)
+#define XFS_QM_QUOTAOFF_END_LOG_RES(mp)        ((mp)->m_reservations.tr_qm_equotaoff)
 
 /*
  * Various log count values.