Skip to content

xfs: log item and transaction cleanups

Dave Chinner linux-xfs 2 versions – In review

Get series with b4
b4 am [email protected]

Versions newest first · linked by author and title

  1. v2 Latest9 patches 25 repliesIn review
  2. v110 patches 20 repliesSuperseded

Patches in v2 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/9xfs: log item flags are racyfs/xfs/xfs_bmap_item.c fs/xfs/xfs_buf_item.c fs/xfs/xfs_dquot.c +16 more+52 -481 reply
2/9xfs: add tracing to high level transaction operationsfs/xfs/xfs_log_cil.c fs/xfs/xfs_trace.h fs/xfs/xfs_trans.c+57 -11 reply
3/9xfs: adder caller IP to xfs_defer* tracepointsfs/xfs/libxfs/xfs_defer.c fs/xfs/xfs_trace.h+17 -121 reply
4/9xfs: don't assert fail with AIL lock heldfs/xfs/xfs_trans_ail.c+32 -113 replies
5/9xfs: fix double ijoin in xfs_inactive_symlink_rmt()fs/xfs/xfs_symlink.c+2 -78 replies
6/9xfs: fix double ijoin in xfs_reflink_cancel_cow_rangefs/xfs/xfs_reflink.c+4 -22 replies
7/9xfs: fix double ijoin in xfs_reflink_clear_inode_flag()fs/xfs/xfs_reflink.c+6 -24 replies
8/9xfs: add some more debug checks to buffer log item reusefs/xfs/xfs_buf_item.c+3 -12 replies
9/9xfs: get rid of the log item descriptorfs/xfs/libxfs/xfs_bmap.c fs/xfs/libxfs/xfs_shared.h fs/xfs/xfs_buf_item.c +15 more+62 -1233 replies

Reviews trailers per patch

All 9 patches have a Reviewed-by or Acked-by.

PatchChristoph HellwigBrian FosterDarrick J. Wong
1/9R-b R-b R-b
2/9R-b R-b R-b
3/9R-b R-b R-b
4/9R-b R-b R-b
5/9R-b R-b R-b
6/9R-b R-b R-b
7/9R-b R-b R-b
8/9R-b R-b R-b
9/9R-b R-b R-b

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes. Scroll the table sideways for all 3 reviewers.

Read the v2 thread

Cover letter [PATCH 0/9 v2] xfs: log item and transaction cleanups

Hi folks,

This is an updated version of the patchset that emerged from trying to remove the log item descriptors. First version was here:

https://marc.info/?l=linux-xfs&m=152524812914200&w=2

This version addresses the initial review comments.

Version 2:
- rebased on current for-next tree
- get rid of XFS_LI_TRANS (was patch 2)
- rework AIL assert failure patch with Brian's suggestions
- reworked removal of log item descriptor patch to add checks that XFS_LI_TRANS was providing.

Cheers,

Dave.