On Thu, Dec 03, 2020 at 08:18:23AM +0200, Amir Goldstein wrote: > Here is a recent example, where during patch review, I requested NOT to include > any stable backport triggers [1]: > "...We should consider sending this to stable, but maybe let's merge > first and let it > run in master for a while before because it is not a clear and > immediate danger..." > > As a developer and as a reviewer, I wish (as Dave implied) that I had a way to > communicate to AUTOSEL that auto backport of this patch has more risk than > the risk of not backporting. My suggestion is that we could put something in the MAINTAINERS file which indicates what the preferred delay time should be for (a) patches explicitly cc'ed to stable, and (b) preferred time should be for patches which are AUTOSEL'ed for stable for that subsystem. That time might be either in days/weeks, or "after N -rc releases", "after the next full release", or, "never" (which would be a way for a subsystem to opt out of the AUTOSEL process). It should also be possible specify the delay in the trailer, e.g.: Stable-Defer: <delayspec> Auto-Stable-Defer: <delayspec> The advantage of specifying the delay relative to when they show up in Linus's tree helps deal with the case where the submaintainer might not be sure when their patches will get pushed to Linus by the maintainer. Cheers, - Ted
