Hi all, Here's v3 of the bli refcount fix. This includes only minor changes from v2, as noted below. Patch 1 is a bug fix and patches 2-3 are refactoring patches. Brian v3: - Rename xfs_buf_item_unref() to xfs_buf_item_put() - Minor logic and comment fixups. v2: https://marc.info/?l=linux-xfs&m=153537995111772&w=2 - Refactor stale bli unlock logic to more closely resemble original. - Use bool for 'freed' variable and clean up type casting. - Reorder asserts/tracepoints and remove unused stale tracepoint. - Added patches 2 and 3. v1: https://marc.info/?l=linux-xfs&m=153486008127962&w=2 Brian Foster (3): xfs: don't unlock invalidated buf on aborted tx commit xfs: clean up xfs_trans_brelse() xfs: refactor xfs_buf_log_item reference count handling fs/xfs/xfs_buf_item.c | 113 +++++++++++++++++++++-------------------- fs/xfs/xfs_buf_item.h | 1 + fs/xfs/xfs_trace.h | 1 - fs/xfs/xfs_trans_buf.c | 99 +++++++++--------------------------- 4 files changed, 82 insertions(+), 132 deletions(-) -- 2.17.1
