8143013: Remove unused DirtyCardQueue::iterate_closure_all_threads
authorkbarrett
Mon, 16 Nov 2015 14:11:36 -0500
changeset 34141 1030e4216817
parent 34140 43b8383b63e2
child 34143 5d2afe31c768
8143013: Remove unused DirtyCardQueue::iterate_closure_all_threads Summary: Remove unused function. Reviewed-by: tbenson, tschatzl, mgerdin
hotspot/src/share/vm/gc/g1/dirtyCardQueue.cpp
hotspot/src/share/vm/gc/g1/dirtyCardQueue.hpp
--- a/hotspot/src/share/vm/gc/g1/dirtyCardQueue.cpp	Sat Nov 14 18:26:35 2015 +0000
+++ b/hotspot/src/share/vm/gc/g1/dirtyCardQueue.cpp	Mon Nov 16 14:11:36 2015 -0500
@@ -119,20 +119,6 @@
   t->dirty_card_queue().handle_zero_index();
 }
 
-void DirtyCardQueueSet::iterate_closure_all_threads(CardTableEntryClosure* cl,
-                                                    bool consume,
-                                                    uint worker_i) {
-  assert(SafepointSynchronize::is_at_safepoint(), "Must be at safepoint.");
-  for (JavaThread* t = Threads::first(); t; t = t->next()) {
-    bool b = t->dirty_card_queue().apply_closure(cl, consume);
-    guarantee(b, "Should not be interrupted.");
-  }
-  bool b = shared_dirty_card_queue()->apply_closure(cl,
-                                                    consume,
-                                                    worker_i);
-  guarantee(b, "Should not be interrupted.");
-}
-
 bool DirtyCardQueueSet::mut_process_buffer(void** buf) {
 
   // Used to determine if we had already claimed a par_id
--- a/hotspot/src/share/vm/gc/g1/dirtyCardQueue.hpp	Sat Nov 14 18:26:35 2015 +0000
+++ b/hotspot/src/share/vm/gc/g1/dirtyCardQueue.hpp	Mon Nov 16 14:11:36 2015 -0500
@@ -112,14 +112,6 @@
 
   static void handle_zero_index_for_thread(JavaThread* t);
 
-  // Apply the given closure to all entries in all currently-active buffers.
-  // This should only be applied at a safepoint. (Currently must not be called
-  // in parallel; this should change in the future.)  If "consume" is true,
-  // processed entries are discarded.
-  void iterate_closure_all_threads(CardTableEntryClosure* cl,
-                                   bool consume = true,
-                                   uint worker_i = 0);
-
   // If there exists some completed buffer, pop it, then apply the
   // specified closure to all its elements, nulling out those elements
   // processed.  If all elements are processed, returns "true".  If no