sched: Ensure that a child can't gain time over it's parent after fork()
authorMike Galbraith <efault@gmx.de>
Tue, 8 Sep 2009 09:12:28 +0000 (11:12 +0200)
committerIngo Molnar <mingo@elte.hu>
Tue, 8 Sep 2009 11:15:34 +0000 (13:15 +0200)
commitb5d9d734a53e0204aab0089079cbde2a1285a38f
treef8c657d213514de7affaab50ed4cfd8b3a17d0f3
parenta8fae3ec5f118dc92517dcbed3ecf69ddb641d0f
sched: Ensure that a child can't gain time over it's parent after fork()

A fork/exec load is usually "pass the baton", so the child
should never be placed behind the parent.  With START_DEBIT we
make room for the new task, but with child_runs_first, that
room comes out of the _parent's_ hide. There's nothing to say
that the parent wasn't ahead of min_vruntime at fork() time,
which means that the "baton carrier", who is essentially the
parent in drag, can gain time and increase scheduling latencies
for waiters.

With NEW_FAIR_SLEEPERS + START_DEBIT + child_runs_first
enabled, we essentially pass the sleeper fairness off to the
child, which is fine, but if we don't base placement on the
parent's updated vruntime, we can end up compounding latency
woes if the child itself then does fork/exec.  The debit
incurred at fork doesn't hurt the parent who is then going to
sleep and maybe exit, but the child who acquires the error
harms all comers.

This improves latencies of make -j<n> kernel build workloads.

Reported-by: Jens Axboe <jens.axboe@oracle.com>
Signed-off-by: Mike Galbraith <efault@gmx.de>
Acked-by: Peter Zijlstra <a.p.zijlstra@chello.nl>
LKML-Reference: <new-submission>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
kernel/sched_fair.c