xfs: remove struct xfs_attr_item.xattri_flags
authorDarrick J. Wong <djwong@kernel.org>
Sun, 22 May 2022 05:59:48 +0000 (15:59 +1000)
committerDave Chinner <david@fromorbit.com>
Sun, 22 May 2022 05:59:48 +0000 (15:59 +1000)
Nobody uses this field, so get rid of it and the unused flag definition.
Rearrange the structure layout to reduce its size from 104 to 96 bytes.
This gets us from 39 to 42 objects per page.

Signed-off-by: Darrick J. Wong <djwong@kernel.org>
Reviewed-by: Allison Henderson <allison.henderson@oracle.com>
Reviewed-by: Dave Chinner <dchinner@redhat.com>
Signed-off-by: Dave Chinner <david@fromorbit.com>
fs/xfs/libxfs/xfs_attr.h

index cb3b3d270569567de4a426e7224fd4d570f002ea..f0b93515c1e8ffd6b43e412634e9c4feddc3459a 100644 (file)
@@ -501,15 +501,19 @@ enum xfs_delattr_state {
        { XFS_DAS_NODE_REMOVE_ATTR,     "XFS_DAS_NODE_REMOVE_ATTR" }, \
        { XFS_DAS_DONE,                 "XFS_DAS_DONE" }
 
-/*
- * Defines for xfs_attr_item.xattri_flags
- */
-#define XFS_DAC_LEAF_ADDNAME_INIT      0x01 /* xfs_attr_leaf_addname init*/
-
 /*
  * Context used for keeping track of delayed attribute operations
  */
 struct xfs_attr_item {
+       /*
+        * used to log this item to an intent containing a list of attrs to
+        * commit later
+        */
+       struct list_head                xattri_list;
+
+       /* Used in xfs_attr_node_removename to roll through removing blocks */
+       struct xfs_da_state             *xattri_da_state;
+
        struct xfs_da_args              *xattri_da_args;
 
        /*
@@ -517,16 +521,7 @@ struct xfs_attr_item {
         */
        struct xfs_buf                  *xattri_leaf_bp;
 
-       /* Used in xfs_attr_rmtval_set_blk to roll through allocating blocks */
-       struct xfs_bmbt_irec            xattri_map;
-       xfs_dablk_t                     xattri_lblkno;
-       int                             xattri_blkcnt;
-
-       /* Used in xfs_attr_node_removename to roll through removing blocks */
-       struct xfs_da_state             *xattri_da_state;
-
        /* Used to keep track of current state of delayed operation */
-       unsigned int                    xattri_flags;
        enum xfs_delattr_state          xattri_dela_state;
 
        /*
@@ -534,11 +529,10 @@ struct xfs_attr_item {
         */
        unsigned int                    xattri_op_flags;
 
-       /*
-        * used to log this item to an intent containing a list of attrs to
-        * commit later
-        */
-       struct list_head                xattri_list;
+       /* Used in xfs_attr_rmtval_set_blk to roll through allocating blocks */
+       xfs_dablk_t                     xattri_lblkno;
+       int                             xattri_blkcnt;
+       struct xfs_bmbt_irec            xattri_map;
 };