diff options
| author | Sherry Yang <sherryy@android.com> | 2017-09-15 21:12:15 -0400 |
|---|---|---|
| committer | Sherry Yang <sherryy@android.com> | 2017-09-20 22:27:11 -0400 |
| commit | 9b9d7cf19106f1b2429ca192b8bfd8ffce7b859b (patch) | |
| tree | 2106a26632df6ffe695d02060e37862b33d78231 /drivers/android/binder_alloc.h | |
| parent | e6fa28a9a9c0f4ad120f1b6d4bfe2300e658d514 (diff) | |
FROMLIST: android: binder: Don't get mm from task
(from https://patchwork.kernel.org/patch/9954125/)
Use binder_alloc struct's mm_struct rather than getting
a reference to the mm struct through get_task_mm to
avoid a potential deadlock between lru lock, task lock and
dentry lock, since a thread can be holding the task lock
and the dentry lock while trying to acquire the lru lock.
Test: ran binderLibTest, throughputtest, interfacetest and
mempressure w/lockdep
Bug: 63926541
Change-Id: Icc661404eb7a4a2ecc5234b1bf8f0104665f9b45
Acked-by: Arve Hjønnevåg <arve@android.com>
Signed-off-by: Sherry Yang <sherryy@android.com>
Diffstat (limited to 'drivers/android/binder_alloc.h')
| -rw-r--r-- | drivers/android/binder_alloc.h | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/drivers/android/binder_alloc.h b/drivers/android/binder_alloc.h index a3a3602c689c..2dd33b6df104 100644 --- a/drivers/android/binder_alloc.h +++ b/drivers/android/binder_alloc.h @@ -100,7 +100,6 @@ struct binder_lru_page { */ struct binder_alloc { struct mutex mutex; - struct task_struct *tsk; struct vm_area_struct *vma; struct mm_struct *vma_vm_mm; void *buffer; |