RFR: 8364235: Fix for JDK-8361447 breaks the alignment requirements for GuardedMemory [v5]
David Holmes
dholmes at openjdk.org
Sun Aug 3 22:54:33 UTC 2025
> The fix for [JDK-8361447](https://bugs.openjdk.org/browse/JDK-8361447) added a new field to the `GuardHeader`, not realizing that the size of the `GuardHeader` must be such that the address of the user-data has the strictest necessary alignment (16-byte).
>
> We need to add a padding field to restore the alignment.
>
> A static assert is added to check the alignment.
>
> Testing:
> - tiers 1-3 (in progress)
>
> Thanks
David Holmes has updated the pull request incrementally with one additional commit since the last revision:
Add "padding" to the layout description
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/26524/files
- new: https://git.openjdk.org/jdk/pull/26524/files/ab19a75d..e671ba73
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=26524&range=04
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=26524&range=03-04
Stats: 3 lines in 1 file changed: 3 ins; 0 del; 0 mod
Patch: https://git.openjdk.org/jdk/pull/26524.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/26524/head:pull/26524
PR: https://git.openjdk.org/jdk/pull/26524
More information about the hotspot-runtime-dev
mailing list