Bug#656813: [PATCH] net/wireless: ipw2200: Fix WARN_ON occurring in wiphy_register called by ipw_pci_probe
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
\x\x\x\x\x\x\x\x\x\x\x\x\x\x\x\x\x
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.
Cc: sta...@kernel.org
Signed-off-by: Stanislav Yakovlev
---
Stefan, can you test it once again?
---
drivers/net/wireless/ipw2x00/ipw2200.c | 21 ++---
1 files changed, 6 insertions(+), 15 deletions(-)
diff --git a/drivers/net/wireless/ipw2x00/ipw2200.c
b/drivers/net/wireless/ipw2x00/ipw2200.c
index f37d315..4130802 100644
--- a/drivers/net/wireless/ipw2x00/ipw2200.c
+++ b/drivers/net/wireless/ipw2x00/ipw2200.c
@@ -11432,20 +11432,6 @@ static void ipw_bg_down(struct work_struct *work)
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;
@@ -11714,7 +11700,6 @@ static void ipw_prom_free(struct ipw_priv *priv)
#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,
@@ -11841,6 +11826,12 @@ static int __devinit ipw_pci_probe(struct pci_dev
*pdev,
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) {
--
1.7.2.5
--
To UNSUBSCRIBE, email to debian-kernel-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive:
http://lists.debian.org/1334572710-10838-1-git-send-email-stas.yakov...@gmail.com
Bug#656813: [PATCH] net/wireless: ipw2200: Fix WARN_ON occurring in wiphy_register called by ipw_pci_probe
Hi
On Monday 16 April 2012, Stanislav Yakovlev wrote:
> 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
> \x\x\x\x\x\x\x\x\x\x\x\x\x\x\x\x\x
> 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]
[…]
>
> 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.
>
> Cc: sta...@kernel.org
> Signed-off-by: Stanislav Yakovlev
>
> ---
>
> Stefan, can you test it once again?
[…]
I've successfully tested this patch on its own and with Ben Hutching's
patch ("Bug#656813: [PATCH 1/2] ipw2200: Fix order of
device registration", Message-ID: <1327201775.8004.83.camel@deadeye>)
applied on top, it's working fine in both cases. The "cfg80211: failed
to add phy80211 symlink to netdev!" warning disappears after applying
both patches to kernel 3.3.2.
Feel free to add a tested-by tag, if you like:
Tested-by: Stefan Lippers-Hollmann
Tested using:
02:02.0 Network controller [0280]: Intel Corporation PRO/Wireless 2200BG
[Calexico2] Network Connection [8086:4220] (rev 05)
Subsystem: Intel Corporation Device [8086:2701]
Control: I/O- Mem+ BusMaster+ SpecCycle- MemWINV- VGASnoop- ParErr-
Stepping- SERR- FastB2B- DisINTx-
Status: Cap+ 66MHz- UDF- FastB2B+ ParErr- DEVSEL=medium >TAbort-
SERR- http://lists.debian.org/201204161945.08522.s@gmx.de
Bug#656813: [PATCH] net/wireless: ipw2200: Fix WARN_ON occurring in wiphy_register called by ipw_pci_probe
On 16 April 2012 10:45, Stefan Lippers-Hollmann wrote:
> On Monday 16 April 2012, Stanislav Yakovlev wrote:
>>
>> Stefan, can you test it once again?
>
> I've successfully tested this patch on its own and with Ben Hutching's
> patch ("Bug#656813: [PATCH 1/2] ipw2200: Fix order of
> device registration", Message-ID: <1327201775.8004.83.camel@deadeye>)
> applied on top, it's working fine in both cases. The "cfg80211: failed
> to add phy80211 symlink to netdev!" warning disappears after applying
> both patches to kernel 3.3.2.
Thanks!
Stanislav.
--
To UNSUBSCRIBE, email to debian-kernel-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive:
http://lists.debian.org/ca++wf2mi7vlf673cnkrty+8nyfrbgred5fan7rmu+dvv81y...@mail.gmail.com
