Closed Bug 1744582 Opened 4 years ago Closed 4 years ago

Intermittent GeckoProfiler.CPUUsage | Expected: (threadCPUDeltaZeroCount) > (0u), actual: 0 vs 0

Categories

(Core :: Gecko Profiler, defect, P5)

defect

Tracking

()

RESOLVED FIXED
97 Branch
Tracking Status
firefox97 --- fixed

People

(Reporter: intermittent-bug-filer, Assigned: mozbugz)

Details

(Keywords: intermittent-failure)

Attachments

(1 file)

Filed by: apavel [at] mozilla.com
Parsed log: https://treeherder.mozilla.org/logviewer?job_id=360274761&repo=mozilla-beta
Full log: https://firefox-ci-tc.services.mozilla.com/api/queue/v1/task/DSHWFh0SQ3KZIBMmgTq6KA/runs/0/artifacts/public/logs/live_backing.log


[task 2021-12-06T19:08:55.581Z] 19:08:55     INFO -  TEST-START | GeckoProfiler.CPUUsage
[task 2021-12-06T19:08:55.637Z] 19:08:55  WARNING -  TEST-UNEXPECTED-FAIL | GeckoProfiler.CPUUsage | Expected: (threadCPUDeltaZeroCount) > (0u), actual: 0 vs 0
[task 2021-12-06T19:08:55.637Z] 19:08:55     INFO -  There should be some zero-CPUs due to the idle loop body @ /builds/worker/checkouts/gecko/tools/profiler/tests/gtest/GeckoProfiler.cpp:4341
[task 2021-12-06T19:08:55.701Z] 19:08:55  WARNING -  TEST-UNEXPECTED-FAIL | GeckoProfiler.CPUUsage | test completed (time: 121ms)
[task 2021-12-06T19:08:55.701Z] 19:08:55     INFO -  TEST-START | GeckoProfiler.AllThreads

First, add the "Busy test" thread to the filters, so it actually get sampled!
Also check that each expected thread is present (this is how the above error was found).

And instead of expecting some zeroes/non-zeroes in idle/busy threads, which can unfortunately intermittently fail, just add up the CPU values for each thread, and we should at least expect that the busy thread will have done more work than the idle thread.

Assignee: nobody → gsquelart
Status: NEW → ASSIGNED
Pushed by gsquelart@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/a09f76aa6703 Rework GeckoProfiler.CPUUsage idle/busy expectations - r=canaltinova
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 97 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: