Hi, Sorry to insist but I would like to get feedback on this Marlin patch soon before going forward on tile-size tuning in java2d accelerated pipelines.
Laurent 2018-03-21 22:56 GMT+01:00 Laurent Bourgès <bourges.laur...@gmail.com>: > Hi, > > Here is the updated webrev: > http://cr.openjdk.java.net/~lbourges/marlin/marlin-091.1/ > > Changes in MarlinProperties only: > - getTileSize_Log2() & getTileWidth_Log2(); 32x32 tiles ie default = 5 > (log2) > > I hope it is good for now as tile settings are the same as in jdk9/10. > > Regards, > Laurent > > > 2018-03-21 21:44 GMT+01:00 Laurent Bourgès <bourges.laur...@gmail.com>: > >> Sergey, >> >> Le mer. 14 mars 2018 à 17:14, Sergey Bylokhov <sergey.bylok...@oracle.com> >> a écrit : >> >>> On 13/03/2018 17:04, Sergey Bylokhov wrote: >>> >>>> I have started to look to this review, will run some closed tests and >>>> send a feedback soon. >>>> >>> >>> No issues found so far, +1. >> >> >> Thanks for your vote. >> I need another approval I suppose ? >> >> I will prepare another review asap reverting only tile size changes as >> using large tiles has performance drop on d3d & ogl that needs more work. >> It can be done later in follow-up issues. >> >> Phil do you agree the proposed plan ? >> >> Laurent >> > >
