Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752863Ab2E0Trw (ORCPT ); Sun, 27 May 2012 15:47:52 -0400 Received: from youngberry.canonical.com ([91.189.89.112]:45799 "EHLO youngberry.canonical.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752121Ab2E0Tru (ORCPT ); Sun, 27 May 2012 15:47:50 -0400 Date: Sun, 27 May 2012 16:47:44 -0300 From: Herton Ronaldo Krzesinski To: Greg KH Cc: linux-kernel@vger.kernel.org, stable@vger.kernel.org, torvalds@linux-foundation.org, akpm@linux-foundation.org, alan@lxorguk.ukuu.org.uk, Stanislav Yakovlev , Stefan Lippers-Hollmann , "John W. Linville" Subject: Re: [ 12/94] net/wireless: ipw2200: Fix WARN_ON occurring in wiphy_register called by ipw_pci_probe Message-ID: <20120527194743.GB2638@herton-Z68MA-D2H-B3> References: <20120527010332.GA11170@kroah.com> <20120527010425.489792331@linuxfoundation.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20120527010425.489792331@linuxfoundation.org> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 5294 Lines: 131 On Sun, May 27, 2012 at 10:04:35AM +0900, Greg KH wrote: > 3.3-stable review patch. If anyone has any objections, please let me know. > > ------------------ Same as what I commented on this being proposed for 3.4.y, this would only be needed if "ipw2200: Fix order of device registration" was applied to 3.3 kernels. > > From: Stanislav Yakovlev > > commit 7ed85b65ab611ba841d508ae4ff8c45fa0c48c80 upstream. > > The problem was found by Stefan Lippers-Hollmann > http://marc.info/?l=linux-wireless&m=132720334512946&w=2 > > WARNING: at /tmp/buildd/linux-aptosid-3.2/debian/build/source_i386_none/net/wireless/core.c:562 wiphy_register+0x45/0x38d [cfg80211]() > Hardware name: TravelMate 290 \xffffffff\xffffffff\xffffffff\xffffffff\xffffffff\xffffffff\xffffffff\xffffffff\xffffffff\xffffffff\xffffffff\xffffffff\xffffffff\xffffffff\xffffffff\xffffffff\xffffffff > Modules linked in: ipw2200(+) iTCO_wdt libipw joydev drm snd_seq snd_timer snd_seq_device iTCO_vendor_support yenta_socket snd intel_agp i2c_i801 pcmcia_rsrc cfg80211 soundcore parport_pc psmouse parport rng_core snd_page_alloc serio_raw pcspkr i2c_algo_bit intel_gtt pcmcia_core evdev irda crc_ccitt rfkill lib80211 processor container ac battery shpchp pci_hotplug button ext4 mbcache jbd2 crc16 dm_mod sd_mod sr_mod crc_t10dif cdrom ata_generic pata_acpi ata_piix libata scsi_mod firewire_ohci firewire_core crc_itu_t 8139too 8139cp mii uhci_hcd ehci_hcd usbcore usb_common [last unloaded: scsi_wait_scan] > Pid: 328, comm: modprobe Not tainted 3.2-1.slh.4-aptosid-686 #1 > Call Trace: > [] ? warn_slowpath_common+0x7c/0x8f > [] ? wiphy_register+0x45/0x38d [cfg80211] > [] ? wiphy_register+0x45/0x38d [cfg80211] > [] ? warn_slowpath_null+0x1b/0x1f > [] ? wiphy_register+0x45/0x38d [cfg80211] > [] ? internal_create_group+0xf5/0xff > [] ? ipw_pci_probe+0xa9a/0xbd0 [ipw2200] > [] ? arch_local_irq_save+0xf/0x14 > [] ? pci_device_probe+0x53/0x9a > [] ? driver_probe_device+0x94/0x124 > [] ? pci_match_id+0x15/0x34 > [] ? __driver_attach+0x40/0x5b > [] ? bus_for_each_dev+0x37/0x60 > [] ? driver_attach+0x17/0x1a > [] ? driver_probe_device+0x124/0x124 > [] ? bus_add_driver+0x92/0x1d1 > [] ? 0xe099cfff > [] ? driver_register+0x7d/0xd4 > [] ? jump_label_module_notify+0xec/0x167 > [] ? 0xe099cfff > [] ? __pci_register_driver+0x32/0x87 > [] ? 0xe099cfff > [] ? ipw_init+0x2e/0x72 [ipw2200] > [] ? do_one_initcall+0x7d/0x132 > [] ? __blocking_notifier_call_chain+0x47/0x4f > [] ? sys_init_module+0x13a4/0x159c > [] ? sysenter_do_call+0x12/0x28 > > This warning appears only if we apply Ben Hutchings' fix > http://marc.info/?l=linux-wireless&m=132720195012653&w=2 > for the bug reported by Cesare Leonardi > http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=656813 > with cfg80211 warning during device registration > ("cfg80211: failed to add phy80211 symlink to netdev!"). > > We separate device bring up and registration with network stack > to avoid the problem. > > After that Ben Hutchings' fix can be applied to fix the bug. > > Signed-off-by: Stanislav Yakovlev > Tested-by: Stefan Lippers-Hollmann > Signed-off-by: John W. Linville > Signed-off-by: Greg Kroah-Hartman > > --- > drivers/net/wireless/ipw2x00/ipw2200.c | 21 ++++++--------------- > 1 file changed, 6 insertions(+), 15 deletions(-) > > --- a/drivers/net/wireless/ipw2x00/ipw2200.c > +++ b/drivers/net/wireless/ipw2x00/ipw2200.c > @@ -11443,20 +11443,6 @@ static void ipw_bg_down(struct work_stru > mutex_unlock(&priv->mutex); > } > > -/* Called by register_netdev() */ > -static int ipw_net_init(struct net_device *dev) > -{ > - int rc = 0; > - struct ipw_priv *priv = libipw_priv(dev); > - > - mutex_lock(&priv->mutex); > - if (ipw_up(priv)) > - rc = -EIO; > - mutex_unlock(&priv->mutex); > - > - return rc; > -} > - > static int ipw_wdev_init(struct net_device *dev) > { > int i, rc = 0; > @@ -11722,7 +11708,6 @@ static void ipw_prom_free(struct ipw_pri > #endif > > static const struct net_device_ops ipw_netdev_ops = { > - .ndo_init = ipw_net_init, > .ndo_open = ipw_net_open, > .ndo_stop = ipw_net_stop, > .ndo_set_rx_mode = ipw_net_set_multicast_list, > @@ -11849,6 +11834,12 @@ static int __devinit ipw_pci_probe(struc > goto out_release_irq; > } > > + if (ipw_up(priv)) { > + mutex_unlock(&priv->mutex); > + err = -EIO; > + goto out_remove_sysfs; > + } > + > mutex_unlock(&priv->mutex); > err = register_netdev(net_dev); > if (err) { > > > -- > To unsubscribe from this list: send the line "unsubscribe stable" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > -- []'s Herton -- 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/