Closed Bug 1342384 Opened 7 years ago Closed 7 years ago

Regression on misc-basic-array-forof due to TraceLogger changes for multithreaded runtimes

Categories

(Core :: JavaScript Engine: JIT, defect, P1)

Product:
Core
Component:
JavaScript Engine: JIT
Type:
defect

Tracking

()

Status:
RESOLVED WONTFIX

People

(Reporter: h4writer, Unassigned)

References

(Blocks 1 open bug)

Details

See bug 1342369. This is about the third regression. Locally it changed score from 45 to 47 (ms):

https://arewefastyet.com/#machine=29&view=single&suite=misc&subtest=basic-array-forof&start=1486531747&end=1486587463

The first regression visible on that graph is (click on jsc in the legend to not see that line):
Bug 1334194 - TraceLogger changes for multithreaded runtimes, r=h4writer.
Blocks: 1342369
Depends on: 1334194
Flags: needinfo?(bhackett1024)
The second regression visible on that graph is:
Brian Hackett <bhackett1024@gmail.com> - Wed, 08 Feb 2017 05:23:32 -0700 - rev 341355
Bug 1334212 - Handle multiple contexts per runtime in compiled JIT code, r=h4writer.
Bug 1334212 shouldn't affect behavior when JS_TRACE_LOGGING is not defined.
Flags: needinfo?(bhackett1024)
(In reply to Brian Hackett (:bhackett) from comment #2)
> Bug 1334212 shouldn't affect behavior when JS_TRACE_LOGGING is not defined.

But I think JS_TRACE_LOGGING is #defined by default nowadays...
Priority: -- → P1
(In reply to Hannes Verschore [:h4writer] from comment #0)
> See bug 1342369. This is about the third regression. Locally it changed
> score from 45 to 47 (ms):
> 
> https://arewefastyet.com/#machine=29&view=single&suite=misc&subtest=basic-
> array-forof&start=1486531747&end=1486587463
> 
> The first regression visible on that graph is (click on jsc in the legend to
> not see that line):
> Bug 1334194 - TraceLogger changes for multithreaded runtimes, r=h4writer.

This will be solved by bug 1345148. The locking was taking some overhead. Bug 1345148 makes sure we need no locking when tracelogger is not active.

(In reply to Hannes Verschore [:h4writer] from comment #1)
> The second regression visible on that graph is:
> Brian Hackett <bhackett1024@gmail.com> - Wed, 08 Feb 2017 05:23:32 -0700 -
> rev 341355
> Bug 1334212 - Handle multiple contexts per runtime in compiled JIT code,
> r=h4writer.

https://hg.mozilla.org/integration/mozilla-inbound/rev/85849dc84129

Seems like this is caused by the need to load values instead of comparing values out of memory immediately.
That part added a lot of these. I don't think we can actually do something about it though.
I'll mark it as WONTFIX
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.