Fix bti support (#830)
authorBill Roberts <152999275+billatarm@users.noreply.github.com>
Tue, 19 Mar 2024 16:44:55 +0000 (11:44 -0500)
committerGitHub <noreply@github.com>
Tue, 19 Mar 2024 16:44:55 +0000 (12:44 -0400)
commitf64141ee3f9e455a060bd09e9ab72b6c94653d7c
tree86320289b7aaa8cd150be5e9d29b696ce959394f
parent3d0ce1e6fcf19f853894862abcbac0ae78a7be60
Fix bti support (#830)

* bti: add identifier to ffi_closure_SYSV_V_alt

This was missing BTI_C identifier.

Old Code:
ffi_closure_SYSV_V_alt:
0000fffff7f70500:   ldr     x17, [sp, #8]

Signed-off-by: Bill Roberts <bill.roberts@arm.com>
* testsuite: fix whitespace in Makefile.am

Signed-off-by: Bill Roberts <bill.roberts@arm.com>
* aarch64: correct comment describing BTI

The comment is incorrect, BTI is enabled per mapping via mprotect with
PROT_BTI flag set, not per-process. When the loader loads the library,
if the GNU Notes section is missing this, PROT_BTI will not be enabled
for that mapping, but is independent of other mappings.

Signed-off-by: Bill Roberts <bill.roberts@arm.com>
---------

Signed-off-by: Bill Roberts <bill.roberts@arm.com>
src/aarch64/sysv.S
testsuite/Makefile.am