diff options
| author | Russell King | 2016-01-26 14:40:42 +0100 |
|---|---|---|
| committer | Ulf Hansson | 2016-02-29 11:03:16 +0100 |
| commit | 054cedff5e025a54ceefff891c6ea42ee8b37eab (patch) | |
| tree | 5f81cb7025cafb4ade9c578658e5b80a3ba56e61 /drivers/mmc/host/sdhci.c | |
| parent | mmc: sdhci: avoid unnecessary mapping/unmapping of align buffer (diff) | |
| download | kernel-qcow2-linux-054cedff5e025a54ceefff891c6ea42ee8b37eab.tar.gz kernel-qcow2-linux-054cedff5e025a54ceefff891c6ea42ee8b37eab.tar.xz kernel-qcow2-linux-054cedff5e025a54ceefff891c6ea42ee8b37eab.zip | |
mmc: sdhci: plug DMA mapping leak on error
If we terminate a command early, we fail to properly clean up the DMA
mappings for the data part of the request. Put this clean up to the
tasklet, which is the common path for finishing a request so we always
clean up after ourselves.
Signed-off-by: Russell King <rmk+kernel@arm.linux.org.uk>
[ Split original patch so that it now contains only the fix ]
Signed-off-by: Adrian Hunter <adrian.hunter@intel.com>
Cc: stable@vger.kernel.org # v4.5+
Tested-by: Gregory CLEMENT <gregory.clement@free-electrons.com>
Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org>
Diffstat (limited to 'drivers/mmc/host/sdhci.c')
| -rw-r--r-- | drivers/mmc/host/sdhci.c | 16 |
1 files changed, 16 insertions, 0 deletions
diff --git a/drivers/mmc/host/sdhci.c b/drivers/mmc/host/sdhci.c index 78359baee369..97e5f40a831f 100644 --- a/drivers/mmc/host/sdhci.c +++ b/drivers/mmc/host/sdhci.c @@ -2208,6 +2208,22 @@ static void sdhci_tasklet_finish(unsigned long param) mrq = host->mrq; /* + * Always unmap the data buffers if they were mapped by + * sdhci_prepare_data() whenever we finish with a request. + * This avoids leaking DMA mappings on error. + */ + if (host->flags & SDHCI_REQ_USE_DMA) { + struct mmc_data *data = mrq->data; + + if (data && data->host_cookie == COOKIE_MAPPED) { + dma_unmap_sg(mmc_dev(host->mmc), data->sg, data->sg_len, + (data->flags & MMC_DATA_READ) ? + DMA_FROM_DEVICE : DMA_TO_DEVICE); + data->host_cookie = COOKIE_UNMAPPED; + } + } + + /* * The controller needs a reset of internal state machines * upon error conditions. */ |