[0/2] btrfs-progs: Metadata preallocation enhancement

Message

Qu Wenruo April 16, 2019, 10:21 a.m. UTC

This patchset will address the github issue #123 to make btrfs-convert
less possible to report false ENOSPC.

The first patch will try to avoid the nested chunk/extent tree
modification in a more explicit way.

The 2nd patch will address the ENOSPC problem, by adding CSUM tree to
the metadata preallocate list.

The 2nd patch is not as aggressive as 7a12d8470e5f ("btrfs-progs:
Do metadata preallocation as long as we're not modifying extent tree").
Even we have the 1st patch I still prefer to make less modification to
avoid possible bugs.

Qu Wenruo (2):
  btrfs-progs: Avoid nested chunk allocation call
  btrfs-progs: Do metadata preallocation for fs trees and csum tree

 check/main.c  |  2 +-
 extent-tree.c | 20 +++++++++++++++++++-
 transaction.c |  3 ++-
 transaction.h |  3 ++-
 4 files changed, 24 insertions(+), 4 deletions(-)

Comments

David Sterba June 5, 2019, 4:39 p.m. UTC | #1

On Tue, Apr 16, 2019 at 06:21:42PM +0800, Qu Wenruo wrote:
> This patchset will address the github issue #123 to make btrfs-convert
> less possible to report false ENOSPC.
> 
> The first patch will try to avoid the nested chunk/extent tree
> modification in a more explicit way.
> 
> The 2nd patch will address the ENOSPC problem, by adding CSUM tree to
> the metadata preallocate list.
> 
> The 2nd patch is not as aggressive as 7a12d8470e5f ("btrfs-progs:
> Do metadata preallocation as long as we're not modifying extent tree").
> Even we have the 1st patch I still prefer to make less modification to
> avoid possible bugs.
> 
> Qu Wenruo (2):
>   btrfs-progs: Avoid nested chunk allocation call
>   btrfs-progs: Do metadata preallocation for fs trees and csum tree

Added to devel, thanks.