JDK 9 RFR of JDK-8035279: Clean up internal deprecations in BigInteger

David Holmes david.holmes at oracle.com
Thu Feb 20 10:20:27 UTC 2014


Hi Brian,

On 20/02/2014 7:44 AM, Brian Burkhalter wrote:
> This patch concerns cleaning up some ugly internal deprecations.
>
> Issue:	https://bugs.openjdk.java.net/browse/JDK-8035279
> Webrev:	http://cr.openjdk.java.net/~bpb/8035279/webrev.00/
>
> All JTREG BigInteger tests pass, and the serialized form has been unaltered as verified by bidirectional read-write testing between Java 7 and this version.
>
> I would appreciate scrutiny of the arithmetic to make sure that I've made no silly errors. Also I would be interested in opinions as to whether the "volatile" keyword should in fact be used for the affected instance variables.

re: volatile

The basic approach being taken with these primitive fields is lazy 
initialization to a value that will always be the same if recomputed. If 
the variables are not volatile then in the worst-case (this is 
theoretical only) they will be recomputed by each thread that accesses 
them. If they are volatile then they will only be recomputed by threads 
that truly race on the initialization logic, but you will thereafter pay 
the price of a volatile read on each access.

Either way you will be functionally correct so it all comes down to 
performance. Practically speaking I would expect you to be better off 
overall without the volatile.

As Paul notes the powerCache array does not need to be volatile as it is 
initialized as part of class initialization.

Cheers,
David

> Thank you for your comments.
>
> Brian
>



More information about the core-libs-dev mailing list