Review Request for 6751338: ZIP inflater/deflater performance

Xueming Shen xueming.shen at oracle.com
Fri Apr 1 23:04:35 UTC 2011


Dave, Alan,

Here is the final webrev based on Dave's patch and the jdk1.5 code that 
does not
have the change for 6206933. JPRT job result suggests no new testing 
failure and
my "non-scientific" benchmark test (to use GZIPOu/InputStream to compress/
decompress the rt.jar) does show relative performance gain. Will try to 
run more
tests the weekend, but here is the webrev.

http://cr.openjdk.java.net/~sherman/6751338/webrev/

Background Info:

This fix is basically to back out the fix for #6206933 we made back to 
jdk5, which
is to use malloc+GetByteArrayuRegion to replace the original 
GetPrimitiveArrayCritical/
ReleasePrimitiveArrayCritical() pair when access the java byte array at 
native code
Inflater/Deflater.c, to mainly workaround the GC/Critical... issue 
discussed in
#6186200.

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.

-Sherman




More information about the core-libs-dev mailing list