8329109: Threads::print_on() tries to print CPU time for terminated GC threads

Liang Mao · RealCLanger · commit fe4a2ae57072 · 2024-05-23T08:47:30.000Z
Backport-of: c1cfb43d8de8d0eed01b9bb6aa1202dc7cbff007
diff --git a/src/hotspot/share/runtime/threads.cpp b/src/hotspot/share/runtime/threads.cpp
@@ -1317,10 +1317,7 @@ void Threads::print_on(outputStream* st, bool print_stacks,
   }
 
   PrintOnClosure cl(st);
-  cl.do_thread(VMThread::vm_thread());
-  Universe::heap()->gc_threads_do(&cl);
-  cl.do_thread(WatcherThread::watcher_thread());
-  cl.do_thread(AsyncLogWriter::instance());
+  non_java_threads_do(&cl);
 
   st->flush();
 }

Original file line number	Diff line number	Diff line change
`@@ -1317,10 +1317,7 @@ void Threads::print_on(outputStream* st, bool print_stacks,`
`1317`	`1317`	`}`
`1318`	`1318`
`1319`	`1319`	`PrintOnClosure cl(st);`
`1320`		`- cl.do_thread(VMThread::vm_thread());`
`1321`		`- Universe::heap()->gc_threads_do(&cl);`
`1322`		`- cl.do_thread(WatcherThread::watcher_thread());`
`1323`		`- cl.do_thread(AsyncLogWriter::instance());`
	`1320`	`+ non_java_threads_do(&cl);`
`1324`	`1321`
`1325`	`1322`	`st->flush();`
`1326`	`1323`	`}`