Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(123)

Issue 2943004: arm: enable laptop-mode-tools (Closed)

Created:
10 years, 5 months ago by piman
Modified:
9 years, 7 months ago
Reviewers:
Sameer Nanda
CC:
chromium-os-reviews_chromium.org, Mandeep Singh Baines, tedbo, awong, adlr, anush
Visibility:
Public.

Description

arm: enable laptop-mode-tools BUG=4631 TEST=build for arm, run on device

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -2 lines) Patch
M app-laptop/laptop-mode-tools/laptop-mode-tools-1.52.ebuild View 1 chunk +1 line, -1 line 0 comments Download
M app-laptop/laptop-mode-tools/laptop-mode-tools-9999.ebuild View 1 chunk +1 line, -1 line 0 comments Download
M chromeos-base/chromeos/chromeos-0.0.1.ebuild View 2 chunks +1 line, -1 line 0 comments Download
A + chromeos-base/chromeos/chromeos-0.0.1-r72.ebuild View 0 chunks +-1 lines, --1 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
piman
10 years, 5 months ago (2010-07-09 03:14:18 UTC) #1
Sameer Nanda
LGTM, although the effectiveness of laptop-mode-tools on ARM still needs to be proven.
10 years, 5 months ago (2010-07-09 16:37:23 UTC) #2
piman
On Fri, Jul 9, 2010 at 9:37 AM, <snanda@chromium.org> wrote: > LGTM, although the effectiveness ...
10 years, 5 months ago (2010-07-09 16:54:59 UTC) #3
Sameer Nanda
10 years, 5 months ago (2010-07-09 17:10:32 UTC) #4
On 2010/07/09 16:54:59, piman wrote:
> On Fri, Jul 9, 2010 at 9:37 AM, <mailto:snanda@chromium.org> wrote:
> 
> > LGTM, although the effectiveness of laptop-mode-tools on ARM still needs to
> > be
> > proven.
> 
> 
> I'm not sure why you're saying that. AFAIK laptop-mode-tools deals with
> things that are orthogonal to the processor architecture (e.g. disk I/O) so
> I'm not sure why it would be relevant for x86 and not ARM. In any case, the
> goal is to "unfork" ARM so that the same tools are available on both
> platforms.

Quite a few of the power management features enabled for x86 by
laptop-mode-tools such as SATA ALPM are not going to be relevant for ARMs. 
Similarly there are going to be power consumption related features that are
relevant to ARM but not x86 that would need to be addressed.

On x86 we measured power draw impact of these features to help decide whether to
enable these features or not. A similar tuning exercise would need to be done on
ARM before laptop-mode-tools improves power draw to the optimal extent.

> 
> >
> >
> > http://codereview.chromium.org/2943004/show
> >
>

Powered by Google App Engine
This is Rietveld 408576698