Closed Bug 1855594 Opened 1 year ago Closed 1 year ago

Use different profiler settings for MotionMark

Categories

(Testing :: Raptor, enhancement, P1)

enhancement

Tracking

(firefox124 fixed)

RESOLVED FIXED
124 Branch
Tracking Status
firefox124 --- fixed

People

(Reporter: mstange, Assigned: canova)

References

(Blocks 1 open bug)

Details

(Whiteboard: [fxp])

Attachments

(1 file)

We automatically gather profiles from MotionMark in CI, e.g. on this job (corresponding profile)

Since this is a test that mostly exercises graphics code, we should change the profiler settings to match those of the "Graphics" preset.

The profile linked above does not have the threads I was looking for (CanvasWorkers in this case). It is also very large because of the included screenshots. Adjusting the features to match the Graphics preset would solve both problems.

Severity: -- → S2
Component: Performance → Raptor
Priority: -- → P2
Whiteboard: [fxp]

from a quick look, getting this implemented should help with Bug 1853434 slightly
does not look like gecko_profiler_* configs are properly parsed by the manifest parsing script when --extra-profiler-run is used

See Also: → 1853434
See Also: → 1870174

MotionMark benchmark exercises graphics code. That's why it's important to see
graphics related threads when we are looking at a MotionMark profile. This
patch changes the profiler features and threads to match the "Graphics" preset
that we have in the profiler popup:
https://searchfox.org/mozilla-central/rev/a2709c2f0e31b2ad1a7b6447117626a34063fe57/devtools/client/performance-new/shared/background.jsm.js#163-177

Assignee: nobody → canaltinova
Status: NEW → ASSIGNED
Priority: P2 → P1
Pushed by canaltinova@gmail.com: https://hg.mozilla.org/integration/autoland/rev/303334fc1459 Use "Graphics" profiler preset settings for MotionMark benchmarks r=kshampur,perftest-reviewers
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 124 Branch
See Also: → 1892834
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: