Re: [PATCH 5.10 113/290] net: dsa: implement a central TX reallocation procedure

[Sasha Levin]

On Tue, Mar 16, 2021 at 05:05:11PM +0100, gregkh@xxxxxxxxxxxxxxxxxxx wrote:
> On Tue, Mar 16, 2021 at 09:54:01AM -0400, Sasha Levin wrote:
> > On Tue, Mar 16, 2021 at 06:46:10AM +0100, gregkh@xxxxxxxxxxxxxxxxxxx wrote:
> > > I cc: everyone on the signed-off-by list on the patch, why would we need
> > > to add more?  A maintainer should always be on that list automatically.
> >
> > Oh, hm, could this be an issue with subsystems that have a shared
> > maintainership model? In that scenario not all maintainers will sign-off
> > on a commit.
>
> So a shared maintainer trusts their co-maintainer for reviewing patches
> for Linus's tree and all future kernels, but NOT into an old backported
> stable tree?  I doubt that, trust should be the same for both.

I don't think it's necessarily a trust issue, but is an availability
issue: one of the reasons shared maintainership models exist is so
that one maintainer can go on vacation (or focus other work) while the
other maintainer(s) take over. If we send a review request to that
maintainer he might be away and we'll never get our review.

--
Thanks,
Sasha