RESOLVED FIXED 140762
EdenCollections unnecessarily visit SmallStrings 
https://bugs.webkit.org/show_bug.cgi?id=140762
Summary EdenCollections unnecessarily visit SmallStrings
Mark Hahnenberg
Reported 2015-01-21 23:17:32 PST
Once SmallStrings have been marked, they no longer need to be visited during EdenCollections. I noticed they were consuming a non-trivial amount of total EdenCollection time, so let's fix it!
Attachments
Patch (5.56 KB, patch)
2015-01-21 23:51 PST, Mark Hahnenberg 		no flags
DetailsFormatted DiffDiff
GC logs before (9.69 KB, text/plain)
2015-01-21 23:53 PST, Mark Hahnenberg 		no flags
Details
GC logs after (9.69 KB, text/plain)
2015-01-21 23:54 PST, Mark Hahnenberg 		no flags
Details
View All Add attachment proposed patch, testcase, etc.
Mark Hahnenberg
Comment 1 2015-01-21 23:51:54 PST
Created attachment 245125 [details] Patch
Mark Hahnenberg
Comment 2 2015-01-21 23:53:56 PST
Created attachment 245126 [details] GC logs before
Mark Hahnenberg
Comment 3 2015-01-21 23:54:12 PST
Created attachment 245127 [details] GC logs after
Mark Hahnenberg
Comment 4 2015-01-21 23:55:02 PST
Attached GC timing logs before and after the change. Time spent in VisitSmallStrings is down significantly, but it didn't seem to affect the overall Eden time as much as I'd expect.
Geoffrey Garen
Comment 5 2015-01-22 09:09:48 PST
Comment on attachment 245125 [details] Patch r=me
WebKit Commit Bot
Comment 6 2015-01-22 23:04:02 PST
Comment on attachment 245125 [details] Patch Clearing flags on attachment: 245125 Committed r178984: <http://trac.webkit.org/changeset/178984>
WebKit Commit Bot
Comment 7 2015-01-22 23:04:08 PST
All reviewed patches have been landed. Closing bug.
Andreas Kling
Comment 8 2015-03-16 13:57:01 PDT
(In reply to comment #4) > Attached GC timing logs before and after the change. Time spent in > VisitSmallStrings is down significantly, but it didn't seem to affect the > overall Eden time as much as I'd expect. Still, that's a really impressive improvement! Thanks for making this change, Mark. :)
Note You need to log in before you can comment on or make changes to this bug.