RFR: 8281314: Rename Stack{Red,Yellow,Reserved,Shadow}Pages multipliers [v2]
Aleksey Shipilev
shade at openjdk.java.net
Mon Feb 7 15:55:48 UTC 2022
On Mon, 7 Feb 2022 15:44:36 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:
>> src/hotspot/share/runtime/stackOverflow.cpp line 45:
>>
>>> 43: // in 4K pages to the actual os page size. We must do this before setting
>>> 44: // up minimal stack sizes etc. in os::init_2().
>>> 45: size_t unit = 4*K;
>>
>> I think we picked 4*k because it was the smallest page size in the platforms that we supported. I might be wrong about that but the name 'unit' should be something else indicative of why it's 4k.
>
> I was thinking that the `globals.hpp` use it in a unit-like fashion: "Number of yellow zone (recoverable overflows) pages of size 4KB". In other words, that 4K is forced by option definition. I could add a blurb to the comment itself, about the motivation for this oddity.
...like in the new commit.
-------------
PR: https://git.openjdk.java.net/jdk/pull/7362
More information about the hotspot-runtime-dev
mailing list