Revision: 20504
Author: [email protected]
Date: Fri Apr 4 09:30:01 2014 UTC
Log: Version 3.25.28.5 (merged r20357)
Do not check for interrupt when allocating stack locals.
BUG=357137
LOG=N
[email protected]
Review URL: https://codereview.chromium.org/225623002
http://code.google.com/p/v8/source/detail?r=20504
Modified:
/branches/3.25/src/arm/full-codegen-arm.cc
/branches/3.25/src/arm64/full-codegen-arm64.cc
/branches/3.25/src/ia32/full-codegen-ia32.cc
/branches/3.25/src/mips/full-codegen-mips.cc
/branches/3.25/src/version.cc
/branches/3.25/src/x64/full-codegen-x64.cc
/branches/3.25/test/cctest/test-heap.cc
=======================================
--- /branches/3.25/src/arm/full-codegen-arm.cc Thu Mar 27 01:04:43 2014 UTC
+++ /branches/3.25/src/arm/full-codegen-arm.cc Fri Apr 4 09:30:01 2014 UTC
@@ -118,10 +118,14 @@
Isolate* isolate = masm_->isolate();
Label ok;
ASSERT(scratch.is(sp) == (pointers == 0));
+ Heap::RootListIndex index;
if (pointers != 0) {
__ sub(scratch, sp, Operand(pointers * kPointerSize));
+ index = Heap::kRealStackLimitRootIndex;
+ } else {
+ index = Heap::kStackLimitRootIndex;
}
- __ LoadRoot(stack_limit_scratch, Heap::kStackLimitRootIndex);
+ __ LoadRoot(stack_limit_scratch, index);
__ cmp(scratch, Operand(stack_limit_scratch));
__ b(hs, &ok);
PredictableCodeSizeScope predictable(masm_, 2 * Assembler::kInstrSize);
=======================================
--- /branches/3.25/src/arm64/full-codegen-arm64.cc Thu Mar 27 01:04:43 2014
UTC
+++ /branches/3.25/src/arm64/full-codegen-arm64.cc Fri Apr 4 09:30:01 2014
UTC
@@ -117,10 +117,14 @@
Label ok;
ASSERT(jssp.Is(__ StackPointer()));
ASSERT(scratch.Is(jssp) == (pointers == 0));
+ Heap::RootListIndex index;
if (pointers != 0) {
__ Sub(scratch, jssp, pointers * kPointerSize);
+ index = Heap::kRealStackLimitRootIndex;
+ } else {
+ index = Heap::kStackLimitRootIndex;
}
- __ CompareRoot(scratch, Heap::kStackLimitRootIndex);
+ __ CompareRoot(scratch, index);
__ B(hs, &ok);
PredictableCodeSizeScope predictable(masm_,
Assembler::kCallSizeWithRelocation);
=======================================
--- /branches/3.25/src/ia32/full-codegen-ia32.cc Thu Mar 27 01:04:43 2014
UTC
+++ /branches/3.25/src/ia32/full-codegen-ia32.cc Fri Apr 4 09:30:01 2014
UTC
@@ -106,12 +106,14 @@
Register scratch = esp) {
Label ok;
Isolate* isolate = masm_->isolate();
- ExternalReference stack_limit =
- ExternalReference::address_of_stack_limit(isolate);
ASSERT(scratch.is(esp) == (pointers == 0));
+ ExternalReference stack_limit;
if (pointers != 0) {
__ mov(scratch, esp);
__ sub(scratch, Immediate(pointers * kPointerSize));
+ stack_limit =
ExternalReference::address_of_real_stack_limit(isolate);
+ } else {
+ stack_limit = ExternalReference::address_of_stack_limit(isolate);
}
__ cmp(scratch, Operand::StaticVariable(stack_limit));
__ j(above_equal, &ok, Label::kNear);
=======================================
--- /branches/3.25/src/mips/full-codegen-mips.cc Thu Mar 27 01:04:43 2014
UTC
+++ /branches/3.25/src/mips/full-codegen-mips.cc Fri Apr 4 09:30:01 2014
UTC
@@ -127,10 +127,14 @@
Isolate* isolate = masm_->isolate();
Label ok;
ASSERT(scratch.is(sp) == (pointers == 0));
+ Heap::RootListIndex index;
if (pointers != 0) {
__ Subu(scratch, sp, Operand(pointers * kPointerSize));
+ index = Heap::kRealStackLimitRootIndex;
+ } else {
+ index = Heap::kStackLimitRootIndex;
}
- __ LoadRoot(stack_limit_scratch, Heap::kStackLimitRootIndex);
+ __ LoadRoot(stack_limit_scratch, index);
__ Branch(&ok, hs, scratch, Operand(stack_limit_scratch));
PredictableCodeSizeScope predictable(masm_, 4 * Assembler::kInstrSize);
__ Call(isolate->builtins()->StackCheck(), RelocInfo::CODE_TARGET);
=======================================
--- /branches/3.25/src/version.cc Thu Apr 3 10:22:31 2014 UTC
+++ /branches/3.25/src/version.cc Fri Apr 4 09:30:01 2014 UTC
@@ -35,7 +35,7 @@
#define MAJOR_VERSION 3
#define MINOR_VERSION 25
#define BUILD_NUMBER 28
-#define PATCH_LEVEL 4
+#define PATCH_LEVEL 5
// Use 1 for candidates and 0 otherwise.
// (Boolean macro values are not supported by all preprocessors.)
#define IS_CANDIDATE_VERSION 0
=======================================
--- /branches/3.25/src/x64/full-codegen-x64.cc Thu Mar 27 01:04:43 2014 UTC
+++ /branches/3.25/src/x64/full-codegen-x64.cc Fri Apr 4 09:30:01 2014 UTC
@@ -107,11 +107,15 @@
Isolate* isolate = masm_->isolate();
Label ok;
ASSERT(scratch.is(rsp) == (pointers == 0));
+ Heap::RootListIndex index;
if (pointers != 0) {
__ movq(scratch, rsp);
__ subq(scratch, Immediate(pointers * kPointerSize));
+ index = Heap::kRealStackLimitRootIndex;
+ } else {
+ index = Heap::kStackLimitRootIndex;
}
- __ CompareRoot(scratch, Heap::kStackLimitRootIndex);
+ __ CompareRoot(scratch, index);
__ j(above_equal, &ok, Label::kNear);
__ call(isolate->builtins()->StackCheck(), RelocInfo::CODE_TARGET);
__ bind(&ok);
=======================================
--- /branches/3.25/test/cctest/test-heap.cc Thu Mar 27 01:04:43 2014 UTC
+++ /branches/3.25/test/cctest/test-heap.cc Fri Apr 4 09:30:01 2014 UTC
@@ -3890,3 +3890,32 @@
heap->CollectAllGarbage(Heap::kAbortIncrementalMarkingMask);
}
#endif
+
+
+static void InterruptCallback357137(v8::Isolate* isolate, void* data) { }
+
+
+static void RequestInterrupt(const v8::FunctionCallbackInfo<v8::Value>&
args) {
+ CcTest::isolate()->RequestInterrupt(&InterruptCallback357137, NULL);
+}
+
+
+TEST(Regress357137) {
+ CcTest::InitializeVM();
+ v8::Isolate* isolate = CcTest::isolate();
+ v8::HandleScope hscope(isolate);
+ v8::Handle<v8::ObjectTemplate> global =v8::ObjectTemplate::New(isolate);
+ global->Set(v8::String::NewFromUtf8(isolate, "interrupt"),
+ v8::FunctionTemplate::New(isolate, RequestInterrupt));
+ v8::Local<v8::Context> context = v8::Context::New(isolate, NULL, global);
+ ASSERT(!context.IsEmpty());
+ v8::Context::Scope cscope(context);
+
+ v8::Local<v8::Value> result = CompileRun(
+ "var locals = '';"
+ "for (var i = 0; i < 512; i++) locals += 'var v' + i + '= 42;';"
+ "eval('function f() {' + locals + 'return function() { return v0; };
}');"
+ "interrupt();" // This triggers a fake stack overflow in f.
+ "f()()");
+ CHECK_EQ(42.0, result->ToNumber()->Value());
+}
--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
---
You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/d/optout.
