fastboot: Make libata initialization even more async
authorArjan van de Ven <arjan@linux.intel.com>
Mon, 5 Jan 2009 23:07:07 +0000 (15:07 -0800)
committerArjan van de Ven <arjan@linux.intel.com>
Wed, 7 Jan 2009 16:47:11 +0000 (08:47 -0800)
As suggested by Linus: Don't do the libata init in 2 separate
steps with a global sync inbetween, but do it as one async step,
with a local sync before registering the device.

This cuts the boottime on my machine with 2 sata controllers down
significantly, and it seems to work. Would be nice if the libata
folks take a good look at this patch though..

Signed-off-by: Arjan van de Ven <arjan@linux.intel.com>
drivers/ata/libata-core.c

index 7d3ae6a..f178a45 100644 (file)
@@ -5951,6 +5951,12 @@ static void async_port_probe(void *data, async_cookie_t cookie)
                         */
                }
        }
+
+       /* in order to keep device order, we need to synchronize at this point */
+       async_synchronize_cookie(cookie);
+
+       ata_scsi_scan_host(ap, 1);
+
 }
 /**
  *     ata_host_register - register initialized ATA host
@@ -6033,15 +6039,7 @@ int ata_host_register(struct ata_host *host, struct scsi_host_template *sht)
                struct ata_port *ap = host->ports[i];
                async_schedule(async_port_probe, ap);
        }
-       async_synchronize_full();
-       /* probes are done, now scan each port's disk(s) */
-       DPRINTK("host probe begin\n");
-       for (i = 0; i < host->n_ports; i++) {
-               struct ata_port *ap = host->ports[i];
-
-               ata_scsi_scan_host(ap, 1);
-       }
-       DPRINTK("host probe end\n");
+       DPRINTK("probe end\n");
 
        return 0;
 }