RFR: 8229147: Linux os::create_thread() overcounts guardpage size with newer glibc (>=2.27)
Thomas Stuefe
stuefe at openjdk.org
Mon Apr 24 08:11:46 UTC 2023
On Sun, 23 Apr 2023 01:55:02 GMT, David Holmes <dholmes at openjdk.org> wrote:
>> src/hotspot/os/linux/os_linux.cpp line 4549:
>>
>>> 4547:
>>> 4548: // Check if we need to adjust the stack size for glibc guard pages.
>>> 4549: init_adjust_stack_for_guard_pages();
>>
>> Please make this glibc only. I agree with @TheRealMDoerr, muslc does not have `__pthread_get_minstack`, no need to pay for a dynamic lookup that will always fail.
>
> To a degree this is pre-existing behaviour - nothing to do with `AdjustStackSizeForTLS` is glibc-only even though it is. So I would have to change that as well, which broadens the scope and is not something I can, or want to, test!
Okay, up to you then. We can fix this in a cleanup for muslc later.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/13571#discussion_r1174929672
More information about the hotspot-runtime-dev
mailing list