[8u] RFR: Backport 8208648: ECC Field Arithmetic Enhancements
Andrew John Hughes
gnu.andrew at redhat.com
Tue Jun 18 19:19:17 UTC 2019
On 18/06/2019 19:37, Andrew John Hughes wrote:
> On 14/06/2019 22:37, Alvarez, David wrote:
>> Hi,
>>
>> Here is the proper RFR for 8208648: ECC Field Arithmetic Enhancements
>>
>> Sorry for the confusion
>>
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8208648
>> Original: http://hg.openjdk.java.net/jdk/jdk/rev/746602d9682f
>> Webrev: http://cr.openjdk.java.net/~phh/8208648/webrev.8u.00/
>>
>> JDK-8208648 is marked as jdk8u-critical-yes
>>
>> This is the second of a chain of three patches, JDK-8181594, JDK-8208648 and JDK-8208698 I will be sending today.
>>
>> The patch did not apply cleanly. The following conflicts appeared:
>>
>> sun/security/util/ArrayUtil.java is not present in jdk8u. ArrayUtil is a utility class with static methods. I created the file but only with the static methods that were required for this patch (all of them were included in the original patch).
>
> That seems fine. The file originally arises from JDK-8179098, which in
> turn fixes a performance regression introduced by JDK-8076112 adding
> bounds checks. That is a JDK & HotSpot change to mark intrinsics, which
> I doubt is suitable for backport, so I'm happy with ArrayUtil being
> created just by this patch.
>
>> sun/security/util/math/intpoly/IntegerPolynomial1305.java had a minor conflict due to mismatching of the context lines
>> sun/security/util/math/intpoly/IntegerPolynomial.java had significant amount of rejections, but they were mostly easy to fix, caused by context mismatching.
>>
>> Additionally, some of the new implementations of IntegerPolynomial contained an @Override for a method (finalCarryReduceLast) that is not present in the jdk8u version of IntegerPolynomial.java, so I removed the annotation.
>>
>
> I think this is because JDK-8203228 & JDK-8201317 are missing, which are
> also part of the 8u222 list. I'll post backports of those two, based on
> your JDK-8181594 work and then those changes should apply cleanly.
>
>> Below are the relevant changes I've done to resolve the rejects and compilation errors.
>>
>> Thanks,
>> David
>>
>>
>
>
Looking at JDK-8203228 & JDK8201317, they both apply cleanly (once
changes to XDHKeyAgreement are dropped from the latter), so they can go
straight in after JDK-8181594 is pushed. You should then be able to
backport JDK-8208648 more simply.
--
Andrew :)
Senior Free Java Software Engineer
Red Hat, Inc. (http://www.redhat.com)
PGP Key: ed25519/0xCFDA0F9B35964222 (hkp://keys.gnupg.net)
Fingerprint = 5132 579D D154 0ED2 3E04 C5A0 CFDA 0F9B 3596 4222
https://keybase.io/gnu_andrew
More information about the jdk8u-dev
mailing list