Skip site navigation (1)Skip section navigation (2)
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 &lt;<a href=3D"mailto:marklmi@yahoo.com">marklmi@yahoo.com</a>&gt; =
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&#39;t have done.</div><div><br></div><div>I&#39;ll be pushing chan=
ges to revert those three changes. I&#39;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&#39;ve =
missed anything else. I did yet another scan of the commits,</div><div>and =
didn&#39;t see any others, but things are easy to miss. We also don&#39;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>