Re: [PATCH v4 00/13] "Task_isolation" mode

Peter Zijlstra
2020-07-23T14:29:20Z

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.