Date: Tue, 28 May 2024 11:09:19 -0600 From: Warner Losh <imp@bsdimp.com> To: Jessica Clarke <jrtc27@freebsd.org> Cc: Mark Johnston <markj@freebsd.org>, src-committers <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: c867ba72889d - main - bnxt: Do not compile on 32-bit platforms Message-ID: <CANCZdfr-3BKRvCAJfhxtTDncpSyUJrDtZtBP8bhEB_d94nd7Yw@mail.gmail.com> In-Reply-To: <EED35A2A-B024-4953-9ED1-AB5355DF4864@freebsd.org> References: <202405281313.44SDD7VM021879@gitrepo.freebsd.org> <EED35A2A-B024-4953-9ED1-AB5355DF4864@freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
--000000000000c9d08e0619878eb9 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable On Tue, May 28, 2024, 10:41=E2=80=AFAM Jessica Clarke <jrtc27@freebsd.org> = wrote: > On 28 May 2024, at 14:13, Mark Johnston <markj@FreeBSD.org> wrote: > > > > The branch main has been updated by markj: > > > > URL: > https://cgit.FreeBSD.org/src/commit/?id=3Dc867ba72889de51b0bd2c6a7049f539= abaafd46e > > > > commit c867ba72889de51b0bd2c6a7049f539abaafd46e > > Author: Mark Johnston <markj@FreeBSD.org> > > AuthorDate: 2024-05-28 13:05:14 +0000 > > Commit: Mark Johnston <markj@FreeBSD.org> > > CommitDate: 2024-05-28 13:12:52 +0000 > > > > bnxt: Do not compile on 32-bit platforms > > > > The new bnxt_re driver doesn't compile on any of them (it uses > writeq() > > from the LinuxKPI, which isn't implemented there), and had already > been > > disconnected from the build on i386. > > > > Reported by: Jenkins > > Fixes: acd884dec99a ("RDMA/bnxt_re: Add bnxt_re RoCE driver") > > --- > > sys/modules/Makefile | 4 ++-- > > 1 file changed, 2 insertions(+), 2 deletions(-) > > > > diff --git a/sys/modules/Makefile b/sys/modules/Makefile > > index 54108097fe1d..d3ef4c63a41c 100644 > > --- a/sys/modules/Makefile > > +++ b/sys/modules/Makefile > > @@ -442,8 +442,8 @@ SUBDIR+=3D dtrace > > SUBDIR+=3D opensolaris > > .endif > > > > -# Requires bus_space_read_8 > > -.if ${MACHINE_ARCH} !=3D "i386" > > +.if ${MACHINE_ARCH} !=3D "i386" && ${MACHINE_CPUARCH} !=3D "arm" && \ > > + ${MACHINE_ARCH} !=3D "powerpc" && ${MACHINE_ARCH} !=3D "powerpcspe= " > > Check ${MACHINE_ABI:Mlong64} or !${MACHINE_ABI:Mlong32} instead? > Read 8 can be orthogonal to either of these... i thought we had it for armv7 at least... For this driver, though, any of the above are fine. Warner > --000000000000c9d08e0619878eb9 Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable <div dir=3D"auto"><div><br><br><div class=3D"gmail_quote"><div dir=3D"ltr" = class=3D"gmail_attr">On Tue, May 28, 2024, 10:41=E2=80=AFAM Jessica Clarke = <<a href=3D"mailto:jrtc27@freebsd.org">jrtc27@freebsd.org</a>> wrote:= <br></div><blockquote class=3D"gmail_quote" style=3D"margin:0 0 0 .8ex;bord= er-left:1px #ccc solid;padding-left:1ex">On 28 May 2024, at 14:13, Mark Joh= nston <markj@FreeBSD.org> wrote:<br> > <br> > The branch main has been updated by markj:<br> > <br> > URL: <a href=3D"https://cgit.FreeBSD.org/src/commit/?id=3Dc867ba72889d= e51b0bd2c6a7049f539abaafd46e" rel=3D"noreferrer noreferrer" target=3D"_blan= k">https://cgit.FreeBSD.org/src/commit/?id=3Dc867ba72889de51b0bd2c6a7049f53= 9abaafd46e</a><br> > <br> > commit c867ba72889de51b0bd2c6a7049f539abaafd46e<br> > Author:=C2=A0 =C2=A0 =C2=A0Mark Johnston <markj@FreeBSD.org><br> > AuthorDate: 2024-05-28 13:05:14 +0000<br> > Commit:=C2=A0 =C2=A0 =C2=A0Mark Johnston <markj@FreeBSD.org><br> > CommitDate: 2024-05-28 13:12:52 +0000<br> > <br> >=C2=A0 =C2=A0 bnxt: Do not compile on 32-bit platforms<br> > <br> >=C2=A0 =C2=A0 The new bnxt_re driver doesn't compile on any of them= (it uses writeq()<br> >=C2=A0 =C2=A0 from the LinuxKPI, which isn't implemented there), an= d had already been<br> >=C2=A0 =C2=A0 disconnected from the build on i386.<br> > <br> >=C2=A0 =C2=A0 Reported by:=C2=A0 =C2=A0 Jenkins<br> >=C2=A0 =C2=A0 Fixes:=C2=A0 acd884dec99a ("RDMA/bnxt_re: Add bnxt_r= e RoCE driver")<br> > ---<br> > sys/modules/Makefile | 4 ++--<br> > 1 file changed, 2 insertions(+), 2 deletions(-)<br> > <br> > diff --git a/sys/modules/Makefile b/sys/modules/Makefile<br> > index 54108097fe1d..d3ef4c63a41c 100644<br> > --- a/sys/modules/Makefile<br> > +++ b/sys/modules/Makefile<br> > @@ -442,8 +442,8 @@ SUBDIR+=3D dtrace<br> > SUBDIR+=3D opensolaris<br> > .endif<br> > <br> > -# Requires bus_space_read_8<br> > -.if ${MACHINE_ARCH} !=3D "i386"<br> > +.if ${MACHINE_ARCH} !=3D "i386" && ${MACHINE_CPUARC= H} !=3D "arm" && \<br> > +=C2=A0 =C2=A0 ${MACHINE_ARCH} !=3D "powerpc" && ${M= ACHINE_ARCH} !=3D "powerpcspe"<br> <br> Check ${MACHINE_ABI:Mlong64} or !${MACHINE_ABI:Mlong32} instead?<br></block= quote></div></div><div dir=3D"auto"><br></div><div dir=3D"auto">Read 8 can = be orthogonal to either of these... i thought we had it for armv7 at least.= ..</div><div dir=3D"auto"><br></div><div dir=3D"auto">For this driver, thou= gh, any of the above are fine.</div><div dir=3D"auto"><br></div><div dir=3D= "auto">Warner</div><div dir=3D"auto"><div class=3D"gmail_quote"><blockquote= class=3D"gmail_quote" style=3D"margin:0 0 0 .8ex;border-left:1px #ccc soli= d;padding-left:1ex"> </blockquote></div></div></div> --000000000000c9d08e0619878eb9--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CANCZdfr-3BKRvCAJfhxtTDncpSyUJrDtZtBP8bhEB_d94nd7Yw>