Date: Tue, 22 Mar 2022 20:12:21 -0700 From: Mark Millard <marklmi@yahoo.com> Cc: bob prohaska <fbsd@www.zefox.net>, "mav@freebsd.org" <mav@FreeBSD.org>, freebsd-arm@freebsd.org Subject: Re: /usr/src/sys/net/if_epair.c:181:6: error: ... Message-ID: <956CA18B-CB65-412D-86D3-60400FD30767@yahoo.com> In-Reply-To: <A618B408-D8AC-493B-91E8-58733168CC8C@yahoo.com> References: <20220323012657.GA82109@www.zefox.net> <A618B408-D8AC-493B-91E8-58733168CC8C@yahoo.com>
next in thread | previous in thread | raw e-mail | index | archive | help
IGNORE THIS: I somehow switched contexts. Wrong commit referenced! > On 2022-Mar-22, at 20:06, Mark Millard <marklmi@yahoo.com> wrote: >=20 > On 2022-Mar-22, at 18:26, bob prohaska <fbsd@www.zefox.net> wrote: >=20 >> A Pi2 running=20 >> FreeBSD www.zefox.net 12.3-STABLE FreeBSD 12.3-STABLE r371495 GENERIC = arm >>=20 >> stops buildkernel with: >> --- if_epair.o --- >> /usr/src/sys/net/if_epair.c:181:6: error: implicit declaration of = function 'atomic_testandclear_long' is invalid in C99 = [-Werror,-Wimplicit-function-declaration] >> if (atomic_testandclear_long(&q->state, BIT_MBUF_QUEUED)) >> ^ >>=20 >> Not sure if this is specific to the Raspberry Pi 2, it didn't show up = on a pair of Pi3's >> and a single Pi4. The system is still using svnlite, info reports >> root@www:/usr/src # svnlite info >> Path: . >> Working Copy Root Path: /usr/src >> URL: svn://svn.freebsd.org/base/stable/12 >> Relative URL: ^/stable/12 >> Repository Root: svn://svn.freebsd.org/base >> Repository UUID: ccf9f872-aa2e-dd11-9fc8-001c23d0bc1f >> Revision: 371771 >> Node Kind: directory >> Schedule: normal >> Last Changed Author: 0mp >> Last Changed Rev: 371771 >> Last Changed Date: 2022-03-22 15:28:40 -0700 (Tue, 22 Mar 2022) >>=20 >>=20 >> Didn't see anything similar on bugs.freebsd.org, if it's worth a bug = report or >> there's a workaround please post. It was built using WITH_META_MODE = if that=20 >> matters.=20 >=20 >=20 > author Alexander Motin <mav@FreeBSD.org> 2022-03-16 = 04:09:09 +0000 > committer Alexander Motin <mav@FreeBSD.org> 2022-03-23 = 01:31:17 +0000 > commit 5f81a4619dcf8026ab0ba12ea0bd1e6e36ae8c6c (patch) > . . . >=20 > Remove "/dev/" from geom name in `gpart add` command.stable/12 >=20 > broke things by adding the atomic_testandclear_long usage without > defining it as well. >=20 > It goes like this: >=20 > path: root/sys/arm/include/atomic.h > Commit message (Expand) Author Age Files Lines > * MFC r341787 by hselasky: Implement atomic_swap_xxx() for all = platforms. Andriy Gapon 2019-10-24 1 -0/+7 > * Remove arm-specific implementations of atomic_load/store_xxx() = now that Ian Lepore 2017-12-20 1 -27/+0 > . . . >=20 > So not updated in a long time. But for armv7 and the like, it = includes: >=20 > path: root/sys/arm/include/atomic-v6.h > Commit message (Expand) Author Age Files Lines > * MFC r352938: Ian Lepore 2019-12-07 1 = -100/+256 > * MFC r341679: Michal Meloun 2018-12-14 1 -1/+1 > . . . >=20 > Also not updated in a long time. >=20 > sys/arm/include/atomic-v6.h has various "atomic_testand" > examples ( sys/arm/include/atomic.h does not ): >=20 > atomic_testandset_32 > atomic_testandset_int > atomic_testandset_long > atomic_testandset_64 >=20 > But no examples of "atomic_testandclear" >=20 =3D=3D=3D Mark Millard marklmi at yahoo.com
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?956CA18B-CB65-412D-86D3-60400FD30767>