diff options
| author | Linus Torvalds <torvalds@g5.osdl.org> | 2005-12-12 16:24:33 -0800 |
|---|---|---|
| committer | Linus Torvalds <torvalds@g5.osdl.org> | 2005-12-12 16:24:33 -0800 |
| commit | 1ff8038988adecfde71d82c0597727fc239d4e8c (patch) | |
| tree | a592cbfad237a5e373724918a0cc17ee9c421237 | |
| parent | 14ee0a1414536d27bc29373bf23c106b8ef8ce0b (diff) | |
| download | lwn-1ff8038988adecfde71d82c0597727fc239d4e8c.tar.gz lwn-1ff8038988adecfde71d82c0597727fc239d4e8c.zip | |
get_user_pages: don't try to follow PFNMAP pages
Nick Piggin points out that a few drivers play games with VM_IO (why?
who knows..) and thus a pfn-remapped area may not have that bit set even
if remap_pfn_range() set it originally.
So make it explicit in get_user_pages() that we don't follow VM_PFNMAP
pages, since pretty much by definition they do not have a "struct page"
associated with them.
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
| -rw-r--r-- | mm/memory.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/mm/memory.c b/mm/memory.c index 47c533eaa072..d22f78c8a381 100644 --- a/mm/memory.c +++ b/mm/memory.c @@ -1009,7 +1009,7 @@ int get_user_pages(struct task_struct *tsk, struct mm_struct *mm, continue; } - if (!vma || (vma->vm_flags & VM_IO) + if (!vma || (vma->vm_flags & (VM_IO | VM_PFNMAP)) || !(vm_flags & vma->vm_flags)) return i ? : -EFAULT; |