Re: [PATCH 3/5] dmaengine: bcm2835-dma: Fix memory leak when stopping a running transfer

2015-03-30 Thread Vinod Koul
On Fri, Mar 27, 2015 at 01:35:53PM +0200, Peter Ujfalusi wrote: > The vd->node is removed from the lists when the transfer started so the > vchan_get_all_descriptors() will not find it. This results memory leak. > > Signed-off-by: Peter Ujfalusi > CC: Stephen Warren > CC: Lee Jones Applied,

Re: [PATCH 3/5] dmaengine: bcm2835-dma: Fix memory leak when stopping a running transfer

2015-03-30 Thread Vinod Koul
On Fri, Mar 27, 2015 at 01:35:53PM +0200, Peter Ujfalusi wrote: The vd-node is removed from the lists when the transfer started so the vchan_get_all_descriptors() will not find it. This results memory leak. Signed-off-by: Peter Ujfalusi peter.ujfal...@ti.com CC: Stephen Warren

Re: [PATCH 3/5] dmaengine: bcm2835-dma: Fix memory leak when stopping a running transfer

2015-03-28 Thread Peter Ujfalusi
Thanks, On 03/28/2015 05:57 AM, Stephen Warren wrote: > On 03/27/2015 05:35 AM, Peter Ujfalusi wrote: >> The vd->node is removed from the lists when the transfer started so the >> vchan_get_all_descriptors() will not find it. This results memory leak. > > Acked-by: Stephen Warren > (I'm just

Re: [PATCH 3/5] dmaengine: bcm2835-dma: Fix memory leak when stopping a running transfer

2015-03-28 Thread Peter Ujfalusi
Thanks, On 03/28/2015 05:57 AM, Stephen Warren wrote: On 03/27/2015 05:35 AM, Peter Ujfalusi wrote: The vd-node is removed from the lists when the transfer started so the vchan_get_all_descriptors() will not find it. This results memory leak. Acked-by: Stephen Warren swar...@wwwdotorg.org

Re: [PATCH 3/5] dmaengine: bcm2835-dma: Fix memory leak when stopping a running transfer

2015-03-27 Thread Stephen Warren
On 03/27/2015 05:35 AM, Peter Ujfalusi wrote: > The vd->node is removed from the lists when the transfer started so the > vchan_get_all_descriptors() will not find it. This results memory leak. Acked-by: Stephen Warren (I'm just assuming the explanation makes sense and is correct; more of a

[PATCH 3/5] dmaengine: bcm2835-dma: Fix memory leak when stopping a running transfer

2015-03-27 Thread Peter Ujfalusi
The vd->node is removed from the lists when the transfer started so the vchan_get_all_descriptors() will not find it. This results memory leak. Signed-off-by: Peter Ujfalusi CC: Stephen Warren CC: Lee Jones --- drivers/dma/bcm2835-dma.c | 1 + 1 file changed, 1 insertion(+) diff --git

[PATCH 3/5] dmaengine: bcm2835-dma: Fix memory leak when stopping a running transfer

2015-03-27 Thread Peter Ujfalusi
The vd-node is removed from the lists when the transfer started so the vchan_get_all_descriptors() will not find it. This results memory leak. Signed-off-by: Peter Ujfalusi peter.ujfal...@ti.com CC: Stephen Warren swar...@wwwdotorg.org CC: Lee Jones l...@kernel.org --- drivers/dma/bcm2835-dma.c

Re: [PATCH 3/5] dmaengine: bcm2835-dma: Fix memory leak when stopping a running transfer

2015-03-27 Thread Stephen Warren
On 03/27/2015 05:35 AM, Peter Ujfalusi wrote: The vd-node is removed from the lists when the transfer started so the vchan_get_all_descriptors() will not find it. This results memory leak. Acked-by: Stephen Warren swar...@wwwdotorg.org (I'm just assuming the explanation makes sense and is