> Hi, > > I need a review of the locking change to the RSA blinding code. The problem > was reported that multithreaded performance suffered because there was one > global lock on the many blindings operation. The change reduces locking by > using a ConcurrentLinkedQueue to store the different BlindingParameters that > other threads maybe using. The queue size is limited to prevent sudden > surges in stored BlindingParameters and a WeakHashMap is still used so the > objects can be freed when no longer used. Performance doubles under high > load. > > thanks > > Tony
Anthony Scarpino has updated the pull request incrementally with one additional commit since the last revision: use computeIfAbsent instead ------------- Changes: - all: https://git.openjdk.java.net/jdk/pull/3296/files - new: https://git.openjdk.java.net/jdk/pull/3296/files/dda3ed69..f8c7baf7 Webrevs: - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=3296&range=03 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=3296&range=02-03 Stats: 19 lines in 1 file changed: 3 ins; 15 del; 1 mod Patch: https://git.openjdk.java.net/jdk/pull/3296.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/3296/head:pull/3296 PR: https://git.openjdk.java.net/jdk/pull/3296
