Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 7 Sep 2023 07:22:07 -0600
From:      Warner Losh <imp@bsdimp.com>
To:        "Bjoern A. Zeeb" <bz@freebsd.org>
Cc:        John Baldwin <jhb@freebsd.org>, src-committers@freebsd.org,  dev-commits-src-all@freebsd.org, dev-commits-src-main@freebsd.org
Subject:   Re: git: cf94672363d6 - main - rtw88/rtw89: remove local firmware.
Message-ID:  <CANCZdfpPx8NmBR5d9H5=GeSSSGGjxZpv=x1EJX-NQNX9gOFpYg@mail.gmail.com>
In-Reply-To: <4sr8n9or-q731-1945-893s-no953r283756@SerrOFQ.bet>
References:  <202309052116.385LGoau059784@gitrepo.freebsd.org> <d9be0f6a-baed-55f1-5ea1-390e1392a631@FreeBSD.org> <92064p8n-134r-9rqo-8038-rpnq4r25p567@SerrOFQ.bet> <5325f22d-9e7f-04bd-ab35-128438eb9a2a@FreeBSD.org> <4sr8n9or-q731-1945-893s-no953r283756@SerrOFQ.bet>

next in thread | previous in thread | raw e-mail | index | archive | help
--000000000000dcd05f0604c4bf55
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

On Thu, Sep 7, 2023 at 6:36=E2=80=AFAM Bjoern A. Zeeb <bz@freebsd.org> wrot=
e:

> On Wed, 6 Sep 2023, John Baldwin wrote:
>
> > On 9/6/23 12:06 PM, Bjoern A. Zeeb wrote:
> >> On Wed, 6 Sep 2023, John Baldwin wrote:
> >>
> >>> On 9/5/23 2:16 PM, Bjoern A. Zeeb wrote:
> >>>> The branch main has been updated by bz:
> >>>>
> >>>> URL:
> >>>>
> https://cgit.FreeBSD.org/src/commit/?id=3Dcf94672363d6638b7268c05b4628b62=
bbfc8dbe5
> >>>>
> >>>> commit cf94672363d6638b7268c05b4628b62bbfc8dbe5
> >>>> Author:     Bjoern A. Zeeb <bz@FreeBSD.org>
> >>>> AuthorDate: 2023-06-08 21:35:09 +0000
> >>>> Commit:     Bjoern A. Zeeb <bz@FreeBSD.org>
> >>>> CommitDate: 2023-09-05 21:13:18 +0000
> >>>>
> >>>>       rtw88/rtw89: remove local firmware.
> >>>>            Remove firmware from src/ in favor of the ports/packages
> and
> >>>> fwget(8).
> >>>>       This will allow us to shrink the size of src (and installed
> >>>> modules).
> >>>>       Update the rtw88 man page to reflect the change.
> >>>
> >>> How does this interfact with doing an install to a machine over one o=
f
> >>> these
> >>> interfaces?
> >>
> >> That's nothing we support.  At least we do not support netboot over wi=
fi
> >> currently, so you'll always need another install media.
> >>
> >> How's the "bootstrapping" going to work if wireless is the only networ=
k
> >> option?
> >
> > To be clear, I was discussing the case you use a USB memstick, but fetc=
h
> > the base system and packages over the network.
>
> Understood.  Not sure we actually support the former;  I guess I am not
> using the installer too much.
>
>
> > Or at least you need to
> > install pkg over the network, but you need pkg to install the firmware
> > so you can bring up the network which you did address, though it seems
> > something we plan to fix in the future so won't be in 14.0?
>
> Yes.  We plan to sort this out but I'd say it's unlikely for 14.0.
>
> And to be clear, it was me who raised exactly these "botstrapping"
> installer concerns in the first place.
>
> At the moment we will first need stable drivers to be able to do any
> networking in first place.   Once that is sorted, installing mobile
> devices with just them and no other means of networking will be
> addressed.
>

Yea, the rtw88 driver is stable enough for me to do a network install
over it. So now people with -current can't do that any more and have
to do hokey things to get the firmware or other packages to the laptop
before they can do anything. This is not really going to make people
happy and will generate a fair number of complaints.

I'd assume some other "fallout" will improve some installer bits along
> when this happens and that we'll want enough and brought testing for it.
>

It sounds rather premature to remove it from -current if there's no support
in the installer, and nothing in place that adds the packages to the
installation
media with no firm plans or timelines to address that.

Warner

--000000000000dcd05f0604c4bf55
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 Thu, Sep 7, 2023 at 6:36=E2=80=AFA=
M Bjoern A. Zeeb &lt;<a href=3D"mailto:bz@freebsd.org">bz@freebsd.org</a>&g=
t; wrote:<br></div><blockquote class=3D"gmail_quote" style=3D"margin:0px 0p=
x 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Wed=
, 6 Sep 2023, John Baldwin wrote:<br>
<br>
&gt; On 9/6/23 12:06 PM, Bjoern A. Zeeb wrote:<br>
&gt;&gt; On Wed, 6 Sep 2023, John Baldwin wrote:<br>
&gt;&gt; <br>
&gt;&gt;&gt; On 9/5/23 2:16 PM, Bjoern A. Zeeb wrote:<br>
&gt;&gt;&gt;&gt; The branch main has been updated by bz:<br>
&gt;&gt;&gt;&gt; <br>
&gt;&gt;&gt;&gt; URL:<br>
&gt;&gt;&gt;&gt; <a href=3D"https://cgit.FreeBSD.org/src/commit/?id=3Dcf946=
72363d6638b7268c05b4628b62bbfc8dbe5" rel=3D"noreferrer" target=3D"_blank">h=
ttps://cgit.FreeBSD.org/src/commit/?id=3Dcf94672363d6638b7268c05b4628b62bbf=
c8dbe5</a><br>
&gt;&gt;&gt;&gt; <br>
&gt;&gt;&gt;&gt; commit cf94672363d6638b7268c05b4628b62bbfc8dbe5<br>
&gt;&gt;&gt;&gt; Author:=C2=A0 =C2=A0 =C2=A0Bjoern A. Zeeb &lt;bz@FreeBSD.o=
rg&gt;<br>
&gt;&gt;&gt;&gt; AuthorDate: 2023-06-08 21:35:09 +0000<br>
&gt;&gt;&gt;&gt; Commit:=C2=A0 =C2=A0 =C2=A0Bjoern A. Zeeb &lt;bz@FreeBSD.o=
rg&gt;<br>
&gt;&gt;&gt;&gt; CommitDate: 2023-09-05 21:13:18 +0000<br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt;=C2=A0 =C2=A0 =C2=A0 =C2=A0rtw88/rtw89: remove local firmwa=
re.<br>
&gt;&gt;&gt;&gt;=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 Remove firmware f=
rom src/ in favor of the ports/packages and<br>
&gt;&gt;&gt;&gt; fwget(8).<br>
&gt;&gt;&gt;&gt;=C2=A0 =C2=A0 =C2=A0 =C2=A0This will allow us to shrink the=
 size of src (and installed <br>
&gt;&gt;&gt;&gt; modules).<br>
&gt;&gt;&gt;&gt;=C2=A0 =C2=A0 =C2=A0 =C2=A0Update the rtw88 man page to ref=
lect the change.<br>
&gt;&gt;&gt; <br>
&gt;&gt;&gt; How does this interfact with doing an install to a machine ove=
r one of <br>
&gt;&gt;&gt; these<br>
&gt;&gt;&gt; interfaces?<br>
&gt;&gt; <br>
&gt;&gt; That&#39;s nothing we support.=C2=A0 At least we do not support ne=
tboot over wifi<br>
&gt;&gt; currently, so you&#39;ll always need another install media.<br>
&gt;&gt; <br>
&gt;&gt; How&#39;s the &quot;bootstrapping&quot; going to work if wireless =
is the only network<br>
&gt;&gt; option?<br>
&gt;<br>
&gt; To be clear, I was discussing the case you use a USB memstick, but fet=
ch<br>
&gt; the base system and packages over the network.<br>
<br>
Understood.=C2=A0 Not sure we actually support the former;=C2=A0 I guess I =
am not<br>
using the installer too much.<br>
<br>
<br>
&gt; Or at least you need to<br>
&gt; install pkg over the network, but you need pkg to install the firmware=
<br>
&gt; so you can bring up the network which you did address, though it seems=
<br>
&gt; something we plan to fix in the future so won&#39;t be in 14.0?<br>
<br>
Yes.=C2=A0 We plan to sort this out but I&#39;d say it&#39;s unlikely for 1=
4.0.<br>
<br>
And to be clear, it was me who raised exactly these &quot;botstrapping&quot=
;<br>
installer concerns in the first place.<br>
<br>
At the moment we will first need stable drivers to be able to do any<br>
networking in first place.=C2=A0 =C2=A0Once that is sorted, installing mobi=
le<br>
devices with just them and no other means of networking will be<br>
addressed.<br></blockquote><div><br></div><div>Yea, the rtw88 driver is sta=
ble enough for me to do a network install</div><div>over it. So now people =
with -current can&#39;t do that any more and have</div><div>to do hokey thi=
ngs to get the firmware or other packages to the laptop</div><div>before th=
ey can do anything. This is not really going to make people</div><div>happy=
 and will generate a fair number of complaints.</div><div><br></div><blockq=
uote class=3D"gmail_quote" style=3D"margin:0px 0px 0px 0.8ex;border-left:1p=
x solid rgb(204,204,204);padding-left:1ex">I&#39;d assume some other &quot;=
fallout&quot; will improve some installer bits along<br>
when this happens and that we&#39;ll want enough and brought testing for it=
.<br></blockquote><div><br></div><div>It sounds rather premature to remove =
it from -current if there&#39;s no support</div><div>in the installer, and =
nothing in place that adds the packages to the installation</div><div>media=
 with no firm plans or timelines to address that.</div><div><br></div><div>=
Warner</div></div></div>

--000000000000dcd05f0604c4bf55--



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CANCZdfpPx8NmBR5d9H5=GeSSSGGjxZpv=x1EJX-NQNX9gOFpYg>