Bug 184019

Summary: memoryStatus() is wrong in certain testing scenarios on iOS
Product: WebKit Reporter: Saam Barati <saam>
Component: JavaScriptCoreAssignee: Saam Barati <saam>
Status: RESOLVED FIXED    
Severity: Normal CC: ap, benjamin, fpizlo, ggaren, gskachkov, jfbastien, joepeck, keith_miller, mark.lam, msaboff, rmorisset, rniwa, simon.fraser, ticaiolima, webkit-bug-importer, ysuzuki
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 184050    
Attachments:
Description Flags
patch mark.lam: review+

Saam Barati
Reported 2018-03-26 11:51:59 PDT
phys_footprint is a number that in my experiments remains constant at around ~9MB. I think what we want is "internal + compressed".
Attachments
patch (2.41 KB, patch)
2018-03-27 11:51 PDT, Saam Barati 		mark.lam: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Saam Barati
Comment 1 2018-03-26 16:20:33 PDT
(In reply to Saam Barati from comment #0) > phys_footprint is a number that in my experiments remains constant at around > ~9MB. I think what we want is "internal + compressed". I'm actually not 100% sure of this. Doing some more investigation now.
Saam Barati
Comment 2 2018-03-27 11:48:56 PDT
<rdar://problem/37959258>
Saam Barati
Comment 3 2018-03-27 11:51:27 PDT
Created attachment 336602 [details] patch
Saam Barati
Comment 4 2018-03-27 11:52:26 PDT
Comment on attachment 336602 [details] patch View in context: https://bugs.webkit.org/attachment.cgi?id=336602&action=review > Source/bmalloc/ChangeLog:3 > + Move back to phys_footprint when it works for our testing scenarios on iOS Will change bug title to match before landing.
Mark Lam
Comment 5 2018-03-27 11:53:40 PDT
Comment on attachment 336602 [details] patch rs=me
Saam Barati
Comment 6 2018-03-27 13:52:53 PDT
landed in: https://trac.webkit.org/changeset/230005/webkit
Note You need to log in before you can comment on or make changes to this bug.