On 7/16/24 11:45, Jonathan Corbet wrote: > Carlos Bilbao <carlos.bilbao.osdev@xxxxxxxxx> writes: > >> Add some documentation regarding the newly introduced scheduler EEVDF. > A nit, but one I wish more people would make note of... > > [...] > >> --- a/Documentation/scheduler/sched-design-CFS.rst >> +++ b/Documentation/scheduler/sched-design-CFS.rst >> @@ -8,10 +8,12 @@ CFS Scheduler >> 1. OVERVIEW >> ============ >> >> -CFS stands for "Completely Fair Scheduler," and is the new "desktop" process >> -scheduler implemented by Ingo Molnar and merged in Linux 2.6.23. It is the >> -replacement for the previous vanilla scheduler's SCHED_OTHER interactivity >> -code. >> +CFS stands for "Completely Fair Scheduler," and is the "desktop" process >> +scheduler implemented by Ingo Molnar and merged in Linux 2.6.23. When >> +originally merged, it was the replacement for the previous vanilla >> +scheduler's SCHED_OTHER interactivity code. Nowadays, CFS is making room >> +for EEVDF, for which documentation can be found in >> +:ref:`sched_design_EEVDF`. > If, here, you just say "can be found in > Documentation/scheduler/sched-eevdf.rst", the right cross-reference will > be created and ... > >> 80% of CFS's design can be summed up in a single sentence: CFS basically models >> an "ideal, precise multi-tasking CPU" on real hardware. >> diff --git a/Documentation/scheduler/sched-eevdf.rst b/Documentation/scheduler/sched-eevdf.rst >> new file mode 100644 >> index 000000000000..019327da333a >> --- /dev/null >> +++ b/Documentation/scheduler/sched-eevdf.rst >> @@ -0,0 +1,44 @@ >> +.. _sched_design_EEVDF: > ...you can take out this unnecessary label. Love it! Labels are a source of warnings. Sending v3 now. > > Thanks, > > jon Thanks, Carlos