- 03 Jun, 2016 1 commit
-
-
ahaas authored
We need the host address in another CL to clear invalid slots. R=ulan@chromium.org Review-Url: https://codereview.chromium.org/2025833002 Cr-Commit-Position: refs/heads/master@{#36696}
-
- 29 Mar, 2016 2 commits
-
-
ulan authored
This avoids redundant casts, loss of precision, and potential overflows. BUG=chromium:597310 LOG=NO Review URL: https://codereview.chromium.org/1841043002 Cr-Commit-Position: refs/heads/master@{#35113}
-
ulan authored
Now instead of saving all event details in the ring buffer, we save only the bytes and duration. This reduces the GCTracer size from 20K to 3K and simplifies code. BUG=chromium:597310 LOG=NO Review URL: https://codereview.chromium.org/1830723004 Cr-Commit-Position: refs/heads/master@{#35104}
-
- 21 Mar, 2016 1 commit
-
-
mlippautz authored
Now that we have page-local remembered sets (due to refilling in page granularity) we can perform all updates on the sets during compaction in parallel without caching slots locally. BUG=chromium:524425 LOG=N NOTRY=true Review URL: https://codereview.chromium.org/1811573002 Cr-Commit-Position: refs/heads/master@{#34952}
-
- 25 Feb, 2016 1 commit
-
-
ulan authored
Reland "Replace slots buffer with remembered set. (patchset #14 id:250001 of https://codereview.chromium.org/1703823002/ )" This reverts commit 9146bc5e. This contains a fix for the following crash: 1. We record slots for a fixed array. 2. We trim the fixed array, so that some recorded slots are now in free space. 3. During mark-compact we sweep the page with the fixed array. Now free list items contain memory with recorded slots. 4. We evacuate a byte array using the new free list items. 5. We iterate slots that are now inside the byte array and crash. BUG=chromium:589413,chromium:578883 LOG=NO Review URL: https://codereview.chromium.org/1735523002 Cr-Commit-Position: refs/heads/master@{#34302}
-
- 24 Feb, 2016 1 commit
-
-
ulan authored
Revert of Replace slots buffer with remembered set. (patchset #14 id:250001 of https://codereview.chromium.org/1703823002/ ) Reason for revert: Revert because of canary crashes: crbug.com/589413 Original issue's description: > Replace slots buffer with remembered set. > > Slots pointing to evacuation candidates are now recorded in the new RememberedSet<OLD_TO_OLD>. > > The remembered set is extended to support typed slots. > > During parallel evacuation all migration slots are recorded in local slots buffers. > After evacuation all local slots are added to the remembered set. > > BUG=chromium:578883 > LOG=NO > > Committed: https://crrev.com/2285a99ef6f7d52f4f0c4d88a7db4224443ee152 > Cr-Commit-Position: refs/heads/master@{#34212} TBR=jochen@chromium.org,hpayer@chromium.org,mlippautz@chromium.org # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=chromium:578883 Review URL: https://codereview.chromium.org/1725073003 Cr-Commit-Position: refs/heads/master@{#34238}
-
- 23 Feb, 2016 1 commit
-
-
ulan authored
Slots pointing to evacuation candidates are now recorded in the new RememberedSet<OLD_TO_OLD>. The remembered set is extended to support typed slots. During parallel evacuation all migration slots are recorded in local slots buffers. After evacuation all local slots are added to the remembered set. BUG=chromium:578883 LOG=NO Review URL: https://codereview.chromium.org/1703823002 Cr-Commit-Position: refs/heads/master@{#34212}
-
- 22 Feb, 2016 1 commit
-
-
ulan authored
BUG=chromium:587574 LOG=NO Review URL: https://codereview.chromium.org/1705183003 Cr-Commit-Position: refs/heads/master@{#34188}
-
- 17 Feb, 2016 1 commit
-
-
ulan authored
If sweeping is in progress then we need to filter out slots in free space after array trimming, because the sweeper will add the free space into free list. This CL also fixes a bug in SlotSet::RemoveRange. BUG=chromium:587004 LOG=NO TBR=hpayer@chromium.org Review URL: https://codereview.chromium.org/1701963003 Cr-Commit-Position: refs/heads/master@{#34071}
-
- 08 Feb, 2016 1 commit
-
-
ulan authored
This replaces the global remembered set with per-page remembered sets. Each page in the old space, map space, and large object space keeps track of the set of slots in the page pointing to the new space. The data structure for storing slot sets is a two-level bitmap, which allows us to remove the store buffer overflow and SCAN_ON_SCAVENGE logic. Design doc: https://goo.gl/sMKCf7 BUG=chromium:578883 LOG=NO Review URL: https://codereview.chromium.org/1608583002 Cr-Commit-Position: refs/heads/master@{#33806}
-
- 27 Oct, 2015 1 commit
-
-
ulan authored
BUG=chromium:490559 LOG=NO Review URL: https://codereview.chromium.org/1420363004 Cr-Commit-Position: refs/heads/master@{#31600}
-
- 08 Oct, 2015 1 commit
-
-
mlippautz authored
See bug report. R=ulan@chromium.org BUG=chromium:538567 LOG=N Review URL: https://codereview.chromium.org/1393293002 Cr-Commit-Position: refs/heads/master@{#31179}
-
- 02 Oct, 2015 1 commit
-
-
ulan authored
This restores size_of_objects and removed unused fields. BUG=chromium:538539 LOG=NO Review URL: https://codereview.chromium.org/1380953003 Cr-Commit-Position: refs/heads/master@{#31072}
-
- 25 Sep, 2015 2 commits
-
-
ulan authored
BUG=chromium:490559 LOG=NO Review URL: https://codereview.chromium.org/1352453004 Cr-Commit-Position: refs/heads/master@{#30944}
-
mstarzinger authored
This prevents the internal gc-idle-time-handler.h to be usable outisde of the "heap" directory. The logic inside that component is only useful within the GC and is now properly encapsulated. R=ulan@chromium.org Review URL: https://codereview.chromium.org/1368983002 Cr-Commit-Position: refs/heads/master@{#30939}
-
- 08 Sep, 2015 1 commit
-
-
ulan authored
This moves incremental marking steps from gc-idle-time-handler and heap to the new incremental marking task. BUG=chromium:490559 LOG=NO Review URL: https://codereview.chromium.org/1265423002 Cr-Commit-Position: refs/heads/master@{#30641}
-
- 06 Aug, 2015 1 commit
-
-
hpayer authored
When allocation rate is low and we are close to the new space limit, we should perform a scavenge during idle time. BUG=chromium:517395 LOG=n Review URL: https://codereview.chromium.org/1272573006 Cr-Commit-Position: refs/heads/master@{#30044}
-
- 03 Aug, 2015 1 commit
-
-
ulan authored
BUG= Review URL: https://codereview.chromium.org/1261373006 Cr-Commit-Position: refs/heads/master@{#29987}
-
- 31 Jul, 2015 1 commit
-
-
ulan authored
BUG= Review URL: https://codereview.chromium.org/1262363002 Cr-Commit-Position: refs/heads/master@{#29950}
-
- 07 Jul, 2015 2 commits
-
-
ulan authored
disregarding the allocation throughput. BUG=chromium:506132 LOG=NO Review URL: https://codereview.chromium.org/1213313004 Cr-Commit-Position: refs/heads/master@{#29515}
-
ulan authored
This reverts commit 26991892. This reverts commit 435b3c87. The failing test is fixing in chromium. BUG=chromium:490559 LOG=NO TBR=hpayer@chromium.org Review URL: https://codereview.chromium.org/1208993009 Cr-Commit-Position: refs/heads/master@{#29512}
-
- 05 Jul, 2015 1 commit
-
-
machenbach authored
Revert of Replace reduce-memory mode in idle notification with delayed clean-up GC. (patchset #17 id:320001 of https://codereview.chromium.org/1218863002/) Reason for revert: [Sheriff] Looks like it blocks the roll (bisected). Speculative revert. https://codereview.chromium.org/1210293003/ Original issue's description: > Replace reduce-memory mode in idle notification with delayed clean-up GC. > > BUG=490559 > LOG=NO > > Committed: https://crrev.com/0ecd9e1bd2c6b519d4e7285f46cb7e844bc2235c > Cr-Commit-Position: refs/heads/master@{#29451} TBR=hpayer@chromium.org,ulan@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=490559 Review URL: https://codereview.chromium.org/1226703002 Cr-Commit-Position: refs/heads/master@{#29470}
-
- 02 Jul, 2015 1 commit
-
-
ulan authored
BUG=490559 LOG=NO Review URL: https://codereview.chromium.org/1218863002 Cr-Commit-Position: refs/heads/master@{#29451}
-
- 15 Jun, 2015 1 commit
-
-
ulan authored
Doc: https://goo.gl/LLGvBs BUG= Review URL: https://codereview.chromium.org/1163143009 Cr-Commit-Position: refs/heads/master@{#29015}
-
- 20 May, 2015 1 commit
-
-
ulan authored
BUG= Review URL: https://codereview.chromium.org/1145103002 Cr-Commit-Position: refs/heads/master@{#28508}
-
- 19 May, 2015 1 commit
-
-
ulan authored
This also adjusts transitioning between modes so that crbug.com/460090 remains fixed. BUG=chromium:489323, chromium:460090 LOG=NO Review URL: https://codereview.chromium.org/1141393002 Cr-Commit-Position: refs/heads/master@{#28490}
-
- 15 May, 2015 2 commits
-
-
hpayer authored
BUG= Review URL: https://codereview.chromium.org/1138643003 Cr-Commit-Position: refs/heads/master@{#28428}
-
hpayer authored
When context disposal rate is high and we cannot perform a full GC, we do nothing until the context disposal rate becomes lower. BUG=chromium:473351 LOG=n Review URL: https://codereview.chromium.org/1118303004 Cr-Commit-Position: refs/heads/master@{#28416}
-
- 12 May, 2015 1 commit
-
-
ulan authored
BUG=chromium:486005 LOG=NO Review URL: https://codereview.chromium.org/1131943004 Cr-Commit-Position: refs/heads/master@{#28378}
-
- 07 May, 2015 1 commit
-
-
ulan authored
While the mutator is active, the idle time handler optimizes for latency by doing only incremental steps and scavenges. When the mutator becomes inactive, the idle time handler forces few incremental GCs to reclaim memory and then stops until mutator is active again. BUG=460090 LOG=N Review URL: https://codereview.chromium.org/1105293004 Cr-Commit-Position: refs/heads/master@{#28300}
-
- 10 Apr, 2015 2 commits
-
-
machenbach authored
Revert of Make full GC reduce memory footprint an explicit event in the idle notification handler. (patchset #2 id:20001 of https://codereview.chromium.org/1072363002/) Reason for revert: [Sheriff] breaks nosnap with timeouts: http://build.chromium.org/p/client.v8/builders/V8%20Linux%20-%20nosnap/builds/2513 http://build.chromium.org/p/client.v8/builders/V8%20Win32%20-%20nosnap%20-%20shared/builds/6220 Original issue's description: > Make full GC reduce memory footprint an explicit event in the idle notification handler. > > BUG= > > Committed: https://crrev.com/845705aa99b6bfa8d264cfda1c3b5f1229802ab5 > Cr-Commit-Position: refs/heads/master@{#27753} TBR=ulan@chromium.org,rmcilroy@chromium.org,hpayer@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG= Review URL: https://codereview.chromium.org/1081443002 Cr-Commit-Position: refs/heads/master@{#27755}
-
hpayer authored
BUG= Review URL: https://codereview.chromium.org/1072363002 Cr-Commit-Position: refs/heads/master@{#27753}
-
- 30 Mar, 2015 3 commits
-
-
rmcilroy authored
The V8::IdleNotification will only return 'True' when the gc idle time handler thinks there is no more GC which can be done. However, the gc idle task handler can end up repeatedly making no progress (e.g., if it can't finalize a sweep) which causes idle tasks to be repeatedly scheduled in Chrome which do nothing but wake up Chrome. Fix this by returning Done if we can't make any progress within an Idle Round. BUG=chromium:470615 LOG=Y Review URL: https://codereview.chromium.org/1042483002 Cr-Commit-Position: refs/heads/master@{#27529}
-
hpayer authored
BUG=chromium:468554 LOG=n Review URL: https://codereview.chromium.org/1034403002 Cr-Commit-Position: refs/heads/master@{#27517}
-
hpayer authored
A follow-up CL will implement incremental sweeping during idle time. BUG= Review URL: https://codereview.chromium.org/1038283003 Cr-Commit-Position: refs/heads/master@{#27513}
-
- 27 Mar, 2015 1 commit
-
-
hpayer authored
The current GC idle time handling heuristics are getting too complicated. Moreover, with longer idle time we are getting more full garbage collections. This CL shrinks the idle round window and reduces complexity in the case where we cause a full garbage collection. BUG=chromium:468554 LOG=n Review URL: https://codereview.chromium.org/1024043003 Cr-Commit-Position: refs/heads/master@{#27493}
-
- 28 Nov, 2014 1 commit
-
-
hpayer authored
BUG=417668 LOG=n Review URL: https://codereview.chromium.org/750813003 Cr-Commit-Position: refs/heads/master@{#25560}
-
- 27 Nov, 2014 1 commit
-
-
Hannes Payer authored
Re-land: Distinguish beween final incremental mark-compact and full mark-compact event in IdleNotification. BUG= R=ulan@chromium.org Review URL: https://codereview.chromium.org/765743002 Cr-Commit-Position: refs/heads/master@{#25545}
-
- 26 Nov, 2014 1 commit
-
-
Benedikt Meurer authored
Revert "Distinguish beween final incremental mark-compact and full mark-compact event in IdleNotification." This reverts commit d15d453f for breaking cctest/test-api/Threading3. TBR=hpayer@chromium.org Review URL: https://codereview.chromium.org/758163003 Cr-Commit-Position: refs/heads/master@{#25506}
-
- 25 Nov, 2014 1 commit
-
-
hpayer authored
BUG= Review URL: https://codereview.chromium.org/727323004 Cr-Commit-Position: refs/heads/master@{#25501}
-