Open
Bug 1357810
Opened 6 years ago
Updated 6 months ago
Investigate source of memory_pressure event in gecko, and whether to repurpose it
Categories
(Core :: JavaScript: GC, enhancement, P3)
Core
JavaScript: GC
Tracking
()
NEW
Performance Impact | low |
People
(Reporter: djvj, Unassigned)
References
(Blocks 1 open bug)
Details
(Keywords: perf, perf:responsiveness, triage-deferred)
Gecko reports a memory_pressure event which initiates GC. This was supposedly introduced due to requirements by B2G, and may not be relevant any more. Can we use a free memory metric and repurpose this event to make it relevant again?
Comment 1•6 years ago
|
||
I thought linux, windows, and osx all had this concept too?
Comment 2•6 years ago
|
||
I guess there's prior art in bug 664291.
Reporter | ||
Updated•6 years ago
|
Whiteboard: [qf]
Updated•6 years ago
|
Whiteboard: [qf] → [qf:investigate]
Comment 3•6 years ago
|
||
Found at least one place where it's fired: http://searchfox.org/mozilla-central/rev/7aa21f3b531ddee90a353215bd86e97d6974e25b/xpcom/base/nsMemoryImpl.cpp#115-149
Comment 4•6 years ago
|
||
(In reply to Mike Conley (:mconley) from comment #3) > Found at least one place where it's fired: > > http://searchfox.org/mozilla-central/rev/ > 7aa21f3b531ddee90a353215bd86e97d6974e25b/xpcom/base/nsMemoryImpl.cpp#115-149 I think that's only triggered by nsIMemory::heapMinimize, which is used in imglib [1] and NPAPI's NPN_memflush [2]. [1] http://searchfox.org/mozilla-central/rev/7aa21f3b531ddee90a353215bd86e97d6974e25b/image/ImageFactory.cpp#225 [2] http://searchfox.org/mozilla-central/rev/7aa21f3b531ddee90a353215bd86e97d6974e25b/dom/plugins/base/nsNPAPIPlugin.cpp#926
Updated•6 years ago
|
Keywords: triage-deferred
Priority: -- → P3
Updated•4 years ago
|
Whiteboard: [qf:investigate] → [qf:js:investigate]
Updated•1 year ago
|
Performance Impact: --- → ?
Whiteboard: [qf:js:investigate]
Comment 5•1 year ago
|
||
Setting keyword perf:responsiveness because this is about GC timing, and GC blocks the main thread.
Performance Impact: ? → P3
Keywords: perf:responsiveness
Updated•6 months ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•