Re: [PATCH 09/18] ipc, shm: make shmem attach/detach wait for mmap_sem killable

[Michal Hocko]

On Tue 08-03-16 11:15:50, Davidlohr Bueso wrote:
> On Mon, 29 Feb 2016, Michal Hocko wrote:
> 
> >From: Michal Hocko <mho...@suse.com>
> >
> >shmat and shmdt rely on mmap_sem for write. If the waiting task
> >gets killed by the oom killer it would block oom_reaper from
> >asynchronous address space reclaim and reduce the chances of timely
> >OOM resolving. Wait for the lock in the killable mode and return with
> >EINTR if the task got killed while waiting.
> >
> >Cc: Davidlohr Bueso <d...@stgolabs.net>
> >Cc: Hugh Dickins <hu...@google.com>
> >Signed-off-by: Michal Hocko <mho...@suse.com>
> 
> I have no objection to this perse, just one comment below.
> 
> Acked-by: Davidlohr Bueso <d...@stgolabs.net>

Thanks!

[...]
> >-    down_write(&current->mm->mmap_sem);
> >+    if (down_write_killable(&current->mm->mmap_sem)) {
> >+            err = -EINVAL;
> >+            goto out_fput;
> >+    }
> 
> This should be EINTR, no?

Of course. Thanks for catching that.
-- 
Michal Hocko
SUSE Labs