Skip to content

Commit 500a512

Browse files
Darrick J. Wongdchinner
authored andcommitted
xfs: remove struct xfs_attr_item.xattri_flags
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 <[email protected]> Reviewed-by: Allison Henderson <[email protected]> Reviewed-by: Dave Chinner <[email protected]> Signed-off-by: Dave Chinner <[email protected]>
1 parent e2c7894 commit 500a512

File tree

1 file changed

+13
-19
lines changed

1 file changed

+13
-19
lines changed

fs/xfs/libxfs/xfs_attr.h

Lines changed: 13 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -501,44 +501,38 @@ enum xfs_delattr_state {
501501
{ XFS_DAS_NODE_REMOVE_ATTR, "XFS_DAS_NODE_REMOVE_ATTR" }, \
502502
{ XFS_DAS_DONE, "XFS_DAS_DONE" }
503503

504-
/*
505-
* Defines for xfs_attr_item.xattri_flags
506-
*/
507-
#define XFS_DAC_LEAF_ADDNAME_INIT 0x01 /* xfs_attr_leaf_addname init*/
508-
509504
/*
510505
* Context used for keeping track of delayed attribute operations
511506
*/
512507
struct xfs_attr_item {
508+
/*
509+
* used to log this item to an intent containing a list of attrs to
510+
* commit later
511+
*/
512+
struct list_head xattri_list;
513+
514+
/* Used in xfs_attr_node_removename to roll through removing blocks */
515+
struct xfs_da_state *xattri_da_state;
516+
513517
struct xfs_da_args *xattri_da_args;
514518

515519
/*
516520
* Used by xfs_attr_set to hold a leaf buffer across a transaction roll
517521
*/
518522
struct xfs_buf *xattri_leaf_bp;
519523

520-
/* Used in xfs_attr_rmtval_set_blk to roll through allocating blocks */
521-
struct xfs_bmbt_irec xattri_map;
522-
xfs_dablk_t xattri_lblkno;
523-
int xattri_blkcnt;
524-
525-
/* Used in xfs_attr_node_removename to roll through removing blocks */
526-
struct xfs_da_state *xattri_da_state;
527-
528524
/* Used to keep track of current state of delayed operation */
529-
unsigned int xattri_flags;
530525
enum xfs_delattr_state xattri_dela_state;
531526

532527
/*
533528
* Attr operation being performed - XFS_ATTRI_OP_FLAGS_*
534529
*/
535530
unsigned int xattri_op_flags;
536531

537-
/*
538-
* used to log this item to an intent containing a list of attrs to
539-
* commit later
540-
*/
541-
struct list_head xattri_list;
532+
/* Used in xfs_attr_rmtval_set_blk to roll through allocating blocks */
533+
xfs_dablk_t xattri_lblkno;
534+
int xattri_blkcnt;
535+
struct xfs_bmbt_irec xattri_map;
542536
};
543537

544538

0 commit comments

Comments
 (0)