RFR (S): 8026043: add regression test for JDK-8000831
Filipp Zhinkin
filipp.zhinkin at oracle.com
Mon Oct 14 10:26:35 UTC 2013
Hi,
I've refactored options parsing, here is updated webrev:
http://cr.openjdk.java.net/~kshefov/8026043/webrev.01/
Note that I'm using ProcessTools.createVerboseJavaProcessBuilder from
8026047
to make sure that VM output will not be suppressed or redirected to file.
Thanks,
Filipp.
On 10/11/2013 08:22 PM, Tao Mao wrote:
> Ok, thank you for heads-up. Then, I'll await your webrev update.
>
> Thanks.
> Tao
>
> On 10/11/13 9:11 AM, Filipp Zhinkin wrote:
>> Hi Tao,
>>
>> Sorry for typo. You're right, the correct one is
>> http://cr.openjdk.java.net/~kshefov/8026043/webrev.00/.
>>
>> And I'll refactor it in the same way as fix for 8026047.
>>
>> Thanks,
>> Filipp.
>>
>> On 10/11/2013 08:11 PM, Tao Mao wrote:
>>> Hi Filipp,
>>>
>>> FYI, it has a wrong webrev. I guess it should be
>>> http://cr.openjdk.java.net/~kshefov/8026043/webrev.00/
>>>
>>> Thanks.
>>> Tao
>>>
>>> On 10/8/13 8:34 AM, Filipp Zhinkin wrote:
>>>> Hi,
>>>>
>>>> I would like to get couple review on test, that cover issue fixed
>>>> by 8000831 - Heap verification output incorrect/incomplete.
>>>>
>>>> Test verifies that options VerifyBeforeGC and VerifyAfterGCemit
>>>> appropriate output if they are turned on and never
>>>> emit corrupted output as it was before 8000831 fixed.
>>>>
>>>> Bug ID: https://bugs.openjdk.java.net/browse/JDK-8026043
>>>> Webrev: http://cr.openjdk.java.net/~kshefov/8026047/webrev.00/
>>>> Testing: manual on local host, automated on various platforms.
>>>>
>>>> Thanks,
>>>> Filipp.
>>
More information about the hotspot-gc-dev
mailing list