On Thu, 6 Oct 2022 18:33:51 GMT, Xue-Lei Andrew Fan <xue...@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

Reply via email to