Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 18 Feb 2021 11:14:51 +0100
From:      rscheff@gmx.at
To:        Navdeep Parhar <nparhar@gmail.com>
Cc:        dev-commits-src-all@freebsd.org, dev-commits-src-main@freebsd.org, Randall Stewart <rrs@freebsd.org>, src-committers <src-committers@freebsd.org>
Subject:   Re: Re: git: 0a4f851074a3 - main - Fix another pesky missing #ifdef TCPHPTS
Message-ID:  <trinity-2863703f-dc2a-4557-91ae-af1e3f9bc600-1613643291020@msvc-mesg-gmx023>
In-Reply-To: <CAPFoGT88OG_JNtYUsOYBaabA4_MN3pBXAGUxyAzkcrGzKiuMow@mail.gmail.com>
References:  <202102180627.11I6Rxa4030228@gitrepo.freebsd.org> <CAPFoGT88OG_JNtYUsOYBaabA4_MN3pBXAGUxyAzkcrGzKiuMow@mail.gmail.com>

next in thread | previous in thread | raw e-mail | index | archive | help
I remember having added this in the driver to expose the tos byte for tso/=
lro operations; interesting that it is unused now.

Hope the IP ECN flags are still propagated in the fastpath....

Gesendet mit der GMX Mail App

Am 18.02.21 um 09:48 schrieb Navdeep Parhar

> I still get build failures in tcp_lro.c for i386/amd64 LINT-NOINET
> builds after this.
>
> --- tcp_lro.o ---
> /storage/src/head/sys/netinet/tcp_lro.c:1359:10: error: unused
> variable 'iptos' [-Werror,-Wunused-variable]
>         uint8_t iptos;
>                 ^
> 1 error generated.
> *** [tcp_lro.o] Error code 1
>
> make[5]: stopped in /storage/obj/storage/src/head/amd64.amd64/sys/LINT-N=
OINET
>
> Regards,
> Navdeep
>
> On Wed, Feb 17, 2021 at 10:28 PM Randall Stewart <rrs@freebsd.org> wrote=
:
> >
> > The branch main has been updated by rrs:
> >
> > URL: https://cgit.FreeBSD.org/src/commit/?id=3D0a4f851074a3ca74cd4859c=
20e7d9807b2aeca65
> >
> > commit 0a4f851074a3ca74cd4859c20e7d9807b2aeca65
> > Author:     Randall Stewart <rrs@FreeBSD.org>
> > AuthorDate: 2021-02-18 06:27:30 +0000
> > Commit:     Randall Stewart <rrs@FreeBSD.org>
> > CommitDate: 2021-02-18 06:27:30 +0000
> >
> >     Fix another pesky missing #ifdef TCPHPTS
> > ---
> >  sys/netinet/tcp_lro.c | 2 ++
> >  1 file changed, 2 insertions(+)
> >
> > diff --git a/sys/netinet/tcp_lro.c b/sys/netinet/tcp_lro.c
> > index 199b7f9a79a0..a8e751b67071 100644
> > --- a/sys/netinet/tcp_lro.c
> > +++ b/sys/netinet/tcp_lro.c
> > @@ -470,6 +470,7 @@ tcp_lro_rx_ipv4(struct lro_ctrl *lc, struct mbuf *=
m, struct ip *ip4,
> >  }
> >  #endif
> >
> > +#ifdef TCPHPTS
> >  static void
> >  tcp_lro_log(struct tcpcb *tp, struct lro_ctrl *lc,
> >             struct lro_entry *le, struct mbuf *m, int frm, int32_t tcp=
_data_len,
> > @@ -520,6 +521,7 @@ tcp_lro_log(struct tcpcb *tp, struct lro_ctrl *lc,
> >                                0, &log, false, &tv);
> >         }
> >  }
> > +#endif
> >
> >  static void
> >  tcp_flush_out_le(struct tcpcb *tp, struct lro_ctrl *lc, struct lro_en=
try *le)
>




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?trinity-2863703f-dc2a-4557-91ae-af1e3f9bc600-1613643291020>