RFR: 8265448: (zipfs): Reduce read contention in ZipFileSystem [v4]
Jason Zaugg
jzaugg at openjdk.java.net
Thu May 6 00:03:20 UTC 2021
> If the given Path represents a file, use the overload of read defined
> in FileChannel that accepts an explicit position and avoid serializing
> reads.
>
> Note: The underlying NIO implementation is not required to implement
> FileChannel.read(ByteBuffer, long) concurrently; Windows still appears
> to lock, as it returns true for NativeDispatcher.needsPositionLock.
>
>
> On MacOS X, the enclosed benchmark improves from:
>
>
> Benchmark Mode Cnt Score Error Units
> ZipFileSystemBenchmark.read avgt 10 75.311 ? 3.301 ms/op
>
>
> To:
>
>
> Benchmark Mode Cnt Score Error Units
> ZipFileSystemBenchmark.read avgt 10 12.520 ? 0.875 ms/op
Jason Zaugg has updated the pull request incrementally with one additional commit since the last revision:
Remove redundant cast from previous commit.
-------------
Changes:
- all: https://git.openjdk.java.net/jdk/pull/3853/files
- new: https://git.openjdk.java.net/jdk/pull/3853/files/bccf5d3c..c9758c58
Webrevs:
- full: https://webrevs.openjdk.java.net/?repo=jdk&pr=3853&range=03
- incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=3853&range=02-03
Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod
Patch: https://git.openjdk.java.net/jdk/pull/3853.diff
Fetch: git fetch https://git.openjdk.java.net/jdk pull/3853/head:pull/3853
PR: https://git.openjdk.java.net/jdk/pull/3853
More information about the nio-dev
mailing list