RFR: 8360791: [ubsan] Adjust signal handling [v2]

Magnus Ihse Bursie ihse at openjdk.org
Mon Jun 30 08:59:41 UTC 2025


On Mon, 30 Jun 2025 07:02:31 GMT, Matthias Baesken <mbaesken at openjdk.org> wrote:

>> A couple of tests e.g. VendorInfoPluginsTest but also some Hotspot like runtime/ErrorHandling/CreateCoredumpOnCrash.java put (write) to special addresses like 0 to provoke crashs.
>> However this does not work well with ubsan-enabled binaries on the clang based platforms (macOS, AIX).
>> The mentioned tests generate a SIGSEGV.
>> 
>> Some other tests generate a SIGBUS, e.g.
>> 
>> runtime/memory/ReserveMemory.java
>> runtime/memory/ReadFromNoaccessArea.java
>> 
>> and this leads to similar issues with ubsan-enabled binaries.
>> 
>> We should adjust the signal handling with the sanitizer options, how to do this is documented here :
>> https://github.com/google/sanitizers/wiki/SanitizerCommonFlags
>
> Matthias Baesken has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Remove the clang check

Looks good. Let's exclude the other signals if we can actually show it is a problem.

-------------

Marked as reviewed by ihse (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/26015#pullrequestreview-2970443722


More information about the build-dev mailing list