1118170 - Simplify GC after removing PJS

Status: RESOLVED FIXED

(Core :: JavaScript: GC, defect)

Description

[Lars T Hansen [:lth]]

Not yet sure what goes into this work, but at least these:

* ThreadSafeContext -> ExclusiveContext
* No more ForkJoinContext, Shu may have touched some bits here already
* No more isThreadLocal() testing
* Probably some things in the GC triggering machinery (callbacks to requestGC
  and requestZoneGC on the ForkJoinContext)

Also not sure how to stage this, but will probably put it on top of some of Shu's changes.

Comment 1

[Lars T Hansen [:lth]]

Attachment: WIP

Probably roughly right but there are some outstanding issues.

(This patch contains a bunch of ugly hacks to files in vm/ that are there just to make things compile w/o ThreadSafeContext, while we're waiting for the removal of that.  Pls ignore.)

Comment 2

[Lars T Hansen [:lth]]

Attachment: Remove the PJS bits in the GC

The thing that makes me most nervous here is that I've introduced an invariant that there are no empty arenas in the arena lists, since that was a facility added for PJS (IIUC).  I've changed code two places to handle that.

Other than that, adoptArenas remains but I have removed some comments that appeared to tie it to PJS.

Comment 3

[Lars T Hansen [:lth]]

Oh, btw, that patch sits on top of Shu's patch queue (bug 1118084 followed by bug 1118107, with some adjustments necessary).

Comment 4

[Terrence Cole [:terrence]]

Comment on attachment 8545917 [details] [diff] [review]
Remove the PJS bits in the GC

Review of attachment 8545917 [details] [diff] [review]:
-----------------------------------------------------------------

Great!

::: js/src/gc/Zone.h
@@ +29,1 @@
>  class Allocator

I believe that the entire Allocator thing was PJS, so we could probably rip even further.

Comment 5

[Lars T Hansen [:lth]]

(In reply to Terrence Cole [:terrence] from comment #4)
> Comment on attachment 8545917 [details] [diff] [review]
> Remove the PJS bits in the GC
> 
> Review of attachment 8545917 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> Great!
> 
> ::: js/src/gc/Zone.h
> @@ +29,1 @@
> >  class Allocator
> 
> I believe that the entire Allocator thing was PJS, so we could probably rip
> even further.

I was wondering about that.  Will look into doing so, probably as a follow-up bug.

Comment 6

[Lars T Hansen [:lth]]

Try: https://treeherder.mozilla.org/ui/#/jobs?repo=try&revision=14e3a123fc5d

Comment 7

[Lars T Hansen [:lth]]

Try and try again: https://treeherder.mozilla.org/ui/#/jobs?repo=try&revision=756a70c50990

Comment 8

[Lars T Hansen [:lth]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/84d9dd565293

Comment 9

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/84d9dd565293