Date: Tue, 27 Dec 2016 11:56:13 +0100 From: Michael Tuexen <tuexen@freebsd.org> To: hiren panchasara <hiren@strugglingcoder.info> Cc: src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: Re: svn commit: r310547 - head/sys/netinet Message-ID: <693D4C85-526B-426C-9685-1D463E3C3903@freebsd.org> In-Reply-To: <20161227081940.GL82166@strugglingcoder.info> References: <201612251737.uBPHbIhd027939@repo.freebsd.org> <20161227081940.GL82166@strugglingcoder.info>
next in thread | previous in thread | raw e-mail | index | archive | help
> On 27 Dec 2016, at 09:19, hiren panchasara = <hiren@strugglingcoder.info> wrote: >=20 > On 12/25/16 at 05:37P, Michael Tuexen wrote: >> Author: tuexen >> Date: Sun Dec 25 17:37:18 2016 >> New Revision: 310547 >> URL: https://svnweb.freebsd.org/changeset/base/310547 >>=20 >> Log: >> Remove a KASSERT which is not always true. >>=20 >> In case of the empty queue tp->snd_holes and tcp_sackhole_insert() >> failing due to memory shortage, tp->snd_holes will be empty. >> This problem was hit when stress tests where performed by pho. >>=20 >> PR: 215513 >> Reported by: pho >> Tested by: pho >> Sponsored by: Netflix, Inc. >>=20 >> Modified: >> head/sys/netinet/tcp_sack.c >=20 > Thanks for the fix, Michael. Can you please MFC this? Sure. Will do it after 3 days or so. Missed that. Best regards Michael >=20 > Cheers, > Hiren
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?693D4C85-526B-426C-9685-1D463E3C3903>