Closed Bug 1569564 Opened 6 years ago Closed 6 years ago

Remove original malloc memory accounting framework

Categories

(Core :: JavaScript: GC, task, P1)

task

Tracking

()

RESOLVED FIXED
mozilla70
Tracking Status
firefox70 --- fixed

People

(Reporter: jonco, Assigned: jonco)

References

Details

Attachments

(4 files)

Now that the more precise malloc tracking work has landed we should be able to remove the original framework.

First of all remove the runtime-wide malloc bytes tracking. There's only a few uses of this so it almost never triggers a collection, and most of these uses are tracked in other ways anyway.

Depends on D39730

This replaces the original JIT code memory counter with one that tracks the allocated JIT code precisely. This now uses a fixed threshold which was the original intention.

This also removes the INCREMENTAL_MALLOC_TRIGGER GC reason and all GCs triggered by malloc allocations use TOO_MUCH_MALLOC, so whether a GC is incremental is separated from the trigger reason.

Depends on D39733

Attachment #9081342 - Attachment description: Bug 1569564 - Remove runtimei-wide malloc bytes counter as all uses are tracked separately r=sfink? → Bug 1569564 - Remove runtime-wide malloc bytes counter as all uses are tracked separately r=sfink
Attachment #9081344 - Attachment description: Bug 1569564 - Remove the original per-zone malloc counter r=sfink? → Bug 1569564 - Remove the original per-zone malloc counter r=sfink
Attachment #9081345 - Attachment description: Bug 1569564 - Replace the JIT code counter using the new precise tracking infrastructure r=sfink? → Bug 1569564 - Replace the JIT code counter using the new precise tracking infrastructure r=sfink
Attachment #9081346 - Attachment description: Bug 1569564 - Remove the original malloc counter infrastructure r=sfink? → Bug 1569564 - Remove the original malloc counter infrastructure r=sfink
Pushed by jcoppeard@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/c0cf078b5497 Remove runtime-wide malloc bytes counter as all uses are tracked separately r=sfink https://hg.mozilla.org/integration/autoland/rev/bda9fb62a8eb Remove the original per-zone malloc counter r=sfink https://hg.mozilla.org/integration/autoland/rev/86ec01cf8dac Replace the JIT code counter using the new precise tracking infrastructure r=sfink https://hg.mozilla.org/integration/autoland/rev/694c0c845fd5 Remove the original malloc counter infrastructure r=sfink
Regressions: 1570896
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: