On Fri, Jan 19, 2018 at 03:08:43PM +0100, Carlos Maiolino wrote: > Hi, > > A few time ago Christoph suggested to use list_head API to handle buffer's > log_item_list. > > This patchset aims to split the current bp->b_fspriv field into a specific field > to hold the xfs_buf_log_item, and another to hold the list of log items attached > to the buffer (3rd patch), and finally replace the singly linked list > implementation by the list_head infra-structure (4th patch). > > The first two patches are just a typedef removal of xfs_buf_log_item_t and > xfs_buf_t, I did while studying how all the buffer I/O mechanism works, I > thought since we plan to get rid of the typedefs in future, this might be > useful. > > I can rebase the 3rd and 4th patch on top of current xfs tree if the typedef > removal patches are useless, you guys call. Typedef removal seems useful... is this series based atop current for-next? --D > This patchset survived several xfstests runs. > > Cheers. > > Carlos Maiolino (4): > Get rid of xfs_buf_log_item_t typedef > Get rid of xfs_buf_t typedef > Split buffer's b_fspriv field > Use list_head infra-structure for buffer's log items list > > fs/xfs/libxfs/xfs_alloc.c | 36 ++++---- > fs/xfs/libxfs/xfs_attr_leaf.c | 2 +- > fs/xfs/libxfs/xfs_bmap.c | 8 +- > fs/xfs/libxfs/xfs_btree.c | 26 +++--- > fs/xfs/libxfs/xfs_btree.h | 2 +- > fs/xfs/libxfs/xfs_da_btree.c | 2 +- > fs/xfs/libxfs/xfs_dir2_block.c | 2 +- > fs/xfs/libxfs/xfs_dir2_data.c | 2 +- > fs/xfs/libxfs/xfs_dir2_leaf.c | 2 +- > fs/xfs/libxfs/xfs_dir2_node.c | 2 +- > fs/xfs/libxfs/xfs_ialloc.c | 18 ++-- > fs/xfs/libxfs/xfs_inode_buf.c | 4 +- > fs/xfs/libxfs/xfs_rtbitmap.c | 22 ++--- > fs/xfs/libxfs/xfs_sb.c | 2 +- > fs/xfs/libxfs/xfs_symlink_remote.c | 2 +- > fs/xfs/xfs_buf.c | 140 +++++++++++++++---------------- > fs/xfs/xfs_buf.h | 21 ++--- > fs/xfs/xfs_buf_item.c | 166 ++++++++++++++++++++----------------- > fs/xfs/xfs_buf_item.h | 7 +- > fs/xfs/xfs_dquot.c | 22 ++--- > fs/xfs/xfs_dquot_item.c | 2 +- > fs/xfs/xfs_fsops.c | 2 +- > fs/xfs/xfs_inode.c | 24 +++--- > fs/xfs/xfs_inode_item.c | 41 +++------ > fs/xfs/xfs_itable.c | 2 +- > fs/xfs/xfs_log.c | 26 +++--- > fs/xfs/xfs_log_recover.c | 46 +++++----- > fs/xfs/xfs_qm.c | 8 +- > fs/xfs/xfs_rtalloc.c | 20 ++--- > fs/xfs/xfs_rtalloc.h | 4 +- > fs/xfs/xfs_symlink.c | 22 ++--- > fs/xfs/xfs_trans.c | 2 +- > fs/xfs/xfs_trans.h | 4 +- > fs/xfs/xfs_trans_buf.c | 104 ++++++++++++----------- > 34 files changed, 397 insertions(+), 398 deletions(-) > > -- > 2.14.3 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-xfs" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-xfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
