New issue
Advanced search Search tips

Issue 913814 link

Starred by 1 user
Status: WontFix
Owner:
alanlxl@google.com
Closed: Dec 12
Cc:
napper@chromium.org
amoylan@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

ML Service: Correct CPU metrics for multi-core

Project Member Reported by amoylan@chromium.org, Dec 11 
Looks like we have a bug here:
cs/chromeos_public/src/platform2/ml/metrics.cc?l=108

Where we should not divide by number of cores as GetCPUUsage is already doing that.

We need to confirm it really is already doing that then remove the divide.

Comment 1 by alanlxl@google.com, Dec 12

Status: WontFix (was: Untriaged)
It turns out that the GetCPUUsage returns a value in the range of 0 to 100 * numCPUCores. See the comments of this func: cs/chromeos_public/src/aosp/external/libchrome/base/process/process_metrics.h?l=169
That's to say, it's not a bug and we should keep the devide.
►

Sign in to add a comment