LKML Archive on lore.kernel.org
help / color / mirror / Atom feed
From: Juri Lelli <juri.lelli@redhat.com>
To: Quentin Perret <quentin.perret@arm.com>
Cc: Viresh Kumar <viresh.kumar@linaro.org>,
Dietmar Eggemann <dietmar.eggemann@arm.com>,
linux-kernel@vger.kernel.org,
Peter Zijlstra <peterz@infradead.org>,
Ingo Molnar <mingo@redhat.com>,
linux-pm@vger.kernel.org, Pavan Kondeti <pkondeti@codeaurora.org>,
"Rafael J . Wysocki" <rafael.j.wysocki@intel.com>,
Joel Fernandes <joelaf@google.com>,
Patrick Bellasi <patrick.bellasi@arm.com>
Subject: Re: [PATCH] Revert "cpufreq: schedutil: Don't restrict kthread to related_cpus unnecessarily"
Date: Tue, 8 May 2018 14:20:40 +0200 [thread overview]
Message-ID: <20180508122040.GE19168@localhost.localdomain> (raw)
In-Reply-To: <20180508112424.GA463@e108498-lin.cambridge.arm.com>
On 08/05/18 12:24, Quentin Perret wrote:
> On Tuesday 08 May 2018 at 16:44:51 (+0530), Viresh Kumar wrote:
> > On 08-05-18, 12:00, Quentin Perret wrote:
> > > Right, I see your point. Now, with the current implementation, why should
> > > we randomly force a CPU to manage the kthread of another ? IIUC deadline
> > > should assign the kthreads to CPUs depending on the state of the system
> > > when the task is created. So, from one boot to another, you could
> > > theoretically end up with varying configurations, and varying power/perf
> > > numbers.
> >
> > Yeah, if it is fixed at boot then there is no good reason to push it
> > to any other CPU. I agree.
> >
>
> To be fair, I think that DL tasks _can_ migrate, but only in special
> conditions (hotplug, or if a DL task wakes up when another DL task is
> running and things like that IIRC) but that probably doesn't matter much
> for our discussion here.
More than "special" I'd say "different" (w.r.t. CFS). DL looks at tasks
deadlines and use that info to migrate tasks around. So, it's correct
that they will currently stay on first CPU they run on if no other DL
tasks are around.
Luca's capacity(/energy) awareness stuff should change that in the
future. But that might take a while I guess. :/
next prev parent reply other threads:[~2018-05-08 12:20 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-05-08 7:33 Dietmar Eggemann
2018-05-08 8:22 ` Viresh Kumar
2018-05-08 9:09 ` Dietmar Eggemann
2018-05-08 9:42 ` Quentin Perret
2018-05-13 5:19 ` Joel Fernandes
2018-05-17 19:10 ` Saravana Kannan
2018-05-17 19:13 ` Joel Fernandes
2018-05-08 9:45 ` Viresh Kumar
2018-05-08 10:02 ` Quentin Perret
2018-05-08 10:34 ` Viresh Kumar
2018-05-08 11:00 ` Quentin Perret
2018-05-08 11:14 ` Viresh Kumar
2018-05-08 11:24 ` Quentin Perret
2018-05-08 12:20 ` Juri Lelli [this message]
2018-05-08 20:36 ` Rafael J. Wysocki
2018-05-09 4:55 ` Viresh Kumar
2018-05-08 10:36 ` Dietmar Eggemann
2018-05-08 10:53 ` Viresh Kumar
2018-05-08 12:17 ` Juri Lelli
2018-05-09 4:55 ` Viresh Kumar
2018-05-17 10:32 ` Rafael J. Wysocki
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=20180508122040.GE19168@localhost.localdomain \
--to=juri.lelli@redhat.com \
--cc=dietmar.eggemann@arm.com \
--cc=joelaf@google.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pm@vger.kernel.org \
--cc=mingo@redhat.com \
--cc=patrick.bellasi@arm.com \
--cc=peterz@infradead.org \
--cc=pkondeti@codeaurora.org \
--cc=quentin.perret@arm.com \
--cc=rafael.j.wysocki@intel.com \
--cc=viresh.kumar@linaro.org \
--subject='Re: [PATCH] Revert "cpufreq: schedutil: Don'\''t restrict kthread to related_cpus unnecessarily"' \
/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
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).