From: Maxime Ripard <maxime.ripard@bootlin.com>
To: Philipp Rossak <embed3d@gmail.com>
Cc: linux@armlinux.org.uk, wens@csie.org, davem@davemloft.net,
arnd@arndb.de, mchehab@kernel.org, clabbe.montjoie@gmail.com,
rask@formelder.dk, krzk@kernel.org, sean@mess.org,
linux-arm-kernel@lists.infradead.org,
linux-kernel@vger.kernel.org, linux-sunxi@googlegroups.com
Subject: Re: [RFC PATCH] ARM: configs: sunxi: Set ondemand govenor as default
Date: Mon, 19 Feb 2018 09:10:50 +0100 [thread overview]
Message-ID: <20180219081050.vvfhnbw4foved2fb@flea.lan> (raw)
In-Reply-To: <20180217142235.28541-1-embed3d@gmail.com>
[-- Attachment #1: Type: text/plain, Size: 1343 bytes --]
On Sat, Feb 17, 2018 at 03:22:35PM +0100, Philipp Rossak wrote:
> Right now the performance govenor is the default frequency govenor on
> sunxi devices. This causes some general problems.
> When the cpu is idle the cpu runs with its maximum frequency.
> This causes a higher cpu temperature in the idle state. When the cpu is
> now under load the cpu gets with that higher idle temperature now faster
> to its thermal limits.
> An other big problem of the performace govenor is the missing
> thermal throttling. Some tests with cpuburn resulted in a system crash
> when the soc reached its thermal limits since no thermal throttling
> occurred.
This won't change anything with cpuburn. While cpuburn will be
running, ondemand will increase the frequency of the cores to the
maximum frequency, putting yourself in the exact same situation.
The only difference is going to be when you're idle or have a rather
small CPU load. But then, you won't heat much in that case either.
> With this patch we set the default frequency govenor to ondemand mode
> and reduce the temperature when the cpu is idle and activate the thermal
> throtteling.
This patch doesn't activate the thermal throttling.
Maxime
--
Maxime Ripard, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
http://bootlin.com
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
next prev parent reply other threads:[~2018-02-19 8:10 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-02-17 14:22 [RFC PATCH] ARM: configs: sunxi: Set ondemand govenor as default Philipp Rossak
2018-02-19 8:10 ` Maxime Ripard [this message]
2018-02-26 21:29 ` Philipp Rossak
2018-02-27 7:12 ` Maxime Ripard
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=20180219081050.vvfhnbw4foved2fb@flea.lan \
--to=maxime.ripard@bootlin.com \
--cc=arnd@arndb.de \
--cc=clabbe.montjoie@gmail.com \
--cc=davem@davemloft.net \
--cc=embed3d@gmail.com \
--cc=krzk@kernel.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-sunxi@googlegroups.com \
--cc=linux@armlinux.org.uk \
--cc=mchehab@kernel.org \
--cc=rask@formelder.dk \
--cc=sean@mess.org \
--cc=wens@csie.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