lkml.org 
[lkml]   [2015]   [Nov]   [27]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] driver:dma bug_fix : access freed memory
On Fri, Nov 27, 2015 at 05:44:53AM +0000, 김정배 wrote:
> From 8f6aeb362d9e44f29d46ae7694cdfee4408406ce Mon Sep 17 00:00:00 2001
> From: "KIM JUGNBAE" <jb09.kim@samsung.com>
> Date: Thu, 26 Nov 2015 16:28:47 +0900
> Subject: [PATCH] bug_fix : access freed memory

This part should not be present in the patch.

>
> sync_fenc_free & fence_check_cb_func would be executed at
> other cpu. fence_check_cb_func access freed fence memory after
> kfree(fence) at sync_fence_free.
> To escaped this issue, atomic_read(&fence->status) need to be
> protected by child_list_lock.
>
> Signed-off-by: "kimjungbae\" " <jb09.kim@samsung.com>"

The From name and the Signed-off-by name shouls be same. Mayvbe you can
consider having Kim Jugnbae (or Jungbae) as both.

> ---
> drivers/dma-buf/fence.c | 13 +++++++++++++
> drivers/staging/android/sync.c | 10 +++++++---
> drivers/staging/android/sync_debug.c | 2 ++
> include/linux/fence.h | 1 +
> 4 files changed, 23 insertions(+), 3 deletions(-)

Usually staging patches can not touch anything outside staging.

regards
sudip


\
 
 \ /
  Last update: 2015-11-27 08:41    [W:2.450 / U:0.020 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site