Commits · 090be8bf152106fb4f42c11769573238195eb52f · Linshizhi / V8

30 Nov, 2011 1 commit

Be more willing to expand old space when evacuating new space at the end of · e2f2c77e

erik.corry@gmail.com authored 13 years ago

a mark-sweep. We have a soft limit on old space size, which is designed to
trigger an old-space collection when we hit it. Unfortunately although the
soft limit had already triggered an old space collection, the soft limit was
preventing objects from new space from being promoted. For every promotion
candidate we were checking 3 different ways to allocate in old space before
giving up and putting the object in the other semispace. This change allows
the promoted objects to go to old space and also makes us more eager to
sweep a page before trying other ways to find space for an object.
Review URL: http://codereview.chromium.org/8748005

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@10092 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

e2f2c77e

29 Nov, 2011 1 commit

Only sweep one page eagerly unless we are running out of space. · 9c18742d

erik.corry@gmail.com authored 13 years ago

Limit the number of pages that are compacted in a given GC.
Review URL: http://codereview.chromium.org/8692002

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@10084 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

9c18742d

11 Nov, 2011 1 commit

Fix filtering of store buffer for large object pages. · 9dcf002f

mstarzinger@chromium.org authored 13 years ago

Our Heap::FreeQueuedChunks generates fake inner chunks in large object
pages queued for freeing, so that StoreBuffer::Filter can recognize them
as pages to be freed. This also relies on MemoryChunk::Contains to work
properly, which is why the size field needs to be initialized as well.

R=vegorov@chromium.org
BUG=v8:1817
TEST=mozilla/js1_5/Regress/regress-360969-05

Review URL: http://codereview.chromium.org/8536009

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9964 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

9dcf002f

10 Nov, 2011 1 commit

Fix Heap::Shrink to ensure that it does not free pages that are still in use. · b8dc77ab

vegorov@chromium.org authored 13 years ago

Heap::Shrink is called from EnsureFromSpaceIsCommitted at the very start of the GC. At this moment live bytes counts on pages are in inconsistent states. Some pages might have been already swept but have not been yet reached by an incremental marker (or incremental marker is not in progress) and have live bytes count set to 0. Thus we can't rely only on LiveBytes to determine which pages can be released to the OS.

R=mstarzinger@chromium.org
BUG=100414

Review URL: http://codereview.chromium.org/8507038

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9953 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

b8dc77ab

09 Nov, 2011 1 commit

Ensure that promotion queue does not overlap with objects relocated to ToSpace. · 9f501470

vegorov@chromium.org authored 13 years ago

R=erik.corry@gmail.com

Review URL: http://codereview.chromium.org/8477030

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9932 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

9f501470

25 Oct, 2011 2 commits

Remove superfluous assert. · 6950fab3

erikcorry authored 13 years ago

Review URL: http://codereview.chromium.org/8332028

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9775 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

6950fab3

Move some heap verification code in under the --verify-heap flag to speed · f6fef241

erik.corry@gmail.com authored 13 years ago

up debug mode tests.
Review URL: http://codereview.chromium.org/8381040

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9774 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

f6fef241

19 Oct, 2011 1 commit

Don't allow large object space to grow over the max oldspace limit (fixes issue 1717) · 1c21a9dc

ricow@chromium.org authored 13 years ago

Review URL: http://codereview.chromium.org/8345040

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9695 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

1c21a9dc

14 Oct, 2011 1 commit

Fix available new space memory statistics reporting. · 2a4245e0

ricow@chromium.org authored 13 years ago

We currently report how much memory is available without changing page, i.e., the actual amount of available memory is not correctly stated for new space when doing --trace-gc.
Review URL: http://codereview.chromium.org/8275026

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9624 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

2a4245e0

13 Oct, 2011 1 commit

Remove some asserts to speed up debug mode. · 1cca5468

erik.corry@gmail.com authored 13 years ago

Review URL: http://codereview.chromium.org/8256012

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9606 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

1cca5468

11 Oct, 2011 1 commit

Remove some unused and unneeded flags. · f900fc9d

erik.corry@gmail.com authored 13 years ago

Review URL: http://codereview.chromium.org/8228004

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9576 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

f900fc9d

29 Sep, 2011 1 commit

Implement shrinking of paged spaces during sweeping. · 279356b4

mstarzinger@chromium.org authored 13 years ago

For each paged space we release all but one of the unused pages after
marking (when we know the number of live bytes) but before actually
sweeping it. This is not yet done for lazy swept pages.

R=erik.corry@gmail.com
BUG=v8:1614

Review URL: http://codereview.chromium.org/7891010

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9481 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

279356b4

22 Sep, 2011 2 commits

GC Cleanup + Set max old generation size to 700MB on ia32 and max executable... · e4c90dc9

ricow@chromium.org authored 13 years ago

GC Cleanup + Set max old generation size to 700MB on ia32 and max executable size to 128 MB (on ia32)
Review URL: http://codereview.chromium.org/7993003

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9406 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

e4c90dc9

Notify collector about lazily deoptimized code objects. · bfd04817

vegorov@chromium.org authored 13 years ago

All slots that were recorded on these objects during incremental marking should be ignored as they are no longer valid.

To filter such invalidated slots out during slots buffers iteration we set all markbits under the invalidated code object to 1 after the code space was swept and before slots buffers are processed.

R=erik.corry@gmail.com
BUG=v8:1713
TEST=test/mjsunit/regress/regress-1713.js

Review URL: http://codereview.chromium.org/7983045

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9402 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

bfd04817

20 Sep, 2011 2 commits

Fix new space shrinking to compute correct capacity. · bffbceff

mstarzinger@chromium.org authored 13 years ago

R=vegorov@chromium.org
BUG=v8:1702
TEST=cctest/test-heap/GrowAndShrinkNewSpace

Review URL: http://codereview.chromium.org/7983001

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9348 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

bffbceff

Fix calculation of live-bytes in pages. · 610281f4

lrn@chromium.org authored 13 years ago

The "live bytes" count is *really* a "marked black" count - i.e., the count of bytes *known* to be live.

Fix aggravating bug on X64 where assembler code used a value that was off
by a factor of 2^31.

Ensure that sweeping clears live-bytes. Added other missing increments.

Added print statements to trace live-byte modifications, under a flag.

Still a few cases of undercounting left.

(New issue to merge from GC branch to bleeding_edge)

Review URL: http://codereview.chromium.org/7970009

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9338 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

610281f4

19 Sep, 2011 1 commit

Merge experimental/gc branch to the bleeding_edge. · ac36cb45

vegorov@chromium.org authored 13 years ago

Review URL: http://codereview.chromium.org/7945009

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9328 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

ac36cb45

08 Sep, 2011 1 commit

Fix presubmit errors caused by updated depot tools · b8cbe08f

ricow@chromium.org authored 13 years ago

This is all blank line before/after linting errors.
Review URL: http://codereview.chromium.org/7754022

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9204 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

b8cbe08f

25 Aug, 2011 2 commits

Two tiny refactorings: Removed a bit of copy-n-paste. Moved... · 3d88d16f

svenpanne@chromium.org authored 13 years ago

Two tiny refactorings: Removed a bit of copy-n-paste. Moved LargeObjectChunk::Free from header to implementation, it does a syscall, anyway.
Review URL: http://codereview.chromium.org/7744023

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9017 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

3d88d16f

Fixed bool <-> Executability confusion and improved typing a bit. · a546e4a8

svenpanne@chromium.org authored 13 years ago

Passing a value of type Executability to a function expecting a bool worked only
by accident (because of the order of values in the enum). But using boolean
parameters is often a bad idea, anyway, so we use Executability directly.

Just another example why implicit type conversions in C++ are a bad idea... :-P
Review URL: http://codereview.chromium.org/7753001

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9013 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

a546e4a8

18 Aug, 2011 1 commit

Fix another handful of memory leaks · c50094bd

jkummerow@chromium.org authored 13 years ago

TEST=valgrind reports 0 bytes definitely lost for cctest/test-api/RunTwoIsolatesOnSingleThread

Review URL: http://codereview.chromium.org/7621064

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@8968 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

c50094bd

04 Aug, 2011 1 commit

Minimize malloc heap allocation on process startup. · 1dcb6e33

vitalyr@chromium.org authored 13 years ago

R=vegorov@chromium.org
BUG=http://b/issue?id=5095592

Review URL: http://codereview.chromium.org/7572018

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@8833 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

1dcb6e33

20 Jul, 2011 1 commit

Add guard pages in front of executable allocations · e372a2dd

ager@chromium.org authored 13 years ago

BUG=89247

Review URL: http://codereview.chromium.org/7379004
Patch from Chris Neckar <cdn@chromium.org>.

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@8687 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

e372a2dd

13 Jul, 2011 1 commit

Remove the ability to compile without logging and profiling · 8a6108de

sgjesse@chromium.org authored 13 years ago

The preprocessor defines ENABLE_LOGGING_AND_PROFILING and ENABLE_VMSTATE_TRACKING has been removed as these where required to be turned on for Crankshaft to work. To re-enable reducing the binary size by leaving out heap and CPU profiler a new set of defines needs to be created.

R=ager@chromium.org

BUG=v8:1271
TEST=all

Review URL: http://codereview.chromium.org//7350014

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@8622 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

8a6108de

11 Jul, 2011 1 commit

Remove heap protection support. · 15429da4

ager@chromium.org authored 13 years ago

It does not currently work and when it did work we never got it fast
enough to be useful.

R=kmillikin@chromium.org

Review URL: http://codereview.chromium.org/7324051

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@8601 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

15429da4

06 May, 2011 1 commit

A tiny contribution for the IWYU day: Include allocation.h in every · 5cd715cb

svenpanne@chromium.org authored 13 years ago

header which uses BASE_EMBEDDED and/or AllStatic. Note that still only
45 out of 135 headers in src/ can be used stand-alone, but at least
this is a little bit more than before...
Review URL: http://codereview.chromium.org/6931031

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@7798 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

5cd715cb

03 May, 2011 1 commit

Simplify include dependencies. · d0fcbb4e

kmillikin@chromium.org authored 13 years ago

Try to make sure that accessors.h, data-flow.h, list-inl.h, and
scopeinfo.h are included only where needed, but without introducing
implicit dependencies.

Review URL: http://codereview.chromium.org/6903175

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@7756 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

d0fcbb4e

22 Mar, 2011 1 commit

Cleanup of Isolate usage (guided by SunSpider profiling) · bb7f8fbe

dimich@chromium.org authored 14 years ago

Review URL: http://codereview.chromium.org/6718023

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@7311 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

bb7f8fbe

18 Mar, 2011 3 commits

Merge isolates to bleeding_edge. · 7976ca2c

vitalyr@chromium.org authored 14 years ago

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@7271 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

7976ca2c

Revert r7268: it borked the history. · 76e226f8

vitalyr@chromium.org authored 14 years ago

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@7269 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

76e226f8

Merge isolates to bleeding_edge. · 6ff7fdeb

vitalyr@chromium.org authored 14 years ago

Review URL: http://codereview.chromium.org/6685088

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@7268 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

6ff7fdeb

25 Feb, 2011 1 commit

Flush watermarks after compacting map space on mark-sweep collection. · 46f57bea

vegorov@chromium.org authored 14 years ago

MapCompact does not use MC-allocation routines so watermarks are not updated automatically and might become inconsistent.

BUG=http://crbug.com/59688

Review URL: http://codereview.chromium.org/6588007

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@6954 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

46f57bea

17 Dec, 2010 1 commit

Remove the atomic/thread safe stuff from the chunk table, since we don't · e24362b5

erik.corry@gmail.com authored 14 years ago

use cross thread synchronization in Crankshaft.
Review URL: http://codereview.chromium.org/5979001

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@6075 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

e24362b5

15 Dec, 2010 1 commit

Add some missing includes and other minor changes intended to keep · d25755ee

erik.corry@gmail.com authored 14 years ago

the distance between bleeding edge and the gc branch minimal.
Review URL: http://codereview.chromium.org/5788002

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@6016 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

d25755ee

08 Dec, 2010 1 commit

Speed up quoting of JSON strings by allocating a string that is big enough · a5c8cab7

erik.corry@gmail.com authored 14 years ago

and then trimming it when the length is known. This way we only have to
traverse the input once.
Review URL: http://codereview.chromium.org/5556012

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@5951 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

a5c8cab7

07 Dec, 2010 3 commits

Update V8 to version 3.0 (re-land r5920). · 90b33703

kasperl@chromium.org authored 14 years ago

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@5922 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

90b33703

Revert r5920. Will re-land shortly. · 51b494d0

kasperl@chromium.org authored 14 years ago

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@5921 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

51b494d0

Update V8 to version 3.0. · e5860bd6

kasperl@chromium.org authored 14 years ago

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@5920 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

e5860bd6

15 Nov, 2010 1 commit

Provide more accurate results about used heap size via GetHeapStatistics. · 2a8eb355

mikhail.naganov@gmail.com authored 14 years ago

I observed that used heap size provided by Heap::SizeOfObjects() is
usually about ~10% bigger than the number calculated by summing up
heap objects sizes.

This aligns DevTools Timeline stats with Heap profiler stats.

Review URL: http://codereview.chromium.org/4888001

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@5825 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

2a8eb355

13 Nov, 2010 1 commit

Fix Win64 build. · bfeca9e0

antonm@chromium.org authored 14 years ago

MemoryAllocator::AvailableExecutable should return intptr_t, the difference of intptr_t's.

TBR=ager@chromium.org

Review URL: http://codereview.chromium.org/4972001

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@5818 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

bfeca9e0