| To: | Dave Chinner <david@xxxxxxxxxxxxx> |
|---|---|
| Subject: | Re: [PATCH 0/4] XFS: replace the mount inode list with radix tree traversals V2 |
| From: | Christoph Hellwig <hch@xxxxxxxxxxxxx> |
| Date: | Wed, 23 Jul 2008 03:17:48 -0400 |
| Cc: | xfs@xxxxxxxxxxx |
| In-reply-to: | <1216773673-3620-1-git-send-email-david@fromorbit.com> |
| References: | <1216773673-3620-1-git-send-email-david@fromorbit.com> |
| Sender: | xfs-bounce@xxxxxxxxxxx |
| User-agent: | Mutt/1.5.18 (2008-05-17) |
On Wed, Jul 23, 2008 at 10:41:09AM +1000, Dave Chinner wrote: > 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. All patches looks good to me. Well, minus the xfsidbg issue in 4 which would be a merge blocker for that patch. |
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | TAKE 981498 - XFS: Kill xfs_vtoi(), Niv Sardi-Altivanik |
|---|---|
| Next by Date: | TAKE 981498 - Use KM_NOFS for incore inode extent tree allocation V2, Niv Sardi-Altivanik |
| Previous by Thread: | Re: [PATCH 0/4] XFS: replace the mount inode list with radix tree traversals V2, Dave Chinner |
| Next by Thread: | Re: [PATCH 0/4] XFS: replace the mount inode list with radix tree traversals V2, Dave Chinner |
| Indexes: | [Date] [Thread] [Top] [All Lists] |