Re: [PATCH] binder: Drop needless locking in binder_mmap()

2017-07-17 Thread Greg Kroah-Hartman
On Mon, Jun 19, 2017 at 05:53:34PM +0300, Dmitry Safonov wrote: > On 05/24/2017 05:51 PM, Dmitry Safonov wrote: > > binder_mmap() protected proc->buffer from concurrent changes. > > It was introduced with commit bd1eff9741af ("Staging: android: binder: > > Fix crashes when sharing a binder file bet

Re: [PATCH] binder: Drop needless locking in binder_mmap()

2017-06-19 Thread Dmitry Safonov
On 05/24/2017 05:51 PM, Dmitry Safonov wrote: binder_mmap() protected proc->buffer from concurrent changes. It was introduced with commit bd1eff9741af ("Staging: android: binder: Fix crashes when sharing a binder file between processes"). The commit fixed such races alike: task1 = fd = open

[PATCH] binder: Drop needless locking in binder_mmap()

2017-05-24 Thread Dmitry Safonov
binder_mmap() protected proc->buffer from concurrent changes. It was introduced with commit bd1eff9741af ("Staging: android: binder: Fix crashes when sharing a binder file between processes"). The commit fixed such races alike: task1 = fd = open("/dev/binder",..) fork() --