Linux Kernel Summit discussions
 help / color / mirror / Atom feed
From: Rodrigo Vivi <rodrigo.vivi@intel.com>
To: Geert Uytterhoeven <geert@linux-m68k.org>,
	Jani Nikula <jani.nikula@intel.com>,
	Simona Vetter <simona.vetter@ffwll.ch>
Cc: Dave Airlie <airlied@gmail.com>, Mark Brown <broonie@kernel.org>,
	"Steven Rostedt" <rostedt@goodmis.org>,
	James Bottomley <James.Bottomley@hansenpartnership.com>,
	"Lorenzo Stoakes (ARM)" <ljs@kernel.org>,
	Linus Torvalds <torvalds@linux-foundation.org>,
	<ksummit@lists.linux.dev>
Subject: Re: [MAINTAINERS SUMMIT] Time to call it quits for the maintainer summit?
Date: Tue, 11 Aug 2026 15:54:38 -0400	[thread overview]
Message-ID: <ant9_jhefKix5TGu@intel.com> (raw)
In-Reply-To: <CAMuHMdVBABOiOWDdCKE0zDffj856ui5qr3UFVptwcAn1_Hx7oQ@mail.gmail.com>

On Tue, Aug 11, 2026 at 09:12:24AM +0200, Geert Uytterhoeven wrote:
> Hi Dave,
> 
> On Tue, 11 Aug 2026 at 05:30, Dave Airlie <airlied@gmail.com> wrote:
> > > It's not even all of DRM, it's specifically the AMD and Intel driver
> > > stacks which as far as I can tell just routinely cherry pick all their
> > > fixes between their development and fixes branches without even
> > > considering the possibility of either merging up the fixes branch or
> > > just waiting and letting the fixes propagate back.  None of the rest of
> > > the DRM trees ever seems to cause these issues.
> >
> > AMD and Intel are just the two largest teams with the longest
> > pipelines of internal engineers and teams. There are between 50 and
> > 100 people in those groups, across multiple disjoint teams feeding
> > into a single driver. It just doesn't scale for all 50-100 people to
> > understand the cycle of the upstream Linus tree at all times for all
> > patches.
> 
> Doesn't this cause internal (to AMD and Intel) issues, too?
> How come we cannot educate contributors at these two large companies,
> while we expect other contributors to be aware?
> I would hope the people who do the actual commits do know?

We currently have 52 devs with commit rights in drm-intel and 47 devs
with commit rights on drm-xe (I'm sorry, but I didn't check the union).

But honestly, I don't believe this is the key argument anyway. The key
argument for the drm-intel flow is the CI, pipeline (like Dave wrote
below) and also for historical reasons. When we didn't have this clear
separation with this quarantine weeks (-rc6->-rc1) many last minute
regressions were introduced causing hic-ups on the -rc1 flow for Linus.

Probably worse in our case because our regression makes the screen to
not appear... I don't know.

But well, let's say we keep the quarantine and go with the -fixes and
-next branches without the direct cherry-pick, but the commiters deciding
for witch branch they apply the patches. (Similar to drm-misc flow).
I still see some potential risk scenarios and not a clear benefit:

1. Increase the risk of hic-ups in other fronts. For instance: patches
that should had been merged to -fixes only going to next and forgotten
to be ported to the previous version. And patches that should be new
features due to the risk of regression, added to the previous version.

But here one could argue that in a distributed commit rights flow, this
is exactly why maintainers role is more critical, tto ensure we are not
missing anything and that the patches are moved to the right place.
Fair enough, but this makes our hashes more likely to change. We try
to keep the non-rebasing tree to avoid disruptions in OSVs and many
other teams that are consuming our trees.

2. The conflicts would happen anyway. I still see a lot of conflicts
in the drm-misc flow.

3. Harder to manage the conflicts. Our cherry-pick flow brings some very
obvious conflict resolution to the table. If you are in a newer kernel
you likely only need to go with it is already in our -next branches,
if you are in the current -rc based you likely need to go with what
it ported to -fixes. It really is not something complicated to solve.
But if you go with a flow that the patches don't have the same baseline
like we have the -next and our -tip, then you get harder conflicts to
solve and likely to make more mistakes.

> 
> > Which means for CI reasons and pipeline reasons, things go into -next
> > is the default, our -next trees are always open, and get disconnected
> > from upstream next from rc6->rc1 so not to mess things up.
> 
> Just merging fixes into next[*], and resolving the conflicts, before
> publishing the latter would help a lot.  That would mean the conflicts
> would no longer impact every user of your next branch.
> [*] This doesn't need to be the real next you will send to Linus
>     later. Many maintainers use a conglomerate "for-next" branch that
>     is never sent upstream as-is, but its sub-branches are.

Well, I was going to tell about the drm-tip, but that you already know.
And yes, we cannot have that for-next because it contains topic branches
we really don't want to send up.

Dave, Sima, Jani, perhaps we could instrument dim to create a drm-for-next
that is the merge of all of our relevant branches excluding the topic
branches?

Thanks,
Rodrigo.

  parent reply	other threads:[~2026-08-11 19:54 UTC|newest]

Thread overview: 85+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-08-06 15:34 [MAINTAINERS SUMMIT] Time to call it quits for the maintainer summit? James Bottomley
2026-08-06 23:41 ` Steven Rostedt
2026-08-07 13:53   ` Rafael J. Wysocki (Intel)
2026-08-07 14:53     ` Steven Rostedt
2026-08-07 16:33     ` James Bottomley
2026-08-07 17:27   ` Linus Torvalds
2026-08-07 19:03     ` James Bottomley
2026-08-07 19:51     ` Steven Rostedt
2026-08-07 20:12       ` Shuah Khan
2026-08-09  1:51     ` Theodore Tso
2026-08-09 18:47       ` Randy Dunlap
2026-08-09 19:04       ` Jonathan Corbet
2026-08-09 21:09         ` Steven Rostedt
2026-08-10  8:14           ` Lorenzo Stoakes (ARM)
2026-08-10 13:20             ` Steven Rostedt
2026-08-10 20:35               ` Liam R. Howlett
2026-08-10 21:05                 ` Steven Rostedt
2026-08-11  0:14                 ` Theodore Tso
2026-08-18 13:09                   ` Liam R. Howlett
2026-08-18 14:02                     ` Theodore Tso
2026-08-18 14:21                       ` David Hildenbrand (Arm)
2026-08-18 15:32                         ` Liam R. Howlett
2026-08-18 17:46                           ` Theodore Tso
2026-08-18 18:18                             ` Lorenzo Stoakes (ARM)
2026-08-18 14:51                   ` Lorenzo Stoakes (ARM)
2026-08-18 14:52                     ` Lorenzo Stoakes (ARM)
2026-08-18 16:45                     ` Theodore Tso
2026-08-18 17:00                       ` James Bottomley
2026-08-18 17:25                       ` Lorenzo Stoakes (ARM)
2026-08-09 21:56         ` Liam R. Howlett
2026-08-10  1:26           ` Theodore Tso
2026-08-10  2:40             ` Liam R. Howlett
2026-08-10  8:08             ` Lorenzo Stoakes (ARM)
2026-08-10 14:56               ` Theodore Tso
2026-08-10 15:49                 ` Lorenzo Stoakes (ARM)
2026-08-09 18:39     ` Lorenzo Stoakes (ARM)
2026-08-09 18:42       ` Lorenzo Stoakes (ARM)
2026-08-09 22:10       ` Dave Airlie
2026-08-10  7:26         ` Lorenzo Stoakes (ARM)
2026-08-10 13:11           ` Steven Rostedt
2026-08-10 13:32             ` James Bottomley
2026-08-10 14:24               ` Steven Rostedt
2026-08-10 15:23                 ` Mark Brown
2026-08-10 15:52                   ` Randy Dunlap
2026-08-11  3:24                     ` Dave Airlie
2026-08-11  5:29                       ` Randy Dunlap
2026-08-11  6:53                         ` Geert Uytterhoeven
2026-08-11 13:29                       ` Mark Brown
2026-08-10 22:47                   ` Mark Brown
2026-08-11  3:29                   ` Dave Airlie
2026-08-11  7:12                     ` Geert Uytterhoeven
2026-08-11  8:16                       ` Geert Uytterhoeven
2026-08-11 19:54                       ` Rodrigo Vivi [this message]
2026-08-11 23:10                         ` Mark Brown
2026-08-12 20:33                           ` Dave Airlie
2026-08-13 11:48                             ` Mark Brown
2026-08-18  9:38                               ` Jani Nikula
2026-08-18 16:50                                 ` Mark Brown
2026-08-19  8:28                                 ` Takashi Iwai
2026-08-19 16:44                                   ` Linus Torvalds
2026-08-11 14:26                     ` Mark Brown
2026-08-12  6:00                     ` Krzysztof Kozlowski
2026-08-12  6:20                       ` Dave Airlie
2026-08-12  7:07                         ` Krzysztof Kozlowski
2026-08-12  7:41                         ` Greg KH
2026-08-12  8:05                           ` Dave Airlie
2026-08-12 12:48                             ` Mark Brown
2026-08-10 21:30           ` Dave Airlie
2026-08-10 21:45             ` Liam R. Howlett
2026-08-11  8:22               ` Lorenzo Stoakes (ARM)
2026-08-11  8:19             ` Lorenzo Stoakes (ARM)
2026-08-11  9:06               ` Jiri Kosina
2026-08-11  9:34                 ` Dave Airlie
2026-08-11 13:47                 ` Steven Rostedt
2026-08-10  7:42         ` Geert Uytterhoeven
2026-08-10 15:07           ` Theodore Tso
2026-08-07 12:58 ` Laurent Pinchart
2026-08-07 13:09   ` James Bottomley
2026-08-07 13:28     ` Laurent Pinchart
2026-08-07 20:15     ` H. Peter Anvin
2026-08-07 19:15 ` Chris Mason
2026-08-07 19:53   ` Steven Rostedt
2026-08-07 23:19 ` Theodore Tso
2026-08-10 13:54   ` James Bottomley
2026-08-10 14:41     ` Steven Rostedt

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=ant9_jhefKix5TGu@intel.com \
    --to=rodrigo.vivi@intel.com \
    --cc=James.Bottomley@hansenpartnership.com \
    --cc=airlied@gmail.com \
    --cc=broonie@kernel.org \
    --cc=geert@linux-m68k.org \
    --cc=jani.nikula@intel.com \
    --cc=ksummit@lists.linux.dev \
    --cc=ljs@kernel.org \
    --cc=rostedt@goodmis.org \
    --cc=simona.vetter@ffwll.ch \
    --cc=torvalds@linux-foundation.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox