From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933975AbXDBL44 (ORCPT ); Mon, 2 Apr 2007 07:56:56 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S933978AbXDBL44 (ORCPT ); Mon, 2 Apr 2007 07:56:56 -0400 Received: from mx3.mail.elte.hu ([157.181.1.138]:60178 "EHLO mx3.mail.elte.hu" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933975AbXDBL4z (ORCPT ); Mon, 2 Apr 2007 07:56:55 -0400 Date: Mon, 2 Apr 2007 13:56:50 +0200 From: Ingo Molnar To: Ayaz Abdulla Cc: Linux Kernel Mailing List , Ayaz Abdulla , Andrew Morton Subject: [patch] forcedeth: improve NAPI logic Message-ID: <20070402115650.GA1703@elte.hu> References: <20070326083146.GA11666@elte.hu> <20070326083920.GA14648@elte.hu> <20070326085856.GA18067@elte.hu> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20070326085856.GA18067@elte.hu> User-Agent: Mutt/1.4.2.2i X-ELTE-VirusStatus: clean X-ELTE-SpamScore: -2.0 X-ELTE-SpamLevel: X-ELTE-SpamCheck: no X-ELTE-SpamVersion: ELTE 2.0 X-ELTE-SpamCheck-Details: score=-2.0 required=5.9 tests=BAYES_00 autolearn=no SpamAssassin version=3.0.3 -2.0 BAYES_00 BODY: Bayesian spam probability is 0 to 1% [score: 0.0000] Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org From: Ingo Molnar Subject: [patch] forcedeth.c: improve NAPI handler another forcedeth.c thing: i noticed that its NAPI handler does not do tx-ring processing. The patch below implements this - tested on DESC_VER_2 hardware, with CONFIG_FORCEDETH_NAPI=y. Signed-off-by: Ingo Molnar Index: linux/drivers/net/forcedeth.c =================================================================== --- linux.orig/drivers/net/forcedeth.c +++ linux/drivers/net/forcedeth.c @@ -3118,9 +3118,17 @@ static int nv_napi_poll(struct net_devic int retcode; if (np->desc_ver == DESC_VER_1 || np->desc_ver == DESC_VER_2) { + spin_lock_irqsave(&np->lock, flags); + nv_tx_done(dev); + spin_unlock_irqrestore(&np->lock, flags); + pkts = nv_rx_process(dev, limit); retcode = nv_alloc_rx(dev); } else { + spin_lock_irqsave(&np->lock, flags); + nv_tx_done_optimized(dev, np->tx_ring_size); + spin_unlock_irqrestore(&np->lock, flags); + pkts = nv_rx_process_optimized(dev, limit); retcode = nv_alloc_rx_optimized(dev); }