linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Eduardo Valentin <eduardo.valentin@ti.com>
To: Amit Kucheria <amit.kucheria@verdurent.com>
Cc: Zoran Markovic <zoran.markovic@linaro.org>,
	Eduardo Valentin <eduardo.valentin@ti.com>,
	lkml <linux-kernel@vger.kernel.org>, <linux-doc@vger.kernel.org>,
	Linux PM list <linux-pm@vger.kernel.org>,
	Zhang Rui <rui.zhang@intel.com>, Rob Landley <rob@landley.net>,
	Amit Daniel Kachhap <amit.daniel@samsung.com>,
	Andrew Morton <akpm@linux-foundation.org>,
	Durgadoss R <durgadoss.r@intel.com>,
	Christian Daudt <bcm@fixthebug.org>,
	James King <james.king@linaro.org>
Subject: Re: [RFC PATCH] thermal: add generic cpu hotplug cooling device
Date: Mon, 16 Dec 2013 08:44:29 -0400	[thread overview]
Message-ID: <52AEF5AD.5000506@ti.com> (raw)
In-Reply-To: <CAHLCerMCjtXNDRceG2u6kuyTrjV=j6S3mUViVMQJyDAwp_6pgA@mail.gmail.com>

[-- Attachment #1: Type: text/plain, Size: 3674 bytes --]

Zoran,

On 16-12-2013 07:26, Amit Kucheria wrote:
> On Fri, Dec 13, 2013 at 5:31 AM, Zoran Markovic
> <zoran.markovic@linaro.org <mailto:zoran.markovic@linaro.org>> wrote:
> 
>     Hi Eduardo,
> 
>     > Yeah, I would like to see it. But what I was more interested in seeing
>     > is how long does it take to offline a CPU?
>     >
>     I profiled this over 70 shutdown/startup cycles of CPU1 on Capri-AP
>     (Cortex-A9x2) board and I get:
>     shutdown:  1445usec (average), 3159usec (maximum), 834usec (minimum)
>     startup:  707usec (average), 3159usec (maximum), 327usec (minimum)
> 
>     It's using a 32KHz clock so time resolution is ~30usec.
> 
>     Regards, Zoran

Thanks for the data points. Based only on the data above, numbers sounds
promising, from thermal perspective. Provided that  3.1ms is the maximum
 transition time, and you stated that cooling effectiveness is around
8C/s (?).

However, I still would like to challenge the data.

> 
> 
> What is the workload you're running besides the proprietary heater code?

Agreed with Amit here, can you please provide better description of your
testing environment? We know, based on your emails the following:
- Homogeneous dual core Cortex-A9 environment.
- They go up to 48C when fully loaded. Can you explain where is your
sensor location? Gradient to hotspot, etc? 48C at A9s or board temperature?
- Hotplug provides you cooling effectiveness  of ~8C/s.
- Shutdown / startup delay:
     shutdown:  1445usec (average), 3159usec (maximum), 834usec(minimum)
     startup:  707usec (average), 3159usec (maximum), 327usec(minimum)
  Can you please explain the work load here? Is it full cpuburn? both
CPUs 100% loaded?
  Might be interesting to have either plots or logs of these experiments.

There are two major points we need to be careful:

- This code looks promising on embedded dual core system. However, it
does not necessarily mean it works fine on, say server side. How about a
system with 8/16/32 cores? How about a more heterogeneous workload? Not
to talk about heterogeneous cores. I think in more complicated scenarios
the data you provided above might even change. The difference between
your minimum and maximum shutdown/startup times are quite considerable,
so I am assuming your variance is not negligible, imaging if we scale
this up, what happens?

- The other point is that this type of cooling device must be taken in
very sensible way. Shutting down circuitry may not be the best strategy
for thermal. In fact, if you think about it, given you have a workload
well balanced between, say, two cores, as same of your environment,
turning one off it means you need to deal the very same load in only one
CPU. In other words, turning of circuitry means, from thermal standpoint
that you are increasing you heat/area ratio. Sometimes, you actually
want to increase this ratio in order to properly cool down your system.

I am not saying I am against the cooling device, I am simply stating
that this needs to be taken with careful consideration. We need to
properly document this. And building and validating thermal policies on
top of this is even harder.

> Something similar to what Vincent did[1] when benchmarking hotplug would
> be nice to see. Due to the kthread work and other optimisations, we
> shouldn't see drastic increases in hotplug latency as the number of
> threads increase any more.
> 
> Regards,
> Amit
> 
> [1] https://wiki.linaro.org/WorkingGroups/PowerManagement/Archives/Hotplug
>  


-- 
You have got to be excited about what you are doing. (L. Lamport)

Eduardo Valentin


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 295 bytes --]

  parent reply	other threads:[~2013-12-16 12:44 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-09-20 22:15 [RFC PATCH] thermal: add generic cpu hotplug cooling device Zoran Markovic
2013-09-23 11:17 ` Hongbo Zhang
2013-10-04 22:52 ` Zoran Markovic
2013-11-27 21:56   ` Zoran Markovic
2013-11-29 14:08     ` Eduardo Valentin
2013-12-02 23:05       ` Zoran Markovic
2013-12-09 14:30         ` Eduardo Valentin
2013-12-13  0:01           ` Zoran Markovic
     [not found]             ` <CAHLCerMCjtXNDRceG2u6kuyTrjV=j6S3mUViVMQJyDAwp_6pgA@mail.gmail.com>
2013-12-16 12:44               ` Eduardo Valentin [this message]
2013-12-30 20:48                 ` Zoran Markovic
2014-02-01  0:21                   ` Zoran Markovic

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=52AEF5AD.5000506@ti.com \
    --to=eduardo.valentin@ti.com \
    --cc=akpm@linux-foundation.org \
    --cc=amit.daniel@samsung.com \
    --cc=amit.kucheria@verdurent.com \
    --cc=bcm@fixthebug.org \
    --cc=durgadoss.r@intel.com \
    --cc=james.king@linaro.org \
    --cc=linux-doc@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-pm@vger.kernel.org \
    --cc=rob@landley.net \
    --cc=rui.zhang@intel.com \
    --cc=zoran.markovic@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).