Note: There are a few cases of duplicates in user autocompletion which are being worked on.

Telemetry for time spent in between GCs

RESOLVED FIXED in mozilla13

Status

()

Core
JavaScript Engine
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: mccr8, Assigned: billm)

Tracking

Trunk
mozilla13
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [Snappy:P2])

Attachments

(1 attachment)

(Reporter)

Description

6 years ago
This could help us find scheduling regressions.
(Reporter)

Updated

6 years ago
Whiteboard: [Snappy]
(Reporter)

Comment 1

6 years ago
Bill said he'd do this as part of his incremental GC stuff, as he's already computing things like what % of the time is in the GC.
Assignee: general → wmccloskey
(Reporter)

Comment 2

6 years ago
And actually, that undersells what he is already computing a bit.  Here's his comment about what he is computing:

/*
 * MMU (minimum mutator utilization) is a measure of how much garbage collection
 * is affecting the responsiveness of the system. MMU measurements are given
 * with respect to a certain window size. If we report MMU(50ms) = 80%, then
 * that means that, for any 50ms window of time, at least 80% of the window is
 * devoted to the mutator. In other words, the GC is running for at most 20% of
 * the window, or 10ms. The GC can run multiple slices during the 50ms window
 * as long as the total time it spends is at most 10ms.
 */

It looks like he is currently computing MMU(20ms) and MMU(50ms).
(Assignee)

Comment 3

6 years ago
Created attachment 591675 [details] [diff] [review]
patch

Here's a patch on top of the incremental GC stuff. I only added MMU(50ms). Our MMU for smaller window sizes is pretty bad even in testing, so it's probably not worth reporting right now.
Attachment #591675 - Flags: review?(continuation)
(Reporter)

Comment 4

6 years ago
Comment on attachment 591675 [details] [diff] [review]
patch

Review of attachment 591675 [details] [diff] [review]:
-----------------------------------------------------------------

Thanks!
Attachment #591675 - Flags: review?(continuation) → review+
(Reporter)

Updated

6 years ago
Whiteboard: [Snappy] → [Snappy:P2]
This was r+ 3 weeks ago. Who can land these new probes?
(Assignee)

Comment 6

6 years ago
The patch is based on incremental GC, so it can't land yet. However, I realize that besides the MMU measurement, it would be nice just to know how many seconds ago the last GC was. So maybe I'll do a separate patch for that. It could land right away.
(Reporter)

Comment 7

6 years ago
Looks like this landed with incremental GC, so I'm marking as fixed.
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Depends on: 641025
Resolution: --- → FIXED
The target milestone isn't set. Did this land in time for 12 or 13?
(Reporter)

Updated

6 years ago
Target Milestone: --- → mozilla13
You need to log in before you can comment on or make changes to this bug.