On Mon, Jun 05, 2023 at 02:11:35PM +0100, Tvrtko Ursulin wrote: > From: Tvrtko Ursulin <tvrtko.ursulin@xxxxxxxxx> > > Add some missing error propagation in live_parallel_switch. > > To avoid needlessly burdening the various backport processes, note I am > not marking it as a fix against any patches and not copying stable since > it is debug/selftests only code. > This patch is unlikely to make a difference in real life, but I don't think avoiding Fixes tags and backports is the right thing. I would add a Fixes tag and not add a stable tag. The real burden with Fixes tags is if it breaks someone's system. But if it's breaking selftests then hopefully those are the people best able to deal with it. Fixes tags are different from stable tags. If the code is very recent then the fixes tag can automatically allow us to filter out that patch from going back to stable. So for new patches Fixes is the opposite of CC'ing stable. If the bug is old, then adding a Fixes tag does increase the chance of a backport though, that's true. My guess is that if the stable maintainers thought that selftests/ were causing too much issue with backports they would add a grep line to their scripts to solve that problem. Instead we were having the opposite discussion the other week where the bpf people didn't want to backport selftest stuff and Greg wanted to. https://lore.kernel.org/all/2023052647-tacking-wince-85c5@gregkh/ regards, dan carpenter
