Changeset 252157 in webkit


Ignore:
Timestamp:
Nov 6, 2019 3:57:52 PM (4 years ago)
Author:
ysuzuki@apple.com
Message:

Unreviewed, roll-out r251268 due to RAMification regression
https://bugs.webkit.org/show_bug.cgi?id=203121

The code was the bug, but it would be possible that this bug accidentally makes our policy better.
We should clean up to make this bug's behavior default. And we should look for a bit larger interval here.

  • bmalloc/Scavenger.cpp:

(bmalloc::Scavenger::threadRunLoop):

Location:
trunk/Source/bmalloc
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • trunk/Source/bmalloc/ChangeLog

    r251268 r252157  
     12019-11-06  Yusuke Suzuki  <ysuzuki@apple.com>
     2
     3        Unreviewed, roll-out r251268 due to RAMification regression
     4        https://bugs.webkit.org/show_bug.cgi?id=203121
     5
     6        The code was the bug, but it would be possible that this bug accidentally makes our policy better.
     7        We should clean up to make this bug's behavior default. And we should look for a bit larger interval here.
     8
     9        * bmalloc/Scavenger.cpp:
     10        (bmalloc::Scavenger::threadRunLoop):
     11
    1122019-10-17  Basuke Suzuki  <Basuke.Suzuki@sony.com>
    213
  • trunk/Source/bmalloc/bmalloc/Scavenger.cpp

    r251268 r252157  
    501501            timeSpentScavenging *= 50;
    502502            newWaitTime = std::chrono::duration_cast<std::chrono::milliseconds>(timeSpentScavenging);
    503             m_waitTime = std::min(std::max(newWaitTime, std::chrono::milliseconds(25)), std::chrono::milliseconds(500));
     503            newWaitTime = std::min(std::max(newWaitTime, std::chrono::milliseconds(25)), std::chrono::milliseconds(500));
    504504        } else {
    505505            timeSpentScavenging *= 150;
Note: See TracChangeset for help on using the changeset viewer.