Re: git: 43e8849bc294 - main - conf: Enable BTI checking in the arm64 kernel

From: Andrew Turner <andrew_at_fubar.geek.nz>
Date: Fri, 30 Aug 2024 08:55:54 UTC

> On 29 Aug 2024, at 17:02, Jessica Clarke <jrtc27@freebsd.org> wrote:
> 
> 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.

See https://github.com/freebsd/freebsd-src/pull/1393 and https://github.com/freebsd/freebsd-src/pull/1399

Andrew