Revert "ART: Ensure plugin is loaded on AttachAgent"

Test fails.
Bug: 31682382

This reverts commit ad2dc875ce29c3fafc1129a2a25184df28b66a10.

Change-Id: Id15d9d5478d2f72a4dc59f119944a311be7bc989

1 files changed, 4 insertions, 44 deletions

diff --git a/runtime/runtime.cc b/runtime/runtime.cc
index df5fc5ce8a..2086d70791 100644
--- a/runtime/runtime.cc
+++ b/runtime/runtime.cc
@@ -1364,39 +1364,6 @@ bool Runtime::Init(RuntimeArgumentMap&& runtime_options_in) {
   return true;
 }
 
-static bool EnsureJvmtiPlugin(Runtime* runtime,
-                              std::vector<Plugin>* plugins,
-                              std::string* error_msg) {
-  constexpr const char* plugin_name = kIsDebugBuild ? "libopenjdkjvmtid.so" : "libopenjdkjvmti.so";
-
-  // Is the plugin already loaded?
-  for (Plugin p : *plugins) {
-    if (p.GetLibrary() == plugin_name) {
-      return true;
-    }
-  }
-
-  // Is the process debuggable? Otherwise, do not attempt to load the plugin.
-  if (!runtime->IsDebuggable()) {
-    *error_msg = "Process is not debuggable.";
-    return false;
-  }
-
-  Plugin new_plugin = Plugin::Create(plugin_name);
-
-  // Suspend all threads to protect ourself somewhat.
-  Thread* self = Thread::Current();
-  ScopedObjectAccess soa(self);      // Now we know we have the shared lock.
-  ScopedThreadSuspension sts(self, art::kWaitingForDebuggerToAttach);
-  ScopedSuspendAll ssa("EnsureJvmtiPlugin");
-  if (!new_plugin.Load(error_msg)) {
-    return false;
-  }
-
-  plugins->push_back(std::move(new_plugin));
-  return true;
-}
-
 // Attach a new agent and add it to the list of runtime agents
 //
 // TODO: once we decide on the threading model for agents,
@@ -1404,25 +1371,18 @@ static bool EnsureJvmtiPlugin(Runtime* runtime,
 //   (and we synchronize access to any shared data structures like "agents_")
 //
 void Runtime::AttachAgent(const std::string& agent_arg) {
-  std::string error_msg;
-  if (!EnsureJvmtiPlugin(this, &plugins_, &error_msg)) {
-    LOG(WARNING) << "Could not load plugin: " << error_msg;
-    ScopedObjectAccess soa(Thread::Current());
-    ThrowIOException("%s", error_msg.c_str());
-    return;
-  }
-
   ti::Agent agent(agent_arg);
 
   int res = 0;
-  ti::Agent::LoadError result = agent.Attach(&res, &error_msg);
+  std::string err;
+  ti::Agent::LoadError result = agent.Attach(&res, &err);
 
   if (result == ti::Agent::kNoError) {
     agents_.push_back(std::move(agent));
   } else {
-    LOG(WARNING) << "Agent attach failed (result=" << result << ") : " << error_msg;
+    LOG(ERROR) << "Agent attach failed (result=" << result << ") : " << err;
     ScopedObjectAccess soa(Thread::Current());
-    ThrowIOException("%s", error_msg.c_str());
+    ThrowWrappedIOException("%s", err.c_str());
   }
 }