On Fri, Nov 17, 2017 at 03:01:08PM +0200, Jani Nikula wrote: >I still have no idea how this autoselect picks up patches that do *not* >have cc: stable nor Fixes: from us. What information do you have that we >don't for making that call? I think there's a difference in views about the stable tag here. I suspect that the way you see it, if you guys choose not to add a stable tag to a commit this means that you, on purpose do *not* want this commit to go into stable tree. However, the stable tag is really just a sign for us saying "hey this might need to go into stable trees", and in the same way that having this tag doesn't guarantee that the commit will go into stable, not adding this tag in no way guarantees that it won't. In reality, authors and maintainers forget to tag, miss fixes, newer authors aren't even aware of the stable process and so on, so we end up with a good chunk of fixes that affect users but don't end up getting fixed in stable trees just because someone forgot a simple tag in his commit message. The autoselection code tries to determine what a "fix" is based on information from the commit message (so, for example, if "fix", "panic" and "oops" appear in a commit message it's quite likely that this commit is a fix), and various code metrics (for example, commits that add a single condition statement and <5 lines of code are likely to be fixes". Past that point, it's all a judgement call on my end of whether some commit, based on it's code, commit message, context, etc would be something that we'd want in stable trees or not. I also accept the fact that I may be wrong, but in those cases I'll do my best to learn from it, fix the issue, and improve my process to prevent similar issues in the future. Also, if we would have strictly followed the "fix a real bug that bothers people" rule, stable trees would mostly likely die since you'd force users (who are usually clueless about the kernel) to do basic triaging and report bugs on LKML, which would never happen. Instead, we try to proactively fix bugs before users have to complain. Consider fuzzers as an example of the above; syzkaller/trinity find a considerable amount of bugs that don't really affect anyone, but they still should get fixed, right? -- Thanks, Sasha _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel
