Closed Bug 1455896 Opened 7 years ago Closed 6 years ago

Fix the races in the available memory tracker

Categories

(Core :: Memory Allocator, enhancement)

Product:
Core
Component:
Memory Allocator
Type:
enhancement
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED INVALID

People

(Reporter: gsvelto, Unassigned)

References

(Blocks 1 open bug)

Details

+++ This bug was initially created as a clone of Bug #1451002 +++ The available memory tracker checks for memory use via DLL hooks. Since we can't use locks but we have to keep some state within the code suffers from at least two different races. Neither is severe as both can only cause extra memory-pressure events to be sent. Still, since the code predates the introduction of atomics and relies on the volatile for certain globals it would be best to clean it up and fix the races as well.
When will this be landed, and with what priority?
Component: General → Memory Allocator
Flags: needinfo?(gsvelto)
This is cleanup work so it's pretty low priority.
Flags: needinfo?(gsvelto)
The old available memory tracker has been removed in bug 1468207 and with it its races. Closing as INVALID.
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.