libata: only ports >= 0 need to synchronize
authorArjan van de Ven <arjan@linux.intel.com>
Sat, 10 Jan 2009 18:18:44 +0000 (10:18 -0800)
committerLinus Torvalds <torvalds@linux-foundation.org>
Sat, 10 Jan 2009 23:06:52 +0000 (15:06 -0800)
In a discussio with Jeff Garzik, he mentioned that the serialization
for the libata port probes only needs to be within the domain of a host.
This means that for the first port of each host (with ID 0), we don't
need to wait, so we can relax our serialization a little.

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

index f810078..71218d7 100644 (file)
@@ -5925,8 +5925,10 @@ static void async_port_probe(void *data, async_cookie_t cookie)
         * If we're not allowed to scan this host in parallel,
         * we need to wait until all previous scans have completed
         * before going further.
+        * Jeff Garzik says this is only within a controller, so we
+        * don't need to wait for port 0, only for later ports.
         */
-       if (!(ap->host->flags & ATA_HOST_PARALLEL_SCAN))
+       if (!(ap->host->flags & ATA_HOST_PARALLEL_SCAN) && ap->port_no != 0)
                async_synchronize_cookie(cookie);
 
        /* probe */