On Tue, Mar 16, 2021 at 05:05:11PM +0100, gregkh@linuxfoundation.org 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@linuxfoundation.org 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.