author | brutisso |
Thu, 07 Aug 2014 09:35:08 +0200 | |
changeset 25909 | 571781915421 |
parent 25908 | 8adb2fb6fc3c |
child 25910 | afebf4c38c3f |
child 25941 | cbe8a62f98ce |
--- a/hotspot/src/share/vm/gc_implementation/g1/g1RemSet.cpp Wed Aug 06 09:55:16 2014 +0200 +++ b/hotspot/src/share/vm/gc_implementation/g1/g1RemSet.cpp Thu Aug 07 09:35:08 2014 +0200 @@ -349,23 +349,8 @@ assert((ParallelGCThreads > 0) || worker_i == 0, "invariant"); - // The two flags below were introduced temporarily to serialize - // the updating and scanning of remembered sets. There are some - // race conditions when these two operations are done in parallel - // and they are causing failures. When we resolve said race - // conditions, we'll revert back to parallel remembered set - // updating and scanning. See CRs 6677707 and 6677708. - if (G1UseParallelRSetUpdating || (worker_i == 0)) { - updateRS(&into_cset_dcq, worker_i); - } else { - _g1p->phase_times()->record_update_rs_processed_buffers(worker_i, 0); - _g1p->phase_times()->record_update_rs_time(worker_i, 0.0); - } - if (G1UseParallelRSetScanning || (worker_i == 0)) { - scanRS(oc, code_root_cl, worker_i); - } else { - _g1p->phase_times()->record_scan_rs_time(worker_i, 0.0); - } + updateRS(&into_cset_dcq, worker_i); + scanRS(oc, code_root_cl, worker_i); // We now clear the cached values of _cset_rs_update_cl for this worker _cset_rs_update_cl[worker_i] = NULL;
--- a/hotspot/src/share/vm/gc_implementation/g1/g1_globals.hpp Wed Aug 06 09:55:16 2014 +0200 +++ b/hotspot/src/share/vm/gc_implementation/g1/g1_globals.hpp Thu Aug 07 09:35:08 2014 +0200 @@ -220,14 +220,6 @@ product(uintx, G1HeapRegionSize, 0, \ "Size of the G1 regions.") \ \ - experimental(bool, G1UseParallelRSetUpdating, true, \ - "Enables the parallelization of remembered set updating " \ - "during evacuation pauses") \ - \ - experimental(bool, G1UseParallelRSetScanning, true, \ - "Enables the parallelization of remembered set scanning " \ - "during evacuation pauses") \ - \ product(uintx, G1ConcRefinementThreads, 0, \ "If non-0 is the number of parallel rem set update threads, " \ "otherwise the value is determined ergonomically.") \