ide: fix probing for hosts with serialized or IRQ sharing interfaces
authorBartlomiej Zolnierkiewicz <bzolnier@gmail.com>
Sat, 26 Jan 2008 19:13:04 +0000 (20:13 +0100)
committerBartlomiej Zolnierkiewicz <bzolnier@gmail.com>
Sat, 26 Jan 2008 19:13:04 +0000 (20:13 +0100)
After hwif_register_devices() IDE devices are ready to be used so
the way in which ide_device_add() works currently is racy for hosts
with serialized / IRQ sharing interfaces.

Fix ide_device_add() by looping over interfaces for probe_hwif(),
hwif_init() and hwif_register_devices() operations.

Signed-off-by: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com>
drivers/ide/ide-probe.c

index bfcc8078b092519105bc44486aea7d47f40c6905..817564fba820b259053a2101db33c990927671b6 100644 (file)
@@ -1382,9 +1382,14 @@ int ide_device_add(u8 idx[4])
                if (idx[i] == 0xff)
                        continue;
 
-               hwif = &ide_hwifs[idx[i]];
+               probe_hwif(&ide_hwifs[idx[i]]);
+       }
 
-               probe_hwif(hwif);
+       for (i = 0; i < 4; i++) {
+               if (idx[i] == 0xff)
+                       continue;
+
+               hwif = &ide_hwifs[idx[i]];
 
                if (hwif_init(hwif) == 0) {
                        printk(KERN_INFO "%s: failed to initialize IDE "
@@ -1392,6 +1397,13 @@ int ide_device_add(u8 idx[4])
                        rc = -1;
                        continue;
                }
+       }
+
+       for (i = 0; i < 4; i++) {
+               if (idx[i] == 0xff)
+                       continue;
+
+               hwif = &ide_hwifs[idx[i]];
 
                if (hwif->present)
                        hwif_register_devices(hwif);