From: Guennadi Liakhovetski Date: Sun, 3 Aug 2014 17:13:03 +0000 (+0200) Subject: dmaengine: nbpfaxi: fix a theoretical race X-Git-Url: https://git.karo-electronics.de/?a=commitdiff_plain;h=67b166847009b009cd9fbcdef6b71558f49e1bd6;p=linux-beck.git dmaengine: nbpfaxi: fix a theoretical race A race possibility exists if a DMA slave driver tries to free channel resources witout waiting for all transfers to complete and without explicitly terminating all requests. In such a case the IRQ processing thread can race with .device_free_chan_resources(). To fix this race empty all descriptor lists before freeing descriptor cache. Signed-off-by: Guennadi Liakhovetski Signed-off-by: Vinod Koul --- diff --git a/drivers/dma/nbpfaxi.c b/drivers/dma/nbpfaxi.c index 77c5a890a30a..5b40ac8d82d1 100644 --- a/drivers/dma/nbpfaxi.c +++ b/drivers/dma/nbpfaxi.c @@ -1054,6 +1054,7 @@ static void nbpf_free_chan_resources(struct dma_chan *dchan) dev_dbg(dchan->device->dev, "Entry %s()\n", __func__); nbpf_chan_halt(chan); + nbpf_chan_idle(chan); /* Clean up for if a channel is re-used for MEMCPY after slave DMA */ nbpf_chan_prepare_default(chan);