bmalloc should implement malloc introspection (to stop false-positive leaks when MallocStackLogging is off)
Created attachment 246900 [details] Patch
Attachment 246900 [details] did not pass style-queue: ERROR: Source/bmalloc/bmalloc/Zone.cpp:26: Found other header before WebCore config.h. Should be: config.h, primary header, blank line, and then alphabetically sorted. [build/include_order] [4] ERROR: Source/bmalloc/bmalloc/Zone.cpp:27: Found header this file implements after other header. Should be: config.h, primary header, blank line, and then alphabetically sorted. [build/include_order] [4] ERROR: Source/bmalloc/bmalloc/Zone.cpp:46: type_mask is incorrectly named. Don't use underscores in your identifier names. [readability/naming/underscores] [4] ERROR: Source/bmalloc/bmalloc/Zone.cpp:46: zone_address is incorrectly named. Don't use underscores in your identifier names. [readability/naming/underscores] [4] Total errors found: 4 in 7 files If any of these errors are false positives, please file a bug against check-webkit-style.
> ERROR: Source/bmalloc/bmalloc/Zone.cpp:46: type_mask is incorrectly named. > Don't use underscores in your identifier names. > [readability/naming/underscores] [4] > ERROR: Source/bmalloc/bmalloc/Zone.cpp:46: zone_address is incorrectly > named. Don't use underscores in your identifier names. > [readability/naming/underscores] [4] I did this on purpose. We're implementing a delegation API, and I want our signature to match the signature declared in the header exactly.
Comment on attachment 246900 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=246900&action=review r=me > Source/bmalloc/bmalloc/Zone.cpp:54 > + auto begin = remoteZone.superChunks().begin(); > + auto end = remoteZone.superChunks().end(); > + > + for (auto it = begin; it != end; ++it) { Could we use range for here? > Source/bmalloc/bmalloc/Zone.h:39 > + // Enough capacity to track a 16GB heap, so probably enough for anything. > + static const size_t capacity = 512; Let's go a bit bigger here. 64GB?
Committed r180359: <http://trac.webkit.org/changeset/180359>
Reopening to attach new patch.
Created attachment 246914 [details] Patch
Comment on attachment 246914 [details] Patch rs=me
Committed r180363: <http://trac.webkit.org/changeset/180363>
(In reply to comment #9) > Committed r180363: <http://trac.webkit.org/changeset/180363> It caused serious regression (at least) on Apple Yosemite bot: https://build.webkit.org/builders/Apple%20Yosemite%20Release%20WK2%20%28Tests%29/builds/2973
Re-opened since this is blocked by bug 141814
<rdar://problem/19799874>
Created attachment 246977 [details] Patch
Comment on attachment 246977 [details] Patch A-ha!
Committed r180430: <http://trac.webkit.org/changeset/180430>
Rolled out in http://trac.webkit.org/changeset/180604 until we can figure out why the PLT is crashing. rdar://problem/19948015
Committed r180954: <http://trac.webkit.org/changeset/180954>
Created attachment 247797 [details] Patch
Committed r180960: <http://trac.webkit.org/changeset/180960>