IronPort-SDR: KaMLN4Q/hDrQpEX9RIIBsss5qHLFVCskBtlyd3veRbl0MQLFnIwJG1exfsOS933vcei/eLtFXx
 wyPHjxJa3S6D4Iq1ypV0LVp2/67vFRf3gOAn0KBfLlpcXMafk5RL13Pll7jcCcBifk4aFuoj4m
 2aRdMQm2+8fPShzY59xINo+qQ0aTih9mg4ebZnA9FoLS/vQmHY3CwLgASpMsdXGTvSm7qju59g
 +X0TO1h1lCvzkHiqrf4oEXs36tDA1/ft/TwbSiAB2xaa2HcBVbDx0yqgL8aX9TYJKW0oaGno8d
 WW8=
IronPort-SDR: SzNb9rhXXtSvRzzZHTjbI5IJEwQAJptPReVOeaT0GQMUisy9WQ+tFstXchRXibsD7K/6Uvafkx
 aJX+qWP7au4V1xYbtGAgUwSQaC8HICv5sm0SyBj5y1lMwQJYBdfWPCUR7zEz9Pir3gfwlrLMjD
 QfLj8t+A9p8vL5KELHwZD8pFwVeAsWGX62SeqszWlfICt5+VCNemhZovqEqqM1WEJaVDPp4Xs1
 jfwg+EbXsy65FUgke7jVaJX7LOIirAorS0K8qU4z15zvdGEbN8Imjzgys42R7twGfULbGMsSC+
 Ccc=
WDCIronportException: Internal
Message-ID: <6073061b-138b-499c-6de1-5196f191b176@opensource.wdc.com>
Date:   Tue, 24 Jan 2023 17:41:01 +0900
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101
 Thunderbird/102.6.0
Subject: Re: linux-next: manual merge of the zonefs tree with the
 vfs-idmapping tree
Content-Language: en-US
To:     Christian Brauner <brauner@kernel.org>
Cc:     Stephen Rothwell <sfr@rothwell.id.au>,
        Stephen Rothwell <sfr@canb.auug.org.au>,
        Damien Le Moal <Damien.LeMoal@wdc.com>,
        Christian Brauner <christian@brauner.io>,
        Seth Forshee <sforshee@kernel.org>,
        Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
        Linux Next Mailing List <linux-next@vger.kernel.org>
References: <20230124085951.59eb8da3@canb.auug.org.au>
 <1631b427-9efd-cd26-5dbc-0143097f859b@opensource.wdc.com>
 <20230124102535.1e21a533@canb.auug.org.au>
 <87958074-baa4-b6c9-da0c-d654e4009611@opensource.wdc.com>
 <20230124110745.5c9b78b0@oak.ozlabs.ibm.com>
 <8871d62a-b87f-75d6-fb5d-d9811a3e5dae@opensource.wdc.com>
 <20230124083235.6fhvnjgl4bzuzwuq@wittgenstein>
From:   Damien Le Moal <damien.lemoal@opensource.wdc.com>
Organization: Western Digital Research
In-Reply-To: <20230124083235.6fhvnjgl4bzuzwuq@wittgenstein>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: bulk

On 1/24/23 17:32, Christian Brauner wrote:
> On Tue, Jan 24, 2023 at 09:11:47AM +0900, Damien Le Moal wrote:
>> On 1/24/23 09:07, Stephen Rothwell wrote:
>>> Hi Damien,
>>>
>>> On Tue, 24 Jan 2023 08:30:29 +0900 Damien Le Moal <damien.lemoal@opensource.wdc.com> wrote:
>>>>
>>>> OK. I think I will merge the 3 patches that create the conflict and rebase
>>>> the patches. I need that for retesting at least. But given the size of the
>>>> conflict resolution, I may push that as an update to my for-6.3/for-next
>>>> branch. Let me see...
>>>>
>>>>> Alternatively, just leave the fix up to Linus (but mention it to him
>>>>> when you send your pull requests).  
>>>>
>>>> Understood. Let me retest first :)
>>>
>>> When I said "merge", I meant literally "git merge <some stable branch
>>> from the vfs-mapping tree that contains the conflicting commit>" not
>>> cherry pick the commits i.e. you would need to coordinate with
>>> Christian about having a common branch (or making sure that the part of
>>> his tree you pull in is immutable).
>>
>> Yep, cherry picking did not work :)
>> I did a merge test and came up with the same resolution as you did. It
>> looks good. It looks big but is in fact fairly simple. I will keep it as
>> is and signal it to Linus when I send my PR.
> 
> I don't rebase branches after they're in -next as soon as someone
> wants to depend on them.

I understand that. Nobody does.

> 
>>
>> But retesting everything to be sure there are no issues.
>>
>> Christian,
>>
>> Next time, when you touch an fs, please cc the maintainer for acks. I had
>> that zonefs series ready for a while and we could have coordinated for the
>> conflict...
> 
> I understand merge conflicts aren't pleasant as I'm dealing with them on
> a regular basis myself and I'm sorry that this is causing churn for you.

The conflict is fine, I can handle. I was more surprised that patches
touching zonefs were applied without acks from me.

> 
> Similar to other large branches such as the initial folio conversion
> that affected a lot of filesystems and other branches of mine it simply
> becomes impractical to generate a massive recipients list.

Fair enough.

> All filesystems were touched in non-semantical ways and simply replace a
> vfs type.

That I saw :)

> 
> One of linux-next's tasks is to find generated merge conflicts so that
> we can coordinate. As usual, I will send a list of merge conflicts
> caused by one of our branches to Linus and point him to the relevant
> threads that Steven reported with proposed resolutions as he prefers to
> fix them up himself.
> 
> Sorry for the inconvenience.

All solved now. My morning was only a little more busy than expected
dealing with this :)
Thanks.

-- 
Damien Le Moal
Western Digital Research