Reviewers: Sven Panne,
Description:
Revert interrupt handling code changed in r21208.
[email protected]
Please review this at https://codereview.chromium.org/277233004/
SVN Base: https://v8.googlecode.com/svn/branches/bleeding_edge
Affected files (+16, -22 lines):
M src/execution.h
M src/execution.cc
Index: src/execution.cc
diff --git a/src/execution.cc b/src/execution.cc
index
ee6cb970cb8bd6d852f3e34a681e82d9f3efb686..ee359e722f68f45490778a6800e7e377c98328f7
100644
--- a/src/execution.cc
+++ b/src/execution.cc
@@ -388,18 +388,6 @@ void StackGuard::ClearInterrupt(int flagbit) {
}
-bool StackGuard::CheckAndClearInterrupt(InterruptFlag flag,
- const ExecutionAccess& lock) {
- int flagbit = 1 << flag;
- bool result = (thread_local_.interrupt_flags_ & flagbit);
- thread_local_.interrupt_flags_ &= ~flagbit;
- if (!should_postpone_interrupts(lock) && !has_pending_interrupts(lock)) {
- reset_limits(lock);
- }
- return result;
-}
-
-
char* StackGuard::ArchiveStackGuard(char* to) {
ExecutionAccess access(isolate_);
OS::MemCopy(to, reinterpret_cast<char*>(&thread_local_),
sizeof(ThreadLocal));
@@ -732,37 +720,44 @@ void Execution::ProcessDebugMessages(Isolate* isolate,
Object* StackGuard::HandleInterrupts() {
- ExecutionAccess access(isolate_);
- if (should_postpone_interrupts(access)) {
- return isolate_->heap()->undefined_value();
+ { ExecutionAccess access(isolate_);
+ if (should_postpone_interrupts(access)) {
+ return isolate_->heap()->undefined_value();
+ }
}
- if (CheckAndClearInterrupt(API_INTERRUPT, access)) {
+ if (CheckApiInterrupt()) {
+ ClearApiInterrupt();
isolate_->InvokeApiInterruptCallback();
}
- if (CheckAndClearInterrupt(GC_REQUEST, access)) {
+ if (CheckGC()) {
isolate_->heap()->CollectAllGarbage(Heap::kNoGCFlags, "GC interrupt");
+ ClearGC();
}
if (CheckDebugBreak() || CheckDebugCommand()) {
Execution::DebugBreakHelper(isolate_);
}
- if (CheckAndClearInterrupt(TERMINATE_EXECUTION, access)) {
+ if (CheckTerminateExecution()) {
+ ClearTerminateExecution();
return isolate_->TerminateExecution();
}
- if (CheckAndClearInterrupt(FULL_DEOPT, access)) {
+ if (CheckFullDeopt()) {
+ ClearFullDeopt();
Deoptimizer::DeoptimizeAll(isolate_);
}
- if (CheckAndClearInterrupt(DEOPT_MARKED_ALLOCATION_SITES, access)) {
+ if (CheckDeoptMarkedAllocationSites()) {
+ ClearDeoptMarkedAllocationSites();
isolate_->heap()->DeoptMarkedAllocationSites();
}
- if (CheckAndClearInterrupt(INSTALL_CODE, access)) {
+ if (CheckInstallCode()) {
ASSERT(isolate_->concurrent_recompilation_enabled());
+ ClearInstallCode();
isolate_->optimizing_compiler_thread()->InstallOptimizedFunctions();
}
Index: src/execution.h
diff --git a/src/execution.h b/src/execution.h
index
6d0b15f214762a65082492ad7fe7af826893dd15..8aaa7b42288993b3bc0279d3302c2bc0f8df5e0c
100644
--- a/src/execution.h
+++ b/src/execution.h
@@ -210,7 +210,6 @@ enum InterruptFlag {
bool CheckInterrupt(int flagbit);
void RequestInterrupt(int flagbit);
void ClearInterrupt(int flagbit);
- bool CheckAndClearInterrupt(InterruptFlag flag, const ExecutionAccess&
lock);
void InvokeApiInterruptCallback();
--
--
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.
