Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 16 Jul 2026 20:51:53 -0700
From:      Adrian Chadd <adrian@freebsd.org>
To:        Enji Cooper <ngie@freebsd.org>
Cc:        src-committers@freebsd.org, dev-commits-src-all@freebsd.org,  dev-commits-src-main@freebsd.org
Subject:   Re: git: 5cafd6213f14 - main - exterr_set: sync the definition with the header declaration
Message-ID:  <CAJ-VmokGa=o%2BS7fAUxmq5wjB8DfR1q_pSaai6i-1eYug0f47cw@mail.gmail.com>
In-Reply-To: <6a59940a.1c11d.5d8c1a96@gitrepo.freebsd.org>

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

Err this broke amd64 builds.

/usr/home/adrian/work/freebsd/head/freebsd-head/sys/kern/sys_generic.c:2356:56:
error: unknown type name 'uintptr64_t'; did you mean 'uintptr_t'?
 2356 | exterr_set(int eerror, int category, const char *mmsg, uintptr64_t pp1,
      |                                                        ^~~~~~~~~~~
      |                                                        uintptr_t
/usr/home/adrian/work/freebsd/head/freebsd-head/sys/sys/_stdint.h:78:22:
note: 'uintptr_t' declared here
   78 | typedef __uintptr_t             uintptr_t;
      |                                 ^
/usr/home/adrian/work/freebsd/head/freebsd-head/sys/kern/sys_generic.c:2357:5:
error: unknown type name 'uintptr64_t'; did you mean 'uintptr_t'?
 2357 |     uintptr64_t pp2, int line)
      |     ^~~~~~~~~~~
      |     uintptr_t
/usr/home/adrian/work/freebsd/head/freebsd-head/sys/sys/_stdint.h:78:22:
note: 'uintptr_t' declared here
   78 | typedef __uintptr_t             uintptr_t;
      |                                 ^

On Thu, 16 Jul 2026 at 19:31, Enji Cooper <ngie@freebsd.org> wrote:
>
> The branch main has been updated by ngie:
>
> URL: https://cgit.FreeBSD.org/src/commit/?id=5cafd6213f145bbe2a70d55eb90a3d894e66115b
>
> commit 5cafd6213f145bbe2a70d55eb90a3d894e66115b
> Author:     Enji Cooper <ngie@FreeBSD.org>
> AuthorDate: 2026-07-17 02:26:17 +0000
> Commit:     Enji Cooper <ngie@FreeBSD.org>
> CommitDate: 2026-07-17 02:31:16 +0000
>
>     exterr_set: sync the definition with the header declaration
>
>     This unbreaks buildkernel with TARGET=armv7 (32-bit arm). More work may be required
>     in order to unbreak `exterr_set` with 32-bit kernels.
>
>     Fixes:  844009378da9 ("exterr: allow exterr to fit pointers on CHERI targets")
> ---
>  sys/kern/sys_generic.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/sys/kern/sys_generic.c b/sys/kern/sys_generic.c
> index c867867f2618..cf152018c689 100644
> --- a/sys/kern/sys_generic.c
> +++ b/sys/kern/sys_generic.c
> @@ -2353,8 +2353,8 @@ kern_exterrctl(struct thread *td, u_int op, u_int flags, void *ptr)
>  }
>
>  int
> -exterr_set(int eerror, int category, const char *mmsg, uintptr_t pp1,
> -    uintptr_t pp2, int line)
> +exterr_set(int eerror, int category, const char *mmsg, uintptr64_t pp1,
> +    uintptr64_t pp2, int line)
>  {
>         struct thread *td;
>
>


home | help

Want to link to this message? Use this
URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAJ-VmokGa=o%2BS7fAUxmq5wjB8DfR1q_pSaai6i-1eYug0f47cw>