| To: | Brian Foster <bfoster@xxxxxxxxxx> |
|---|---|
| Subject: | Re: [PATCH v3 05/11] xfs: insert newly allocated inode chunks into the finobt |
| From: | Dave Chinner <david@xxxxxxxxxxxxx> |
| Date: | Tue, 11 Feb 2014 17:48:39 +1100 |
| Cc: | xfs@xxxxxxxxxxx |
| Delivered-to: | xfs@xxxxxxxxxxx |
| In-reply-to: | <1391536182-9048-6-git-send-email-bfoster@xxxxxxxxxx> |
| References: | <1391536182-9048-1-git-send-email-bfoster@xxxxxxxxxx> <1391536182-9048-6-git-send-email-bfoster@xxxxxxxxxx> |
| User-agent: | Mutt/1.5.21 (2010-09-15) |
On Tue, Feb 04, 2014 at 12:49:36PM -0500, Brian Foster wrote: > A newly allocated inode chunk, by definition, has at least one > free inode, so a record is always inserted into the finobt. > > Create the xfs_inobt_insert() helper from existing code to insert > a record in an inobt based on the provided BTNUM. Update > xfs_ialloc_ag_alloc() to invoke the helper for the existing > XFS_BTNUM_INO tree and XFS_BTNUM_FINO tree, if enabled. > > Signed-off-by: Brian Foster <bfoster@xxxxxxxxxx> Looks good. Reviewed-by: Dave Chinner <dchinner@xxxxxxxxxx> -- Dave Chinner david@xxxxxxxxxxxxx |
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | Re: [PATCH v3 04/11] xfs: update inode allocation/free transaction reservations for finobt, Dave Chinner |
|---|---|
| Next by Date: | Re: [PATCH v3 06/11] xfs: use and update the finobt on inode allocation, Dave Chinner |
| Previous by Thread: | [PATCH v3 05/11] xfs: insert newly allocated inode chunks into the finobt, Brian Foster |
| Next by Thread: | [PATCH v3 02/11] xfs: reserve v5 superblock read-only compat. feature bit for finobt, Brian Foster |
| Indexes: | [Date] [Thread] [Top] [All Lists] |