Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 30 Aug 2024 09:55:54 +0100
From:      Andrew Turner <andrew@fubar.geek.nz>
To:        Jessica Clarke <jrtc27@freebsd.org>
Cc:        John Baldwin <jhb@FreeBSD.org>, "src-committers@freebsd.org" <src-committers@FreeBSD.org>, "dev-commits-src-all@freebsd.org" <dev-commits-src-all@FreeBSD.org>, "dev-commits-src-main@freebsd.org" <dev-commits-src-main@FreeBSD.org>
Subject:   Re: git: 43e8849bc294 - main - conf: Enable BTI checking in the arm64 kernel
Message-ID:  <F794FC8A-6997-4D79-A5BD-72ACC937EFEC@fubar.geek.nz>
In-Reply-To: <19DBDC11-EB0F-48C0-9AAE-9EED087F0AB6@freebsd.org>
References:  <202408200902.47K92BsJ078096@gitrepo.freebsd.org> <77cbcd47-1778-4e71-b824-4e75e0e4b2d6@FreeBSD.org> <19DBDC11-EB0F-48C0-9AAE-9EED087F0AB6@freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help


> On 29 Aug 2024, at 17:02, Jessica Clarke <jrtc27@freebsd.org> wrote:
>=20
> On 21 Aug 2024, at 15:28, John Baldwin <jhb@FreeBSD.org> wrote:
>>=20
>> 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=3D43e8849bc29414036ccaef7788de95a0=
7ad32ab5
>>> 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
>>=20
>> 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).
>=20
> This is still broken. If it=E2=80=99s not fixed promptly I will just =
revert
> this change; we can=E2=80=99t leave CI broken, especially when it gets =
used to
> test GitHub PRs. Please stop breaking building with older toolchains,
> this isn=E2=80=99t the first time it=E2=80=99s happened.

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

Andrew




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?F794FC8A-6997-4D79-A5BD-72ACC937EFEC>