On Tue, 6 Apr 2021 19:43:50 GMT, Anthony Scarpino <ascarp...@openjdk.org> wrote:

>> 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

+1 (non-binding)

-------------

Marked as reviewed by djelin...@github.com (no known OpenJDK username).

PR: https://git.openjdk.java.net/jdk/pull/3296

Reply via email to