From owner-svn-src-head@FreeBSD.ORG Sat Dec 1 10:23:16 2012 Return-Path: Delivered-To: svn-src-head@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [69.147.83.52]) by hub.freebsd.org (Postfix) with ESMTP id D7FFAA81 for ; Sat, 1 Dec 2012 10:23:16 +0000 (UTC) (envelope-from andre@freebsd.org) Received: from c00l3r.networx.ch (c00l3r.networx.ch [62.48.2.2]) by mx1.freebsd.org (Postfix) with ESMTP id 420068FC0C for ; Sat, 1 Dec 2012 10:23:15 +0000 (UTC) Received: (qmail 78104 invoked from network); 1 Dec 2012 11:54:11 -0000 Received: from c00l3r.networx.ch (HELO [127.0.0.1]) ([62.48.2.2]) (envelope-sender ) by c00l3r.networx.ch (qmail-ldap-1.03) with SMTP for ; 1 Dec 2012 11:54:11 -0000 Message-ID: <50B9DA84.8070701@freebsd.org> Date: Sat, 01 Dec 2012 11:23:00 +0100 From: Andre Oppermann User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:16.0) Gecko/20121026 Thunderbird/16.0.2 MIME-Version: 1.0 To: Jack Vogel Subject: Re: svn commit: r243714 - in head/sys/dev: ixgbe netmap References: <201211302219.qAUMJIkI065470@svn.freebsd.org> <50B9C291.4030107@freebsd.org> In-Reply-To: Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Cc: Jack F Vogel , svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: SVN commit messages for the src tree for head/-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 01 Dec 2012 10:23:16 -0000 On 01.12.2012 10:48, Jack Vogel wrote: > > > On Sat, Dec 1, 2012 at 12:40 AM, Andre Oppermann > wrote: > > On 30.11.2012 23:19, Jack F Vogel wrote: > > Author: jfv > Date: Fri Nov 30 22:19:18 2012 > New Revision: 243714 > URL: http://svnweb.freebsd.org/__changeset/base/243714 > > > Log: > First of a series of 11 patches leading to new ixgbe version 2.5.0 > This removes the header split and supporting code from the driver. > > > You did this because the datasheet says the header split feature is > not supported and using it gives unpredictable results? > > > I'm not sure what the datasheet says, but yes there is a hardware issue > on some of the 10G adapters if using header split. I have left the code in > in a disabled state for a long time, thinking it would be there as example > code, but I think it best to remove it and simplify the code at this point. It says so in section 7.1.10 Header Splitting: Note: Header Splitting mode might cause unpredictable behavior and should not be used with the 82599. For more information, see the product specification update errata on this subject. > Oh, and yes, results have always been mixed with the feature in my > experience anyway. I can imagine. Considering how we work with the packet through the stack and in the socket buffers there isn't much benefit from header splitting. -- Andre