[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <alpine.WNT.2.00.0904281648300.5352@jbrandeb-desk1.amr.corp.intel.com>
Date: Tue, 28 Apr 2009 17:12:10 -0700 (Pacific Daylight Time)
From: "Brandeburg, Jesse" <jesse.brandeburg@...el.com>
To: David Miller <davem@...emloft.net>
cc: "Kirsher, Jeffrey T" <jeffrey.t.kirsher@...el.com>,
"netdev@...r.kernel.org" <netdev@...r.kernel.org>,
"gospo@...hat.com" <gospo@...hat.com>,
"Duyck, Alexander H" <alexander.h.duyck@...el.com>
Subject: Re: [net-next PATCH 1/3] ixgbe: enable HW RSC for 82599
On Tue, 28 Apr 2009, David Miller wrote:
> > This patch enables hardware receive side coalescing for 82599 hardware.
> > 82599 can merge multiple frames from the same TCP/IP flow into a single
> > structure that can span one ore more descriptors. The accumulated data is
> > arranged similar to how jumbo frames are arranged with the exception that
> > other packets can be interlaced inbetween. To overcome this issue a next
> > pointer is included in the written back descriptor which indicates the next
> > descriptor in the writeback sequence.
> >
> > This feature sets the NETIF_F_LRO flag and clearing it via the ethtool set
> > flags operation will also disable hardware RSC.
> >
> > Signed-off-by: Alexander Duyck <alexander.h.duyck@...el.com>
> > Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@...el.com>
>
> Applied, but.........
>
> Since IXGBE has been converted to GRO, all of this fiddling with
> LRO is a complete NOP. All of that code can be removed completely.
ah, you're right, but this features enables our new hardware that can do
LRO in hardware, and then GRO can work on top of it if necessary. We
wanted some way to turn it off and overloaded the LRO flag to do so.
FYI This is not a TOE, it just recognizes a flow and will pack data frames
together in a list of descriptors, which the driver then hands to the
stack as a large receive.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists