Date: Tue, 26 Jan 2021 16:15:02 +0000 From: "Scheffenegger, Richard" <Richard.Scheffenegger@netapp.com> To: Michael Tuexen <Michael.Tuexen@macmic.franken.de>, Richard Scheffenegger <rscheff@FreeBSD.org> Cc: "src-committers@freebsd.org" <src-committers@FreeBSD.org>, "dev-commits-src-all@freebsd.org" <dev-commits-src-all@FreeBSD.org>, "dev-commits-src-main@freebsd.org" <dev-commits-src-main@FreeBSD.org> Subject: AW: git: 84761f3df508 - main - Adjust line length in tcp_prr_partialack Message-ID: <SN4PR0601MB3728AD4716E9EE783F0B7DCA86BC9@SN4PR0601MB3728.namprd06.prod.outlook.com> In-Reply-To: <84A0BAE9-0553-4D19-9F48-535A4B9473D3@macmic.franken.de> References: <202101261501.10QF16hV000586@gitrepo.freebsd.org> <84A0BAE9-0553-4D19-9F48-535A4B9473D3@macmic.franken.de>
next in thread | previous in thread | raw e-mail | index | archive | help
Let's discuss this during the next @transport call on thursday; I would wan= t to hold off for next weekend at least for the MFC to apply all related pa= tches in close succession. (and quite frankly didn't think about it) Richard Scheffenegger -----Urspr=FCngliche Nachricht----- Von: Michael Tuexen <Michael.Tuexen@macmic.franken.de>=20 Gesendet: Dienstag, 26. J=E4nner 2021 17:09 An: Richard Scheffenegger <rscheff@FreeBSD.org> Cc: src-committers@freebsd.org; dev-commits-src-all@freebsd.org; dev-commit= s-src-main@freebsd.org Betreff: Re: git: 84761f3df508 - main - Adjust line length in tcp_prr_parti= alack NetApp Security WARNING: This is an external email. Do not click links or o= pen attachments unless you recognize the sender and know the content is saf= e. > On 26. Jan 2021, at 16:01, Richard Scheffenegger <rscheff@FreeBSD.org> wr= ote: > > The branch main has been updated by rscheff: > > URL:=20 > https://cgit.FreeBSD.org/src/commit/?id=3D84761f3df508aed50783b60f028af9 > d98a684b41 > > commit 84761f3df508aed50783b60f028af9d98a684b41 > Author: Richard Scheffenegger <rscheff@FreeBSD.org> > AuthorDate: 2021-01-26 13:47:19 +0000 > Commit: Richard Scheffenegger <rscheff@FreeBSD.org> > CommitDate: 2021-01-26 13:47:19 +0000 > > Adjust line length in tcp_prr_partialack > > Summary: > Wrap lines before column 80 in new prr code checked in recently. > > No functional changes. > > Reviewers: tuexen, rrs, jtl, mm, kbowling, #transport > > Reviewed By: tuexen, mm, #transport > > Subscribers: imp, melifaro > > Differential Revision: https://reviews.freebsd.org/D28329 Any plans to MFC this to stable/13? Best regards Michael > --- > sys/netinet/tcp_input.c | 13 ++++++++----- > 1 file changed, 8 insertions(+), 5 deletions(-) > > diff --git a/sys/netinet/tcp_input.c b/sys/netinet/tcp_input.c index=20 > 75718352da00..4b8f91ed9d0b 100644 > --- a/sys/netinet/tcp_input.c > +++ b/sys/netinet/tcp_input.c > @@ -3949,14 +3949,17 @@ tcp_prr_partialack(struct tcpcb *tp, struct tcphd= r *th) > * Proportional Rate Reduction > */ > if (pipe > tp->snd_ssthresh) > - snd_cnt =3D (tp->sackhint.prr_delivered * tp->snd_ssthresh = / tp->sackhint.recover_fs) - > - tp->sackhint.sack_bytes_rexmit; > + snd_cnt =3D (tp->sackhint.prr_delivered * tp->snd_ssthresh = / > + tp->sackhint.recover_fs) -=20 > + tp->sackhint.sack_bytes_rexmit; > else { > if (V_tcp_do_prr_conservative) > - limit =3D tp->sackhint.prr_delivered - tp->sackhint= .sack_bytes_rexmit; > + limit =3D tp->sackhint.prr_delivered - > + tp->sackhint.sack_bytes_rexmit; > else > - if ((tp->sackhint.prr_delivered - tp->sackhint.sack= _bytes_rexmit) > del_data) > - limit =3D tp->sackhint.prr_delivered - tp->= sackhint.sack_bytes_rexmit + maxseg; > + if ((tp->sackhint.prr_delivered - > + tp->sackhint.sack_bytes_rexmit) > del_data) > + limit =3D tp->sackhint.prr_delivered - > + tp->sackhint.sack_bytes_rexmit +=20 > + maxseg; > else > limit =3D del_data + maxseg; > snd_cnt =3D min((tp->snd_ssthresh - pipe), limit);
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?SN4PR0601MB3728AD4716E9EE783F0B7DCA86BC9>