Closed Bug 1332819 Opened 7 years ago Closed 6 years ago

Enable multiprocessor support in Stopwatch.(h|cpp)

Categories

(Toolkit :: Performance Monitoring, defect)

Product:
Toolkit
Component:
Performance Monitoring
Version:
50 Branch
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED WONTFIX

People

(Reporter: emk, Unassigned)

Details

The code path is long untested and turn the tree orange when enabled. See bug 1325299 for details.
Flags: needinfo?(dteller)
Surprisingly, it works without problems under Linux.
Yes, fails only with Win7 VM. Even Win8 buildbot did not fail.
And I could not reproduce the failure locally (Win10).
Mmmh... By any chance is there a difference in number of bits between the Win7 and Win8+ VMs?

Otherwise, is there a way to selectively disable on Win7?
Flags: needinfo?(dteller) → needinfo?(VYV03354)
Win7 tests 32bit and Win8 tests 64bit in our CI.
Oh, I just realized that there simply is no meaningful code on Linux, so that's not a good check.
(In reply to David Teller [:Yoric] (please use "needinfo") from comment #4)
> Mmmh... By any chance is there a difference in number of bits between the
> Win7 and Win8+ VMs?
> 
> Otherwise, is there a way to selectively disable on Win7?

I changed the #if condition to disable the branch only on Win32. Let's see what's going on.
Flags: needinfo?(VYV03354)
Could it be that this bug is in the wrong component?
Component: JavaScript Engine: JIT → Performance Monitoring
Product: Core → Toolkit
about:performance is being redesigned; mass closing the bugs related to parts of the current about:performance page that we are not keeping.

Our goals with the redesign are to reduce the overhead caused by having the page opened, increase the reliability of the displayed information, and make the offered information actionable for most users. The back-end work is being tracked in bug 1419681.
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.