Closed
Bug 551728
Opened 15 years ago
Closed 15 years ago
Update MMgc documentation with blacklist debugging info
Categories
(Tamarin Graveyard :: Garbage Collection (mmGC), defect, P3)
Tamarin Graveyard
Garbage Collection (mmGC)
Tracking
(Not tracked)
RESOLVED
FIXED
Q3 11 - Serrano
People
(Reporter: mike, Assigned: treilly)
References
Details
Attachments
(1 file)
2.83 KB,
patch
|
stejohns
:
review+
lhansen
:
superreview+
|
Details | Diff | Splinter Review |
https://developer.mozilla.org/en/MMgc needs some updates: We need to add description of blacklist debugging technique (GC::addToBlacklist() etc.)
I tried the GC::WhosPointingAtMe() function that is described in https://developer.mozilla.org/en/MMgc#Allocation_traces.2c_deletion_traces_etc. , but it caused an assertion. Tommy says it may not work any more, and that I should use the newer blacklist stuff; but it is not documented on the wiki or in the code.
Reporter | ||
Comment 1•15 years ago
|
||
This changeset describes what I think it needs to say:
changeset: 3039:30e3807a2100
user: Tommy Reilly <treilly@adobe.com>
date: Tue Nov 10 10:39:46 2009 -0500
files: MMgc/FixedAlloc.cpp MMgc/FixedAlloc.h MMgc/GC.cpp MMgc/GC.h MMgc/GCHeap.cpp MMgc/GCHeap
description:
New blacklist facility. Use GC::addToBlacklist to put something on
the list and then when a Sweep happens we print out the back pointer
chain if the thing is still around. (r=lhansen)
Assignee | ||
Updated•15 years ago
|
Assignee: nobody → treilly
Assignee | ||
Updated•15 years ago
|
Status: NEW → ASSIGNED
Reporter | ||
Comment 2•15 years ago
|
||
Doing that works in that it prints out a series of hex addresses, but then I'm not sure what to do with those hex addresses.
Flags: flashplayer-qrb+
Priority: -- → P3
Target Milestone: --- → flash10.2
Updated•15 years ago
|
Priority: P3 → --
Target Milestone: flash10.2 → Future
Assignee | ||
Comment 3•15 years ago
|
||
Attachment #504526 -
Flags: superreview?(lhansen)
Attachment #504526 -
Flags: review?(stejohns)
Comment 4•15 years ago
|
||
Comment on attachment 504526 [details] [diff] [review]
Somethings better than nothing...
Indeed. I have an index.html pending, I'll make sure to link this one into it.
Attachment #504526 -
Flags: superreview?(lhansen) → superreview+
Assignee | ||
Comment 5•15 years ago
|
||
Assignee | ||
Updated•15 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Comment 6•15 years ago
|
||
Comment on attachment 504526 [details] [diff] [review]
Somethings better than nothing...
Good start. Other stuff I've noted:
-- AVMFEATURE_MEMORY_PROFILER seems to need enabling.
-- for mac, uncomment startAtosProcess in MacPortMMgc
Attachment #504526 -
Flags: review?(stejohns) → review+
Updated•15 years ago
|
Flags: flashplayer-bug-
You need to log in
before you can comment on or make changes to this bug.
Description
•