dmaengine/ste_dma40: fix Oops due to double free of client descriptor
authorPer Forlin <per.forlin@linaro.org>
Mon, 29 Aug 2011 11:33:34 +0000 (13:33 +0200)
committerVinod Koul <vinod.koul@intel.com>
Mon, 5 Sep 2011 11:41:21 +0000 (16:41 +0530)
commit70a207ad4db2f0c60308b3f32086263c438c67a3
tree875fbbdfe57a044516f722766e4185d9fcc05938
parent270e779036ff144d6c6904ce9480f0d70ff93e86
dmaengine/ste_dma40: fix Oops due to double free of client descriptor

The client list may exist in two lists at the same time. This makes free
fail since the same desc is freed multiple times. Remove desc from
client list when adding it to the pending queue. Move free of client owned
descriptors from free_dma() to terminate_all().

Unable to handle kernel paging request at virtual address 00100104
pgd = dea8c000
[00100104] *pgd=1ea62831, *pte=00000000, *ppte=00000000
Internal error: Oops: 817 [#1] PREEMPT SMP
Modules linked in:
CPU: 0    Not tainted  (3.1.0-rc3+ #58)
PC is at d40_free_chan_resources+0x64/0x330

Signed-off-by: Per Forlin <per.forlin@linaro.org>
Acked-by: Linus Walleij <linus.walleij@linaro.org>
Signed-off-by: Vinod Koul <vinod.koul@intel.com>
drivers/dma/ste_dma40.c