[8u] RFR Backport 8057003: Large reference arrays cause extremely long synchronization times
Andrew Hughes
gnu.andrew at redhat.com
Wed Aug 19 05:39:26 UTC 2020
On 14:46 Fri 17 Jul , Hohensee, Paul wrote:
> I’d change the oops_do() comment to
>
> // G1 does its own checking
>
> because the assert has a general G1 check (i.e., UseG1GC) rather than a specific one. All uses of oops_do() by G1 will have to do their own checks, not just the array slice case that your current comment says.
>
> Other than that, lgtm. I’ll sponsor the patch and add a ‘reviewed’ comment to the issue.
>
> Thanks,
> Paul
>
This was a bit difficult to follow at first, as a few hunks seem to
have moved files, but I agree it looks ok.
FWIW, JDK-8079082: "VerifyNoCSetOopsClosure is derived twice from Closure"
introduced the additional assert as part of its refactoring.
Thanks,
--
Andrew :)
Senior Free Java Software Engineer
OpenJDK Package Owner
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
More information about the jdk8u-dev
mailing list