[8u40] Request for approval: 8064391: More thread safety problems in core reflection

Martin Buchholz martinrb at google.com
Thu Nov 13 16:30:19 UTC 2014


On Thu, Nov 13, 2014 at 7:17 AM, Joel Borggrén-Franck
<joel.franck at oracle.com> wrote:
> Hi Martin,
>
> I still like to backport your patch as is, because:
>
> - It is small and "obviously correct"
> - Fixing an existing, but hard to repro race
> - I was assuming "Here are changes we made locally at Google" implied it
>   was in production

Yes, those are correct.

> If any of this was incorrect, I agree we should wait and reiterate this
> some more on core-libs-dev. If my assumptions were correct I think this
> should go in as is, potentially followed by more fixes later on.

Yes, it is perfectly fine to commit these changes now.  A reason to
wait might be to consolidate these fixes with additional possible
fixes.  But they should surely make the next update releases - not
sure what your schedule is for that.


More information about the jdk8u-dev mailing list