RFR: 8294731: Improve multiplicative inverse for secp256r1 implementation [v2]

Xue-Lei Andrew Fan xuelei at openjdk.org
Fri Oct 7 17:48:20 UTC 2022


On Thu, 6 Oct 2022 18:33:51 GMT, Xue-Lei Andrew Fan <xuelei at openjdk.org> wrote:

>> It seems to me the scalar multiplication enhancement should be done first, or maybe integrated with this fix.
>> Do you have a bug number for the scalar multiplication enhancement?
>
>> It seems to me the scalar multiplication enhancement should be done first, or maybe integrated with this fix. Do you have a bug number for the scalar multiplication enhancement?
> 
> I did not file the scalar multiplication enhancement in JBS yet.  There are a few places that could be improved for the EC performance.  However, the update is big if having them all in one PR.  In order to simplify the code review and implementation, I would like to break it down into small enhancements.  I filed an [umbrella RFE](https://bugs.openjdk.org/browse/JDK-8294188) for the performance improvement in EC.  The goal to make the common EC crypto operations (key generation/exchange/signature) 3+ times faster, and make the TLS connections 20%+ faster .  
> 
> I may have to wait for a few more weeks so that I can come up with the scalar multiplication pull request.

> @XueleiFan tests are failing after the last commit; see `sun/security/ec/TestEC.java` for example.
> 
@djelinski Thank you very much for help for the testing.  The test passed in my testing, but I may made something wrong in the commit.  Anyway, I'm working on further improvement, similar to your comments.  I will make sure the test passed for the next commit.

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

PR: https://git.openjdk.org/jdk/pull/10544


More information about the security-dev mailing list