Get rid of recursive locks for job based recompilation

src/optimizing-compiler-thread.h

 // Copyright 2012 the V8 project authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef V8_OPTIMIZING_COMPILER_THREAD_H_
 #define V8_OPTIMIZING_COMPILER_THREAD_H_
 
 #include "src/base/atomicops.h"
 #include "src/base/platform/mutex.h"
 #include "src/base/platform/platform.h"
@@ skipping 17 matching lines @@
         thread_id_(0),
 #endif
         isolate_(isolate),
         stop_semaphore_(0),
         input_queue_semaphore_(0),
         input_queue_capacity_(FLAG_concurrent_recompilation_queue_length),
         input_queue_length_(0),
         input_queue_shift_(0),
         osr_buffer_capacity_(FLAG_concurrent_recompilation_queue_length + 4),
         osr_buffer_cursor_(0),
-        task_count_(0),
         osr_hits_(0),
         osr_attempts_(0),
         blocked_jobs_(0),
+        ref_count_(1),
         tracing_enabled_(FLAG_trace_concurrent_recompilation),
         job_based_recompilation_(FLAG_job_based_recompilation),
         recompilation_delay_(FLAG_concurrent_recompilation_delay) {
     base::NoBarrier_Store(&stop_thread_,
                           static_cast<base::AtomicWord>(CONTINUE));
     input_queue_ = NewArray<OptimizedCompileJob*>(input_queue_capacity_);
     if (FLAG_concurrent_osr) {
       // Allocate and mark OSR buffer slots as empty.
       osr_buffer_ = NewArray<OptimizedCompileJob*>(osr_buffer_capacity_);
       for (int i = 0; i < osr_buffer_capacity_; i++) osr_buffer_[i] = NULL;
@@ skipping 81 matching lines @@
 
   // Cyclic buffer of recompilation tasks for OSR.
   OptimizedCompileJob** osr_buffer_;
   int osr_buffer_capacity_;
   int osr_buffer_cursor_;
 
   volatile base::AtomicWord stop_thread_;
   base::TimeDelta time_spent_compiling_;
   base::TimeDelta time_spent_total_;
 
-  int task_count_;
-  // TODO(jochen): This is currently a RecursiveMutex since both Flush/Stop and
-  // Unblock try to get it, but the former methods both can call Unblock. Once
-  // job based recompilation is on by default, and the dedicated thread can be
-  // removed, this should be refactored to not use a RecursiveMutex.
-  base::RecursiveMutex task_count_mutex_;
-
   int osr_hits_;
   int osr_attempts_;
 
   int blocked_jobs_;
 
+  volatile base::AtomicWord ref_count_;
+
   // Copies of FLAG_trace_concurrent_recompilation,
   // FLAG_concurrent_recompilation_delay and
   // FLAG_job_based_recompilation that will be used from the background thread.
   //
   // Since flags might get modified while the background thread is running, it
   // is not safe to access them directly.
   bool tracing_enabled_;
   bool job_based_recompilation_;
   int recompilation_delay_;
 };
 
 } }  // namespace v8::internal
 
 #endif  // V8_OPTIMIZING_COMPILER_THREAD_H_