Re: [PATCH 03/16] xfs: rmap btree add more reserved blocks

To:	Dave Chinner <david@xxxxxxxxxxxxx>
Subject:	Re: [PATCH 03/16] xfs: rmap btree add more reserved blocks
From:	Christoph Hellwig <hch@xxxxxxxxxxxxx>
Date:	Thu, 10 Mar 2016 23:32:45 -0800
Cc:	Christoph Hellwig <hch@xxxxxxxxxxxxx>, xfs@xxxxxxxxxxx
Delivered-to:	xfs@xxxxxxxxxxx
In-reply-to:	<20160310220945.GY30721@dastard>
References:	<1457410578-30233-1-git-send-email-david@xxxxxxxxxxxxx> <1457410578-30233-4-git-send-email-david@xxxxxxxxxxxxx> <20160310142207.GD29058@xxxxxxxxxxxxx> <20160310220945.GY30721@dastard>
User-agent:	Mutt/1.5.24 (2015-08-30)

On Fri, Mar 11, 2016 at 09:09:45AM +1100, Dave Chinner wrote:
> The root btree blocks are not a fixed location. The moment the tree
> splits to the root we get a new root block and the old root is now
> at a lower level of the tree. IOWs, it doesn't matter if there is a
> hole at the time of growfs/mkfs laying out the initial tree roots
> because they are going to move around anyway.

Yeah, once we update the root it moves anyway.  Objection retracted..

<Prev in Thread]	Current Thread	[Next in Thread>
[PATCH 0/16] xfs: first part of rmapbt functionality, Dave Chinner [PATCH 02/16] xfs: add rmap btree stats infrastructure, Dave Chinner [PATCH 04/16] libxfs: rearrange xfs_bmap_add_free parameters, Dave Chinner Re: [PATCH 04/16] libxfs: rearrange xfs_bmap_add_free parameters, Christoph Hellwig [PATCH 03/16] xfs: rmap btree add more reserved blocks, Dave Chinner Re: [PATCH 03/16] xfs: rmap btree add more reserved blocks, Christoph Hellwig Re: [PATCH 03/16] xfs: rmap btree add more reserved blocks, Christoph Hellwig Re: [PATCH 03/16] xfs: rmap btree add more reserved blocks, Dave Chinner Re: [PATCH 03/16] xfs: rmap btree add more reserved blocks, Christoph Hellwig <= [PATCH 01/16] xfs: introduce rmap btree definitions, Dave Chinner [PATCH 08/16] xfs: add rmap btree growfs support, Dave Chinner [PATCH 07/16] xfs: define the on-disk rmap btree format, Dave Chinner [PATCH 15/16] xfs: add rmap btree insert and delete helpers, Dave Chinner [PATCH 11/16] xfs: add rmap btree operations, Dave Chinner [PATCH 12/16] xfs: add tracepoints for the rmap-mirrors-bmbt functions, Dave Chinner [PATCH 05/16] xfs: add owner field to extent allocation and freeing, Dave Chinner Re: [PATCH 05/16] xfs: add owner field to extent allocation and freeing, Christoph Hellwig Re: [PATCH 05/16] xfs: add owner field to extent allocation and freeing, Darrick J. Wong [PATCH 13/16] xfs: add an extent to the rmap btree, Dave Chinner

Previous by Date:	First Time Ever: Official Mayo Clinic Diet is Online!, Mayo Clinic
Next by Date:	Re: [PATCH] xfs: ensure committed is initialized in xfs_trans_roll, Christoph Hellwig
Previous by Thread:	Re: [PATCH 03/16] xfs: rmap btree add more reserved blocks, Dave Chinner
Next by Thread:	[PATCH 01/16] xfs: introduce rmap btree definitions, Dave Chinner
Indexes:	[Date] [Thread] [Top] [All Lists]