From: Lukasz Majewski <l.majewski@samsung.com>
To: Viresh Kumar <viresh.kumar@linaro.org>,
Vincent Guittot <vincent.guittot@linaro.org>
Cc: Jonghwa Lee <jonghwa3.lee@samsung.com>,
"Rafael J. Wysocki" <rjw@sisk.pl>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
Linux PM list <linux-pm@vger.kernel.org>,
"cpufreq@vger.kernel.org" <cpufreq@vger.kernel.org>,
MyungJoo Ham <myungjoo.ham@samsung.com>,
Kyungmin Park <kyungmin.park@samsung.com>,
Chanwoo Choi <cw00.choi@samsung.com>,
sw0312.kim@samsung.com,
Marek Szyprowski <m.szyprowski@samsung.com>
Subject: Re: [RFC PATCH 0/2] cpufreq: Introduce LAB cpufreq governor.
Date: Tue, 09 Apr 2013 18:44:40 +0200 [thread overview]
Message-ID: <20130409184440.4cd87c1b@amdc308.digital.local> (raw)
In-Reply-To: <CAKohpomL-mdx6DdFiGwJzDSeWr6Gw-_F4T-D-Jz9TNH5MSgjbw@mail.gmail.com>
Hi Viresh and Vincent,
> On 9 April 2013 16:07, Lukasz Majewski <l.majewski@samsung.com> wrote:
> >> On Mon, Apr 1, 2013 at 1:54 PM, Jonghwa Lee
> > Our approach is a bit different than cpufreq_ondemand one. Ondemand
> > takes the per CPU idle time, then on that basis calculates per cpu
> > load. The next step is to choose the highest load and then use this
> > value to properly scale frequency.
> >
> > On the other hand LAB tries to model different behavior:
> >
> > As a first step we applied Vincent Guittot's "pack small tasks" [*]
> > patch to improve "race to idle" behavior:
> > http://article.gmane.org/gmane.linux.kernel/1371435/match=sched+pack+small+tasks
>
> Luckily he is part of my team :)
>
> http://www.linaro.org/linux-on-arm/meet-the-team/power-management
>
> BTW, he is using ondemand governor for all his work.
>
> > Afterwards, we decided to investigate different approach for power
> > governing:
> >
> > Use the number of sleeping CPUs (not the maximal per-CPU load) to
> > change frequency. We thereof depend on [*] to "pack" as many tasks
> > to CPU as possible and allow other to sleep.
>
> He packs only small tasks.
What's about packing not only small tasks? I will investigate the
possibility to aggressively pack (even with a cost of performance
degradation) as many tasks as possible to a single CPU.
It seems a good idea for a power consumption reduction.
> And if there are many small tasks we are
> packing, then load must be high and so ondemand gov will increase
> freq.
This is of course true for "packing" all tasks to a single CPU. If we
stay at the power consumption envelope, we can even overclock the
frequency.
But what if other - lets say 3 CPUs - are under heavy workload?
Ondemand will switch frequency to maximum, and as Jonghwa pointed out
this can cause dangerous temperature increase.
>
> > Contrary, when all cores are heavily loaded, we decided to reduce
> > frequency by around 30%. With this approach user experience
> > recution is still acceptable (with much less power consumption).
>
> Don't know.. running many cpus at lower freq for long duration will
> probably take more power than running them at high freq for short
> duration and making system idle again.
>
> > We have posted this "RFC" patch mainly for discussion, and I think
> > it fits its purpose :-).
>
> Yes, no issues with your RFC idea.. its perfect..
>
> @Vincent: Can you please follow this thread a bit and tell us what
> your views are?
>
> --
> viresh
--
Best regards,
Lukasz Majewski
Samsung R&D Poland (SRPOL) | Linux Platform Group
next prev parent reply other threads:[~2013-04-09 16:44 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-04-01 8:24 [RFC PATCH 0/2] cpufreq: Introduce LAB cpufreq governor Jonghwa Lee
2013-04-01 8:24 ` [RFC PATCH 1/2] cpuidle: Add idle enter/exit time stamp for notifying current idle state Jonghwa Lee
2013-04-02 5:00 ` Daniel Lezcano
2013-04-02 6:17 ` jonghwa3.lee
2013-04-02 7:34 ` Daniel Lezcano
2013-04-02 9:37 ` jonghwa3.lee
2013-04-02 10:08 ` Daniel Lezcano
2013-04-02 11:07 ` jonghwa3.lee
2013-04-02 11:18 ` Daniel Lezcano
2013-04-03 6:10 ` jonghwa3.lee
2013-04-01 8:24 ` [RFC PATCH 2/2] cpufreq: Introduce new cpufreq governor, LAB(Legacy Application Boost) Jonghwa Lee
2013-04-01 15:37 ` [RFC PATCH 0/2] cpufreq: Introduce LAB cpufreq governor Viresh Kumar
2013-04-09 10:37 ` Lukasz Majewski
2013-04-09 12:02 ` Viresh Kumar
2013-04-09 16:44 ` Lukasz Majewski [this message]
[not found] ` <CAKfTPtD6MK9ogq7mOijSxLSsH0n65Xra48XfRSB3DFs35GT=2g@mail.gmail.com>
2013-04-10 6:56 ` Vincent Guittot
2013-04-10 8:44 ` Lukasz Majewski
2013-04-10 9:13 ` Vincent Guittot
2013-04-10 9:38 ` Lukasz Majewski
2013-04-10 10:45 ` Vincent Guittot
2013-04-23 7:28 ` Lukasz Majewski
2013-04-23 7:53 ` Vincent Guittot
2013-04-10 10:14 ` Lorenzo Pieralisi
2013-04-09 12:25 ` jonghwa3.lee
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=20130409184440.4cd87c1b@amdc308.digital.local \
--to=l.majewski@samsung.com \
--cc=cpufreq@vger.kernel.org \
--cc=cw00.choi@samsung.com \
--cc=jonghwa3.lee@samsung.com \
--cc=kyungmin.park@samsung.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pm@vger.kernel.org \
--cc=m.szyprowski@samsung.com \
--cc=myungjoo.ham@samsung.com \
--cc=rjw@sisk.pl \
--cc=sw0312.kim@samsung.com \
--cc=vincent.guittot@linaro.org \
--cc=viresh.kumar@linaro.org \
/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 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).