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
