> 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 ReentrantLock for put and get ------------- Changes: - all: https://git.openjdk.java.net/jdk/pull/3296/files - new: https://git.openjdk.java.net/jdk/pull/3296/files/e948ecac..dda3ed69 Webrevs: - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=3296&range=02 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=3296&range=01-02 Stats: 21 lines in 1 file changed: 9 ins; 0 del; 12 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