Mercurial > hg > graal-compiler
comparison src/share/vm/gc_implementation/g1/g1CollectorPolicy.cpp @ 3927:f1b4e0e0bdad
7089625: G1: policy for how many old regions to add to the CSet (when young gen is fixed) is broken
Summary: When refactoring the code for a previous fix, a condition was not correctly negated which prevents the G1 policy from adding the correct number of old regions to the CSet when the young gen size is fixed. The changeset also fixes a small syntactical issue in g1ErgoVerbose.hpp which is causing compiler warnings.
Reviewed-by: brutisso, ysr
| author | tonyp |
|---|---|
| date | Tue, 13 Sep 2011 12:40:14 -0400 |
| parents | af2ab04e0038 |
| children | d912b598c6c3 |
comparison
equal
deleted
inserted
replaced
| 3926:140317da459a | 3927:f1b4e0e0bdad |
|---|---|
| 3040 ergo_format_ms("remaining time"), | 3040 ergo_format_ms("remaining time"), |
| 3041 time_remaining_ms); | 3041 time_remaining_ms); |
| 3042 should_continue = false; | 3042 should_continue = false; |
| 3043 } | 3043 } |
| 3044 } else { | 3044 } else { |
| 3045 if (_collection_set_size < _young_list_fixed_length) { | 3045 if (_collection_set_size >= _young_list_fixed_length) { |
| 3046 ergo_verbose2(ErgoCSetConstruction, | 3046 ergo_verbose2(ErgoCSetConstruction, |
| 3047 "stop adding old regions to CSet", | 3047 "stop adding old regions to CSet", |
| 3048 ergo_format_reason("CSet length lower than target") | 3048 ergo_format_reason("CSet length reached target") |
| 3049 ergo_format_region("CSet") | 3049 ergo_format_region("CSet") |
| 3050 ergo_format_region("young target"), | 3050 ergo_format_region("young target"), |
| 3051 _collection_set_size, _young_list_fixed_length); | 3051 _collection_set_size, _young_list_fixed_length); |
| 3052 should_continue = false; | 3052 should_continue = false; |
| 3053 } | 3053 } |