diff options
author | <herbert@gondor.apana.org.au> | 2005-05-12 20:11:55 -0400 |
---|---|---|
committer | Jeff Garzik <jgarzik@pobox.com> | 2005-05-12 20:11:55 -0400 |
commit | b1fc5505e0dbcc3fd7c75bfe6bee39ec50080963 (patch) | |
tree | f8c0bb679dc8e72816e16820c3dbcccf924097eb /drivers/net/e2100.c | |
parent | 88d7bd8cb9eb8d64bf7997600b0d64f7834047c5 (diff) |
[netdrvr] Fix register_netdev() races in older ISA net drivers
Diffstat (limited to 'drivers/net/e2100.c')
-rw-r--r-- | drivers/net/e2100.c | 15 |
1 files changed, 5 insertions, 10 deletions
diff --git a/drivers/net/e2100.c b/drivers/net/e2100.c index 51c9fa260830..f5a4dd7d8564 100644 --- a/drivers/net/e2100.c +++ b/drivers/net/e2100.c | |||
@@ -162,12 +162,7 @@ struct net_device * __init e2100_probe(int unit) | |||
162 | err = do_e2100_probe(dev); | 162 | err = do_e2100_probe(dev); |
163 | if (err) | 163 | if (err) |
164 | goto out; | 164 | goto out; |
165 | err = register_netdev(dev); | ||
166 | if (err) | ||
167 | goto out1; | ||
168 | return dev; | 165 | return dev; |
169 | out1: | ||
170 | cleanup_card(dev); | ||
171 | out: | 166 | out: |
172 | free_netdev(dev); | 167 | free_netdev(dev); |
173 | return ERR_PTR(err); | 168 | return ERR_PTR(err); |
@@ -286,6 +281,9 @@ static int __init e21_probe1(struct net_device *dev, int ioaddr) | |||
286 | #endif | 281 | #endif |
287 | NS8390_init(dev, 0); | 282 | NS8390_init(dev, 0); |
288 | 283 | ||
284 | retval = register_netdev(dev); | ||
285 | if (retval) | ||
286 | goto out; | ||
289 | return 0; | 287 | return 0; |
290 | out: | 288 | out: |
291 | release_region(ioaddr, E21_IO_EXTENT); | 289 | release_region(ioaddr, E21_IO_EXTENT); |
@@ -453,11 +451,8 @@ init_module(void) | |||
453 | dev->mem_start = mem[this_dev]; | 451 | dev->mem_start = mem[this_dev]; |
454 | dev->mem_end = xcvr[this_dev]; /* low 4bits = xcvr sel. */ | 452 | dev->mem_end = xcvr[this_dev]; /* low 4bits = xcvr sel. */ |
455 | if (do_e2100_probe(dev) == 0) { | 453 | if (do_e2100_probe(dev) == 0) { |
456 | if (register_netdev(dev) == 0) { | 454 | dev_e21[found++] = dev; |
457 | dev_e21[found++] = dev; | 455 | continue; |
458 | continue; | ||
459 | } | ||
460 | cleanup_card(dev); | ||
461 | } | 456 | } |
462 | free_netdev(dev); | 457 | free_netdev(dev); |
463 | printk(KERN_WARNING "e2100.c: No E2100 card found (i/o = 0x%x).\n", io[this_dev]); | 458 | printk(KERN_WARNING "e2100.c: No E2100 card found (i/o = 0x%x).\n", io[this_dev]); |