comparison src/share/vm/gc_implementation/parNew/parCardTableModRefBS.cpp @ 3256:c69b1043dfb1

7036482: clear argument is redundant and unused in cardtable methods Summary: Removed the unused clear argument to various cardtbale methods and unused mod_oop_in_space_iterate method. Unrelated to synopsis, added a pair of clarifying parens in AllocationStats constructor. Reviewed-by: brutisso, jcoomes
author ysr
date Thu, 14 Apr 2011 12:10:15 -0700
parents f95d63e2154a
children c48ad6ab8bdf
comparison
equal deleted inserted replaced
3254:59766fd005ff 3256:c69b1043dfb1
1 /* 1 /*
2 * Copyright (c) 2007, 2010 Oracle and/or its affiliates. All rights reserved. 2 * Copyright (c) 2007, 2011 Oracle and/or its affiliates. All rights reserved.
3 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. 3 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
4 * 4 *
5 * This code is free software; you can redistribute it and/or modify it 5 * This code is free software; you can redistribute it and/or modify it
6 * under the terms of the GNU General Public License version 2 only, as 6 * under the terms of the GNU General Public License version 2 only, as
7 * published by the Free Software Foundation. 7 * published by the Free Software Foundation.
34 #include "runtime/virtualspace.hpp" 34 #include "runtime/virtualspace.hpp"
35 35
36 void CardTableModRefBS::par_non_clean_card_iterate_work(Space* sp, MemRegion mr, 36 void CardTableModRefBS::par_non_clean_card_iterate_work(Space* sp, MemRegion mr,
37 DirtyCardToOopClosure* dcto_cl, 37 DirtyCardToOopClosure* dcto_cl,
38 MemRegionClosure* cl, 38 MemRegionClosure* cl,
39 bool clear,
40 int n_threads) { 39 int n_threads) {
41 if (n_threads > 0) { 40 if (n_threads > 0) {
42 assert((n_threads == 1 && ParallelGCThreads == 0) || 41 assert((n_threads == 1 && ParallelGCThreads == 0) ||
43 n_threads <= (int)ParallelGCThreads, 42 n_threads <= (int)ParallelGCThreads,
44 "# worker threads != # requested!"); 43 "# worker threads != # requested!");
55 pst->set_n_threads(n_threads); 54 pst->set_n_threads(n_threads);
56 pst->set_n_tasks(n_strides); 55 pst->set_n_tasks(n_strides);
57 56
58 int stride = 0; 57 int stride = 0;
59 while (!pst->is_task_claimed(/* reference */ stride)) { 58 while (!pst->is_task_claimed(/* reference */ stride)) {
60 process_stride(sp, mr, stride, n_strides, dcto_cl, cl, clear, 59 process_stride(sp, mr, stride, n_strides, dcto_cl, cl,
61 lowest_non_clean, 60 lowest_non_clean,
62 lowest_non_clean_base_chunk_index, 61 lowest_non_clean_base_chunk_index,
63 lowest_non_clean_chunk_size); 62 lowest_non_clean_chunk_size);
64 } 63 }
65 if (pst->all_tasks_completed()) { 64 if (pst->all_tasks_completed()) {
81 process_stride(Space* sp, 80 process_stride(Space* sp,
82 MemRegion used, 81 MemRegion used,
83 jint stride, int n_strides, 82 jint stride, int n_strides,
84 DirtyCardToOopClosure* dcto_cl, 83 DirtyCardToOopClosure* dcto_cl,
85 MemRegionClosure* cl, 84 MemRegionClosure* cl,
86 bool clear,
87 jbyte** lowest_non_clean, 85 jbyte** lowest_non_clean,
88 uintptr_t lowest_non_clean_base_chunk_index, 86 uintptr_t lowest_non_clean_base_chunk_index,
89 size_t lowest_non_clean_chunk_size) { 87 size_t lowest_non_clean_chunk_size) {
90 // We don't have to go downwards here; it wouldn't help anyway, 88 // We don't have to go downwards here; it wouldn't help anyway,
91 // because of parallelism. 89 // because of parallelism.
127 used, 125 used,
128 lowest_non_clean, 126 lowest_non_clean,
129 lowest_non_clean_base_chunk_index, 127 lowest_non_clean_base_chunk_index,
130 lowest_non_clean_chunk_size); 128 lowest_non_clean_chunk_size);
131 129
132 non_clean_card_iterate_work(chunk_mr, cl, clear); 130 non_clean_card_iterate_work(chunk_mr, cl);
133 131
134 // Find the next chunk of the stride. 132 // Find the next chunk of the stride.
135 chunk_card_start += CardsPerStrideChunk * n_strides; 133 chunk_card_start += CardsPerStrideChunk * n_strides;
136 } 134 }
137 } 135 }