Closed Bug 538440 Opened 14 years ago Closed 14 years ago

We never jit DOM workers

Categories

(Core :: JavaScript Engine, defect)

x86
macOS
defect
Not set
normal

Tracking

()

VERIFIED FIXED
Tracking Status
status1.9.2 --- .2-fixed

People

(Reporter: bzbarsky, Assigned: jorendorff)

References

Details

(Keywords: relnote, verified1.9.2, Whiteboard: [3.6.x][rc-ridealong][fixed-in-tracemonkey])

Attachments

(1 file)

Bug 533481 made it so workers use separate debugger hooks so we don't call into jsd.  Bug 519719 made it so that we disable the jit on contexts that don't have the runtime's debug hooks.  Result: no jit in workers.
Flags: blocking1.9.2?
Not a release blocker, but we'll relnote it for sure and take it if we do a RC2 or otherwise in 3.6.1, so let's get the fix together.
Flags: blocking1.9.2? → blocking1.9.2-
Keywords: relnote
Whiteboard: [3.6.x][rc-ridealong]
Attached patch v1Splinter Review
Assignee: general → jorendorff
That looks like it should do the trick, yes.  Want testing or reviews or anything?
Yes, a test is needed for this.
Comment on attachment 420832 [details] [diff] [review]
v1

Nobody can think of a good way to test this. Let's get it in, I guess. :-P
Attachment #420832 - Flags: review?(mrbkap)
Attachment #420832 - Flags: review?(mrbkap) → review+
Add a js_OnTrace friend API, add a method to DOMWindowUtils that gets called in a loop to check that JITting happens?  Hackish, but adequate.
So this didn't make to rc2 after all?
http://hg.mozilla.org/tracemonkey/rev/502c0b415e10
Whiteboard: [3.6.x][rc-ridealong] → [3.6.x][rc-ridealong][fixed-in-tracemonkey]
http://hg.mozilla.org/mozilla-central/rev/502c0b415e10
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Comment on attachment 420832 [details] [diff] [review]
v1

a1922=beltzner
Attachment #420832 - Flags: approval1.9.2.2? → approval1.9.2.2+
Is there an easy way for QA to verify this for 1.9.2 and Trunk?
You could run some script with loops in a worker and see how jitstats are affected (or not) in a debug build....  Or just measure the performance of that script with jit on and off and see whether there's an effect.
my addon that uses worker threads runs over 2x faster in 3.6.2 than 3.6 (non-debug), so I can only assume they're being jited.
Verified! :)
Status: RESOLVED → VERIFIED
I agree at this point.
Keywords: verified1.9.2
so I guess there were other things that could have sped up workers in 3.6.2, but yeah, trying with jit on/off on the same build has the same speedup.
Hi harthur,

Just out of curiosity, which addon is it?
https://addons.mozilla.org/en-US/firefox/addon/14328, the thing that uses web workers is the 'website analyzer'.
Hi,

The issue still exist in the current release 3.6.10. Javascript engine can't capture the user change efficiently. 

Only version 3.5.9 firefox is running fine but all above versions have the same issue.

Zafar
Here is the case:

If user remove the character or characters from text field and then enter character and save them (repeat quickly several time), it won't capture the last change.

Zafar
Zafar, that sounds completely unrelated to this bug.  Please file a separate bug on your problem.
You need to log in before you can comment on or make changes to this bug.