Closed Bug 720045 Opened 12 years ago Closed 10 years ago

simplify conservative stack scanner with single-threaded runtime


(Core :: JavaScript Engine, defect)

Not set





(Reporter: igor, Assigned: igor)




(1 file)

+++ This bug was initially created as a clone of Bug #675078 +++

With 675078 fixed we no longer need to record the native stack boundaries in the runtime. As the GC only need to deal with one thread and that thread is current one, we can always query for the native stack bounds directly during the GC.
The patch in bug 675078 does this.
(In reply to Luke Wagner [:luke] from comment #1)
> The patch in bug 675078 does this.

No, it is not - the patch still stores the base of the native thread in JSRuntime. It is not necessary.
Blocks: 722345
Attached patch v1Splinter Review
The patch always scans the native stack unless it is explicitly disabled via AutoDisableConservativeStackScan class. The latter is a temporary measure to keep the current logic in nsXPConnect::Collect. For now I want to keep this to make sure that in the current form the patch does not increase orange factor due to false positives roots.

To minimize the recorded stack depth the patch requires that any code that needs to scan the stack uses the callback to implements its logic. This way RecordNativeStackTopForGC can use the shallowest stack point when recording the stack. Again, this complication can be reverted if it does not cause extra orange, but for now the main goal is to decouple the conservative stack scanner from the JS request infrastructure.
Assignee: general → igor
Attachment #592878 - Flags: review?(anygregor)
Comment on attachment 592878 [details] [diff] [review]

Try server shows a leak with the patch
Attachment #592878 - Flags: review?(anygregor)
I suspect the conservative stack scanner is no longer of much interest at this point.
Generational GC (bug 619558) will obsolete the conservative stack scanner.
Closed: 10 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.