Review Request for 6751338: ZIP inflater/deflater performance
frances.ho at oracle.com
Sat Apr 2 00:48:51 UTC 2011
Regarding micro benchmark, did you ever get together with monica from performance team so that she can start building suites of micro benchmarks? I think we need to start doing this so that they help(take over) performance validation in the future. Of course developers should still do it before checkin but benchmarks need to be run regularly.
From: Xueming Shen <xueming.shen at oracle.com>
Sender: core-libs-dev-bounces at openjdk.java.net
Date: Fri, 01 Apr 2011 16:04:35
To: Dave Bristor<bristor at yahoo.com>; BATEMAN, ALAN<alan.bateman at oracle.com>; core-libs-dev<core-libs-dev at openjdk.java.net>
Subject: Review Request for 6751338: ZIP inflater/deflater performance
Here is the final webrev based on Dave's patch and the jdk1.5 code that
have the change for 6206933. JPRT job result suggests no new testing
my "non-scientific" benchmark test (to use GZIPOu/InputStream to compress/
decompress the rt.jar) does show relative performance gain. Will try to
tests the weekend, but here is the webrev.
This fix is basically to back out the fix for #6206933 we made back to
is to use malloc+GetByteArrayuRegion to replace the original
ReleasePrimitiveArrayCritical() pair when access the java byte array at
Inflater/Deflater.c, to mainly workaround the GC/Critical... issue
The change for #6206933 itself has triggered lots of performance issues
since its integration, some fixed, some still outstanding. The GC rfe#6186200 has
been fixed long time ago, after couple weeks of discussion/debating, we all agreed
that it's the time to back out#6206933.
More information about the core-libs-dev