src/hotspot/share/runtime/os.cpp

rev 47287 : Port 09.17.Thread_SMR_logging_update from JDK9 to JDK10
rev 47289 : eosterlund, stefank CR - refactor code into threadSMR.cpp and threadSMR.hpp
rev 47292 : stefank, coleenp CR - refactor most JavaThreadIterator usage to use JavaThreadIteratorWithHandle.

@@ -1091,14 +1091,11 @@
     st->print_cr(INTPTR_FORMAT " is a local jni handle", p2i(addr));
     return;
   }
 #endif
 
-  {
-    ThreadsListHandle tlh;
-    JavaThreadIterator jti(tlh.list());
-    for (JavaThread *thread = jti.first(); thread != NULL; thread = jti.next()) {
+  for (JavaThreadIteratorWithHandle jtiwh; JavaThread *thread = jtiwh.next(); ) {
       // Check for privilege stack
       if (thread->privileged_stack_top() != NULL &&
           thread->privileged_stack_top()->contains(addr)) {
         st->print_cr(INTPTR_FORMAT " is pointing into the privilege stack "
                      "for thread: " INTPTR_FORMAT, p2i(addr), p2i(thread));

@@ -1121,11 +1118,10 @@
                      INTPTR_FORMAT, p2i(addr), p2i(thread));
         if (verbose) thread->print_on(st);
         return;
       }
     }
-  } // ThreadsListHandle destroyed here.
 
   // Check if in metaspace and print types that have vptrs (only method now)
   if (Metaspace::contains(addr)) {
     if (Method::has_method_vptr((const void*)addr)) {
       ((Method*)addr)->print_value_on(st);