Bug 1337073 - Rename IPC_SYNC_LATENCY_MS to IPC_LINEAR_SYNC_LATENCY_MS and make it linear. r?billm,r?francois
This will make it easier to interpret, as we'll have consistent ranges in the
histogram boxes.
MozReview-Commit-ID: LN3FQO6qeGD
--- a/ipc/glue/MessageChannel.cpp
+++ b/ipc/glue/MessageChannel.cpp
@@ -1275,17 +1275,17 @@ MessageChannel::Send(Message* aMsg, Mess
*aReply = Move(*reply);
if (aReply->size() >= kMinTelemetryMessageSize) {
Telemetry::Accumulate(Telemetry::IPC_REPLY_SIZE,
nsDependentCString(msgName), aReply->size());
}
if (latencyMs >= kMinTelemetrySyncIPCLatencyMs) {
- Telemetry::Accumulate(Telemetry::IPC_SYNC_LATENCY_MS,
+ Telemetry::Accumulate(Telemetry::IPC_LINEAR_SYNC_LATENCY_MS,
nsDependentCString(msgName), latencyMs);
}
return true;
}
bool
MessageChannel::Call(Message* aMsg, Message* aReply)
{
--- a/toolkit/components/telemetry/Histograms.json
+++ b/toolkit/components/telemetry/Histograms.json
@@ -10303,23 +10303,23 @@
"bug_numbers": [1264820],
"expires_in_version": "55",
"kind": "exponential",
"high": 8000000,
"n_buckets": 50,
"keyed": true,
"description": "Measures the size of IPC messages by message name"
},
- "IPC_SYNC_LATENCY_MS": {
+ "IPC_LINEAR_SYNC_LATENCY_MS": {
"alert_emails": ["cpearce@mozilla.com"],
"bug_numbers": [1333489],
"expires_in_version": "60",
- "kind": "exponential",
+ "kind": "linear",
"high": 2000,
- "n_buckets": 10,
+ "n_buckets": 20,
"keyed": true,
"description": "Measures the number of milliseconds we spend waiting for sync IPC messages to finish sending, keyed by message name. Note: only messages that wait for more than 500 microseconds are included in this probe."
},
"MESSAGE_MANAGER_MESSAGE_SIZE2": {
"alert_emails": ["wmccloskey@mozilla.com","amccreight@mozilla.com"],
"bug_numbers": [1260908],
"expires_in_version": "55",
"kind": "exponential",