[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