linux-btrfs.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: David Sterba <dsterba@suse.cz>
To: Nikolay Borisov <nborisov@suse.com>
Cc: Josef Bacik <josef@toxicpanda.com>,
	linux-btrfs@vger.kernel.org, kernel-team@fb.com
Subject: Re: [PATCH v2 10/11] btrfs: add code to support the block group root
Date: Wed, 26 Jan 2022 16:34:32 +0100	[thread overview]
Message-ID: <20220126153432.GY14046@twin.jikos.cz> (raw)
In-Reply-To: <37128edf-7bdd-0d36-3103-8e12805ed761@suse.com>

On Wed, Jan 12, 2022 at 05:16:29PM +0200, Nikolay Borisov wrote:
> 
> 
> On 15.12.21 г. 22:40, Josef Bacik wrote:
> > This code adds the on disk structures for the block group root, which
> > will hold the block group items for extent tree v2.
> > 
> > Signed-off-by: Josef Bacik <josef@toxicpanda.com>
> 
> Ok, this is formally not needed for the extent v2 but the major rework
> for extent v2 is used to introduce this change and it's idea is to speed
> up fs mount right? If so I'd like this to be stated more explicitly in
> the changelog of the patch. I guess this can be done by David as well.

You mean that extent tree v2 is mainly for speeding up mount? AFAIK it's
only one of the improvements it brings. I'm not sure what you'd like to
see in the commit message but could update it eventually.

  reply	other threads:[~2022-01-26 15:35 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-12-15 20:39 [PATCH v2 00/11] btrfs: extent tree v2, support for global roots Josef Bacik
2021-12-15 20:39 ` [PATCH v2 01/11] btrfs: add definition for EXTENT_TREE_V2 Josef Bacik
2021-12-15 20:39 ` [PATCH v2 02/11] btrfs: disable balance for extent tree v2 for now Josef Bacik
2021-12-15 20:40 ` [PATCH v2 03/11] btrfs: disable device manipulation ioctl's EXTENT_TREE_V2 Josef Bacik
2021-12-15 20:40 ` [PATCH v2 04/11] btrfs: disable qgroups in extent tree v2 Josef Bacik
2021-12-15 20:40 ` [PATCH v2 05/11] btrfs: disable scrub for extent-tree-v2 Josef Bacik
2021-12-15 20:40 ` [PATCH v2 06/11] btrfs: disable snapshot creation/deletion for extent tree v2 Josef Bacik
2021-12-15 20:40 ` [PATCH v2 07/11] btrfs: disable space cache related mount options " Josef Bacik
2021-12-15 20:40 ` [PATCH v2 08/11] btrfs: tree-checker: don't fail on empty extent roots " Josef Bacik
2021-12-15 20:40 ` [PATCH v2 09/11] btrfs: abstract out loading the tree root Josef Bacik
2022-01-13  9:20   ` Nikolay Borisov
2022-01-26 15:32     ` David Sterba
2021-12-15 20:40 ` [PATCH v2 10/11] btrfs: add code to support the block group root Josef Bacik
2022-01-12 15:16   ` Nikolay Borisov
2022-01-26 15:34     ` David Sterba [this message]
2022-01-26 15:58       ` Nikolay Borisov
2021-12-15 20:40 ` [PATCH v2 11/11] btrfs: add support for multiple global roots Josef Bacik
2022-01-12 15:09 ` [PATCH v2 00/11] btrfs: extent tree v2, support for " Nikolay Borisov
2022-01-26 15:52   ` David Sterba
2022-01-26 15:54 ` David Sterba

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=20220126153432.GY14046@twin.jikos.cz \
    --to=dsterba@suse.cz \
    --cc=josef@toxicpanda.com \
    --cc=kernel-team@fb.com \
    --cc=linux-btrfs@vger.kernel.org \
    --cc=nborisov@suse.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;
as well as URLs for NNTP newsgroup(s).