From: Brian Foster <bfoster@redhat.com>
To: xfs@oss.sgi.com
Subject: [PATCH v4 00/11] xfs: introduce the free inode btree
Date: Mon, 24 Feb 2014 09:29:52 -0500 [thread overview]
Message-ID: <1393252203-7423-1-git-send-email-bfoster@redhat.com> (raw)
Hi all,
Here's the latest update of the finobt work. This series includes fixes
based on review of v3:
http://oss.sgi.com/archives/xfs/2014-02/msg00121.html
The primary changes in v4 are fixes to the transaction reservation
calculations for inode management, a refactoring of the finobt-based
inode allocation algorithm into smaller helper functions and the
addition of a warning in the unlikely scenario of reserve block pool
exhaustion due to ifree transactions (inode eviction). Minor fixes and
comment cleanups are included as requested in review for v3. Thoughts,
reviews and flames appreciated.
Brian
v4:
- Fixed up comments associated with separation of the AGI into separate
logging regions.
- Added warning on ENOSPC failure of ifree reservation and comment
describing reasoning for ifree being reserve-enabled.
- Removed finobt res. from xfs_calc_create_resv_alloc() (v4 sb only).
- Updated xfs_calc_finobt_res() to account for free space btrees and
enable for inode alloc. record modification scenario.
- Renamed xfs_dialloc_ag_slow() to xfs_dialloc_ag_inobt().
- Refactored the inode allocation algorithm into several smaller helper
functions.
- Fixed up comments in xfs_difree_finobt().
v3:
- Rebased to latest master.
- Fixed up unused variable warning in xfs_difree_inobt().
- Replaced a few more typedefs.
v2:
- Rebase to latest xfs tree (minor shifting around of some header bits).
- Added "xfs: report finobt status in fs geometry" patch to series.
v1:
- Separate patch to enable rw finobt support at end of series.
- Rework xfs_ialloc_log_agi() to log the agi in two distinct regions.
- Rework xfs_ialloc_btree.c changes to use separate finobt handlers
where appropriate.
- Fix bug to show fibt2 stats data in stat proc file.
- Move finobt log reservation calculations into separate helper, made
conditional and merged to a single patch.
- Use reserved block pool in xfs_inactive() codepath instead of flush.
- Moved and cleaned up xfs_inobt_insert() to use inobt helpers.
- Enhanced lookup algorithm for allocation (xfs_dialloc_ag()).
- Refactored xfs_difree() to use xfs_difree_inobt() and
xfs_difree_finobt(), cleaned up the latter.
Brian Foster (11):
xfs: refactor xfs_ialloc_btree.c to support multiple inobt numbers
xfs: reserve v5 superblock read-only compat. feature bit for finobt
xfs: support the XFS_BTNUM_FINOBT free inode btree type
xfs: update inode allocation/free transaction reservations for finobt
xfs: insert newly allocated inode chunks into the finobt
xfs: use and update the finobt on inode allocation
xfs: refactor xfs_difree() inobt bits into xfs_difree_inobt() helper
xfs: update the finobt on inode free
xfs: add finobt support to growfs
xfs: report finobt status in fs geometry
xfs: enable the finobt feature on v5 superblocks
fs/xfs/xfs_ag.h | 36 ++-
fs/xfs/xfs_btree.c | 6 +-
fs/xfs/xfs_btree.h | 3 +
fs/xfs/xfs_format.h | 14 +-
fs/xfs/xfs_fs.h | 1 +
fs/xfs/xfs_fsops.c | 36 ++-
fs/xfs/xfs_ialloc.c | 695 +++++++++++++++++++++++++++++++++++++++-------
fs/xfs/xfs_ialloc_btree.c | 68 ++++-
fs/xfs/xfs_ialloc_btree.h | 3 +-
fs/xfs/xfs_inode.c | 25 +-
fs/xfs/xfs_itable.c | 6 +-
fs/xfs/xfs_log_recover.c | 2 +
fs/xfs/xfs_sb.h | 10 +-
fs/xfs/xfs_stats.c | 1 +
fs/xfs/xfs_stats.h | 18 +-
fs/xfs/xfs_trans_resv.c | 53 +++-
fs/xfs/xfs_trans_space.h | 7 +-
fs/xfs/xfs_types.h | 2 +-
18 files changed, 850 insertions(+), 136 deletions(-)
--
1.8.3.1
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
next reply other threads:[~2014-02-24 15:20 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-02-24 14:29 Brian Foster [this message]
2014-02-24 14:29 ` [PATCH v4 01/11] xfs: refactor xfs_ialloc_btree.c to support multiple inobt numbers Brian Foster
2014-02-24 14:29 ` [PATCH v4 02/11] xfs: reserve v5 superblock read-only compat. feature bit for finobt Brian Foster
2014-02-24 14:29 ` [PATCH v4 03/11] xfs: support the XFS_BTNUM_FINOBT free inode btree type Brian Foster
2014-02-24 14:29 ` [PATCH v4 04/11] xfs: update inode allocation/free transaction reservations for finobt Brian Foster
2014-02-24 14:29 ` [PATCH v4 05/11] xfs: insert newly allocated inode chunks into the finobt Brian Foster
2014-02-24 14:29 ` [PATCH v4 06/11] xfs: use and update the finobt on inode allocation Brian Foster
2014-02-24 14:29 ` [PATCH v4 07/11] xfs: refactor xfs_difree() inobt bits into xfs_difree_inobt() helper Brian Foster
2014-02-24 14:30 ` [PATCH v4 08/11] xfs: update the finobt on inode free Brian Foster
2014-02-24 14:30 ` [PATCH v4 09/11] xfs: add finobt support to growfs Brian Foster
2014-02-24 14:30 ` [PATCH v4 10/11] xfs: report finobt status in fs geometry Brian Foster
2014-02-24 14:30 ` [PATCH v4 11/11] xfs: enable the finobt feature on v5 superblocks Brian Foster
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1393252203-7423-1-git-send-email-bfoster@redhat.com \
--to=bfoster@redhat.com \
--cc=xfs@oss.sgi.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox