On Thu, Jul 23, 2020 at 03:17:04PM +0200, Thomas Gleixner wrote:
8) Changelogs
Most of the changelogs have something along the lines:
'task isolation does not want X, so do Y to make it not do X'
without any single line of explanation why this approach was chosen
and why it is correct under all circumstances and cannot have nasty
side effects.
It's not the job of the reviewers/maintainers to figure this out.
Please come up with a coherent design first and then address the
identified issues one by one in a way which is palatable and reviewable.
Throwing a big pile of completely undocumented 'works for me' mess over
the fence does not get you anywhere, not even to the point that people
are willing to review it in detail.
This.. as presented it is an absolutely unreviewable pile of junk. It
presents code witout any coherent problem description and analysis. And
the patches are not split sanely either.