Closed Bug 927355 Opened 9 years ago Closed 8 years ago

Print fractional milliseconds in event tracer samples

Categories

(Toolkit :: General, defect)

Product:
Toolkit
Component:
General
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla29

People

(Reporter: ted, Assigned: ted)

References

Details

(Whiteboard: [qa-])

Attachments

(1 file)

Print fractional milliseconds in event tracer samples
3.12 KB, patch
benjamin
: review+
Details | Diff | Splinter Review 
In bug 904642 mbrubeck noted that we're rounding our Talos responsiveness results, resulting in aliasing in the final numbers. Turns out we're rounding them at the sample level as well, so we need to fix that to fix the whole problem.

The other printf changes in this file are just to silence compiler warnings on my Linux64 build.
Blocks: 904642
Attachment #817776 - Flags: review?(benjamin) → review+
I forgot about this while waiting for the Talos changes to go live:
https://hg.mozilla.org/integration/mozilla-inbound/rev/4faf6493c9ea
https://hg.mozilla.org/mozilla-central/rev/4faf6493c9ea
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla29
Whiteboard: [qa-]
You need to log in before you can comment on or make changes to this bug.