From nobody Sun May 3 19:31:50 2026 X-Original-To: dev-commits-src-all@mlmmj.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mlmmj.nyi.freebsd.org (Postfix) with ESMTP id 4g7vzm3cc4z6bxbY for ; Sun, 03 May 2026 19:32:04 +0000 (UTC) (envelope-from adrian.chadd@gmail.com) Received: from mail-qk1-f177.google.com (mail-qk1-f177.google.com [209.85.222.177]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256 client-signature RSA-PSS (2048 bits) client-digest SHA256) (Client CN "smtp.gmail.com", Issuer "WR4" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 4g7vzl1tFGz3v05 for ; Sun, 03 May 2026 19:32:03 +0000 (UTC) (envelope-from adrian.chadd@gmail.com) Authentication-Results: mx1.freebsd.org; none Received: by mail-qk1-f177.google.com with SMTP id af79cd13be357-8cbc593a67aso314442785a.2 for ; Sun, 03 May 2026 12:32:03 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1777836722; cv=none; d=google.com; s=arc-20240605; b=T27oien+PP39BRJfcT7WzkJ4+2UD/dXPBmqc9ioFbPexi2SY0sye+AMc9CNuh40ikb rXfCiWJLUArZiv97vfHhpNw8MC0no8dUh8ImYNwd2bmVT6f/K1uOcAoTwNmL1Vi7+6vR JcX9/DzA1EP41dMhdH+ZYhL48GO475MTzyAtV1vOeymumkY1logpQlYiBKkpSEPqpYsd 95olriZpI5uOxE3YIAkH8yAXDe+WGBFcurnw89EI+s8LgCb7lQtYa+gqK6UDE9h+MX/d OHvpgVWGV3jkJ6Pr+9ktjndR/gS/4lwdGbMwSlPPFeh4XX7ts2VQN8G5Enc5aVUnerKc mHtQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20240605; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version; bh=KEM6l1hcVDQVqX+aukKW64DKb2oprVq/tvvIlVvRiX8=; fh=dyWhlWnPnDf3U7petwi4e28Z/iDDC1Kqlfvjb3hZkek=; b=i0o1mKL99eLm53q3S0DlLm6rRPd8/cOYqVUpMTwulr47DIQBKlaCopDi26NktHSXMB zmEI5N7JRolXjam7d5kDZZ9nP3amLDuH8Yk3k/1VRmoChod8jyuguq76xEceDlAHdAXi fUOeYHxHx1V4uHaxuXT9oOawpwhGtv3snQf3yoZJhVIzsFSs9lWH066B6m1j9BwEFE3f TXlOpJS+nCLlsmJYrC1DPlK7RFRLWd0XDzLODI/VnZHGJMpc3wJW6UnarcK4+lmSA+py YNvKoz6a9lx8oeymP1tgBdz/N9fVK9hI9ON/nLQN86hvA5xKG61xUDwBHOmxVndLjvIV +b5g==; darn=freebsd.org ARC-Authentication-Results: i=1; mx.google.com; arc=none X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1777836722; x=1778441522; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:x-gm-gg:x-gm-message-state:from :to:cc:subject:date:message-id:reply-to; bh=KEM6l1hcVDQVqX+aukKW64DKb2oprVq/tvvIlVvRiX8=; b=EqVKgwqvyf9CnZ65JZJkNBm3rPp/sGNfgSgeWKu3/kb4WRzDxscoYmmA42vlhMF834 hFEfvrJC3bKh2riPMzV064JqqVkejLsMiEKx+WbsjS8Baibzkn/qLJwIT/kuyqW0r/nN l3uQSQx679QHCNXFOkImVcaQHkuIFpE+LLUSW0p9hfUEj2nANixAKQy/QjppmUziaZSH HtjK/xf2OBaKVJBHDazhcBqNYb7Xj3ftbLg7a1sHPRPsPVMAD0FT4Rdf3aLVAFw4ZCeZ 4b3fNGVlimwXHGzF5cLebHwuUq7DgQnWjYvfJcY8VhE1enI0TbIRcW9yFJMb4hLy1BWs 2z/A== X-Forwarded-Encrypted: i=1; AFNElJ/J/WNBbX6brnec45cyRXrYtHTGypKV5eIrWwQMKVVxtLFntzN90rHZIuuKxKimfShtP8Yg0AP+vTN6+vdeIT4kMwhM@freebsd.org X-Gm-Message-State: AOJu0YyQpuOUo/Q5/r+PyRv2pXRf4JpD6DGfF5um0cCa5foWkl0ZOizI DJzG8JLzF3dkMVStK7qu6v5q0aDAgrLVxhVGtcdIand+rjpPsLm8qt1aPsVEUexu3+8iGXfke+i +9gROqJmNQ4Ix16PE6t8f2a+ULlOSSg0= X-Gm-Gg: AeBDietJpR9XSvErX9w3yxhYTwa4z4Efpssg0/AJoRgPE9SiJyw+e7mCNjmrZaSNZsq S4oVUa7cjXDRK/NkadzVlD5kMXjpku2dHa1bhPfQ9NuqEFmupzLSJ59Kt7Pz9XtIdkFpjkc938l YEfYJ175mVLRj/8Css50QJDbRwco9lBLgxiZqQVn8A6uUd3jNO8X0oLET7/AM61HeADc2L4TpeD Hn5RSSOsUcHzZDMDZh7sUreTq8kGJ5UAd1SWzmU6tMbOyN1q8Saflzs9qkgvh4V5qNAJm2lLsZE 29A8w4UemwaPhF/cVkH0w3FMUH3CXH4iAMVA2VSfThWBAbyJD/HFD+wtfzJ0vYYyY/CYrvLR21j jjK4n5wbWFULRz/msHGnW2QXp6fe8MgiqsQAt58pjImPW09QXLaM6ZMWVQ7/Tg0JIvemx X-Received: by 2002:ad4:5ce4:0:b0:8b3:fb6a:d35d with SMTP id 6a1803df08f44-8b6696db9b5mr137706956d6.47.1777836722101; Sun, 03 May 2026 12:32:02 -0700 (PDT) List-Id: Commit messages for all branches of the src repository List-Archive: https://lists.freebsd.org/archives/dev-commits-src-all List-Help: List-Post: List-Subscribe: List-Unsubscribe: X-BeenThere: dev-commits-src-all@freebsd.org Sender: owner-dev-commits-src-all@FreeBSD.org List-Id: List-Post: List-Help: List-Subscribe: List-Unsubscribe: List-Owner: Precedence: list MIME-Version: 1.0 References: <69f6b1ec.181e8.27d0b938@gitrepo.freebsd.org> <6C5FEC1A-D8E1-45B8-A80A-B32F81392058@freebsd.org> In-Reply-To: <6C5FEC1A-D8E1-45B8-A80A-B32F81392058@freebsd.org> From: Adrian Chadd Date: Sun, 3 May 2026 12:31:50 -0700 X-Gm-Features: AVHnY4L5lWsj72fWPxZd59ctEUa51Rs4cxCohA6FO3a1TC1luqDawdesJW4QnHY Message-ID: Subject: Re: git: 01674e15dedb - main - bsd.endian.mk: Optimize the handling of big/little endian determination. To: Jessica Clarke Cc: "Stephen J. Kiernan" , "src-committers@freebsd.org" , "dev-commits-src-all@freebsd.org" , "dev-commits-src-main@freebsd.org" Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-Spamd-Result: default: False [-4.00 / 15.00]; REPLY(-4.00)[]; TAGGED_FROM(0.00)[]; ASN(0.00)[asn:15169, ipnet:209.85.128.0/17, country:US] X-Rspamd-Queue-Id: 4g7vzl1tFGz3v05 X-Spamd-Bar: ---- X-Rspamd-Pre-Result: action=no action; module=replies; Message is reply to one we originated I just hit this too. can we revert the diff for now until we figure out why it breaks ports building at least? -a On Sun, 3 May 2026 at 11:49, Jessica Clarke wrote: > > On 3 May 2026, at 03:24, Stephen J. Kiernan wrote: > > > > The branch main has been updated by stevek: > > > > URL: https://cgit.FreeBSD.org/src/commit/?id=3D01674e15dedb15f8ce498af0= 680c50bb075c5482 > > > > commit 01674e15dedb15f8ce498af0680c50bb075c5482 > > Author: Stephen J. Kiernan > > AuthorDate: 2024-04-03 23:25:19 +0000 > > Commit: Stephen J. Kiernan > > CommitDate: 2026-05-03 02:21:56 +0000 > > > > bsd.endian.mk: Optimize the handling of big/little endian determinat= ion. > > > > Add variables to contain lists of MACHINE_ARCH values to use > > to determine little or big endian. > > > > Only error out about not being able to determine endianess if > > TARGET_ENDIANNESS is empty and not cross-compiling. > > > > Reviewed by: sjg > > Obtained from: Hewlett Packard Enterprise Development LP > > Differential Revision: https://reviews.freebsd.org/D44629 > > --- > > share/mk/bsd.endian.mk | 39 +++++++++++++++++++++++++++++---------- > > 1 file changed, 29 insertions(+), 10 deletions(-) > > > > diff --git a/share/mk/bsd.endian.mk b/share/mk/bsd.endian.mk > > index 0c5ebb7aeba2..2a617a1d2794 100644 > > --- a/share/mk/bsd.endian.mk > > +++ b/share/mk/bsd.endian.mk > > @@ -1,20 +1,35 @@ > > > > -.if ${MACHINE_CPUARCH} =3D=3D "aarch64" || \ > > - ${MACHINE_CPUARCH} =3D=3D "arm" || \ > > - ${MACHINE_ARCH} =3D=3D "amd64" || \ > > - ${MACHINE_ARCH} =3D=3D "i386" || \ > > - ${MACHINE_ARCH} =3D=3D "powerpc64le" || \ > > - ${MACHINE_CPUARCH} =3D=3D "riscv" > > Losing MACHINE_CPUARCH makes me sad as a downstream that adds new > MACHINE_ARCH for aarch64 and riscv. For riscv at least it=E2=80=99s now r= iscv* > which I guess is close enough, but for aarch64 it=E2=80=99s not wildcarde= d. > Similarly arm though I don=E2=80=99t care about that personally. > > > +MACHINE_ARCH_LIST.little =3D \ > > + aarch64 \ > > + amd64 \ > > + armv7 \ > > + i386 \ > > + powerpc64le \ > > + riscv* > > + > > +MACHINE_ARCH_LIST.big =3D \ > > + powerpc \ > > + powerpc64 > > + > > +.for e in big little > > +N_$e:=3D ${MACHINE_ARCH_LIST.$e:${M_ListToSkip}} > > Should we be worried about catching MACHINE(_ARCH) being empty? > > Jessica > > > +.endfor > > + > > +# For the host, we need to look at the host architecture > > +.if ${MACHINE:Nhost*} =3D=3D "" > > +_ENDIAN_ARCH=3D${_HOST_ARCH} > > +.else > > +_ENDIAN_ARCH=3D${MACHINE_ARCH} > > +.endif > > + > > +.if ${_ENDIAN_ARCH:${N_little}} =3D=3D "" > > TARGET_ENDIANNESS=3D 1234 > > CAP_MKDB_ENDIAN=3D -l > > LOCALEDEF_ENDIAN=3D -l > > -.elif ${MACHINE_ARCH} =3D=3D "powerpc" || \ > > - ${MACHINE_ARCH} =3D=3D "powerpc64" > > +.elif ${_ENDIAN_ARCH:${N_big}} =3D=3D "" > > TARGET_ENDIANNESS=3D 4321 > > CAP_MKDB_ENDIAN=3D -b > > LOCALEDEF_ENDIAN=3D -b > > -.elif ${.MAKE.OS} =3D=3D "FreeBSD" > > -.error Don't know the endian of this architecture > > .else > > # > > # During bootstrapping on !FreeBSD OSes, we need to define some value. = Short of > > @@ -33,3 +48,7 @@ TARGET_ENDIANNESS=3D 1234 > > CAP_MKDB_ENDIAN=3D -B # Poisoned value, invalid flags for both cap_mkdb > > LOCALEDEF_ENDIAN=3D -B # and localedef. > > .endif > > + > > +.if empty(TARGET_ENDIANNESS) && ${.MAKE.OS} =3D=3D "FreeBSD" > > +.error Don't know the endianness of this architecture > > +.endif > > > >