The default bug view has changed. See this FAQ.

Workers: Should respect JS JIT pref

RESOLVED WORKSFORME

Status

()

Core
DOM
RESOLVED WORKSFORME
7 years ago
6 years ago

People

(Reporter: cjones, Unassigned)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Mr. bent tells me that currently they don't.
Created attachment 414192 [details] [diff] [review]
Make Workers listen for changes to jit.content

This patch was pretty easy ... perhaps too easy.  I hope I didn't violate any threadsafety rules, please check carefully!

I tested it by flipping the pref during long worker computations, and it didn't seem to break anything.  (And wow, the JIT makes a huge difference for bz's mandelbrot demo.)  Interestingly enough, the JIT-enabledness state change seemed to be applied nearly instantaneously while the workers were still executing.  Kudos to the spidermonkey guys for getting that to work.

One issue is that this patch sets JIT-enabledness for all Worker JSContexts based on jit.content; it doesn't attempt to determine whether chrome or content JS is executing on the JSContext, and so ignores jit.chrome.  Is that going to be an issue?  Does browser.js use workers at all?
Assignee: nobody → jones.chris.g
Attachment #414192 - Flags: review?(bent.mozilla)

Comment 2

6 years ago
This would be very useful to have (could have used it in bug 653967 right now, for example).

Did this get forgotten, or have we given up on it?
This was a four-day project and I just ran out of time.  I'd still like to take this patch if it wasn't fixed in the meantime.
Assignee: jones.chris.g → nobody

Comment 4

6 years ago
This hasn't been fixed in the meantime, and would be useful to have I think.
We need this to enable TI for web workers. We can't hard code it since TI will be disabled by default when it lands on TM.

Chris, still interested in finishing this patch?
Blocks: 619433
All this code is gone soon, we'll need a different approach as soon as bug 649537 lands.
Will be fixed in bug 649537.
Attachment #414192 - Flags: review?(bent.mozilla) → review-
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.