commit 226805d9b81ba442251d0e2c4baedc36fcda6592
author Nicolas Geoffray <ngeoffray@google.com> Fri Dec 14 10:59:02 2018 +0000
committer Nicolas Geoffray <ngeoffray@google.com> Wed Dec 19 09:22:01 2018 +0000
tree 3a256b1d36894d8d43ec9650f2a3f55c90c4730c
parent 96453b7d6aa445c6609aa465df5d66ae0beb7956

Deoptimize zygote compiled methods in DeoptimizeBootImage.

Those methods don't get compiled with the "debuggable" flag,
so we need to deoptimize them.

Also fix a bug revealed by the new test where a concurrent
JIT collection happens when trying to disable it.

Also make DeoptimizeBootImage truly mutator lock exclusive.

Test: 689-zygote-jit-deopt
Change-Id: I00607dbe100350c5328293c35c87946fa97924b8

openjdkjvmti/deopt_manager.h

diff --git a/openjdkjvmti/deopt_manager.h b/openjdkjvmti/deopt_manager.h
index d9f34a5..065baa7 100644
--- a/openjdkjvmti/deopt_manager.h
+++ b/openjdkjvmti/deopt_manager.h
@@ -107,7 +107,7 @@
 
   void FinishSetup()
       REQUIRES(!deoptimization_status_lock_, !art::Roles::uninterruptible_)
-      REQUIRES_SHARED(art::Locks::mutator_lock_);
+      REQUIRES(art::Locks::mutator_lock_);
 
   static DeoptManager* Get();