Log and transaction cleanups, factoring and bug fixes V2
Get series with b4
b4 am [email protected]Versions newest first
- v1 Latest9 patches 4 repliesIn review
Patches in v1 No applies recorded yet
| # | Subject and files | Size | Replies | Applied |
|---|---|---|---|---|
| 1/9 | xfs: factor log item initialisationfs/ fs/ fs/ +3 more | +29 | 0 replies | |
| 2/9 | xfs: Add inode pin counts to tracesfs/ fs/ fs/ | +12 | 0 replies | |
| 3/9 | xfs: remove stale parameter from ->iop_unpin methodfs/ fs/ fs/ +4 more | +39 | 0 replies | |
| 4/9 | xfs: fix reservation release commit flag in xfs_bmap_add_attrfork()fs/ | +1 | 0 replies | |
| 5/9 | xfs: split out iclog writing from xfs_trans_commit()fs/ | +199 | 0 replies | |
| 6/9 | xfs: clean up xfs_trans_commit logic even morefs/ | +37 | 0 replies | |
| 7/9 | xfs: update and factor xfs_trans_committed()fs/ | +95 | 0 replies | |
| 8/9 | xfs: Clean up xfs_trans_committed code after factoringfs/ | +166 | 1 reply | |
| 9/9 | xfs: log ticket reservation underestimates the number of iclogsfs/ | +42 | 3 replies |
Reviews trailers per patch
8 of 9 patches have a Reviewed-by or Acked-by.
| Patch | Christoph Hellwig | Dave Chinner |
|---|---|---|
| 1/9 | R-b | |
| 2/9 | R-b | |
| 3/9 | R-b | |
| 4/9 | R-b | |
| 5/9 | R-b | |
| 6/9 | R-b | |
| 7/9 | R-b | |
| 8/9 | R-b | |
| 9/9 |
Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.
Cover letter [PATCH 0/9] Log and transaction cleanups, factoring and bug fixes V2
The following series is preparation for delayed logging patches. The cleanups and factoring patches are mainly to make it simple to introduce the delayed logging patches, especially as these changes really have nothing to do with delayed logging. The cleanups and factoring patches should be no-ops from a functionality point of view.
A couple of the patches are fixes for bugs that I've found as I've been working through delayed logging prototypes.
Patches 8 (reservation bug fix) and 9 (new cleanup) of the series still require review.
Version 2:
- drop log vector changes from the series
- fix bogus assert in xfs_trans_committed() changes
- Added Christoph's _xfs_trans_commit() cleanup patch
- Added further cleanup of the xfs_trans_committed() code as suggested by Christoph.
These changes are also available in the git repository at:
git://git.kernel.org/pub/scm/linux/kernel/git/dgc/xfs.git trans-cleanup
Christoph Hellwig (1):
xfs: clean up xfs_trans_commit logic even more
Dave Chinner (8):
xfs: factor log item initialisation
xfs: Add inode pin counts to traces
xfs: remove stale parameter from ->iop_unpin method
xfs: fix reservation release commit flag in xfs_bmap_add_attrfork()
xfs: split out iclog writing from xfs_trans_commit()
xfs: update and factor xfs_trans_committed()
xfs: Clean up xfs_trans_committed code after factoring
xfs: log ticket reservation underestimates the number of iclogs
fs/xfs/linux-2.6/xfs_trace.h | 9 +- fs/xfs/quota/xfs_dquot_item.c | 28 +- fs/xfs/xfs_bmap.c | 2 +- fs/xfs/xfs_buf_item.c | 55 ++-- fs/xfs/xfs_extfree_item.c | 18 +- fs/xfs/xfs_inode.c | 2 + fs/xfs/xfs_inode_item.c | 21 +- fs/xfs/xfs_log.c | 68 +++- fs/xfs/xfs_log.h | 7 + fs/xfs/xfs_trans.c | 760 +++++++++++++++++++---------------------- fs/xfs/xfs_trans.h | 5 +- fs/xfs/xfs_trans_buf.c | 3 +- 12 files changed, 481 insertions(+), 497 deletions(-)
_______________________________________________
xfs mailing list
[email protected]
http://oss.sgi.com/mailman/listinfo/xfs