lkml.org 
[lkml]   [2017]   [Nov]   [14]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
Date
SubjectRe: [PATCH v2] sched/deadline: fix runtime accounting in documentation
Hi Claudio,

On 14 November 2017 at 04:19, Claudio Scordino <claudio@evidence.eu.com> wrote:
> Signed-off-by: Claudio Scordino <claudio@evidence.eu.com>
> Signed-off-by: Luca Abeni <luca.abeni@santannapisa.it>
> Acked-by: Daniel Bristot de Oliveira <bristot@redhat.com>
> CC: Jonathan Corbet <corbet@lwn.net>
> CC: "Peter Zijlstra (Intel)" <peterz@infradead.org>
> CC: Ingo Molnar <mingo@redhat.com>
> CC: linux-doc@vger.kernel.org
> Cc: Tommaso Cucinotta <tommaso.cucinotta@sssup.it>
> Cc: Mathieu Poirier <mathieu.poirier@linaro.org>
> ---

Please add a short description for your change, even if it is trivial.


> Documentation/scheduler/sched-deadline.txt | 13 ++++++++++---
> 1 file changed, 10 insertions(+), 3 deletions(-)
>
> diff --git a/Documentation/scheduler/sched-deadline.txt b/Documentation/scheduler/sched-deadline.txt
> index e89e36e..8ce78f8 100644
> --- a/Documentation/scheduler/sched-deadline.txt
> +++ b/Documentation/scheduler/sched-deadline.txt
> @@ -204,10 +204,17 @@ CONTENTS
> It does so by decrementing the runtime of the executing task Ti at a pace equal
> to
>
> - dq = -max{ Ui, (1 - Uinact) } dt
> + dq = -max{ Ui / Umax, (1 - Uinact - Uextra) } dt
>
> - where Uinact is the inactive utilization, computed as (this_bq - running_bw),
> - and Ui is the bandwidth of task Ti.
> + where:
> +
> + - Ui is the bandwidth of task Ti;
> + - Umax is the maximum reclaimable utilization (subjected to RT throttling
> + limits);
> + - Uinact is the (per runqueue) inactive utilization, computed as
> + (this_bq - running_bw);
> + - Uextra is the (per runqueue) extra reclaimable utilization
> + (subjected to RT throttling limits).

I think there would be value in defining 'dq' and 'dt'. That way
people know exactly what they are and it leaves no room for
interpretation.

Thanks,
Mathieu

>
>
> Let's now see a trivial example of two deadline tasks with runtime equal
> --
> 2.7.4
>

\
 
 \ /
  Last update: 2017-11-14 19:42    [W:0.174 / U:0.092 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site