RFR: 8231953: Wrong assumption in assertion in oop::register_oop

coleen.phillimore at oracle.com coleen.phillimore at oracle.com
Thu Oct 10 14:50:58 UTC 2019


This seems fine to me.  The pc was only there to help with debugging.
Coleen

On 10/10/19 10:28 AM, christoph.goettschkes at microdoc.com wrote:
> Hi,
>
> please review the following changeset. This patch makes the
> CheckUnhandledOops feature work on 32-bit ARM platforms. On this platform,
> the os::current_frame() function does not return a valid frame in thumb
> mode [1]. I removed the failing assertion, since it does not hold on all
> supported platforms. I didn't want to introduce architecture specific code
> into that feature, just for an assertion.
>
> This fixes the hotspot tier1 test
> "runtime/CheckUnhandledOops/TestOutOfMemory.java" mentioned in the issue.
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8231953
> Webrev: https://cr.openjdk.java.net/~bulasevich/8231953/webrev.00/
>
> Thanks,
> Christoph
>
> [1]
> https://hg.openjdk.java.net/jdk/jdk/file/30a9612a657d/src/hotspot/os_cpu/linux_arm/os_linux_arm.cpp#l224
>



More information about the hotspot-runtime-dev mailing list