Re: git: 43e8849bc294 - main - conf: Enable BTI checking in the arm64 kernel
Date: Thu, 29 Aug 2024 16:02:09 UTC
On 21 Aug 2024, at 15:28, John Baldwin <jhb@FreeBSD.org> wrote: > > On 8/20/24 05:02, Andrew Turner wrote: >> The branch main has been updated by andrew: >> URL: https://cgit.FreeBSD.org/src/commit/?id=43e8849bc29414036ccaef7788de95a07ad32ab5 >> commit 43e8849bc29414036ccaef7788de95a07ad32ab5 >> Author: Andrew Turner <andrew@FreeBSD.org> >> AuthorDate: 2024-08-19 12:59:49 +0000 >> Commit: Andrew Turner <andrew@FreeBSD.org> >> CommitDate: 2024-08-20 08:49:15 +0000 >> conf: Enable BTI checking in the arm64 kernel >> To ensure new code has BTI support make it an error to not have the >> BTI ELF note when linking the kernel and kernel modules. >> Reviewed by: kib, emaste >> Sponsored by: Arm Ltd >> Differential Revision: https://reviews.freebsd.org/D45469 > > This has broken two of the GitHub CI actions using clang 12 and clang 13. > Please fix this to be conditional on a supported linker version (or perhaps > add a new linker feature to bsd.linker.mk). This is still broken. If it’s not fixed promptly I will just revert this change; we can’t leave CI broken, especially when it gets used to test GitHub PRs. Please stop breaking building with older toolchains, this isn’t the first time it’s happened. Jess