184019 2018-03-26 11:51:59 -0700 memoryStatus() is wrong in certain testing scenarios on iOS 2018-03-27 13:52:53 -0700 1 1 1 Unclassified WebKit JavaScriptCore WebKit Nightly Build Unspecified Unspecified RESOLVED FIXED InRadar P2 Normal --- 184050 1 saam saam ap benjamin fpizlo ggaren gskachkov jfbastien joepeck keith_miller mark.lam msaboff rmorisset rniwa simon.fraser ticaiolima webkit-bug-importer ysuzuki oldest_to_newest 1409359 0 saam 2018-03-26 11:51:59 -0700 phys_footprint is a number that in my experiments remains constant at around ~9MB. I think what we want is "internal + compressed". 1409438 1 saam 2018-03-26 16:20:33 -0700 (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. 1409601 2 saam 2018-03-27 11:48:56 -0700 <rdar://problem/37959258> 1409603 3 336602 saam 2018-03-27 11:51:27 -0700 Created attachment 336602 patch 1409604 4 336602 saam 2018-03-27 11:52:26 -0700 Comment on attachment 336602 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. 1409607 5 336602 mark.lam 2018-03-27 11:53:40 -0700 Comment on attachment 336602 patch rs=me 1409661 6 saam 2018-03-27 13:52:53 -0700 landed in: https://trac.webkit.org/changeset/230005/webkit 336602 2018-03-27 11:51:27 -0700 2018-03-27 11:53:40 -0700 patch c-backup.diff text/plain 2463 saam SW5kZXg6IFNvdXJjZS9ibWFsbG9jL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvYm1h bGxvYy9DaGFuZ2VMb2cJKHJldmlzaW9uIDIyOTk5OCkKKysrIFNvdXJjZS9ibWFsbG9jL0NoYW5n ZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDI0IEBACisyMDE4LTAzLTI3ICBTYWFtIEJh cmF0aSAgPHNiYXJhdGlAYXBwbGUuY29tPgorCisgICAgICAgIE1vdmUgYmFjayB0byBwaHlzX2Zv b3RwcmludCB3aGVuIGl0IHdvcmtzIGZvciBvdXIgdGVzdGluZyBzY2VuYXJpb3Mgb24gaU9TCisg ICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0xODQwNTAKKyAg ICAgICAgPHJkYXI6Ly9wcm9ibGVtLzM3OTU5MjU4PgorCisgICAgICAgIFJldmlld2VkIGJ5IE5P Qk9EWSAoT09QUyEpLgorCisgICAgICAgIFRoaXMgc3dpdGNoZXMgdXMgZnJvbSB1c2luZyAicGh5 c19mb290cHJpbnQiIHRvIHVzaW5nICJpbnRlcm5hbCArIGNvbXByZXNzZWQiCisgICAgICAgIHdo ZW4gY29tcHV0aW5nIHRoZSBkaXJ0eSBtZW1vcnkgaW4gdGhlIGN1cnJlbnQgcHJvY2Vzcy4gVGhl cmUgYXJlIGlPUyB0ZXN0aW5nCisgICAgICAgIHNjZW5hcmlvcyB3aGVyZSBwaHlzX2Zvb3Rwcmlu dCBkb2Vzbid0IGdpdmUgdXMgYSByZWxpYWJsZSBhbnN3ZXIuIEluIG15IHRlc3RpbmcsCisgICAg ICAgICJpbnRlcm5hbCArIGNvbXByZXNzZWQiIHRyYWNrcyBwaHlzX2Zvb3RwcmludCBjbG9zZWx5 ICh3aGVuIHBoeXNfZm9vdHByaW50IGlzCisgICAgICAgIHdvcmtpbmcpLiBUaGV5J3JlIHVzdWFs bHkgd2l0aGluIG11Y2ggbGVzcyB0aGFuIDElIG9mIGVhY2ggb3RoZXIuIFdlJ3JlIG1ha2luZwor ICAgICAgICB0aGlzIGNoYW5nZSB0byBlbnN1cmUgdGVzdGluZyBpbiBvdXIgaU9TIGluZnJhc3Ry dWN0dXJlIGlzIHZhbGlkLgorICAgICAgICAKKyAgICAgICAgSSBvcGVuZWQgYSBidWcgdG8gbW92 ZSBiYWNrIHRvIHBoeXNfZm9vdHByaW50IHdoZW4gaXQncyBmZWFzaWJsZToKKyAgICAgICAgaHR0 cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTE4NDA1MAorCisgICAgICAgICog Ym1hbGxvYy9BdmFpbGFibGVNZW1vcnkuY3BwOgorICAgICAgICAoYm1hbGxvYzo6bWVtb3J5U3Rh dHVzKToKKwogMjAxOC0wMy0yMCAgVGltIEhvcnRvbiAgPHRpbW90aHlfaG9ydG9uQGFwcGxlLmNv bT4KIAogICAgICAgICBBZGQgYW5kIGFkb3B0IFdLX1BMQVRGT1JNX05BTUUgYW5kIGFkanVzdCBk ZWZhdWx0IGZlYXR1cmUgZGVmaW5lcwpJbmRleDogU291cmNlL2JtYWxsb2MvYm1hbGxvYy9BdmFp bGFibGVNZW1vcnkuY3BwCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFNvdXJjZS9ibWFsbG9jL2JtYWxsb2MvQXZh aWxhYmxlTWVtb3J5LmNwcAkocmV2aXNpb24gMjI5OTk4KQorKysgU291cmNlL2JtYWxsb2MvYm1h bGxvYy9BdmFpbGFibGVNZW1vcnkuY3BwCSh3b3JraW5nIGNvcHkpCkBAIC05OSw4ICs5OSwxMiBA QCBNZW1vcnlTdGF0dXMgbWVtb3J5U3RhdHVzKCkKICAgICBtYWNoX21zZ190eXBlX251bWJlcl90 IHZtU2l6ZSA9IFRBU0tfVk1fSU5GT19DT1VOVDsKICAgICAKICAgICBzaXplX3QgbWVtb3J5Rm9v dHByaW50ID0gMDsKLSAgICBpZiAoS0VSTl9TVUNDRVNTID09IHRhc2tfaW5mbyhtYWNoX3Rhc2tf c2VsZigpLCBUQVNLX1ZNX0lORk8sICh0YXNrX2luZm9fdCkoJnZtSW5mbyksICZ2bVNpemUpKQot ICAgICAgICBtZW1vcnlGb290cHJpbnQgPSBzdGF0aWNfY2FzdDxzaXplX3Q+KHZtSW5mby5waHlz X2Zvb3RwcmludCk7CisgICAgaWYgKEtFUk5fU1VDQ0VTUyA9PSB0YXNrX2luZm8obWFjaF90YXNr X3NlbGYoKSwgVEFTS19WTV9JTkZPLCAodGFza19pbmZvX3QpKCZ2bUluZm8pLCAmdm1TaXplKSkg eworICAgICAgICAvLyBGSVhNRTogTW92ZSBiYWNrIHRvIHBoeXNfZm9vdHByaW50IHdoZW4gaXQg d29ya3MgaW4gYWxsIG9mIG91cgorICAgICAgICAvLyBpT1MgdGVzdGluZyBpbmZyYXN0cnVjdHVy ZToKKyAgICAgICAgLy8gaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTE4 NDA1MAorICAgICAgICBtZW1vcnlGb290cHJpbnQgPSBzdGF0aWNfY2FzdDxzaXplX3Q+KHZtSW5m by5pbnRlcm5hbCkgKyBzdGF0aWNfY2FzdDxzaXplX3Q+KHZtSW5mby5jb21wcmVzc2VkKTsKKyAg ICB9CiAKICAgICBkb3VibGUgcGVyY2VudEluVXNlID0gc3RhdGljX2Nhc3Q8ZG91YmxlPihtZW1v cnlGb290cHJpbnQpIC8gc3RhdGljX2Nhc3Q8ZG91YmxlPihhdmFpbGFibGVNZW1vcnkoKSk7CiAg ICAgZG91YmxlIHBlcmNlbnRBdmFpbGFibGVNZW1vcnlJblVzZSA9IHN0ZDo6bWluKHBlcmNlbnRJ blVzZSwgMS4wKTsK