New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 676723 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Dec 2016
Cc:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 2
Type: Bug



Sign in to add a comment

chromeos-4.4: Non-SMP builds may fail due to missing 'capacity_curr_of'

Project Member Reported by groeck@chromium.org, Dec 22 2016

Issue description

Non-SMP builds with CPU_FREQ enabled fail to build as follows.

drivers/cpufreq/cpufreq.c: In function 'cpufreq_freq_transition_begin':
drivers/cpufreq/cpufreq.c:509:3: error: implicit declaration of function 'capacity_curr_of' [

Also see Android commit 6c7e03dde14e ("DEBUG: cpufreq: fix cpu_capacity tracing build for non-smp systems").

 

Comment 1 by groeck@chromium.org, Dec 22 2016

Description: Show this description

Comment 2 by groeck@chromium.org, Dec 22 2016

Status: Started (was: Assigned)
Project Member

Comment 3 by bugdroid1@chromium.org, Dec 23 2016

Labels: merge-merged-chromeos-4.4
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/kernel/+/c38e7b4c2dbce8e041ded9a8736945e5ed0a7510

commit c38e7b4c2dbce8e041ded9a8736945e5ed0a7510
Author: Amit Pundir <amit.pundir@linaro.org>
Date: Mon Aug 29 14:18:17 2016

ANDROID: DEBUG: cpufreq: fix cpu_capacity tracing build for non-smp systems

cpu curr capacity can only be traced for SMP systems. Non-SMP builds
will fail with:

drivers/cpufreq/cpufreq.c: In function cpufreq_freq_transition_begin:
drivers/cpufreq/cpufreq.c:438:22: error: implicit declaration of function capacity_curr_of [-Werror=implicit-function-declaration]
   trace_cpu_capacity(capacity_curr_of(cpu), cpu);
                      ^
BUG= chromium:676723 
TEST=Build with CPU_FREQ enabled and SMP disabled

Fixes: Change-Id: Icd0930d11068fcb7d2b6a9a48e7ed974904e1081
       ("DEBUG: sched,cpufreq: add cpu_capacity change tracepoint")
Signed-off-by: Amit Pundir <amit.pundir@linaro.org>
[jstultz: Cherry-picked from common/android-3.18]
Signed-off-by: John Stultz <john.stultz@linaro.org>
(cherry picked from commit 6c7e03dde14e5074134f47d9ec3dea7de24e4e58)
Signed-off-by: Guenter Roeck <groeck@chromium.org>

Change-Id: I68511af87351c731cd6596856fbb9ffb7185af2d
Reviewed-on: https://chromium-review.googlesource.com/423332
Commit-Ready: Guenter Roeck <groeck@chromium.org>
Tested-by: Guenter Roeck <groeck@chromium.org>
Reviewed-by: Sonny Rao <sonnyrao@chromium.org>

[modify] https://crrev.com/c38e7b4c2dbce8e041ded9a8736945e5ed0a7510/drivers/cpufreq/cpufreq.c

Comment 4 by groeck@chromium.org, Dec 23 2016

Status: Fixed (was: Started)

Sign in to add a comment