RFR: 8057135: [TESTBUG] Need a test to cover JDK-8050167
Coleen Phillimore
coleen.phillimore at oracle.com
Tue Oct 14 19:37:00 UTC 2014
Boris,
Test seems ok.
Coleen
On 10/8/14, 7:02 AM, Boris Molodenkov wrote:
> 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