Commit 520bd7a8 authored by Ulf Hansson's avatar Ulf Hansson

mmc: core: Optimize boot time by detecting cards simultaneously

The mmc workqueue is an ordered workqueue, allowing only one work to
execute per given time. As this workqueue is used for card detection, the
conseqeunce is that cards will be detected one by one waiting for each
other.

Moreover, most of the time spent during card initialization is waiting for
the card's internal firmware to be ready. From a CPU perspective this
typically means waiting for a completion variable to be kicked via an
IRQ-handler or waiting for a sleep timer to finish.

This behaviour of detecting/initializing cards is sub-optimal, especially
for SOCs having several controllers/cards.

Let's convert to use the system_freezable_wq for the mmc detect works.
This enables several works to be executed simultaneously and thus also
cards to be detected like so.

Tests on UX500, which holds two eMMC cards and an SD-card (actually also
an SDIO card, currently not detected), shows a significant improved
behaviour due to this change.

Before this change, both the eMMC cards waited for the SD card to be
initialized as its detect work entered the workqueue first. In some cases,
depending on the characteristic of the SD-card, they got delayed 1-1.5 s.

Additionally for the second eMMC, it needed to wait for the first eMMC to
be initialized which added another 120-190 ms.

Converting to the system_freezable_wq, removed these delays and made both
the eMMC cards available far earlier in the boot sequence.

Selecting the system_freezable_wq, in favour of for example the system_wq,
is because we need card detection mechanism to be disabled once userspace
are frozen during system PM. Currently the mmc core deal with this via PM
notifiers, but following patches may utilize the behaviour of the
system_freezable_wq, to simplify the use of the PM notifiers.
Signed-off-by: default avatarUlf Hansson <ulf.hansson@linaro.org>
Tested-by: default avatarAlan Cooper <alcooperx@gmail.com>
Tested-by: default avatarShawn Lin <shawn.lin@rock-chips.com>
parent 260b3164
...@@ -55,7 +55,6 @@ ...@@ -55,7 +55,6 @@
*/ */
#define MMC_BKOPS_MAX_TIMEOUT (4 * 60 * 1000) /* max time to wait in ms */ #define MMC_BKOPS_MAX_TIMEOUT (4 * 60 * 1000) /* max time to wait in ms */
static struct workqueue_struct *workqueue;
static const unsigned freqs[] = { 400000, 300000, 200000, 100000 }; static const unsigned freqs[] = { 400000, 300000, 200000, 100000 };
/* /*
...@@ -66,21 +65,16 @@ static const unsigned freqs[] = { 400000, 300000, 200000, 100000 }; ...@@ -66,21 +65,16 @@ static const unsigned freqs[] = { 400000, 300000, 200000, 100000 };
bool use_spi_crc = 1; bool use_spi_crc = 1;
module_param(use_spi_crc, bool, 0); module_param(use_spi_crc, bool, 0);
/*
* Internal function. Schedule delayed work in the MMC work queue.
*/
static int mmc_schedule_delayed_work(struct delayed_work *work, static int mmc_schedule_delayed_work(struct delayed_work *work,
unsigned long delay) unsigned long delay)
{ {
return queue_delayed_work(workqueue, work, delay); /*
} * We use the system_freezable_wq, because of two reasons.
* First, it allows several works (not the same work item) to be
/* * executed simultaneously. Second, the queue becomes frozen when
* Internal function. Flush all scheduled work from the MMC work queue. * userspace becomes frozen during system PM.
*/ */
static void mmc_flush_scheduled_work(void) return queue_delayed_work(system_freezable_wq, work, delay);
{
flush_workqueue(workqueue);
} }
#ifdef CONFIG_FAIL_MMC_REQUEST #ifdef CONFIG_FAIL_MMC_REQUEST
...@@ -2669,7 +2663,6 @@ void mmc_stop_host(struct mmc_host *host) ...@@ -2669,7 +2663,6 @@ void mmc_stop_host(struct mmc_host *host)
host->rescan_disable = 1; host->rescan_disable = 1;
cancel_delayed_work_sync(&host->detect); cancel_delayed_work_sync(&host->detect);
mmc_flush_scheduled_work();
/* clear pm flags now and let card drivers set them as needed */ /* clear pm flags now and let card drivers set them as needed */
host->pm_flags = 0; host->pm_flags = 0;
...@@ -2852,13 +2845,9 @@ static int __init mmc_init(void) ...@@ -2852,13 +2845,9 @@ static int __init mmc_init(void)
{ {
int ret; int ret;
workqueue = alloc_ordered_workqueue("kmmcd", 0);
if (!workqueue)
return -ENOMEM;
ret = mmc_register_bus(); ret = mmc_register_bus();
if (ret) if (ret)
goto destroy_workqueue; return ret;
ret = mmc_register_host_class(); ret = mmc_register_host_class();
if (ret) if (ret)
...@@ -2874,9 +2863,6 @@ static int __init mmc_init(void) ...@@ -2874,9 +2863,6 @@ static int __init mmc_init(void)
mmc_unregister_host_class(); mmc_unregister_host_class();
unregister_bus: unregister_bus:
mmc_unregister_bus(); mmc_unregister_bus();
destroy_workqueue:
destroy_workqueue(workqueue);
return ret; return ret;
} }
...@@ -2885,7 +2871,6 @@ static void __exit mmc_exit(void) ...@@ -2885,7 +2871,6 @@ static void __exit mmc_exit(void)
sdio_unregister_bus(); sdio_unregister_bus();
mmc_unregister_host_class(); mmc_unregister_host_class();
mmc_unregister_bus(); mmc_unregister_bus();
destroy_workqueue(workqueue);
} }
subsys_initcall(mmc_init); subsys_initcall(mmc_init);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment