mmc: tmio: fix a deadlock
authorGuennadi Liakhovetski <g.liakhovetski@gmx.de>
Thu, 14 Jul 2011 16:39:10 +0000 (18:39 +0200)
committerChris Ball <cjb@laptop.org>
Thu, 21 Jul 2011 14:35:07 +0000 (10:35 -0400)
Currently the tmio-mmc driver contains a recursive runtime PM method
invocation, which leads to a deadlock on a mutex. Avoid it by taking
care not to request DMA too early.

Signed-off-by: Guennadi Liakhovetski <g.liakhovetski@gmx.de>
Signed-off-by: Chris Ball <cjb@laptop.org>
drivers/mmc/host/tmio_mmc.h
drivers/mmc/host/tmio_mmc_dma.c
drivers/mmc/host/tmio_mmc_pio.c

index ba0d8e66001d63b84025e1e2f25374b94185f9fd..087d88023ba1737aebb436bb2d43f5ffbdb9add7 100644 (file)
@@ -109,6 +109,7 @@ static inline void tmio_mmc_kunmap_atomic(struct scatterlist *sg,
 
 #if defined(CONFIG_MMC_SDHI) || defined(CONFIG_MMC_SDHI_MODULE)
 void tmio_mmc_start_dma(struct tmio_mmc_host *host, struct mmc_data *data);
+void tmio_mmc_enable_dma(struct tmio_mmc_host *host, bool enable);
 void tmio_mmc_request_dma(struct tmio_mmc_host *host, struct tmio_mmc_data *pdata);
 void tmio_mmc_release_dma(struct tmio_mmc_host *host);
 #else
@@ -117,6 +118,10 @@ static inline void tmio_mmc_start_dma(struct tmio_mmc_host *host,
 {
 }
 
+static inline void tmio_mmc_enable_dma(struct tmio_mmc_host *host, bool enable)
+{
+}
+
 static inline void tmio_mmc_request_dma(struct tmio_mmc_host *host,
                                 struct tmio_mmc_data *pdata)
 {
index 7e86662717d6fffe63835dba03b2d68407610d04..2aa616dec32dfe87d92a30e4097678593bb6e2fe 100644 (file)
 
 #define TMIO_MMC_MIN_DMA_LEN 8
 
-static void tmio_mmc_enable_dma(struct tmio_mmc_host *host, bool enable)
+void tmio_mmc_enable_dma(struct tmio_mmc_host *host, bool enable)
 {
+       if (!host->chan_tx || !host->chan_rx)
+               return;
+
 #if defined(CONFIG_SUPERH) || defined(CONFIG_ARCH_SHMOBILE)
        /* Switch DMA mode on or off - SuperH specific? */
        sd_ctrl_write16(host, CTL_DMA_ENABLE, enable ? 2 : 0);
index 221ffb71c7a307f77eb0c95f1b52a0ff6936b35f..1f16357e7301d1f4ea9a61f994f094bc71e0785b 100644 (file)
@@ -984,7 +984,7 @@ int tmio_mmc_host_resume(struct device *dev)
        if (host->pm_global) {
                /* Runtime PM resume callback didn't run */
                tmio_mmc_reset(host);
-               tmio_mmc_request_dma(host, host->pdata);
+               tmio_mmc_enable_dma(host, true);
                host->pm_global = false;
        }
 
@@ -1007,7 +1007,7 @@ int tmio_mmc_host_runtime_resume(struct device *dev)
        struct tmio_mmc_data *pdata = host->pdata;
 
        tmio_mmc_reset(host);
-       tmio_mmc_request_dma(host, host->pdata);
+       tmio_mmc_enable_dma(host, true);
 
        if (pdata->power) {
                /* Only entered after a card-insert interrupt */