RFR: 8057135: [TESTBUG] Need a test to cover JDK-8050167

Boris Molodenkov boris.molodenkov at oracle.com
Tue Oct 7 06:56:36 UTC 2014


Fixed according to Leonid's and Christian's comments.
New webrev: http://cr.openjdk.java.net/~fzhinkin/bmoloden/8057135/webrev.01/

Thanks,
Boris

On 10/07/2014 09:30 AM, Leonid Mesnik wrote:
> Boris
>    29  * @run main/othervm -Xmx64m -XX:-TransmitErrorReport ProblematicFrameTest
>
> It would be better to us driver instead of run here.
>
>    45         ProcessBuilder pb = ProcessTools.createJavaProcessBuilder(new String[] { Crasher.class.getName() });
>
>
> ProcessBuilder ignores any external flags by default. You need to run  
> ProcessTools.createJavaProcessBuilder(true, ...  ) to preserve flags.
>
> Leonid
>
> On 06.10.2014 18:05, Boris Molodenkov wrote:
>> Hi All,
>>
>> Please review this regression test for runtime issue [1].
>> https://bugs.openjdk.java.net/browse/JDK-8057135
>> http://cr.openjdk.java.net/~fzhinkin/bmoloden/8057135/webrev.00/
>>
>> Testing:
>> 1. Verified locally that test fails before fix integration
>> 2. jprt
>>
>> Thanks,
>> Boris
>>
>> [1] https://bugs.openjdk.java.net/browse/JDK-8050167
>>
>



More information about the hotspot-runtime-dev mailing list