[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date: Wed, 9 Sep 2020 10:37:14 +0200 (CEST)
From: Lukas Bulwahn <lukas.bulwahn@...il.com>
To: John Mathew <john.mathew@...kie.com>
cc: linux-doc@...r.kernel.org, linux-kernel@...r.kernel.org,
corbet@....net, mingo@...hat.com, peterz@...radead.org,
juri.lelli@...hat.com, vincent.guittot@...aro.org,
dietmar.eggemann@....com, rostedt@...dmis.org, bsegall@...gle.com,
mgorman@...e.de, bristot@...hat.com, tsbogend@...ha.franken.de,
lukas.bulwahn@...il.com, x86@...nel.org,
linux-mips@...r.kernel.org, tglx@...utronix.de,
willy@...radead.org, valentin.schneider@....com
Subject: Re: [RFC PATCH v8 0/3] Add scheduler overview documentation
On Wed, 2 Sep 2020, John Mathew wrote:
> This patch series updates the scheduler documentation to add more topics
> wrt to scheduler overview. New sections are added to provide a brief
> overview of the kernel structs used by the scheduler, scheduler invocation,
> and context switch. Previous version of the patch was reviewed at:
> https://lore.kernel.org/lkml/20200527084421.4673-1-John.Mathew@unikie.com/
>
John, here is some first feedback to get the ball rolling:
I tried to apply your patches on v5.9-rc4, and I got those warnings:
Applying: docs: scheduler: Restructure scheduler documentation.
.git/rebase-apply/patch:30: new blank line at EOF.
+
.git/rebase-apply/patch:137: new blank line at EOF.
+
warning: 2 lines add whitespace errors.
Applying: docs: scheduler: Add scheduler overview documentation
.git/rebase-apply/patch:73: new blank line at EOF.
+
warning: 1 line adds whitespace errors.
Applying: docs: scheduler: Add introduction to scheduler context-switch
.git/rebase-apply/patch:153: new blank line at EOF.
+
.git/rebase-apply/patch:260: new blank line at EOF.
+
warning: 2 lines add whitespace errors.
You might want to look into this. I also checked that the patch also
applies on linux-next, i.e., next-20200908; so, it does not clash in an
obvious way with other changes at the moment.
I did run checkpatch.pl and it warned about:
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
No action required here.
Documentation generation (make htmldocs) shows these two new warnings with
your patches applied to v5.9-rc4:
./kernel/sched/core.c:17: WARNING: Definition list ends without a blank
line; unexpected unindent.
./kernel/sched/core.c:21: WARNING: Unexpected indentation.
You might want to put those minor fixes on your remaining TODO list for
this patchset as well.
I will continue to comment with more editorial points in the next hours
and days, while proof-reading your additions to the documentation.
Lukas
Powered by blists - more mailing lists