|
Last change
on this file since 48593 was 47022, checked in by [email protected], 16 years ago |
|
Stack overflow crash in JavaScript garbage collector mark pass
https://bugs.webkit.org/show_bug.cgi?id=12216
Reviewed by Gavin Barraclough and Sam Weinig
Make the GC mark phase iterative by using an explicit mark stack.
To do this marking any single object is performed in multiple stages
- The object is appended to the MarkStack, this sets the marked
bit for the object using the new markDirect() function, and then
returns
- When the MarkStack is drain()ed the object is popped off the stack
and markChildren(MarkStack&) is called on the object to collect
all of its children. drain() then repeats until the stack is empty.
Additionally I renamed a number of methods from 'mark' to 'markAggregate'
in order to make it more clear that marking of those object was not
going to result in an actual recursive mark.
|
-
Property svn:eol-style
set to
native
|
|
File size:
1.1 KB
|
| Line | |
|---|
| 1 | /*
|
|---|
| 2 | * Copyright (C) 2006 Maks Orlovich
|
|---|
| 3 | * Copyright (C) 2006, 2009 Apple, Inc.
|
|---|
| 4 | *
|
|---|
| 5 | * This library is free software; you can redistribute it and/or
|
|---|
| 6 | * modify it under the terms of the GNU Library General Public
|
|---|
| 7 | * License as published by the Free Software Foundation; either
|
|---|
| 8 | * version 2 of the License, or (at your option) any later version.
|
|---|
| 9 | *
|
|---|
| 10 | * This library is distributed in the hope that it will be useful,
|
|---|
| 11 | * but WITHOUT ANY WARRANTY; without even the implied warranty of
|
|---|
| 12 | * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
|
|---|
| 13 | * Library General Public License for more details.
|
|---|
| 14 | *
|
|---|
| 15 | * You should have received a copy of the GNU Library General Public License
|
|---|
| 16 | * along with this library; see the file COPYING.LIB. If not, write to
|
|---|
| 17 | * the Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor,
|
|---|
| 18 | * Boston, MA 02110-1301, USA.
|
|---|
| 19 | *
|
|---|
| 20 | */
|
|---|
| 21 |
|
|---|
| 22 | #include "config.h"
|
|---|
| 23 | #include "JSWrapperObject.h"
|
|---|
| 24 |
|
|---|
| 25 | namespace JSC {
|
|---|
| 26 |
|
|---|
| 27 | ASSERT_CLASS_FITS_IN_CELL(JSWrapperObject);
|
|---|
| 28 |
|
|---|
| 29 | void JSWrapperObject::markChildren(MarkStack& markStack)
|
|---|
| 30 | {
|
|---|
| 31 | JSObject::markChildren(markStack);
|
|---|
| 32 | if (m_internalValue)
|
|---|
| 33 | markStack.append(m_internalValue);
|
|---|
| 34 | }
|
|---|
| 35 |
|
|---|
| 36 | } // namespace JSC
|
|---|
Note:
See
TracBrowser
for help on using the repository browser.
