xfs: fix xfs_init_attr_trans not handling explicit operation codes
authorDarrick J. Wong <djwong@kernel.org>
Wed, 22 May 2024 06:02:01 +0000 (23:02 -0700)
committerChandan Babu R <chandanbabu@kernel.org>
Mon, 27 May 2024 10:25:52 +0000 (15:55 +0530)
When we were converting the attr code to use an explicit operation code
instead of keying off of attr->value being null, we forgot to change the
code that initializes the transaction reservation.  Split the function
into two helpers that handle the !remove and remove cases, then fix both
callsites to handle this correctly.

Fixes: c27411d4c640 ("xfs: make attr removal an explicit operation")
Signed-off-by: "Darrick J. Wong" <djwong@kernel.org>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Chandan Babu R <chandanbabu@kernel.org>
fs/xfs/libxfs/xfs_attr.c
fs/xfs/libxfs/xfs_attr.h
fs/xfs/xfs_attr_item.c

index 430cd3244c143dc065f0490b1ec3c3feef47e2e3..f30bcc64100d56b7199fc6b73ee777f6a6e06aa9 100644 (file)
@@ -329,26 +329,20 @@ xfs_attr_calc_size(
        return nblks;
 }
 
-/* Initialize transaction reservation for attr operations */
-void
-xfs_init_attr_trans(
-       struct xfs_da_args      *args,
-       struct xfs_trans_res    *tres,
-       unsigned int            *total)
+/* Initialize transaction reservation for an xattr set/replace/upsert */
+inline struct xfs_trans_res
+xfs_attr_set_resv(
+       const struct xfs_da_args        *args)
 {
-       struct xfs_mount        *mp = args->dp->i_mount;
-
-       if (args->value) {
-               tres->tr_logres = M_RES(mp)->tr_attrsetm.tr_logres +
-                                M_RES(mp)->tr_attrsetrt.tr_logres *
-                                args->total;
-               tres->tr_logcount = XFS_ATTRSET_LOG_COUNT;
-               tres->tr_logflags = XFS_TRANS_PERM_LOG_RES;
-               *total = args->total;
-       } else {
-               *tres = M_RES(mp)->tr_attrrm;
-               *total = XFS_ATTRRM_SPACE_RES(mp);
-       }
+       struct xfs_mount                *mp = args->dp->i_mount;
+       struct xfs_trans_res            ret = {
+               .tr_logres = M_RES(mp)->tr_attrsetm.tr_logres +
+                           M_RES(mp)->tr_attrsetrt.tr_logres * args->total,
+               .tr_logcount            = XFS_ATTRSET_LOG_COUNT,
+               .tr_logflags            = XFS_TRANS_PERM_LOG_RES,
+       };
+
+       return ret;
 }
 
 /*
@@ -1006,7 +1000,7 @@ xfs_attr_set(
        struct xfs_trans_res    tres;
        int                     error, local;
        int                     rmt_blks = 0;
-       unsigned int            total;
+       unsigned int            total = 0;
 
        ASSERT(!args->trans);
 
@@ -1033,10 +1027,15 @@ xfs_attr_set(
 
                if (!local)
                        rmt_blks = xfs_attr3_rmt_blocks(mp, args->valuelen);
+
+               tres = xfs_attr_set_resv(args);
+               total = args->total;
                break;
        case XFS_ATTRUPDATE_REMOVE:
                XFS_STATS_INC(mp, xs_attr_remove);
                rmt_blks = xfs_attr3_max_rmt_blocks(mp);
+               tres = M_RES(mp)->tr_attrrm;
+               total = XFS_ATTRRM_SPACE_RES(mp);
                break;
        }
 
@@ -1044,7 +1043,6 @@ xfs_attr_set(
         * Root fork attributes can use reserved data blocks for this
         * operation if necessary
         */
-       xfs_init_attr_trans(args, &tres, &total);
        error = xfs_trans_alloc_inode(dp, &tres, total, 0, rsvd, &args->trans);
        if (error)
                return error;
index 088cb7b301680ca024d58cf2ca8d7bbdc5a4ef8d..0e51d0723f9aa36c1519f6ad2c1881489a26dd8a 100644 (file)
@@ -565,8 +565,7 @@ bool xfs_attr_check_namespace(unsigned int attr_flags);
 bool xfs_attr_namecheck(unsigned int attr_flags, const void *name,
                size_t length);
 int xfs_attr_calc_size(struct xfs_da_args *args, int *local);
-void xfs_init_attr_trans(struct xfs_da_args *args, struct xfs_trans_res *tres,
-                        unsigned int *total);
+struct xfs_trans_res xfs_attr_set_resv(const struct xfs_da_args *args);
 
 /*
  * Check to see if the attr should be upgraded from non-existent or shortform to
index 2b10ac4c5fce24d0df38b1fb2445e491c1aa389b..f683b7a9323f16146b1c8cad6e54a40ca056997d 100644 (file)
@@ -746,7 +746,7 @@ xfs_attr_recover_work(
        struct xfs_attri_log_format     *attrp;
        struct xfs_attri_log_nameval    *nv = attrip->attri_nameval;
        int                             error;
-       int                             total;
+       unsigned int                    total = 0;
 
        /*
         * First check the validity of the attr described by the ATTRI.  If any
@@ -763,7 +763,20 @@ xfs_attr_recover_work(
                return PTR_ERR(attr);
        args = attr->xattri_da_args;
 
-       xfs_init_attr_trans(args, &resv, &total);
+       switch (xfs_attr_intent_op(attr)) {
+       case XFS_ATTRI_OP_FLAGS_PPTR_SET:
+       case XFS_ATTRI_OP_FLAGS_PPTR_REPLACE:
+       case XFS_ATTRI_OP_FLAGS_SET:
+       case XFS_ATTRI_OP_FLAGS_REPLACE:
+               resv = xfs_attr_set_resv(args);
+               total = args->total;
+               break;
+       case XFS_ATTRI_OP_FLAGS_PPTR_REMOVE:
+       case XFS_ATTRI_OP_FLAGS_REMOVE:
+               resv = M_RES(mp)->tr_attrrm;
+               total = XFS_ATTRRM_SPACE_RES(mp);
+               break;
+       }
        resv = xlog_recover_resv(&resv);
        error = xfs_trans_alloc(mp, &resv, total, 0, XFS_TRANS_RESERVE, &tp);
        if (error)