RFR(XS): JDK-8152679 DeadlockDetectionTest.java fails due to expected output missing

Staffan Larsen staffan.larsen at oracle.com
Fri Apr 8 10:27:49 UTC 2016


I see - thanks!

> On 8 apr. 2016, at 11:54, Dmitry Samersoff <dmitry.samersoff at oracle.com> wrote:
> 
> Staffan,
> 
> I suspect its because of root/non-root. OS X attach doesn't work for
> non-root user. Test check it and skips silently. But if the test run
> under root, it tries to run and then fail.
> 
> -Dmitry
> 
> On 2016-04-08 11:10, Staffan Larsen wrote:
>> Why did this test start failing?
>> 
>>> On 7 apr. 2016, at 20:35, serguei.spitsyn at oracle.com wrote:
>>> 
>>> Dmitry,
>>> 
>>> I agree with Tim.
>>> The message at L85 should not be alarming but look similar to the one at L107.
>>> Something, like this:
>>> 85        System.out.println("This test is not expected to work on OS X. Skipping");
>>> 
>>> 
>>> Thanks,
>>> Serguei
>>> 
>>> 
>>> On 4/7/16 11:13, Tim Bell wrote:
>>>> On 04/07/16 10:56, Dmitry Samersoff wrote:
>>>>> Everybody,
>>>>> 
>>>>> Please review small changes.
>>>> 
>>>> Note:  I am not a "R"eviewer in Serviceability.  Free advice follows:
>>>> 
>>>>> http://cr.openjdk.java.net/~dsamersoff/JDK-8152679/webrev.01/
>>>>> 
>>>>> Test that is not expected to work on OS X, detect OS X and exits.
>>>> 
>>>> 84             // Coredump stackwalking is not implemented for Darwin
>>>> 85             System.err.println("Error! This test is not expected to work on OS X.");
>>>> 
>>>> 
>>>> It is not implemented, so landing here during a test run on Mac OS should not be surprising.
>>>> 
>>>> Rather than printing 'Error!' I suggest removing that alarming string, as on line 107   .
>>>> 
>>>> My 0x02 cents.
>>>> 
>>>> Tim
>>>> 
>>>> 
>>> 
>> 
> 
> 
> -- 
> Dmitry Samersoff
> Oracle Java development team, Saint Petersburg, Russia
> * I would love to change the world, but they won't give me the sources.



More information about the serviceability-dev mailing list