Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:5 as permitted sender) client-ip=2620:137:e000::3:5;
Date:   Tue, 24 Oct 2023 17:46:20 +0200
From:   David Sterba <dsterba@suse.cz>
To:     Christian Brauner <brauner@kernel.org>
Cc:     David Sterba <dsterba@suse.cz>,
        Stephen Rothwell <sfr@canb.auug.org.au>,
        Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
        Linux Next Mailing List <linux-next@vger.kernel.org>,
        Linus Torvalds <torvalds@linux-foundation.org>,
        Christoph Hellwig <hch@lst.de>, Jan Kara <jack@suse.cz>
Subject: Re: upcoming merge window: Re: linux-next: manual merge of the
 vfs-brauner tree with the btrfs tree
Message-ID: <20231024154620.GQ26353@twin.jikos.cz>
Reply-To: dsterba@suse.cz
References: <20231009104840.1bdadc80@canb.auug.org.au>
 <20231009-bauch-gedanken-e02e35804e03@brauner>
 <20231011083754.45a9ed53@canb.auug.org.au>
 <20231011092004.GE2211@suse.cz>
 <20231012154210.GI2211@suse.cz>
 <20231023175513.GL26353@twin.jikos.cz>
 <20231024082543.575b3edd@canb.auug.org.au>
 <20231024-kolossal-ungelegen-f95c436de174@brauner>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20231024-kolossal-ungelegen-f95c436de174@brauner>
User-Agent: Mutt/1.5.23.1-rc1 (2014-03-12)
Precedence: bulk

On Tue, Oct 24, 2023 at 10:59:39AM +0200, Christian Brauner wrote:
> On Tue, Oct 24, 2023 at 08:25:43AM +1100, Stephen Rothwell wrote:
> > Hi David,
> > 
> > On Mon, 23 Oct 2023 19:55:13 +0200 David Sterba <dsterba@suse.cz> wrote:
> > >
> > > I have updated my for-next branch again, sorry (top commit 1a4dc97c883a4f763cbaf50).
> > > There are some fixes I don't want to miss from the 6.7 pull request.
> > > There should be minimal change to the VFS tree conflict resolution so
> > > the diff should be reusable.
> > 
> > So, why did you not just merge in v6.6-rc7 (or better yet, the branch
> > that contains the fix(es) that Linus merged) and then apply your new
> > commits on top of that?  All the commits that were in the btrfs tree
> > have been rebased unchanged.
> 
> Please reconsider that and follow Stephen's suggestion. I'm sending pull
> requests this week and it'd be really annoying having to rebase
> vfs.super right before sending them.
> 
> We let you carry the required patches in btrfs on your insistence even
> though this effectively blocked two patchsets for a whole cycle

I hope I explained my reasons already under that series, core btrfs
changes should not go via VFS tree.

> and then
> merged in btrfs into vfs.super for that. Rebasing on such short notice
> is really not very nice.

Like said in the my other reply, the amount of VFS changes asks for
stopping taking new patches to btrfs and not continuing the patch
workflow that I've been doing. I understand that the inter-tree
dependencies are never easy so it's about finding some common way and
splitting the work over more releases eventually.

A resync of our branches a week before merge window, when there are no
significant changes on my side does not sound like too short notice, but
you can feel otherwise of course.

> I'm going to wait with the rebase for a bit.

Ok, don't rebase. I'll push to linux-next the previous snapshot and will
find a way how to deliver the new patches.