sched/fair: Remove distribute_running from CFS bandwidth
authorJosh Don <joshdon@google.com>
Fri, 10 Apr 2020 22:52:08 +0000 (15:52 -0700)
committerPeter Zijlstra <peterz@infradead.org>
Thu, 30 Apr 2020 18:14:38 +0000 (20:14 +0200)
commitab93a4bc955b3980c699430bc0b633f0d8b607be
tree06c7c6724eb51685931e630d70f5e3663914d428
parente98fa02c4f2ea4991dae422ac7e34d102d2f0599
sched/fair: Remove distribute_running from CFS bandwidth

This is mostly a revert of commit:

  baa9be4ffb55 ("sched/fair: Fix throttle_list starvation with low CFS quota")

The primary use of distribute_running was to determine whether to add
throttled entities to the head or the tail of the throttled list. Now
that we always add to the tail, we can remove this field.

The other use of distribute_running is in the slack_timer, so that we
don't start a distribution while one is already running. However, even
in the event that this race occurs, it is fine to have two distributions
running (especially now that distribute grabs the cfs_b->lock to
determine remaining quota before assigning).

Signed-off-by: Josh Don <joshdon@google.com>
Signed-off-by: Peter Zijlstra (Intel) <peterz@infradead.org>
Reviewed-by: Phil Auld <pauld@redhat.com>
Tested-by: Phil Auld <pauld@redhat.com>
Link: https://lkml.kernel.org/r/20200410225208.109717-3-joshdon@google.com
kernel/sched/fair.c
kernel/sched/sched.h