Date: Sun, 26 Mar 2023 19:08:38 +0200 From: Warner Losh <imp@bsdimp.com> To: Jessica Clarke <jrtc27@freebsd.org> Cc: "Alexander V. Chernikov" <melifaro@freebsd.org>, Ed Maste <emaste@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 Subject: Re: git: a85dcd4ac4b0 - main - netlink: restrict default userland switch to netlink to i386/amd64. Message-ID: <CANCZdfqKrAiDP359ipUti6yDZ8-xHLw0U3ef27t4GvaC-mBfhw@mail.gmail.com> In-Reply-To: <486C839F-181F-4CE8-8510-4A7D4D100FDC@freebsd.org> References: <202303261107.32QB7T7N058867@gitrepo.freebsd.org> <CAPyFy2Bdhi7xWs9-5JzudbVj62GarWeeJ9BwBEK=0smvcT2Dgw@mail.gmail.com> <486C839F-181F-4CE8-8510-4A7D4D100FDC@freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
--00000000000014576405f7d0aed5 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable On Sun, Mar 26, 2023, 6:45 PM Jessica Clarke <jrtc27@freebsd.org> wrote: > On 26 Mar 2023, at 17:17, Ed Maste <emaste@freebsd.org> wrote: > > > > > > On Sun, 26 Mar 2023 at 07:07, Alexander V. Chernikov > > <melifaro@freebsd.org> wrote: > >> > >> The branch main has been updated by melifaro: > >> > >> URL: > https://cgit.FreeBSD.org/src/commit/?id=3Da85dcd4ac4b0612b63b5a71eeaaa707= a09ae0003 > >> > >> commit a85dcd4ac4b0612b63b5a71eeaaa707a09ae0003 > >> Author: Alexander V. Chernikov <melifaro@FreeBSD.org> > >> AuthorDate: 2023-03-26 11:05:21 +0000 > >> Commit: Alexander V. Chernikov <melifaro@FreeBSD.org> > >> CommitDate: 2023-03-26 11:06:53 +0000 > >> > >> netlink: restrict default userland switch to netlink to i386/amd64. > > > > We should keep default options the same for all Tier-1 architectures, > > so amd64 and arm64 should be in sync. > > But also, what=E2=80=99s the technical justification for this change? The= re > should not be gratuitous differences between architectures like this, > only when there is a reason something shouldn=E2=80=99t be enabled. Why s= hould > x86 (and possibly arm64) get netlink, but not arm/powerpc/riscv? We > have very few machine-dependent defaults in src.opts.mk. If there is no > such justification then please revert this in its entirety, and if > there is such a justification it belongs in a comment above the if. > So if it's a matter of testing, enable it everywhere. We'll get testing. If it's something known to be bad on say riscv64 we should have a bug and reference it in src.opts.mk to document why. If it's just a conservative approach to a roll out, then I'd suggest being less conservative. Warner > --00000000000014576405f7d0aed5 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 Sun, Mar 26, 2023, 6:45 PM Jessica Clarke <<a hr= ef=3D"mailto:jrtc27@freebsd.org">jrtc27@freebsd.org</a>> wrote:<br></div= ><blockquote class=3D"gmail_quote" style=3D"margin:0 0 0 .8ex;border-left:1= px #ccc solid;padding-left:1ex">On 26 Mar 2023, at 17:17, Ed Maste <<a h= ref=3D"mailto:emaste@freebsd.org" target=3D"_blank" rel=3D"noreferrer">emas= te@freebsd.org</a>> wrote:<br> > <br> > <br> > On Sun, 26 Mar 2023 at 07:07, Alexander V. Chernikov<br> > <<a href=3D"mailto:melifaro@freebsd.org" target=3D"_blank" rel=3D"n= oreferrer">melifaro@freebsd.org</a>> wrote:<br> >> <br> >> The branch main has been updated by melifaro:<br> >> <br> >> URL: <a href=3D"https://cgit.FreeBSD.org/src/commit/?id=3Da85dcd4a= c4b0612b63b5a71eeaaa707a09ae0003" rel=3D"noreferrer noreferrer" target=3D"_= blank">https://cgit.FreeBSD.org/src/commit/?id=3Da85dcd4ac4b0612b63b5a71eea= aa707a09ae0003</a><br> >> <br> >> commit a85dcd4ac4b0612b63b5a71eeaaa707a09ae0003<br> >> Author:=C2=A0 =C2=A0 =C2=A0Alexander V. Chernikov <melifaro@Fre= eBSD.org><br> >> AuthorDate: 2023-03-26 11:05:21 +0000<br> >> Commit:=C2=A0 =C2=A0 =C2=A0Alexander V. Chernikov <melifaro@Fre= eBSD.org><br> >> CommitDate: 2023-03-26 11:06:53 +0000<br> >> <br> >>=C2=A0 =C2=A0 netlink: restrict default userland switch to netlink = to i386/amd64.<br> > <br> > We should keep default options the same for all Tier-1 architectures,<= br> > so amd64 and arm64 should be in sync.<br> <br> But also, what=E2=80=99s the technical justification for this change? There= <br> should not be gratuitous differences between architectures like this,<br> only when there is a reason something shouldn=E2=80=99t be enabled. Why sho= uld<br> x86 (and possibly arm64) get netlink, but not arm/powerpc/riscv? We<br> have very few machine-dependent defaults in <a href=3D"http://src.opts.mk" = rel=3D"noreferrer noreferrer" target=3D"_blank">src.opts.mk</a>. If there i= s no<br> such justification then please revert this in its entirety, and if<br> there is such a justification it belongs in a comment above the if.<br></bl= ockquote></div></div><div dir=3D"auto"><br></div><div dir=3D"auto">So if it= 's a matter of testing, enable it everywhere. We'll get testing. If= it's something known to be bad on say riscv64 we should have a bug and= reference it in <a href=3D"http://src.opts.mk">src.opts.mk</a>; to document= why. If it's just a conservative approach to a roll out, then I'd = suggest being less conservative.=C2=A0</div><div dir=3D"auto"><br></div><di= v dir=3D"auto">Warner</div><div dir=3D"auto"><div class=3D"gmail_quote"><bl= ockquote class=3D"gmail_quote" style=3D"margin:0 0 0 .8ex;border-left:1px #= ccc solid;padding-left:1ex"> </blockquote></div></div></div> --00000000000014576405f7d0aed5--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CANCZdfqKrAiDP359ipUti6yDZ8-xHLw0U3ef27t4GvaC-mBfhw>