The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/platform2/+/b46c237acf466ec2de810d8c4e9a4522f4bda47c

commit b46c237acf466ec2de810d8c4e9a4522f4bda47c
Author: Andrew Moylan <amoylan@chromium.org>
Date: Tue Sep 11 17:02:12 2018

metrics: Add Max op to CumulativeMetrics

This operation can be used to keep running track of the peak value of
some quantity over the reporting timescale.

BUG= chromium:881186 
TEST=Updated existing unit tests to include Max

Change-Id: I0724a52744c49f73ae843639cb1d5cc9aa7bff0a
Reviewed-on: https://chromium-review.googlesource.com/1209021
Commit-Ready: Andrew Moylan <amoylan@chromium.org>
Tested-by: Andrew Moylan <amoylan@chromium.org>
Reviewed-by: Luigi Semenzato <semenzato@chromium.org>

[modify] https://crrev.com/b46c237acf466ec2de810d8c4e9a4522f4bda47c/metrics/cumulative_metrics_test.cc
[modify] https://crrev.com/b46c237acf466ec2de810d8c4e9a4522f4bda47c/metrics/cumulative_metrics.h
[modify] https://crrev.com/b46c237acf466ec2de810d8c4e9a4522f4bda47c/metrics/cumulative_metrics.cc
[modify] https://crrev.com/b46c237acf466ec2de810d8c4e9a4522f4bda47c/metrics/persistent_integer_test.cc
[modify] https://crrev.com/b46c237acf466ec2de810d8c4e9a4522f4bda47c/metrics/persistent_integer.h
[modify] https://crrev.com/b46c237acf466ec2de810d8c4e9a4522f4bda47c/metrics/persistent_integer.cc

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/chromiumos-overlay/+/be34c8662f78e5076eef3cc42b725f60fe0bab79

commit be34c8662f78e5076eef3cc42b725f60fe0bab79
Author: Andrew Moylan <amoylan@chromium.org>
Date: Thu Sep 13 08:23:09 2018

ml: Add metrics dependency

Also stop using out-of-tree build which causes the wrong version of
platform2/ includes to be found

TEST=Check that setup_board --force then build_packages works
BUG= chromium:881186 

Change-Id: Ia53ba211cfd28690bfa2b53259a5193206ba613a
Reviewed-on: https://chromium-review.googlesource.com/1215502
Commit-Ready: Andrew Moylan <amoylan@chromium.org>
Tested-by: Andrew Moylan <amoylan@chromium.org>
Reviewed-by: Chirantan Ekbote <chirantan@chromium.org>

[modify] https://crrev.com/be34c8662f78e5076eef3cc42b725f60fe0bab79/chromeos-base/ml/ml-9999.ebuild

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/platform2/+/40ee4fc920ffda053130717750fccff69b2b8dab

commit 40ee4fc920ffda053130717750fccff69b2b8dab
Author: Andrew Moylan <amoylan@chromium.org>
Date: Thu Sep 20 12:15:19 2018

ml: Collect basic metrics

This CL adds some basic metrics collection ML Service. Future CLs will
expand metrics to cover events and resource-usage of the
TensorFlow-related operations (model loading, inference).

Metrics added:
- D-Bus -> Mojo bootstrap event
- Memory usage and peak memory usage per day
- CPU usage

Seccomp policy additions are for ProcessMetrics, UMA metrics, and
cumulative metrics backing storage (pread64/pwrite64).

CQ-DEPEND=CL:1215502
BUG= chromium:881186 
TEST=manual (checked chrome://histograms)

Change-Id: If6791a4ee0b0ed0e3cb7f57d8e59eeadd1d2c6ea
Reviewed-on: https://chromium-review.googlesource.com/1209022
Commit-Ready: Andrew Moylan <amoylan@chromium.org>
Tested-by: Andrew Moylan <amoylan@chromium.org>
Reviewed-by: Andrew Moylan <amoylan@chromium.org>

[modify] https://crrev.com/40ee4fc920ffda053130717750fccff69b2b8dab/ml/README.md
[modify] https://crrev.com/40ee4fc920ffda053130717750fccff69b2b8dab/ml/seccomp/ml_service-seccomp-arm.policy
[add] https://crrev.com/40ee4fc920ffda053130717750fccff69b2b8dab/ml/metrics.h
[modify] https://crrev.com/40ee4fc920ffda053130717750fccff69b2b8dab/ml/daemon.h
[modify] https://crrev.com/40ee4fc920ffda053130717750fccff69b2b8dab/ml/seccomp/ml_service-seccomp-amd64.policy
[modify] https://crrev.com/40ee4fc920ffda053130717750fccff69b2b8dab/ml/init/ml-service.conf
[add] https://crrev.com/40ee4fc920ffda053130717750fccff69b2b8dab/ml/metrics.cc
[modify] https://crrev.com/40ee4fc920ffda053130717750fccff69b2b8dab/ml/ml.gyp
[modify] https://crrev.com/40ee4fc920ffda053130717750fccff69b2b8dab/ml/daemon.cc

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/03140b03f0821388708c52c1fc98d3f2091886f1

commit 03140b03f0821388708c52c1fc98d3f2091886f1
Author: Andrew Moylan <amoylan@chromium.org>
Date: Mon Sep 24 01:17:01 2018

Add UMA histograms for Chrome OS ML Service

Add histograms for the following metrics added in Chrome OS CL
crrev.com/c/1209022.
* Numerical histograms for CPU and RAM usage of ML Service
* Enum histogram for Mojo connection/disconnection events

Bug:  881186 
Change-Id: Id649c3bce16bf6ca65871feee1ab741372530669
Reviewed-on: https://chromium-review.googlesource.com/1237017
Reviewed-by: Jesse Doherty <jwd@chromium.org>
Commit-Queue: Andrew Moylan <amoylan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#593465}
[modify] https://crrev.com/03140b03f0821388708c52c1fc98d3f2091886f1/tools/metrics/histograms/enums.xml
[modify] https://crrev.com/03140b03f0821388708c52c1fc98d3f2091886f1/tools/metrics/histograms/histograms.xml

Remaining metrics needed are:
- For model load
- For graph executor creation
- For inference

For each of the above, record:
Events enum: requested, success, failures
Resource cost: cpu time
Resource cost: maybe delta of memory usage

Split off the remaining work into crbug.com/892999.