New issue
Advanced search Search tips

Issue 726868 link

Starred by 2 users

Issue metadata

Status: Assigned
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Feature



Sign in to add a comment

gru: should we use ddrfreq as a passive cooling device?

Project Member Reported by diand...@chromium.org, May 26 2017

Issue description

In veyron, we used DDRFreq as a passive cooling device.  It was found that lowering the CPU speed without also lowering the DDRFreq ended up just wasting power.  It was much more effective to lower both together.

It seems like we should do the same on gru based devices, like bob and kevin.

On gru-based devices, this should actually be much easier than it was on veyron.  On veyron we got into trouble when we tried to scale DDRFreq when the CPU was too slow.  Presumably the more advanced DDRFreq architecture on gru should mean we don't need to worry so much.

---

I'd be kinda curious if they helps on any of our benchmarks.  Do PageCycler and/or powerLoadTest end up thermally constrained?  If so, this might be a nice help.
 

Comment 1 by derat@chromium.org, May 26 2017

Cc: coconutruben@chromium.org
Indeed, I have a look at the thermal of Bob in recently.

And run the scripts to check, the dmc always run 928M when we turn on the targer temperature.

/sys/class/thermal/thermal_zone[0-1]/temp /sys/devices/sy
stem/cpu/cpu[0-5]/cpufreq/scaling_cur_freq /sys/devices/platform/ff9a0000.gpu/devfreq/ff9a0000.gpu/cur_freq /sys/class/devfreq/dmc/cur_freq; date; sleep .5; done 


@wxt: do you want to try adding some patches for this?  I don't remember it being very difficult to make it a cooling device.  Trying to figure out how to integrate it with IPA might be harder, though.
Okay, I try to do this week and will update something here. 
Copy from the Doug's solution on mickey of rk3288. Posted CL as following.
      
https://chromium-review.googlesource.com/521003 WIP: CHROMIUM: dmcfreq: Allow the DMC to be used as a passive cooling device 
https://chromium-review.googlesource.com/521004 WIP: arm64: dts: rockchip: add the dmc thermal for rk3399

It seem works, but no well now.



Comment 6 by tbroch@chromium.org, Aug 18 2017

Status: Assigned (was: Untriaged)

Sign in to add a comment