Re: [GIT PULL] tracing: Updates for v6.9
From: Borislav Petkov
Date: Sat Mar 16 2024 - 17:10:56 EST
On Sat, Mar 16, 2024 at 01:42:06PM -0700, Linus Torvalds wrote:
> But if it's a "one pull to fix a single-line issue in resource
> control, and another pull to fix a single-line issue in objtool", then
> those make perfect sense to keep separate, even if they are both
> trivial and small.
Yeah, that's what they are - we do topic branches so it is always
separate topics - just sometimes they don't have too much going on
during a particular cycle.
Ok, understood - separate pulls, even if small ones, are fine.
> And on the other hand, if you have a couple of trivial branches with
> no real pattern, and decide to just merge them into one that fixes
> "misc x86 problems", and the end result is still completely trivial
> and there are no surprises or gotchas, that's not wrong either.
Ack.
> And sometimes, merging and sending me just one pull request is
> absolutely the right thing.
>
> For example, the ARM SoC trees tend to just merge "umbrella" updates
> into one single pull request, and I prefer that - because I see no
> point in getting ten different "this is the drivers for SoC xyz"
> thing.
Right.
I do a similar thing with the EDAC tree because it is all EDAC - no need
for separate pulls. We just keep them separate in case we have to rebase
early in the game to keep history clean. Yeah, well, rebase only if it
would get relatively ugly if not.
And then Tony or I merge them all into a single pull.
> So then it's still a clear topic branch ("ARM SoC drivers"), but they
> kept multiple branches for different SoC's and sent me just one pull
> request.
>
> End result: there's no one right thing. Make it make sense. Probably
> the only real rule is
>
> - try to keep conceptually different things separate just for cleanliness
>
> - definitely keep fundamental new features or anything that _might_
> be questionable in a branch of its own
Ok, understood. Makes sense.
> but there aren't some kind of black-and-white rules for "this is so
> small that it's not worth sending on its own".
>
> This merge window, I think I currently have something like ~15 merges
> that ended up being literally just a couple of lines (maybe spread
> over two or three files). I don't mind at all. If that's all that
> happened, that's fine.
.. and that is also some sort of documenting it: this area didn't see
that much development this cycle.
Thanks for explaining.
--
Regards/Gruss,
Boris.
https://people.kernel.org/tglx/notes-about-netiquette