Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932082AbXA0VCN (ORCPT ); Sat, 27 Jan 2007 16:02:13 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S932080AbXA0VCN (ORCPT ); Sat, 27 Jan 2007 16:02:13 -0500 Received: from imf21aec.mail.bellsouth.net ([205.152.59.69]:59256 "EHLO imf21aec.mail.bellsouth.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932074AbXA0VCL (ORCPT ); Sat, 27 Jan 2007 16:02:11 -0500 Message-ID: <45BBBDD0.7030809@bellsouth.net> Date: Sat, 27 Jan 2007 15:02:08 -0600 From: Jay Cliburn User-Agent: Thunderbird 1.5.0.9 (X11/20061223) MIME-Version: 1.0 To: jeff@garzik.org CC: Luca Tettamanti , Stephen Hemminger , Randy Dunlap , csnook@redhat.com, netdev@vger.kernel.org, linux-kernel@vger.kernel.org, atl1-devel@lists.sourceforge.net Subject: Re: [PATCH 4/4] atl1: Ancillary C files for Attansic L1 driver References: <20070121210737.GE2702@osprey.hogchain.net> <20070121183151.4be61ebf.randy.dunlap@oracle.com> <45B43093.6060500@bellsouth.net> <20070122200004.GA12553@dreamland.darkstar.lan> <20070123112522.65aac61b@freekitty> <20070123211917.GA22109@dreamland.darkstar.lan> In-Reply-To: <20070123211917.GA22109@dreamland.darkstar.lan> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3937 Lines: 129 Luca Tettamanti wrote: [snip] > > Anyway... > > Unconditionally enable MSI in atl1 driver. Also remove some useless > #ifdef since pci_{en,dis}able_msi() are no-op when MSI support is not > configured in. > > Signed-off-by: Luca Tettamanti Acked-by: Jay Cliburn I tested this patch today. Works fine on my Asus M2V mainboard (Via K8T890). Here are the eth0 entries in /proc/interrupts before and after the patch. BEFORE: 36: 93 322091 IO-APIC-fasteoi eth0 AFTER: 2298: 85 7289 PCI-MSI-edge eth0 Jeff, shall I add this to the larger patch I'm working on for submittal later this weekend, or do you just add it directly to netdev? (I prefer to do the former if it's okay with you.) Jay > --- > Patch against current netdev tree. > > drivers/net/atl1/atl1.h | 1 - > drivers/net/atl1/atl1_main.c | 22 +++++++--------------- > drivers/net/atl1/atl1_param.c | 13 ------------- > 3 files changed, 7 insertions(+), 29 deletions(-) > > diff --git a/drivers/net/atl1/atl1.h b/drivers/net/atl1/atl1.h > index 1d13e8f..0b30e1c 100644 > --- a/drivers/net/atl1/atl1.h > +++ b/drivers/net/atl1/atl1.h > @@ -281,7 +281,6 @@ struct atl1_adapter { > struct atl1_smb smb; > struct atl1_cmb cmb; > > - int enable_msi; > u32 pci_state[16]; > }; > > diff --git a/drivers/net/atl1/atl1_main.c b/drivers/net/atl1/atl1_main.c > index c0b1555..68e6cd1 100644 > --- a/drivers/net/atl1/atl1_main.c > +++ b/drivers/net/atl1/atl1_main.c > @@ -1793,18 +1793,12 @@ s32 atl1_up(struct atl1_adapter *adapter) > goto err_up; > } > > -#ifdef CONFIG_PCI_MSI > - if (adapter->enable_msi) { > - err = pci_enable_msi(adapter->pdev); > - if (err) { > - dev_info(&adapter->pdev->dev, "Unable to enable MSI: %d\n", err); > - adapter->enable_msi = 0; > - } > - } > -#endif > - if (!adapter->enable_msi) > + err = pci_enable_msi(adapter->pdev); > + if (err) { > + dev_info(&adapter->pdev->dev, "Unable to enable MSI: %d\n", err); > irq_flags |= IRQF_SHARED; > - > + } > + > err = request_irq(adapter->pdev->irq, &atl1_intr, irq_flags, > netdev->name, netdev); > if (unlikely(err)) > @@ -1821,6 +1815,7 @@ s32 atl1_up(struct atl1_adapter *adapter) > free_irq(adapter->pdev->irq, netdev); > > err_up: > + pci_disable_msi(adapter->pdev); > /* free rx_buffers */ > atl1_clean_rx_ring(adapter); > return err; > @@ -1836,10 +1831,7 @@ void atl1_down(struct atl1_adapter *adapter) > > atl1_irq_disable(adapter); > free_irq(adapter->pdev->irq, netdev); > -#ifdef CONFIG_PCI_MSI > - if (adapter->enable_msi) > - pci_disable_msi(adapter->pdev); > -#endif > + pci_disable_msi(adapter->pdev); > atl1_reset_hw(&adapter->hw); > adapter->cmb.cmb->int_stats = 0; > > diff --git a/drivers/net/atl1/atl1_param.c b/drivers/net/atl1/atl1_param.c > index 4732f43..caa2d83 100644 > --- a/drivers/net/atl1/atl1_param.c > +++ b/drivers/net/atl1/atl1_param.c > @@ -68,10 +68,6 @@ static int num_flash_vendor = 0; > module_param_array_named(flash_vendor, flash_vendor, int, &num_flash_vendor, 0); > MODULE_PARM_DESC(flash_vendor, "SPI flash vendor"); > > -int enable_msi; > -module_param(enable_msi, int, 0444); > -MODULE_PARM_DESC(enable_msi, "Enable PCI MSI"); > - > #define DEFAULT_INT_MOD_CNT 100 /* 200us */ > #define MAX_INT_MOD_CNT 65000 > #define MIN_INT_MOD_CNT 50 > @@ -211,13 +207,4 @@ void __devinit atl1_check_options(struct atl1_adapter *adapter) > adapter->hw.flash_vendor = (u8) (opt.def); > } > } > - > - { /* PCI MSI usage */ > - > -#ifdef CONFIG_PCI_MSI > - adapter->enable_msi = enable_msi; > -#else > - adapter->enable_msi = 0; > -#endif > - } > } - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/