- 23 Jul, 2015 1 commit
-
-
hpayer authored
Use a lock in pages to synchronize sweeper threads to allow others to wait on concurrently swept pages. BUG= Review URL: https://codereview.chromium.org/1244353002 Cr-Commit-Position: refs/heads/master@{#29804}
-
- 06 Jul, 2015 2 commits
-
-
machenbach authored
Revert of Reland: Fix logic for incremental marking steps on tenured allocation (patchset #4 id:60001 of https://codereview.chromium.org/1077153004/) Reason for revert: [Sheriff] Speculative revert, see: https://code.google.com/p/chromium/issues/detail?id=506875 Original issue's description: > Reland: Fix logic for incremental marking steps on tenured allocation > > BUG= > > Committed: https://crrev.com/5000650bde2ec0bc90d959b529c97aea20385043 > Cr-Commit-Position: refs/heads/master@{#29442} TBR=hpayer@chromium.org,erikcorry@chromium.org BUG=chromium:506875 LOG=n NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG= Review URL: https://codereview.chromium.org/1212063005 Cr-Commit-Position: refs/heads/master@{#29494}
-
machenbach authored
Revert of Concurrent sweeping of code space. (patchset #4 id:60001 of https://codereview.chromium.org/1222013002/) Reason for revert: [Sheriff] Increased flaky crashes. See: https://code.google.com/p/v8/issues/detail?id=4275 Original issue's description: > Concurrent sweeping of code space. > > BUG= > > Committed: https://crrev.com/3050b52f57d652dc45c8baf416e174f22dc2c159 > Cr-Commit-Position: refs/heads/master@{#29456} TBR=jochen@chromium.org,hpayer@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG= Review URL: https://codereview.chromium.org/1223763003 Cr-Commit-Position: refs/heads/master@{#29475}
-
- 02 Jul, 2015 2 commits
-
-
hpayer authored
BUG= Review URL: https://codereview.chromium.org/1222013002 Cr-Commit-Position: refs/heads/master@{#29456}
-
erikcorry authored
BUG= Review URL: https://codereview.chromium.org/1077153004 Cr-Commit-Position: refs/heads/master@{#29442}
-
- 26 Jun, 2015 1 commit
-
-
hpayer authored
Revert of Re-land new insertion write barrier. (patchset #1 id:1 of https://codereview.chromium.org/1211513002/) Reason for revert: Still crashing in the wild. Original issue's description: > Re-land new insertion write barrier. > > BUG= > > Committed: https://crrev.com/6e6af7e7c70050cdd5ce56cdbaee3b6e8a7caa89 > Cr-Commit-Position: refs/heads/master@{#29258} TBR=ulan@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG= Review URL: https://codereview.chromium.org/1217443002 Cr-Commit-Position: refs/heads/master@{#29320}
-
- 24 Jun, 2015 1 commit
-
-
hpayer authored
BUG= Review URL: https://codereview.chromium.org/1211513002 Cr-Commit-Position: refs/heads/master@{#29258}
-
- 18 Jun, 2015 1 commit
-
-
hpayer authored
Revert of Re-land new insertion write barrier. (patchset #3 id:40001 of https://codereview.chromium.org/1153233003/) Reason for revert: Deoptimizer crashes. Original issue's description: > Re-land new insertion write barrier. > > BUG= > > Committed: https://crrev.com/72f850458cb530d871cf3fb6442660555dc51bc2 > Cr-Commit-Position: refs/heads/master@{#29052} TBR=ulan@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG= Review URL: https://codereview.chromium.org/1176243015 Cr-Commit-Position: refs/heads/master@{#29123}
-
- 16 Jun, 2015 1 commit
-
-
hpayer authored
BUG= Review URL: https://codereview.chromium.org/1153233003 Cr-Commit-Position: refs/heads/master@{#29052}
-
- 29 May, 2015 1 commit
-
-
bbudge authored
Moves alignment fill calculations into two static Heap methods. Adds a Heap method to handle the complex case where filler is potentially needed before and after a heap object. Makes DoubleAlignForDeserialization explicitly fill after an already aligned object. LOG=N BUG=v8:4124 Committed: https://crrev.com/fcfb080eb9a637f0ae066bed4c45095e60df8a84 Cr-Commit-Position: refs/heads/master@{#28687} Review URL: https://codereview.chromium.org/1150593003 Cr-Commit-Position: refs/heads/master@{#28702}
-
- 28 May, 2015 2 commits
-
-
bbudge authored
Revert of Clean up aligned allocation code in preparation for SIMD alignments. (patchset #14 id:300001 of https://codereview.chromium.org/1150593003/) Reason for revert: Breaks mjsunit, webkit, mozilla, benchmarks. TBR=hpayer@chromium.org Original issue's description: > Clean up aligned allocation code in preparation for SIMD alignments. > > Moves alignment fill calculations into two static Heap methods. > Adds a Heap method to handle the complex case where filler is potentially needed before and after a heap object. > Makes DoubleAlignForDeserialization explicitly fill after an already > aligned object. > > LOG=N > BUG=v8:4124 > > Committed: https://crrev.com/fcfb080eb9a637f0ae066bed4c45095e60df8a84 > Cr-Commit-Position: refs/heads/master@{#28687} TBR=hpayer@chromium.org,bmeurer@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=v8:4124 Review URL: https://codereview.chromium.org/1159123002 Cr-Commit-Position: refs/heads/master@{#28688}
-
bbudge authored
Moves alignment fill calculations into two static Heap methods. Adds a Heap method to handle the complex case where filler is potentially needed before and after a heap object. Makes DoubleAlignForDeserialization explicitly fill after an already aligned object. LOG=N BUG=v8:4124 Review URL: https://codereview.chromium.org/1150593003 Cr-Commit-Position: refs/heads/master@{#28687}
-
- 27 May, 2015 1 commit
-
-
ulan authored
This lifts the sqrt(n) limit on number of evacuation candidates, replaces O(n * sqrt(n)) algorithm with O(n*log(n)) algorithm, and removes hard-coded constants. Evacuation candidates are selected as follows: 1) Sort pages from the most free to the least free. 2) Select the first m pages as evacuation candidates such that m is as large as possible under the two conditions: - The total size of live objects in the first m pages does not exceed the given limit. This is based on the assumption that the evacuation cost is proportional to the total size of moved objects. - The fragmentation of the (m+1)-th page does not exceed the given limit. Review URL: https://codereview.chromium.org/1038313003 Cr-Commit-Position: refs/heads/master@{#28651}
-
- 22 May, 2015 1 commit
-
-
hpayer authored
BUG=chromium:481811 LOG=n Review URL: https://codereview.chromium.org/1148633005 Cr-Commit-Position: refs/heads/master@{#28585}
-
- 21 May, 2015 1 commit
-
-
ulan authored
Sample new space allocation throughput at scavenge and at idle notificatioon. This will allow better estimation of mutator idleness for switching between latency and memory modes in idle notification handler. BUG=chromium:486005 LOG=NO TEST=cctest/test-heap/NewSpaceAllocationThroughput Review URL: https://codereview.chromium.org/1125193005 Cr-Commit-Position: refs/heads/master@{#28537}
-
- 15 May, 2015 2 commits
-
-
hpayer authored
BUG= Review URL: https://codereview.chromium.org/1138643005 Cr-Commit-Position: refs/heads/master@{#28430}
-
jochen authored
We want to move to a world where there's no Isolate::Current but we always knows which isolate we're in. There's no way we can teach this info to the C++ allocator. BUG=none R=hpayer@chromium.org LOG=n Review URL: https://codereview.chromium.org/1128023005 Cr-Commit-Position: refs/heads/master@{#28414}
-
- 12 May, 2015 1 commit
-
-
hpayer authored
In follow-up CLs the scavenger and the MC collector should also respect the unalignment of heap numbers. BUG= Review URL: https://codereview.chromium.org/1141523002 Cr-Commit-Position: refs/heads/master@{#28360}
-
- 06 May, 2015 2 commits
-
-
hpayer authored
Revert of New insertion write barrier. (patchset #3 id:200001 of https://codereview.chromium.org/1073953006/) Reason for revert: Various performance regressions. Original issue's description: > New insertion write barrier. > > BUG= > > Committed: https://crrev.com/6e9e2c08292b553602c74b16b8ccff4e4ccac003 > Cr-Commit-Position: refs/heads/master@{#28199} TBR=ulan@chromium.org,erikcorry@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG= Review URL: https://codereview.chromium.org/1131583002 Cr-Commit-Position: refs/heads/master@{#28265}
-
hpayer authored
BUG=chromium:436911 Review URL: https://codereview.chromium.org/1127993002 Cr-Commit-Position: refs/heads/master@{#28262}
-
- 04 May, 2015 1 commit
-
-
hpayer authored
BUG= Review URL: https://codereview.chromium.org/1073953006 Cr-Commit-Position: refs/heads/master@{#28199}
-
- 20 Apr, 2015 1 commit
-
-
jkummerow authored
Revert of Fix logic for doing incremental marking steps on tenured allocation. (patchset #4 id:60001 of https://codereview.chromium.org/1040233003/) Reason for revert: Suspected of triggering memory corruption issues, e.g. crbug.com/478401. Original issue's description: > Fix logic for doing incremental marking steps on tenured allocation. > > R=hpayer@chromium.org > BUG= > > Committed: https://crrev.com/9716468ae63500adb74f5188c47de847e195d71b > Cr-Commit-Position: refs/heads/master@{#27883} TBR=hpayer@chromium.org,erikcorry@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG= Review URL: https://codereview.chromium.org/1099633002 Cr-Commit-Position: refs/heads/master@{#27944}
-
- 17 Apr, 2015 1 commit
-
-
ssid authored
This is first step towards adding V8 heap statistics to the memory tracing infrastructure. For being able to get useful memory number into the memory dump, v8 needs to provide an external api needs to obtain more information about the heap. So, this Cl extends the api to give information about the memory allocated and used in the spaces. BUG=466141, 476013 LOG=Y Review URL: https://codereview.chromium.org/1058253003 Cr-Commit-Position: refs/heads/master@{#27919}
-
- 16 Apr, 2015 2 commits
-
-
erikcorry authored
R=hpayer@chromium.org BUG= Review URL: https://codereview.chromium.org/1040233003 Cr-Commit-Position: refs/heads/master@{#27883}
-
ulan authored
This breaks the (evacuation -> slots buffer overflow -> abort -> new GC -> evacuation) cycle for popular pages. BUG= Review URL: https://codereview.chromium.org/1037433002 Cr-Commit-Position: refs/heads/master@{#27868}
-
- 10 Apr, 2015 1 commit
-
-
verwaest authored
This fixes the arm(64) and mips(64) write barriers BUG= Review URL: https://codereview.chromium.org/1073133002 Cr-Commit-Position: refs/heads/master@{#27751}
-
- 09 Apr, 2015 3 commits
-
-
adamk authored
Revert of Merge cellspace into old pointer space (patchset #8 id:180001 of https://codereview.chromium.org/1010803012/) Reason for revert: Causes test failures on ARM bots related to cells and write barriers. Original issue's description: > Merge cellspace into old pointer space > > BUG= > > Committed: https://crrev.com/4e7163ce05f135918205c7855ae60a48e5d46cc5 > Cr-Commit-Position: refs/heads/master@{#27707} TBR=hpayer@chromium.org,balazs.kilvady@imgtec.com,yangguo@chromium.org,verwaest@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG= Review URL: https://codereview.chromium.org/1053243003 Cr-Commit-Position: refs/heads/master@{#27711}
-
verwaest authored
BUG= Review URL: https://codereview.chromium.org/1010803012 Cr-Commit-Position: refs/heads/master@{#27707}
-
hpayer authored
MarkBit cleanup: They have to be accessed through Marking accessors. Avoid arbitrary abuse of mark bits and make marking explicit. Added DCHECKs to mark bit transitions to check source state. BUG= Review URL: https://codereview.chromium.org/1040443002 Cr-Commit-Position: refs/heads/master@{#27689}
-
- 07 Apr, 2015 1 commit
-
-
hpayer authored
This reverts commit cbfcee55. BUG= Review URL: https://codereview.chromium.org/1051233002 Cr-Commit-Position: refs/heads/master@{#27623}
-
- 27 Mar, 2015 2 commits
-
-
ishell authored
This CL also adds useful machinery that helps triggering incremental write barriers. BUG=chromium:469146 LOG=Y Review URL: https://codereview.chromium.org/1039733003 Cr-Commit-Position: refs/heads/master@{#27503}
-
ishell authored
Revert of Reland "Filter invalid slots out from the SlotsBuffer after marking." (patchset #2 id:2 of https://codereview.chromium.org/1032833002/) Reason for revert: Reverting risky GC changes that block v8 roll. Original issue's description: > Reland "Filter invalid slots out from the SlotsBuffer after marking." > > > There are two reasons that could cause invalid slots appearance in SlotsBuffer: > > 1) If GC trims "tail" of an array for which it has already recorded a slots and then migrate another object to the "tail". > > 2) Tagged slot could become a double slot after migrating of an object to another map with "shifted" fields (for example as a result of generalizing immutable data property to a data field). > > > This CL also adds useful machinery that helps triggering incremental write barriers. > > > BUG=chromium:454297 > > LOG=Y > > NOTRY=true > > Committed: https://crrev.com/f86aadd1d45c756467dff8e08a055b462d7a060b > Cr-Commit-Position: refs/heads/master@{#27433} TBR=machenbach@chromium.org,ulan@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true Review URL: https://codereview.chromium.org/1041593002 Cr-Commit-Position: refs/heads/master@{#27491}
-
- 25 Mar, 2015 1 commit
-
-
ulan authored
> There are two reasons that could cause invalid slots appearance in SlotsBuffer: > 1) If GC trims "tail" of an array for which it has already recorded a slots and then migrate another object to the "tail". > 2) Tagged slot could become a double slot after migrating of an object to another map with "shifted" fields (for example as a result of generalizing immutable data property to a data field). > This CL also adds useful machinery that helps triggering incremental write barriers. > BUG=chromium:454297 > LOG=Y NOTRY=true Review URL: https://codereview.chromium.org/1032833002 Cr-Commit-Position: refs/heads/master@{#27433}
-
- 24 Mar, 2015 3 commits
-
-
machenbach authored
Revert of Filter invalid slots out from the SlotsBuffer after marking. (patchset #6 id:220001 of https://codereview.chromium.org/1010363005/) Reason for revert: Need to revert in order to revert https://codereview.chromium.org/1029323003/ Original issue's description: > Filter invalid slots out from the SlotsBuffer after marking. > > There are two reasons that could cause invalid slots appearance in SlotsBuffer: > 1) If GC trims "tail" of an array for which it has already recorded a slots and then migrate another object to the "tail". > 2) Tagged slot could become a double slot after migrating of an object to another map with "shifted" fields (for example as a result of generalizing immutable data property to a data field). > > This CL also adds useful machinery that helps triggering incremental write barriers. > > BUG=chromium:454297 > LOG=Y > > Committed: https://crrev.com/5c47c1c0d3e4a488f190c16a64ee02f5a14e6561 > Cr-Commit-Position: refs/heads/master@{#27423} TBR=hpayer@chromium.org,erik.corry@gmail.com,ishell@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=chromium:454297 Review URL: https://codereview.chromium.org/1033453005 Cr-Commit-Position: refs/heads/master@{#27426}
-
ishell authored
There are two reasons that could cause invalid slots appearance in SlotsBuffer: 1) If GC trims "tail" of an array for which it has already recorded a slots and then migrate another object to the "tail". 2) Tagged slot could become a double slot after migrating of an object to another map with "shifted" fields (for example as a result of generalizing immutable data property to a data field). This CL also adds useful machinery that helps triggering incremental write barriers. BUG=chromium:454297 LOG=Y Review URL: https://codereview.chromium.org/1010363005 Cr-Commit-Position: refs/heads/master@{#27423}
-
erikcorry authored
Also introduce --trace-fragmentation-verbose, and fix --always-compact. R=ulan@chromium.org BUG=v8:3976 LOG=y Review URL: https://codereview.chromium.org/1024823002 Cr-Commit-Position: refs/heads/master@{#27414}
-
- 19 Mar, 2015 1 commit
-
-
hpayer authored
TBR=verwaest@chromium.org,ulan@chromium.org,ishell@chromium.org NOTRY=true Review URL: https://codereview.chromium.org/1027463002 Cr-Commit-Position: refs/heads/master@{#27323}
-
- 18 Mar, 2015 2 commits
-
-
verwaest authored
Replaces StoreGlobalCell / LoadGlobalCell with NamedField variants that use write barriers. BUG= Review URL: https://codereview.chromium.org/1016803002 Cr-Commit-Position: refs/heads/master@{#27269}
-
hpayer authored
BUG= Review URL: https://codereview.chromium.org/1012023002 Cr-Commit-Position: refs/heads/master@{#27259}
-
- 16 Mar, 2015 1 commit
-
-
ishell authored
Revert of Reland of Remove slots that point to unboxed doubles from the StoreBuffer/SlotsBuffer. (patchset #3 id:40001 of https://codereview.chromium.org/988363002/) Reason for revert: Increased rate of Chrome crashes. Requires further investigation. Original issue's description: > Reland of Remove slots that point to unboxed doubles from the StoreBuffer/SlotsBuffer. > > The problem is that tagged slot could become a double slot after migrating of an object to another map with "shifted" fields (for example as a result of generalizing immutable data property to a data field). > This CL also adds useful machinery that helps triggering incremental write barriers. > > BUG=chromium:454297, chromium:465273 > LOG=Y > > Committed: https://crrev.com/6d0677d845c47ab9fa297de61d0e3d8e5480a02a > Cr-Commit-Position: refs/heads/master@{#27141} TBR=hpayer@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=chromium:454297, chromium:465273 Review URL: https://codereview.chromium.org/1004623003 Cr-Commit-Position: refs/heads/master@{#27207}
-