# HG changeset patch # User kbarrett # Date 1447701096 18000 # Node ID 1030e421681781d8d900675d35dedecb555bf7bf # Parent 43b8383b63e29d57b5c1fd2c4504f6ad5a577026 8143013: Remove unused DirtyCardQueue::iterate_closure_all_threads Summary: Remove unused function. Reviewed-by: tbenson, tschatzl, mgerdin diff -r 43b8383b63e2 -r 1030e4216817 hotspot/src/share/vm/gc/g1/dirtyCardQueue.cpp --- 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 diff -r 43b8383b63e2 -r 1030e4216817 hotspot/src/share/vm/gc/g1/dirtyCardQueue.hpp --- 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