> @@ -4348,7 +4348,9 @@ int __access_remote_vm(struct task_struct *tsk, struct > mm_struct *mm, > void *old_buf = buf; > int write = gup_flags & FOLL_WRITE; > > - down_read(&mm->mmap_sem); > + if (down_read_killable(&mm->mmap_sem)) > + return 0; > +
I too think that "return 0" looks a bit strange even if correct, to me "return -EINTR" would look better. But I won't insist, this is cosmetic. Acked-by: Oleg Nesterov <o...@redhat.com>
