| To: | Jeff Liu <jeff.liu@xxxxxxxxxx> |
|---|---|
| Subject: | Re: [PATCH v2 04/10] xfs: introduce per allocation group inumbers |
| From: | Christoph Hellwig <hch@xxxxxxxxxxxxx> |
| Date: | Thu, 24 Apr 2014 23:43:33 -0700 |
| Cc: | "xfs@xxxxxxxxxxx" <xfs@xxxxxxxxxxx> |
| Delivered-to: | xfs@xxxxxxxxxxx |
| In-reply-to: | <535078B5.4010700@xxxxxxxxxx> |
| References: | <535078B5.4010700@xxxxxxxxxx> |
| User-agent: | Mutt/1.5.21 (2010-09-15) |
On Fri, Apr 18, 2014 at 08:58:29AM +0800, Jeff Liu wrote: > From: Jie Liu <jeff.liu@xxxxxxxxxx> > > Introduce xfs_perag_inumbers(), it could be used to fetch the inode number > tables per allocation group via a new ioctl(2) in the future. Also, that > would be a net win considering the scalability for a file system with huge > number of inodes as multiple allocation groups can be scanned in parallel, > refactor xfs_inumbers() with it. Given that you don't actually introduce a per-ag inumbers interface in this series I'd prefer if you just refactor the inumbers code to add a helper that operates on an AG with xfs_inumbers moving on to the next one, but not adding the xfs_aginumbers structure that just makes the move less obvious for now. |
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | Re: [PATCH] xfstests: introduce FSX_AVOID env var, Christoph Hellwig |
|---|---|
| Next by Date: | Re: [PATCH v2 05/10] xfs: fix error handling in xfs_bulkstat, Christoph Hellwig |
| Previous by Thread: | [PATCH v2 04/10] xfs: introduce per allocation group inumbers, Jeff Liu |
| Next by Thread: | Re: [PATCH v2 04/10] xfs: introduce per allocation group inumbers, Jeff Liu |
| Indexes: | [Date] [Thread] [Top] [All Lists] |