[PATCH] dma driver: fix potential oom issue of fsldma

[Li Yang]

The change looks to be correct.  But we need better formatting and description.

Make the title something like:

dmaengine: fsldma: fix memory leak

On Thu, Dec 24, 2015 at 1:26 AM, Xuelin Shi <xuelin.shi at nxp.com> wrote:
> From: Xuelin Shi <xuelin.shi at freescale.com>
>
> missing unmap sources and destinations while doing dequeue.

How can this describe your change?

Regards,
Leo
>
> Signed-off-by: Xuelin Shi <xuelin.shi at nxp.com>
> ---
>  drivers/dma/fsldma.c | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/drivers/dma/fsldma.c b/drivers/dma/fsldma.c
> index 2209f75..aac85c3 100644
> --- a/drivers/dma/fsldma.c
> +++ b/drivers/dma/fsldma.c
> @@ -522,6 +522,8 @@ static dma_cookie_t fsldma_run_tx_complete_actions(struct fsldma_chan *chan,
>                         chan_dbg(chan, "LD %p callback\n", desc);
>                         txd->callback(txd->callback_param);
>                 }
> +
> +               dma_descriptor_unmap(txd);
>         }
>
>         /* Run any dependencies */
> --
> 1.8.4
>
> _______________________________________________
> Linuxppc-dev mailing list
> Linuxppc-dev at lists.ozlabs.org
> https://lists.ozlabs.org/listinfo/linuxppc-dev