RFR: JDK-8327971: Multiple ASAN errors reported for metaspace [v6]

[Roman Kennke]

On Wed, 20 Mar 2024 06:42:31 GMT, Thomas Stuefe <stuefe at openjdk.org> wrote:

>> We have multiple issues that cause the ASAN build in fastdebug to not work, or to fail gtests.
>> 
>> 1) When adding ASAN support for metaspace, we decided to track poisoning not at the OS border (map) but at the border between metaspace and outer JVM code. More precisely, at the border of ChunkManager chunk allocation/deallocation. Chunks handed out by the chunk manager are unpoisoned and poisoned when handed back.
>> 
>> This causes some problems since within metaspace, we access memory that is committed but not yet handed to the user (for zapping or for verification in debug builds). So, from the ASANs' view, those accesses are invalid, whereas for Metaspace, those accesses are perfectly valid.
>> 
>> In these cases, we need to unpoison the memory or exclude the code section, if possible, for ASAN builds. This affects both JVM code and gtests.
>> 
>> 2) NMT gtests sometimes do dangerous things deliberately (accessing invalid memory to track error recognition), and these tests should be disabled with ASAN
>
> Thomas Stuefe has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Revert accidentally added patch

Looks good, just one small question.

src/hotspot/share/runtime/os.cpp line 1292:

> 1290: 
> 1291:   st->print_cr(INTPTR_FORMAT " is an unknown value", p2i(addr));
> 1292: 

Would it make sense to move this a few lines up and print 'unknown' instead of nothing?

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

PR Review: https://git.openjdk.org/jdk/pull/18230#pullrequestreview-1959878099
PR Review Comment: https://git.openjdk.org/jdk/pull/18230#discussion_r1538918692