RFR: 8280056: gtest/LargePageGtests.java#use-large-pages failed "os.release_one_mapping_multi_commits_vm" [v2]

Afshin Zafari azafari at openjdk.org
Mon Oct 23 11:38:32 UTC 2023


On Wed, 18 Oct 2023 14:58:15 GMT, Afshin Zafari <azafari at openjdk.org> wrote:

>> The `attempt_allocate_memory_at(addr, size)` may fail for some reasons that `::mmap`  will report. To find out why an attempt failed, an `int *` is passed down deep to the `::mmap` to return the `ERRNO` back to the callers.
>> This error code is used in gTest test case to show a proper message.
>> 
>> The changed test case never failed after 10K+ repetitions, but in case it happens in future the reason is also printed out.
>
> Afshin Zafari has updated the pull request incrementally with one additional commit since the last revision:
> 
>   some pd_xxx calls were missed.

One more question, can we use the `/proc/self/map` (like what is done in PRINT_MAPPINGS in the tests) for checking the success/failure of `release_memory`? In the log outputs of all failed tests that are attached to this issue, the "B" part of the test report shows a successful release operation.

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

PR Comment: https://git.openjdk.org/jdk/pull/16240#issuecomment-1774999732


More information about the hotspot-runtime-dev mailing list