Re: [PATCH] dmaengine: pl330: use lock of dma channel in pl330_update

2016-09-09 Thread Hsin-yu Chao
It turns out this patch could cause lock dependency issue. The other patch "Acquire dmac's spinlock in pl330_tx_status" that I've uploaded is the correct fix to replace this one. Thanks, Hsin-yu On Fri, Sep 9, 2016 at 7:48 PM, Vinod Koul wrote: > On Fri, Aug 19, 2016 at 09:03:48PM +0800, Hsin-Yu

Re: [PATCH] dmaengine: pl330: use lock of dma channel in pl330_update

2016-09-09 Thread Vinod Koul
On Fri, Aug 19, 2016 at 09:03:48PM +0800, Hsin-Yu Chao wrote: > The content of pl330_thread could be modified in pl330_update > without protection by the lock from the dma channel who currently > holding this thread. This could cause bug to the calculation of > in pl330_tx_status, if the running re

[PATCH] dmaengine: pl330: use lock of dma channel in pl330_update

2016-08-19 Thread Hsin-Yu Chao
The content of pl330_thread could be modified in pl330_update without protection by the lock from the dma channel who currently holding this thread. This could cause bug to the calculation of in pl330_tx_status, if the running request has just been done and moving to next request while calculating