From: Mark Brown <broonie@kernel.org>
To: Luiz Augusto von Dentz <luiz.dentz@gmail.com>
Cc: Jakub Kicinski <kuba@kernel.org>,
davem@davemloft.net, linux-bluetooth@vger.kernel.org,
netdev@vger.kernel.org
Subject: Re: [GIT PULL] bluetooth 2026-09-07
Date: Tue, 15 Sep 2026 23:54:49 +0100 [thread overview]
Message-ID: <aqnMueXiCztLUn-w@sirena.org.uk> (raw)
In-Reply-To: <CABBYNZJWcrvkx=6DekHxZzcrGjHwiqmSuwXqzHNV1=7BctThGA@mail.gmail.com>
[-- Attachment #1: Type: text/plain, Size: 2459 bytes --]
On Tue, Sep 15, 2026 at 03:36:50PM -0400, Luiz Augusto von Dentz wrote:
> On Tue, Sep 15, 2026 at 3:24 PM Mark Brown <broonie@kernel.org> wrote:
> > On Tue, Sep 15, 2026 at 03:16:21PM -0400, Luiz Augusto von Dentz wrote:
> > Yes, the routine cherry picking is certainly part of it - it's not so
> > much the fact that things get moved between branches as the fact that
> > when you have the same commit is in multiple branches and then build on
> > top of one or both of them this creates extra conflicts, often harder to
> > understand, when the branches are merged. The more normal thing would
> > be to apply once to the right place, or the standard fallback from that
> > would be to drop the commit from branch A at the time it is moved to
> > branch B.
> If I merge to bluetooth, then bluetooth-next won't contain the fixes
> meant for stable/rc, so I will need to merge them back to
> bluetooth-next immediately it order for our CI to stay current with
> the fixes.
Or have your CI merge the branches and run on that (that's a fairly
widely deployed approach).
> > You do seem to end up removing the cherry picked commits from their
> > original branches at some point as part of your workflow (since there
> > are hardly any duplicate bluetooth commits in mainline) so they do wind
> > up being moves AFAICT but the duplicates get left sitting there for
> > quite a while.
> Yep, once before sending the pull request for net-next, bluetooth-next
> is rebased on top of net-next which brings all the fixes already
> merged back to bluetooth-next. Git is absolutely fine with that, I
> don't think I ever run into a conflict doing this. Git detects the
> fixes already present in net-next as duplicates and eliminates them,
> so it really surprises me when you say you are finding conflicts, not
> just duplicates.
The issue is not your rebase working, the issue is that until you get
round to doing that rebase anyone that merges both bluetooth and the net
trees (ot Linus' tree for that matter, often the conflicts are with his
tree) sees the duplicated commits and that routinely generates
conflicts. This obviously comes up an awful lot with -next, I am not
rebasing the trees but rather merging them, but do I gather from some
conversations that other people are seeing this as part of their
workflows.
Anyway, to return to my prior question: should you be a contact for the
bluetooth tree?
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
next prev parent reply other threads:[~2026-09-15 22:54 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-09-07 17:26 [GIT PULL] bluetooth 2026-09-07 Luiz Augusto von Dentz
2026-09-08 20:57 ` Jakub Kicinski
2026-09-08 20:59 ` Luiz Augusto von Dentz
2026-09-15 17:46 ` Mark Brown
2026-09-15 18:07 ` Luiz Augusto von Dentz
2026-09-15 18:26 ` Mark Brown
2026-09-15 19:16 ` Luiz Augusto von Dentz
2026-09-15 19:24 ` Mark Brown
2026-09-15 19:36 ` Luiz Augusto von Dentz
2026-09-15 22:54 ` Mark Brown [this message]
2026-09-16 18:12 ` Luiz Augusto von Dentz
2026-09-16 18:15 ` Mark Brown
2026-09-16 18:17 ` Luiz Augusto von Dentz
2026-09-16 18:27 ` Mark Brown
2026-09-16 18:30 ` Luiz Augusto von Dentz
2026-09-16 18:33 ` Mark Brown
2026-09-16 18:34 ` Luiz Augusto von Dentz
2026-09-16 18:40 ` Mark Brown
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=aqnMueXiCztLUn-w@sirena.org.uk \
--to=broonie@kernel.org \
--cc=davem@davemloft.net \
--cc=kuba@kernel.org \
--cc=linux-bluetooth@vger.kernel.org \
--cc=luiz.dentz@gmail.com \
--cc=netdev@vger.kernel.org \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.