New issue
Advanced search Search tips

Issue 832162 link

Starred by 1 user

Issue metadata

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



Sign in to add a comment

Enhance UKM to understand source of IPCs in background

Project Member Reported by panicker@chromium.org, Apr 12 2018

Issue description

IPC task queue is doing a ton of work (38+%) in background after 5mins.
We agreed to annotate metadata for IPCs in UKM to understand what are the sources queueing this up.

altimin - are you the right owner for this?

 
Project Member

Comment 1 by bugdroid1@chromium.org, May 31 2018

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

commit 78445d3e14d4f8ae1b60f0e7f54f41a2c359eae6
Author: Eric Seckler <eseckler@chromium.org>
Date: Thu May 31 08:20:45 2018

blink scheduler: Add TimeSinceBackgrounded to RendererSchedulerTask UKM.

This was previously approved at [1] by msramek@.

[1] https://docs.google.com/document/d/14c_oWM2B02_mp43bPLcUiN6-094djfnzpbu9SAwH5-4/edit?ts=5a54f254#

Bug: 832162
Change-Id: I8f0f29af540a2e6f817b5e0b470683ad68a581b2
Reviewed-on: https://chromium-review.googlesource.com/1079068
Reviewed-by: Alexander Timin <altimin@chromium.org>
Reviewed-by: Robert Kaplow <rkaplow@chromium.org>
Commit-Queue: Eric Seckler <eseckler@chromium.org>
Cr-Commit-Position: refs/heads/master@{#563181}
[modify] https://crrev.com/78445d3e14d4f8ae1b60f0e7f54f41a2c359eae6/third_party/blink/renderer/platform/scheduler/main_thread/main_thread_scheduler_impl.cc
[modify] https://crrev.com/78445d3e14d4f8ae1b60f0e7f54f41a2c359eae6/tools/metrics/ukm/ukm.xml

Cc: altimin@chromium.org
Owner: eseckler@chromium.org
Status: Started (was: Assigned)
Labels: -Type-Bug Type-Feature
Status: Assigned (was: Started)
There's a (privacy-approved) proposal for adding symbols identifying tasks and IPCs here: https://docs.google.com/document/d/1i-zEWcA9DlBTmFmvvyKa2dgyDh4euA0-HZpOSoNtIm4/edit

But its implementation is on the back-burner for now, as the background IPC task load has improved AFAIU.

Comment 6 by eseckler@chromium.org, Jan 21 (2 days ago)

Cc: eseckler@chromium.org
Owner: ----
Status: Available (was: Assigned)

Sign in to add a comment