Messages in this thread | | | Date | Fri, 27 Nov 2015 12:57:08 +0530 | From | Sudip Mukherjee <> | Subject | Re: [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
| |