RFR: 8009723 CMS logs "concurrent mode failure" twice when using (disabling) -XX:-UseCMSCompactAtFullCollection

[Kevin Walls]

Thanks Jesper, Erik, Bengt,

I'll go ahead with this now so I can then come back with a refreshed 
webrev for the CMS Concurrent Mode Failure event (8008917).

Thanks
Kevin


#On 11/03/13 11:02, Jesper Wilhelmsson wrote:
> Looks good!
> I agree with not adding the test.
> /Jesper
>
> On 9/3/13 12:39 AM, Kevin Walls wrote:
>> Hi,
>>
>> This is a review request for a small change:
>>
>> http://cr.openjdk.java.net/~kevinw/8009723/webrev/
>>
>> Using CMS and disabling UseCMSCompactAtFullCollection on the command 
>> line easily
>> produces a double-logging of the "(concurrent mode failure)" 
>> message.  This was
>> spotted by Erik Helin.
>>
>> This is like removing a print statement.
>>
>> The webrev contains a testcase, but I am currently suggesting I 
>> commit this
>> without the test as:
>>
>> 1) it's fairly trivial: remove a print statement, you don't the see 
>> output of
>> that print statement.
>> 2) it requires -XX:-UseCMSCompactAtFullCollection which I don't 
>> believe is
>> commonly used
>> 3) it is a CMS-specific and the current test framework is quite 
>> awkward to make
>> the test specific (manual work in the java test, or re-exec.  If 
>> re-execing do
>> we ignore TESTVMOPTS or parse it and add UseConcMarkSweepGC?... a few 
>> issues
>> there we should solve but this may not be the test change that needs 
>> that...)
>>
>> So the test is possible, and useful for verifying it if anybody wants 
>> to run it
>> right now, but does not seem a necessity and for this small change 
>> adding to the
>> test harness library to make it more elegant seems unnecessary for 
>> this minor
>> problem...
>>
>> Thanks
>> Kevin
>>