All users were logged out of Bugzilla on October 13th, 2018

Clean up descriptions in TelemetryHistograms.h

RESOLVED FIXED

Status

()

RESOLVED FIXED
7 years ago
6 years ago

People

(Reporter: justin.lebar+bug, Unassigned)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Reporter)

Description

7 years ago
TelemetryHistograms.h:

HISTOGRAM(MEMORY_JS_GC_HEAP, 1024, 512 * 1024, 10, EXPONENTIAL, "Memory(MB) used by the JavaScript GC")
HISTOGRAM(MEMORY_RESIDENT, 32 * 1024, 1024 * 1024, 10, EXPONENTIAL, "Resident memory(MB) reported by OS")
HISTOGRAM(MEMORY_LAYOUT_ALL, 1024, 64 * 1024, 10, EXPONENTIAL, "Memory(MB) reported used by layout")

But TelemetryPing.js:

name = "Memory:" + mr.path + " + (KB)";
val = Math.floor(mr.amount / 1024);
(Reporter)

Updated

7 years ago
Attachment #541442 - Flags: review?(tglek)
(Reporter)

Comment 2

7 years ago
Now that I have a closer look at the file, I think many of these descriptions need to be cleaned up (and my patch doesn't follow the existing convention).  I'll try again in a moment.
Summary: Update TelemetryHistograms.h to indicate that memory is reported in KB, not MB → Clean up descriptions in TelemetryHistograms.h
(Reporter)

Updated

7 years ago
Attachment #541442 - Attachment is obsolete: true
Attachment #541442 - Flags: review?(tglek)
(Reporter)

Updated

7 years ago
Attachment #541444 - Flags: review?(tglek)

Updated

7 years ago
Depends on: 664486

Comment 4

7 years ago
Comment on attachment 541444 [details] [diff] [review]
Patch v2

Please also get rid of 
let name = "Memory:" + mr.path + " (KB)";
 and just use mr.name as a key in _histograms
Attachment #541444 - Flags: review?(tglek) → review+
> HISTOGRAM(HARD_PAGE_FAULTS, 1024, 1024 * 1024, 10, EXPONENTIAL, "Hard page faults (cumulative)")

where does that come from?

Comment 6

7 years ago
(In reply to comment #5)
> > HISTOGRAM(HARD_PAGE_FAULTS, 1024, 1024 * 1024, 10, EXPONENTIAL, "Hard page faults (cumulative)")
> 
> where does that come from?

bug 664486
http://hg.mozilla.org/mozilla-central/rev/a6cdfd70d6a2
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
Whiteboard: [inbound]

Updated

6 years ago
Depends on: 666666

Updated

6 years ago
No longer depends on: 666666
You need to log in before you can comment on or make changes to this bug.