RFR: 8264130: PAC-RET protection for Linux/AArch64 [v2]

Alan Hayward duke at openjdk.java.net
Thu Nov 11 11:47:36 UTC 2021


On Thu, 11 Nov 2021 11:34:09 GMT, Andrew Dinn <adinn at openjdk.org> wrote:

>> As far as the AArch64 docs are concerned the relevant details are provided in ARM-ARM D
>> 
>> - The PAC functionality is described in ARM-ARM Section D5.1.5
>> - Overview of the PAC instructions is provided in section C3.1.9
>> - Detailed PAC instruction descriptions are provided in C6.2.195 - C6.2.199
>> 
>> n.b. I am specifically referring to my (possibly out of date) copy ARM-DDI 0487D.a (ID103018) which is the Initial v8.4 EAC release from 2018.
>> 
>> That said, I agree that a description of how these functions use the underlying PAC support and what, effectively, they achieve via that usage would be necessary. A reference to the relevant sections of the ARM doc in the code would be helpful.
>
> Correction:
> Using the most up to date ARM ARM G  [ARM DDI 0487G.a (ID011921)]
> 
> - The PAC functionality is described in ARM-ARM Section D5.1.5
> - Overview of the PAC instructions is provided in section C3.1.10
> - Detailed PAC instruction descriptions are provided in C6.2.208 - C6.2.212

I'm thinking for references to the Arm Arm to use header titles instead of section numbers, as the titles should be more stable.

Also probably need some description around the code in the pauth_aarch64.hpp too. But I want to make sure I'm not duplicating comments - maybe the macroassembler comments should point to the pauth_aarch64 comments.

It didn't seen common in the code to describe instruction functionality, which is why I didn't add any. Agreed it needs something added though.

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

PR: https://git.openjdk.java.net/jdk/pull/6334



More information about the build-dev mailing list