lkml.org 
[lkml]   [2018]   [May]   [8]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 06/10] ASoC: amd: memory release for rtd structure
    Date
    rtd structure freed early may result in kernel panic in dma close
    call back. moved releasing memory for rtd structure to the end of
    dma close callback.

    Signed-off-by: Vijendar Mukunda <Vijendar.Mukunda@amd.com>
    Reviewed-by: Daniel Kurtz <djkurtz@chromium.org>
    ---
    sound/soc/amd/acp-pcm-dma.c | 4 +---
    1 file changed, 1 insertion(+), 3 deletions(-)

    diff --git a/sound/soc/amd/acp-pcm-dma.c b/sound/soc/amd/acp-pcm-dma.c
    index 8236116..ec9cab3 100644
    --- a/sound/soc/amd/acp-pcm-dma.c
    +++ b/sound/soc/amd/acp-pcm-dma.c
    @@ -998,8 +998,6 @@ static int acp_dma_close(struct snd_pcm_substream *substream)
    DRV_NAME);
    struct audio_drv_data *adata = dev_get_drvdata(component->dev);

    - kfree(rtd);
    -
    if (substream->stream == SNDRV_PCM_STREAM_PLAYBACK) {
    adata->play_i2ssp_stream = NULL;
    /*
    @@ -1028,7 +1026,7 @@ static int acp_dma_close(struct snd_pcm_substream *substream)
    */
    if (!adata->play_i2ssp_stream && !adata->capture_i2ssp_stream)
    acp_reg_write(0, adata->acp_mmio, mmACP_EXTERNAL_INTR_ENB);
    -
    + kfree(rtd);
    return 0;
    }

    --
    2.7.4
    \
     
     \ /
      Last update: 2018-05-08 06:46    [W:4.350 / U:0.004 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site