[OpenJDK 2D-Dev] RFR 8144446: Automate the Marlin crash test

Jim Graham james.graham at oracle.com
Tue Dec 8 23:12:02 UTC 2015


Hi Laurent,

If this were testing a bug that crossed a number of implementation 
boundaries such that any number of engineers might make a change that 
could cause it to fail, then I might agree with the need to run it 
whenever any automated run of tests occurs.

But, the only thing that will cause this to fail would be if someone 
modified Marlin - which is basically you, and maybe 1 or 2 other 
engineers who would only go in for mostly logistical coding reasons, not 
for any changes that might affect the algorithm.  Those engineers should 
be running the tests with the -manual option specified for testing their 
changes.

But, if someone changes some code in core libs, or Swing, or somewhere 
else not related to Marlin - you've just added 30 seconds to the amount 
of time that it takes them to test for any regressions.  Is that 
appropriate?

			...jim

On 12/7/15 11:54 PM, Laurent Bourgès wrote:
> Jim,
>
> This test is the most important for Marlin as it checks off-heap array
> overflows to detect possible SEG FAULT (critical) so I really want it to
> be run automatically to detect a possible regression asap.
>
> I could add the jtreg timeout tag if you want ~ 120s (x4 typical time).
>
> Laurent
>
> Le 8 déc. 2015 03:17, "Jim Graham" <james.graham at oracle.com
> <mailto:james.graham at oracle.com>> a écrit :
>>
>> Prior to pushing it I ran the test and it takes over 30 seconds to complete.  Is this something we want to make a manual test?
>>
>>                         ...jim
>>
>>
>> On 12/4/15 2:57 PM, Phil Race wrote:
>>>
>>> Looks good.
>>>
>>> -phil.
>>>
>>> On 12/04/2015 02:55 PM, Jim Graham wrote:
>>>>
>>>> Looks good to me.  Are jigsaw issues good Phil?
>>>>
>>>>             ...jim
>>>>
>>>> On 12/3/15 12:58 PM, Laurent Bourgès wrote:
>>>>>
>>>>> Phil,
>>>>>
>>>>>     If you were to run this on a jigsaw EA build it would fail due to
>>>>>     accessing
>>>>>     sun.java2d.pipe.RenderingEngine. I know this is "pre-existing" but
>>>>>     jigsaw is coming ...
>>>>>
>>>>>
>>>>> You're right, sorry !
>>>>>
>>>>> Here is a new webrev:
>>>>>http://cr.openjdk.java.net/~lbourges/marlin/marlin-8144446.1/
>>>>>
>>>>> Changes:
>>>>> - set System property "sun.java2d.renderer.verbose" to true (to get the
>>>>> renderer in use)
>>>>> - removed any RenderingEngine usage
>>>>>
>>>>> Laurent
>>>
>>>
>



More information about the 2d-dev mailing list