From: Srivatsa Vaddagiri <vatsa@linux.vnet.ibm.com>
To: Peter Zijlstra <peterz@infradead.org>
Cc: Ingo Molnar <mingo@elte.hu>, Mike Galbraith <efault@gmx.de>,
Suresh Siddha <suresh.b.siddha@intel.com>,
Paul Turner <pjt@google.com>,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH v1] sched: steer waking task to empty cfs_rq for better latencies
Date: Tue, 24 Apr 2012 23:05:18 +0530 [thread overview]
Message-ID: <20120424173518.GB18799@linux.vnet.ibm.com> (raw)
In-Reply-To: <1335287547.28150.210.camel@twins>
* Peter Zijlstra <peterz@infradead.org> [2012-04-24 19:12:27]:
> > The workload I am up against does experience tons of wakeup (after very short
> > (microsecond range) bursts of sleeps) and so am skeptical how the global
> > vruntime would keep up with this kind of workload. I'd be happy to test patches
> > and give feedback!
>
> Thing is, global vruntime might fix that flaw/property you're exploiting
> to get that preemption.
Am curious to see how global vruntime influences task placement (esp. on
wakeup). Going back to the previous example:
N0 N1
(0,1,2,3) (4,5,6,7)
rq.nr_run -> 2 1 1 1 2 2 1 1
/a cfs_rq.nr_run -> 0 0 0 0 0 0 0 1
CPU7 is running a task A0 belonging to "/a". Another task A1 belonging
to "/a" is waking up and its prev_cpu was 7. Currently we let it wake on
cpu7 itself and wait behind A0. With global vruntime, are you saying
that we let A1 wake on cpu7 and it would preempt A0? That would still not be
nice as A0 now incurs some additional latencies?
- vatsa
next prev parent reply other threads:[~2012-04-24 17:35 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-04-24 16:56 [PATCH v1] sched: steer waking task to empty cfs_rq for better latencies Srivatsa Vaddagiri
2012-04-24 16:58 ` Peter Zijlstra
2012-04-24 17:07 ` Srivatsa Vaddagiri
2012-04-24 17:12 ` Peter Zijlstra
2012-04-24 17:35 ` Srivatsa Vaddagiri [this message]
2012-04-24 18:03 ` Rakib Mullick
2012-04-24 17:09 ` Peter Zijlstra
2012-04-24 17:26 ` Srivatsa Vaddagiri
2012-05-02 14:01 ` Srivatsa Vaddagiri
2012-05-03 5:43 ` Nikunj A Dadhania
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20120424173518.GB18799@linux.vnet.ibm.com \
--to=vatsa@linux.vnet.ibm.com \
--cc=efault@gmx.de \
--cc=linux-kernel@vger.kernel.org \
--cc=mingo@elte.hu \
--cc=peterz@infradead.org \
--cc=pjt@google.com \
--cc=suresh.b.siddha@intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.