Date: Tue, 24 Jan 2023 21:40:59 -0700 From: Warner Losh <imp@bsdimp.com> To: Mark Millard <marklmi@yahoo.com> Cc: dev-commits-src-branches@freebsd.org Subject: Re: git: e006859bce01 and git: 8cc2156f85bb (and more?) for stable/13: removing some mips support and using 1400000 in one place? Message-ID: <CANCZdfrUjc5qBD5MF--09X4tpaFdYZHvHVsF0M=M3BXKP3uT0Q@mail.gmail.com> In-Reply-To: <F84F3F96-9A2E-4698-89AC-948BF47442C7@yahoo.com> References: <F84F3F96-9A2E-4698-89AC-948BF47442C7.ref@yahoo.com> <F84F3F96-9A2E-4698-89AC-948BF47442C7@yahoo.com>
next in thread | previous in thread | raw e-mail | index | archive | help
--00000000000096272705f30f3d0d Content-Type: text/plain; charset="UTF-8" On Tue, Jan 24, 2023 at 8:47 PM Mark Millard <marklmi@yahoo.com> wrote: > https://www.freebsd.org/platforms/ shows mips as Tier 2 for 13.x but: > > > https://lists.freebsd.org/archives/dev-commits-src-branches/2023-January/008721.html > > is: > > QUOTE > The branch stable/13 has been updated by imp: > . . . > stand: remove mips support > This was a mistake on my part. > As part of decommissioning mips support, remove the boot loader > support. Do this in advance of other boot loader work to limit the > amount of work that will be thrown away. > END QUOTE > > But earlier there was also a change to use a: > > #define __FreeBSD_version 1400000 > Ditto. And there was another removal for BERI support I shouldn't have done. I'll be pushing changes to revert those three changes. I've also decided to push my MFC notes to make this easier in the future. This merge was hard enough and even though I checked it multiple times, I still missed these. This should help make it easier in the future (which I plan on doing more often). Please let me know if I've missed anything else. I did yet another scan of the commits, and didn't see any others, but things are easy to miss. We also don't have any mips booting tests, though we have them for most of our other supported platforms (though I need to finish moving from rootgen.sh to full-test.sh). Warner > in stable/13: > > > https://lists.freebsd.org/archives/dev-commits-src-branches/2023-January/008705.html > > QUOTE > The branch stable/13 has been updated by imp: > . . . > Bump the FreeBSD kernel version in kernel boot shim. > . . . > -#define __FreeBSD_version 1300000 > +#define __FreeBSD_version 1400000 > . . . > END QUOTE > > === > Mark Millard > marklmi at yahoo.com > > --00000000000096272705f30f3d0d Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable <div dir=3D"ltr"><div dir=3D"ltr"><br></div><br><div class=3D"gmail_quote">= <div dir=3D"ltr" class=3D"gmail_attr">On Tue, Jan 24, 2023 at 8:47 PM Mark = Millard <<a href=3D"mailto:marklmi@yahoo.com">marklmi@yahoo.com</a>> = wrote:<br></div><blockquote class=3D"gmail_quote" style=3D"margin:0px 0px 0= px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><a href= =3D"https://www.freebsd.org/platforms/" rel=3D"noreferrer" target=3D"_blank= ">https://www.freebsd.org/platforms/</a> shows mips as Tier 2 for 13.x but:= <br> <br> <a href=3D"https://lists.freebsd.org/archives/dev-commits-src-branches/2023= -January/008721.html" rel=3D"noreferrer" target=3D"_blank">https://lists.fr= eebsd.org/archives/dev-commits-src-branches/2023-January/008721.html</a><br= > <br> is:<br> <br> QUOTE<br> The branch stable/13 has been updated by imp:<br> . . .<br> =C2=A0 =C2=A0 stand: remove mips support<br></blockquote><div><br></div><di= v>This was a mistake on my part.<br></div><div>=C2=A0</div><blockquote clas= s=3D"gmail_quote" style=3D"margin:0px 0px 0px 0.8ex;border-left:1px solid r= gb(204,204,204);padding-left:1ex"> As part of decommissioning mips support, remove the boot loader<br> support. Do this in advance of other boot loader work to limit the<br> amount of work that will be thrown away.<br> END QUOTE<br> <br> But earlier there was also a change to use a:<br> <br> #define __FreeBSD_version 1400000<br></blockquote><div><br></div><div>Ditto= .</div><div><br></div><div>And there was another removal for BERI support I= shouldn't have done.</div><div><br></div><div>I'll be pushing chan= ges to revert those three changes. I've also decided to push</div><div>= my MFC notes to make this easier in the future. This merge was hard enough<= /div><div>and even though I checked it multiple times, I still missed these= . This should help</div><div>make it easier in the future (which I plan on = doing more often).</div><div><br></div><div>Please let me know if I've = missed anything else. I did yet another scan of the commits,</div><div>and = didn't see any others, but things are easy to miss. We also don't h= ave any mips</div><div>booting tests, though we have them for most of our o= ther supported platforms (though</div><div>I need to finish moving from roo= tgen.sh to full-test.sh).<br></div><div><br></div><div>Warner<br></div><div= >=C2=A0</div><blockquote class=3D"gmail_quote" style=3D"margin:0px 0px 0px = 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"> in stable/13:<br> <br> <a href=3D"https://lists.freebsd.org/archives/dev-commits-src-branches/2023= -January/008705.html" rel=3D"noreferrer" target=3D"_blank">https://lists.fr= eebsd.org/archives/dev-commits-src-branches/2023-January/008705.html</a><br= > <br> QUOTE<br> The branch stable/13 has been updated by imp:<br> . . .<br> =C2=A0 =C2=A0 Bump the FreeBSD kernel version in kernel boot shim.<br> . . .<br> -#define __FreeBSD_version 1300000<br> +#define __FreeBSD_version 1400000<br> . . .<br> END QUOTE<br> <br> =3D=3D=3D<br> Mark Millard<br> marklmi at <a href=3D"http://yahoo.com" rel=3D"noreferrer" target=3D"_blank= ">yahoo.com</a><br> <br> </blockquote></div></div> --00000000000096272705f30f3d0d--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CANCZdfrUjc5qBD5MF--09X4tpaFdYZHvHVsF0M=M3BXKP3uT0Q>