From patchwork Sun Sep 21 19:38:09 2008 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Subject: Mark IPW2100 as BROKEN: Fatal interrupt. Scheduling firmware restart. Date: Sun, 21 Sep 2008 09:38:09 -0000 From: Evgeniy Polyakov X-Patchwork-Id: 797 Message-Id: <20080921193809.GA8735@2ka.mipt.ru> To: Johannes Berg Cc: Arjan van de Ven , netdev@vger.kernel.org, linux-kernel@vger.kernel.org, ipw2100-devel@lists.sourceforge.net, linux-wireless@vger.kernel.org, yi.zhu@intel.com, reinette.chatre@intel.com, jgarzik@pobox.com, linville@tuxdriver.com, davem@davemloft.net Hi. On Sun, Sep 21, 2008 at 09:14:04PM +0200, Johannes Berg (johannes@sipsolutions.net) wrote: > > Do you want me to implement ipw2100 driver as a big work structure > > which will run ipw2100_init()/wait/ipw2100_exit() in a loop? > > And that will be the fix suggested by Intel? That would explain a lot. > > I think what Arjan is saying is that it would be better to put pressure > on the responsible folks (I don't think Arjan is anywhere near them at Both maintainers were added to the copy list. > all) if you'd put in a WARN_ON() for this error and that would make the > top entry on kerneloops.org all the time... And additionally put in a > workaround for yourself for now. As I pointed, I can rewrite the whole driver's initialization process, so that it looked like init/wait/exit loop, which can be processed at the module load and when fatal interrupt fires. Do this a fix? This is not even a remotely workaround. We can just add rmmod/modprobe/ifdown/ifup to the crontab job. Another users reported in bugzilla that they needed to reboot a machine to make card working again. I'm not sure that user tried to do a rmmod/modprobe though. > And can we keep the flames off this list please? That comment from Wei > Weng was absolutely uncalled for, and inciting a flamewar (as you have > already blogged) was not really productive either. If we will keep silence, no one will notice that problem exists. I do hope this will result in a progress. Arjan, do you aggree to add this patch to the current tree? diff --git a/drivers/net/wireless/ipw2100.c b/drivers/net/wireless/ipw2100.c index 19a401c..9a7b64c 100644 --- a/drivers/net/wireless/ipw2100.c +++ b/drivers/net/wireless/ipw2100.c @@ -206,6 +206,8 @@ MODULE_PARM_DESC(disable, "manually disable the radio (default 0 [radio on])"); static u32 ipw2100_debug_level = IPW_DL_NONE; +static int ipw2100_max_fatal_ints = 10; + #ifdef CONFIG_IPW2100_DEBUG #define IPW_DEBUG(level, message...) \ do { \ @@ -3174,6 +3176,10 @@ static void ipw2100_irq_tasklet(struct ipw2100_priv *priv) if (inta & IPW2100_INTA_FATAL_ERROR) { printk(KERN_WARNING DRV_NAME ": Fatal interrupt. Scheduling firmware restart.\n"); + WARN_ON(1); + + BUG_ON(ipw2100_max_fatal_ints-- <= 0); + priv->inta_other++; write_register(dev, IPW_REG_INTA, IPW2100_INTA_FATAL_ERROR);