Thanks for the suggestion! I've submitted a new CR to security-dev. The new CR
is at https://cr.openjdk.java.net/~xliu/8229958/01/webrev/ I've run all tier-1
tests again with the change.
Letu
On 8/21/19, 12:39 AM, "Alan Bateman" <alan.bate...@oracle.com> wrote:
On 21/08/2019 08:32, Yang, Letu wrote:
> Hi Ivan,
>
> You are right! I should make legacyChanged volatile as well. I will get
it tested it with tier-1 and get back to you tomorrow.
>
> Letu
>
>
Probably best to being the discussion and the next version to
security-dev as that is where the JCA code is maintained. There have
also been several recent discussions and patches related to
synchronization issues and performance in this code.
-Alan
