RFR (S): 8076995: gc/ergonomics/TestDynamicNumberOfGCThreads.java failed with java.lang.RuntimeException: 'new_active_workers' missing from stdout/stderr
derek.white at oracle.com
Mon Apr 27 21:02:10 UTC 2015
On 4/27/15 4:04 AM, Bengt Rutisson wrote:
> Hi Derek,
> On 2015-04-23 23:55, Derek White wrote:
>> 2nd webrev:
>> Please review this fix for:
>> - Updated TEST.groups to only run this test if G1, CMS, and Parallel
>> GCs are enabled.
>> - Also searched for similar GC tests that specify a GC to use and
>> added to TESTS.groups:
>> * gc/TestSmallHeap.java
>> * gc/logging/TestGCId.java
>> * gc/TestCardTablePageCommits.java
>> * gc/arguments/TestParallelHeapSizeFlags.java
> I think I would prefer this as a separate change. I'm not very good
> with TEST.groups, but it looks like your suggested changes are
> correct. However, I would prefer the get more attention to that change
> and thus it would be better to send it out as a separate review.
I read this as a request to split out the changes in TEST.groups for the
other tests to a separate bug, review, and checkin. Is that what you
meant? But I've seen some fancy partial reviews floating around, so
maybe I misunderstood.
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the hotspot-gc-dev