Skip to content

XFS: replace the mount inode list with radix tree traversals V4

Dave Chinner linux-xfs 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest6 patches 1 replyIn review

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/6XFS: move sync code to its own filefs/xfs/Makefile fs/xfs/linux-2.6/xfs_super.c fs/xfs/linux-2.6/xfs_sync.c +3 more+615 -5600 replies
2/6XFS: move xfssyncd code to xfs_sync.cfs/xfs/linux-2.6/xfs_super.c fs/xfs/linux-2.6/xfs_super.h fs/xfs/linux-2.6/xfs_sync.c +3 more+223 -1820 replies
3/6XFS: Remove xfs_iflush_all and clean up xfs_finish_reclaim_all() V3fs/xfs/linux-2.6/xfs_sync.c fs/xfs/xfs_inode.c fs/xfs/xfs_inode.h +3 more+22 -640 replies
4/6XFS: Use the inode tree for finding dirty inodes V3fs/xfs/linux-2.6/xfs_sync.c+101 -2600 replies
5/6XFS: Traverse inode trees when releasing dquots V3fs/xfs/quota/xfs_qm_syscalls.c+59 -680 replies
6/6XFS: remove the mount inode listfs/xfs/xfs_iget.c fs/xfs/xfs_inode.h fs/xfs/xfs_mount.c +1 more+1 -550 replies

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.

Read the v1 thread

Cover letter [PATCH 0/6] XFS: replace the mount inode list with radix tree traversals V4

The list of all inodes on a mount is superfluous. We can traverse all inodes now by walking the per-AG inode radix trees without needing a separate list. This enables us to remove a bunch of complex list traversal code and remove another two pointers from the xfs_inode.

Also, by replacing the sync traversal with an ascending inode number traversal, we will issue better inode I/O patterns for writeback triggered by xfssyncd or unmount.

Before we make this change, move all the relevant sync code into it's own file in the linux-2.6/ directory. This aggregates VFS specific sync interfacing in the one file and will allow all the subsequent change history to be associated with this file so it is easy to find in future.

Version 4:
o revert xfs_syncsub -> xfs_sync change in xfs_quiesce_fs and rediff patch series