[OpenJDK Rasterizer] Marlin #4

[Laurent Bourgès]

Jim,

I am ok with your changes to use the new Unsafe and all others also.

PS: I added recently the enableLogs flag to make Marlin silent to mimic
Phil's approach to disable stdout logs 6 months ago.

I am going to sleep now.
Good luck & good night,

Laurent
Le 20 nov. 2015 23:53, "Jim Graham" <james.graham at oracle.com> a écrit :

> I will be pushing this code cleanup changeset momentarily followed by one
> more push to accomplish all of the changing of the various defaults...
>
> Second code cleanup task - switching to jdk.internal.misc.Unsafe...
>
> webrev: http://cr.openjdk.java.net/~flar/Marlin/NewUnsafeClass.0/
>
>                         ...jim
>
> On 11/20/15 7:05 AM, Jim Graham wrote:
>
>> Hi Laurent,
>>
>> I ran a bunch of tests on 4.2 and saw no issues and the performance
>> looked good.  There were still some things that Ductus was faster on,
>> but I just did a brief run of a few tests I cobbled together so I don't
>> know how representative they are.  Marlin beat Ductus on a number of
>> tests as well, so they are right there fighting neck and neck.  I was
>> using the defaults which have the RLE code turned off I believe, so
>> maybe I wasn't running it in its fastest configuration?
>>
>> I'm code reading now:
>>
>> ArrayCache.java, line 205 - should that be needSize there?  Also, should
>> these tests be > or >=?
>>
>> MarlinCache.java, line 181 - in the comment, shouldn't those be maxx and
>> minx?
>>
>> Renderer.java, line 1446 - initial value of useBlkFlags will be false if
>> heuristics is enabled
>> Renderer.java, line 1329 - we send a row to the cache before we update
>> heuristics at line 1332
>> - taken together that means that with heuristics the very first cache
>> line sent out will always be non-RLE?
>>
>> None of those are blocking so I'm going to go through with pushing this
>> to the sandbox under your ID, then follow it up with a small cleanup:
>>
>> - blessed modifier order script
>> - switch to new jdk.internal.misc.Unsafe class
>> - turning off verbose
>> - switch default renderer to Marlin
>> - add RenderingEngine flag to print out RenderingEngine class chosen
>> (-Dsun.java2d.renderer.verbose=true)
>>
>> Then I'll generate a webrev against the client JDK and send that out for
>> review to the 2D list...
>>
>>              ...jim
>>
>> On 10/19/15 7:06 AM, Laurent Bourgès wrote:
>>
>>> Hi Jim,
>>>
>>> Here is the new webrev:
>>> http://cr.openjdk.java.net/~lbourges/marlin/marlin-s4.2/
>>>
>>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/graphics-rasterizer-dev/attachments/20151121/abc3e8b0/attachment-0001.html>