Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 5 Jan 2022 14:07:28 -0700
From:      Warner Losh <imp@bsdimp.com>
To:        Mateusz Guzik <mjguzik@gmail.com>
Cc:        Warner Losh <imp@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: 5a73a6c178f6 - main - Add lio back to NOTES
Message-ID:  <CANCZdfpmzBxwR20M8U34vYmZmV0AwOARyXJEMPp3m7SniEu4kg@mail.gmail.com>
In-Reply-To: <CAGudoHENRFPB=voAzszEAPTx1c0Rc5WXqDMeqQe%2B2QPSgMPEEg@mail.gmail.com>
References:  <202112312157.1BVLvLhd040939@gitrepo.freebsd.org> <CAGudoHF_g6f2FDx_DnGCpkNRF6mK1FCPf9vet46RPUKQ618Pcg@mail.gmail.com> <CANCZdfpYRxdgaxVbc1xEozy7w8tbSMNYmXJTGE3veuO2VAOasQ@mail.gmail.com> <CAGudoHENRFPB=voAzszEAPTx1c0Rc5WXqDMeqQe%2B2QPSgMPEEg@mail.gmail.com>

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

On Wed, Jan 5, 2022 at 1:53 PM Mateusz Guzik <mjguzik@gmail.com> wrote:

> On 1/5/22, Warner Losh <imp@bsdimp.com> wrote:
> > On Wed, Jan 5, 2022, 1:26 PM Mateusz Guzik <mjguzik@gmail.com> wrote:
> >
> >> On 12/31/21, Warner Losh <imp@freebsd.org> wrote:
> >> > The branch main has been updated by imp:
> >> >
> >> > URL:
> >> >
> >>
> https://cgit.FreeBSD.org/src/commit/?id=5a73a6c178f65b91fe60e52e476a88137e40f0e4
> >> >
> >> > commit 5a73a6c178f65b91fe60e52e476a88137e40f0e4
> >> > Author:     Warner Losh <imp@FreeBSD.org>
> >> > AuthorDate: 2021-12-31 21:56:16 +0000
> >> > Commit:     Warner Losh <imp@FreeBSD.org>
> >> > CommitDate: 2021-12-31 21:56:16 +0000
> >> >
> >> >     Add lio back to NOTES
> >> >
> >> >     lio appears to be partially removed from this file. It's a modern
> >> > enough
> >> >     card, so add it back. It builds everywhere.
> >> >
> >>
> >> This breaks NOIP kernels:
> >>
> >
> > NOIP kernels were already broken when I made this commit. I'll take a
> look
> > now that emaste committed fixes for those...
> >
>
> It builds fine without the change.
>

Only because of a6668e31aa3cc855bfb37d6edc05122514592a47

Warner


> > Warner
> >
> > ld: error: undefined symbol: tcp_lro_flush_all
> >> >>> referenced by lio_droq.c
> >> >>>               lio_droq.o:(lio_droq_process_packets)
> >>
> >> ld: error: undefined symbol: tcp_lro_rx
> >> >>> referenced by lio_core.c
> >> >>>               lio_core.o:(lio_push_packet)
> >>
> >> ld: error: undefined symbol: tcp_lro_init
> >> >>> referenced by lio_main.c
> >> >>>               lio_main.o:(lio_attach)
> >>
> >> ld: error: undefined symbol: tcp_lro_free
> >> >>> referenced by lio_main.c
> >> >>>               lio_main.o:(lio_attach)
> >> >>> referenced by lio_main.c
> >> >>>               lio_main.o:(lio_destroy_nic_device)
> >>
> >> >     Sponsored by:           Netflix
> >> > ---
> >> >  sys/conf/NOTES | 1 +
> >> >  1 file changed, 1 insertion(+)
> >> >
> >> > diff --git a/sys/conf/NOTES b/sys/conf/NOTES
> >> > index 65e088cfe7c9..eded586b4491 100644
> >> > --- a/sys/conf/NOTES
> >> > +++ b/sys/conf/NOTES
> >> > @@ -1931,6 +1931,7 @@ envvar          hint.fxp.0.prefer_iomap="0"
> >> >  device               gem             # Apple GMAC/Sun ERI/Sun GEM
> >> >  device               jme             # JMicron JMC250 Gigabit/JMC260
> >> Fast Ethernet
> >> >  device               lge             # Level 1 LXT1001 gigabit
> >> > Ethernet
> >> > +device               lio             # Support for Cavium 23XX
> >> > Ethernet
> >> adapters
> >> >  device               mlxfw           # Mellanox firmware update
> module
> >> >  device               mlx5            # Shared code module between IB
> >> and Ethernet
> >> >  device               mlx5en          # Mellanox ConnectX-4 and
> >> ConnectX-4 LX
> >> >
> >>
> >>
> >> --
> >> Mateusz Guzik <mjguzik gmail.com>
> >>
> >
>
>
> --
> Mateusz Guzik <mjguzik gmail.com>
>

--000000000000dafc8d05d4dc24d6
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 Wed, Jan 5, 2022 at 1:53 PM Mateus=
z Guzik &lt;<a href=3D"mailto:mjguzik@gmail.com">mjguzik@gmail.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">On 1/5/22=
, Warner Losh &lt;<a href=3D"mailto:imp@bsdimp.com" target=3D"_blank">imp@b=
sdimp.com</a>&gt; wrote:<br>
&gt; On Wed, Jan 5, 2022, 1:26 PM Mateusz Guzik &lt;<a href=3D"mailto:mjguz=
ik@gmail.com" target=3D"_blank">mjguzik@gmail.com</a>&gt; wrote:<br>
&gt;<br>
&gt;&gt; On 12/31/21, Warner Losh &lt;<a href=3D"mailto:imp@freebsd.org" ta=
rget=3D"_blank">imp@freebsd.org</a>&gt; wrote:<br>
&gt;&gt; &gt; The branch main has been updated by imp:<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt; URL:<br>
&gt;&gt; &gt;<br>
&gt;&gt; <a href=3D"https://cgit.FreeBSD.org/src/commit/?id=3D5a73a6c178f65=
b91fe60e52e476a88137e40f0e4" rel=3D"noreferrer" target=3D"_blank">https://c=
git.FreeBSD.org/src/commit/?id=3D5a73a6c178f65b91fe60e52e476a88137e40f0e4</=
a><br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt; commit 5a73a6c178f65b91fe60e52e476a88137e40f0e4<br>
&gt;&gt; &gt; Author:=C2=A0 =C2=A0 =C2=A0Warner Losh &lt;imp@FreeBSD.org&gt=
;<br>
&gt;&gt; &gt; AuthorDate: 2021-12-31 21:56:16 +0000<br>
&gt;&gt; &gt; Commit:=C2=A0 =C2=A0 =C2=A0Warner Losh &lt;imp@FreeBSD.org&gt=
;<br>
&gt;&gt; &gt; CommitDate: 2021-12-31 21:56:16 +0000<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt;=C2=A0 =C2=A0 =C2=A0Add lio back to NOTES<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt;=C2=A0 =C2=A0 =C2=A0lio appears to be partially removed from t=
his file. It&#39;s a modern<br>
&gt;&gt; &gt; enough<br>
&gt;&gt; &gt;=C2=A0 =C2=A0 =C2=A0card, so add it back. It builds everywhere=
.<br>
&gt;&gt; &gt;<br>
&gt;&gt;<br>
&gt;&gt; This breaks NOIP kernels:<br>
&gt;&gt;<br>
&gt;<br>
&gt; NOIP kernels were already broken when I made this commit. I&#39;ll tak=
e a look<br>
&gt; now that emaste committed fixes for those...<br>
&gt;<br>
<br>
It builds fine without the change.<br></blockquote><div><br></div><div>Only=
 because of=C2=A0a6668e31aa3cc855bfb37d6edc05122514592a47</div><div><br></d=
iv><div>Warner</div><div>=C2=A0</div><blockquote class=3D"gmail_quote" styl=
e=3D"margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);paddin=
g-left:1ex">
&gt; Warner<br>
&gt;<br>
&gt; ld: error: undefined symbol: tcp_lro_flush_all<br>
&gt;&gt; &gt;&gt;&gt; referenced by lio_droq.c<br>
&gt;&gt; &gt;&gt;&gt;=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=
=A0lio_droq.o:(lio_droq_process_packets)<br>
&gt;&gt;<br>
&gt;&gt; ld: error: undefined symbol: tcp_lro_rx<br>
&gt;&gt; &gt;&gt;&gt; referenced by lio_core.c<br>
&gt;&gt; &gt;&gt;&gt;=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=
=A0lio_core.o:(lio_push_packet)<br>
&gt;&gt;<br>
&gt;&gt; ld: error: undefined symbol: tcp_lro_init<br>
&gt;&gt; &gt;&gt;&gt; referenced by lio_main.c<br>
&gt;&gt; &gt;&gt;&gt;=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=
=A0lio_main.o:(lio_attach)<br>
&gt;&gt;<br>
&gt;&gt; ld: error: undefined symbol: tcp_lro_free<br>
&gt;&gt; &gt;&gt;&gt; referenced by lio_main.c<br>
&gt;&gt; &gt;&gt;&gt;=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=
=A0lio_main.o:(lio_attach)<br>
&gt;&gt; &gt;&gt;&gt; referenced by lio_main.c<br>
&gt;&gt; &gt;&gt;&gt;=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=
=A0lio_main.o:(lio_destroy_nic_device)<br>
&gt;&gt;<br>
&gt;&gt; &gt;=C2=A0 =C2=A0 =C2=A0Sponsored by:=C2=A0 =C2=A0 =C2=A0 =C2=A0 =
=C2=A0 =C2=A0Netflix<br>
&gt;&gt; &gt; ---<br>
&gt;&gt; &gt;=C2=A0 sys/conf/NOTES | 1 +<br>
&gt;&gt; &gt;=C2=A0 1 file changed, 1 insertion(+)<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt; diff --git a/sys/conf/NOTES b/sys/conf/NOTES<br>
&gt;&gt; &gt; index 65e088cfe7c9..eded586b4491 100644<br>
&gt;&gt; &gt; --- a/sys/conf/NOTES<br>
&gt;&gt; &gt; +++ b/sys/conf/NOTES<br>
&gt;&gt; &gt; @@ -1931,6 +1931,7 @@ envvar=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=
=A0 hint.fxp.0.prefer_iomap=3D&quot;0&quot;<br>
&gt;&gt; &gt;=C2=A0 device=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =
=C2=A0gem=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0# Apple GMAC/Sun E=
RI/Sun GEM<br>
&gt;&gt; &gt;=C2=A0 device=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =
=C2=A0jme=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0# JMicron JMC250 G=
igabit/JMC260<br>
&gt;&gt; Fast Ethernet<br>
&gt;&gt; &gt;=C2=A0 device=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =
=C2=A0lge=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0# Level 1 LXT1001 =
gigabit<br>
&gt;&gt; &gt; Ethernet<br>
&gt;&gt; &gt; +device=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=
=A0lio=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0# Support for Cavium =
23XX<br>
&gt;&gt; &gt; Ethernet<br>
&gt;&gt; adapters<br>
&gt;&gt; &gt;=C2=A0 device=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =
=C2=A0mlxfw=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0# Mellanox firmware upd=
ate module<br>
&gt;&gt; &gt;=C2=A0 device=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =
=C2=A0mlx5=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 # Shared code module be=
tween IB<br>
&gt;&gt; and Ethernet<br>
&gt;&gt; &gt;=C2=A0 device=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =
=C2=A0mlx5en=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 # Mellanox ConnectX-4 and<br=
>
&gt;&gt; ConnectX-4 LX<br>
&gt;&gt; &gt;<br>
&gt;&gt;<br>
&gt;&gt;<br>
&gt;&gt; --<br>
&gt;&gt; Mateusz Guzik &lt;mjguzik <a href=3D"http://gmail.com" rel=3D"nore=
ferrer" target=3D"_blank">gmail.com</a>&gt;<br>
&gt;&gt;<br>
&gt;<br>
<br>
<br>
-- <br>
Mateusz Guzik &lt;mjguzik <a href=3D"http://gmail.com" rel=3D"noreferrer" t=
arget=3D"_blank">gmail.com</a>&gt;<br>
</blockquote></div></div>

--000000000000dafc8d05d4dc24d6--



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CANCZdfpmzBxwR20M8U34vYmZmV0AwOARyXJEMPp3m7SniEu4kg>