[lworld] Integrated: 8271324: [lworld] java/foreign/* tests fail with "guarantee(sect->end() <= tend) failed: sanity"
Tobias Hartmann
thartmann at openjdk.java.net
Wed Jul 28 08:07:55 UTC 2021
On Wed, 28 Jul 2021 07:24:24 GMT, Tobias Hartmann <thartmann at openjdk.org> wrote:
> We hit an assert because the code generated for "upcall_stub_linkToNative" does not fit into the code buffer. That's because we are currently saving all registers before the slow call in the GC barriers:
> https://github.com/openjdk/valhalla/blob/8efeb68ef0594c2c7c20bc2e45c112d6ef623b2b/src/hotspot/cpu/x86/gc/g1/g1BarrierSetAssembler_x86.cpp#L206-L211
>
> https://github.com/openjdk/valhalla/blob/8efeb68ef0594c2c7c20bc2e45c112d6ef623b2b/src/hotspot/cpu/x86/gc/g1/g1BarrierSetAssembler_x86.cpp#L332-L337
>
> I've doubled the buffer size to be on the safe side and filed [JDK-8271370](https://bugs.openjdk.java.net/browse/JDK-8271370) to investigate if we can avoid saving all registers in all cases.
>
> Thanks,
> Tobias
This pull request has now been integrated.
Changeset: 7ac76624
Author: Tobias Hartmann <thartmann at openjdk.org>
URL: https://git.openjdk.java.net/valhalla/commit/7ac76624ae0a89d33adb46660025f996883f2798
Stats: 2 lines in 2 files changed: 0 ins; 0 del; 2 mod
8271324: [lworld] java/foreign/* tests fail with "guarantee(sect->end() <= tend) failed: sanity"
-------------
PR: https://git.openjdk.java.net/valhalla/pull/503
More information about the valhalla-dev
mailing list