mbox series

[v3,0/7] xfs: random fixes and code cleanup

Message ID 1600844358-16601-1-git-send-email-kaixuxia@tencent.com (mailing list archive)
Headers show
Series xfs: random fixes and code cleanup | expand

Message

Kaixu Xia Sept. 23, 2020, 6:59 a.m. UTC
From: Kaixu Xia <kaixuxia@tencent.com>

Hi all,

This patchset include random fixes and code cleanups, and there are no
connections among these patches. In order to make it easier to track,
I bundle them up and put all the scattered patches into a single patchset.

Changes for v3:
 -move the log intent/intent-done items detection helpers
  to xfs_trans.h.
 -fix the overly long line.

Changes for v2: 
 -detect intent-done items by their item ops.
 -update the commit messages.
 -code cleanup for xfs_attr_leaf_entsize_{remote,local}.

Kaixu Xia (7):
  xfs: remove the unused SYNCHRONIZE macro
  xfs: use the existing type definition for di_projid
  xfs: remove the unnecessary xfs_dqid_t type cast
  xfs: do the assert for all the log done items in xfs_trans_cancel
  xfs: remove the redundant crc feature check in xfs_attr3_rmt_verify
  xfs: code cleanup in xfs_attr_leaf_entsize_{remote,local}
  xfs: fix some comments

 fs/xfs/libxfs/xfs_attr_remote.c |  2 --
 fs/xfs/libxfs/xfs_da_format.h   | 12 ++++++------
 fs/xfs/libxfs/xfs_inode_buf.h   |  2 +-
 fs/xfs/xfs_dquot.c              |  4 ++--
 fs/xfs/xfs_linux.h              |  1 -
 fs/xfs/xfs_log_recover.c        |  7 -------
 fs/xfs/xfs_qm.c                 |  2 +-
 fs/xfs/xfs_trans.c              |  2 +-
 fs/xfs/xfs_trans.h              | 16 ++++++++++++++++
 9 files changed, 27 insertions(+), 21 deletions(-)

Comments

Darrick J. Wong Sept. 23, 2020, 4:28 p.m. UTC | #1
On Wed, Sep 23, 2020 at 02:59:11PM +0800, xiakaixu1987@gmail.com wrote:
> From: Kaixu Xia <kaixuxia@tencent.com>
> 
> Hi all,
> 
> This patchset include random fixes and code cleanups, and there are no
> connections among these patches. In order to make it easier to track,
> I bundle them up and put all the scattered patches into a single patchset.
> 
> Changes for v3:
>  -move the log intent/intent-done items detection helpers
>   to xfs_trans.h.
>  -fix the overly long line.
> 

Applied, thanks.

--D

> Changes for v2: 
>  -detect intent-done items by their item ops.
>  -update the commit messages.
>  -code cleanup for xfs_attr_leaf_entsize_{remote,local}.
> 
> Kaixu Xia (7):
>   xfs: remove the unused SYNCHRONIZE macro
>   xfs: use the existing type definition for di_projid
>   xfs: remove the unnecessary xfs_dqid_t type cast
>   xfs: do the assert for all the log done items in xfs_trans_cancel
>   xfs: remove the redundant crc feature check in xfs_attr3_rmt_verify
>   xfs: code cleanup in xfs_attr_leaf_entsize_{remote,local}
>   xfs: fix some comments
> 
>  fs/xfs/libxfs/xfs_attr_remote.c |  2 --
>  fs/xfs/libxfs/xfs_da_format.h   | 12 ++++++------
>  fs/xfs/libxfs/xfs_inode_buf.h   |  2 +-
>  fs/xfs/xfs_dquot.c              |  4 ++--
>  fs/xfs/xfs_linux.h              |  1 -
>  fs/xfs/xfs_log_recover.c        |  7 -------
>  fs/xfs/xfs_qm.c                 |  2 +-
>  fs/xfs/xfs_trans.c              |  2 +-
>  fs/xfs/xfs_trans.h              | 16 ++++++++++++++++
>  9 files changed, 27 insertions(+), 21 deletions(-)
> 
> -- 
> 2.20.0
>