RFR: 8312585: Rename DisableTHPStackMitigation flag to THPStackMitigation
David Holmes
dholmes at openjdk.org
Mon Aug 7 02:44:31 UTC 2023
On Mon, 24 Jul 2023 09:02:59 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:
> The flag newly added by [JDK-8312182](https://bugs.openjdk.org/browse/JDK-8312182) is prompting the use of `-XX:+DisableTHPStackMitigation` to disable the THP stack mitigation, thus allowing THP in thread stacks. This double negation does not read well, and not in line with other mitigation flags like `IntelJccErratumMitigation`.
>
> It would be better to rename the flag to avoid double-negation, before it proliferates to other JDK releases.
>
> (I would have the same comment during the original review, but missed it :P)
>
> Additional testing:
> - [ ] GHA
> - [x] Linux x86_64 fastdebug, affected test
Marked as reviewed by dholmes (Reviewer).
We've had some internal discussion and the general position is "no negative naming" and a preference I believe for "Use" as a prefix if a prefix is needed/desired. There will probably be a style-guide update proposed for the general case. So my objection is overridden and so withdrawn. :)
-------------
PR Review: https://git.openjdk.org/jdk/pull/14992#pullrequestreview-1564471024
PR Comment: https://git.openjdk.org/jdk/pull/14992#issuecomment-1667103128
More information about the hotspot-runtime-dev
mailing list