From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Miller Subject: Re: [RFC] gro: Is it ok to share a single napi from several devs ? Date: Sat, 28 Aug 2010 14:41:30 -0700 (PDT) Message-ID: <20100828.144130.189711704.davem@davemloft.net> References: <20100828143132.GA3211@del.dom.local> <1283006909.2277.22.camel@edumazet-laptop> <20100828101424.69c4c782@s6510> Mime-Version: 1.0 Content-Type: Text/Plain; charset=us-ascii Content-Transfer-Encoding: 7bit Cc: eric.dumazet@gmail.com, jarkao2@gmail.com, herbert@gondor.apana.org.au, netdev@vger.kernel.org To: shemminger@vyatta.com Return-path: Received: from 74-93-104-97-Washington.hfc.comcastbusiness.net ([74.93.104.97]:59865 "EHLO sunset.davemloft.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752492Ab0H1VlO (ORCPT ); Sat, 28 Aug 2010 17:41:14 -0400 In-Reply-To: <20100828101424.69c4c782@s6510> Sender: netdev-owner@vger.kernel.org List-ID: From: Stephen Hemminger Date: Sat, 28 Aug 2010 10:14:24 -0700 > The Marvell Yukon2 hardware supports two interfaces sharing a common interrupt. > Therfore the sky2 driver has up to two net devices and a single NAPI per board. > It is possible in a single invocation of the poll loop to process frames > for both ports. GRO works by combining received packets from identical > flows over one NAPI interval. It is possible on sky2 that one packet > could be processed for the first port, and the next packet processed was for > second port and the two packets were related so that GRO would combine them. > The check for the same dev is required to prevent this. Yes it is an unlikely > corner case, but the purpose of GRO is to do aggregation but preserve the > flow characteristics of the incoming traffic. If that is true then GRO is going to refuse to merge every single frame that arrives on the second port of a SKY2 device. :-) This is because for the two ports, you allocate and register one NAPI instance which uses only the first port's netdev pointer. Therefore, when GRO compares napi->dev to skb->dev it will always not match for packets coming from the second port since the netdev pointer in skb->dev will be different. Since netpoll does similar things, this means both NAPI and netpoll cannot function properly with SKY2's second port. It will only work right on the first port.