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

Boris Molodenkov boris.molodenkov at oracle.com
Wed Oct 8 11:02:35 UTC 2014


Please continue review.

On 10/07/2014 10:56 AM, Boris Molodenkov wrote:
> 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