On Fri, 7 Oct 2022 15:37:24 GMT, Daniel Jeliński <djelin...@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?
>> 
>> 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.
> 
> @ferakocz biginteger math is not constant-time, which is why it can't be used 
> here.

@djelinski for this purpose, it doesn't matter if the exponentiation is not 
constant time, as its running time only depends on the value of the exponent, 
which is a known (public) value.

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

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

Reply via email to