Thread (32 messages) flat view 32 messages, 6 authors, 2019-06-06

Re: [PATCH 03/16] mm: simplify gup_fast_permitted

From: Linus Torvalds <torvalds@linux-foundation.org>
Date: 2019-06-03 16:14:11
Also in: linux-mips, linux-mm, linux-sh, lkml, sparclinux

On Mon, Jun 3, 2019 at 12:41 AM Christoph Hellwig [off-list ref] wrote:
I only removed a duplicate of it.
I don't see any remaining cases.
The full (old) code in get_user_pages_fast() looks like this:

        if (nr_pages <= 0)
                return 0;

        if (unlikely(!access_ok((void __user *)start, len)))
                return -EFAULT;

        if (gup_fast_permitted(start, nr_pages)) {
Yes, and that code was correct.

The new code has no test at all for "nr_pages == 0", afaik.

                 Linus
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help