Skip site navigation (1)Skip section navigation (2)
Date:      Sun, 28 Oct 2012 02:01:37 +0400
From:      Gleb Smirnoff <glebius@FreeBSD.org>
To:        Andre Oppermann <andre@FreeBSD.org>
Cc:        svn-src-head@FreeBSD.org, svn-src-all@FreeBSD.org, src-committers@FreeBSD.org
Subject:   Re: svn commit: r242161 - in head/sys: net netinet netpfil/pf
Message-ID:  <20121027220137.GJ70741@FreeBSD.org>
In-Reply-To: <508BBE6C.7010409@freebsd.org>
References:  <201210262106.q9QL6YgY000943@svn.freebsd.org> <508BBE6C.7010409@freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Sat, Oct 27, 2012 at 12:58:52PM +0200, Andre Oppermann wrote:
A> On 26.10.2012 23:06, Gleb Smirnoff wrote:
A> > Author: glebius
A> > Date: Fri Oct 26 21:06:33 2012
A> > New Revision: 242161
A> > URL: http://svn.freebsd.org/changeset/base/242161
A> >
A> > Log:
A> >    o Remove last argument to ip_fragment(), and obtain all needed information
A> >      on checksums directly from mbuf flags. This simplifies code.
A> >    o Clear CSUM_IP from the mbuf in ip_fragment() if we did checksums in
A> >      hardware. Some driver may not announce CSUM_IP in theur if_hwassist,
A> >      although try to do checksums if CSUM_IP set on mbuf. Example is em(4).
A> 
A> I'm not getting your description here?  Why work around a bug in a driver
A> in ip_fragment() when we can fix the bug in the driver?

Well, that was actually bug in the stack and a very special driver that
demonstrates it. I may even agree that driver is incorrect, but the stack was
incorrect, too.

-- 
Totus tuus, Glebius.



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