diff options
author | Lorenzo Stoakes <lstoakes@gmail.com> | 2016-10-12 20:20:16 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2016-10-19 11:11:43 -0400 |
commit | 768ae309a96103ed02eb1e111e838c87854d8b51 (patch) | |
tree | 59f9070c68cc87d37e5edf6439b5074a96f5d3dd /arch/x86/mm | |
parent | 7f23b3504a0df63b724180262c5f3f117f21bcae (diff) |
mm: replace get_user_pages() write/force parameters with gup_flags
This removes the 'write' and 'force' from get_user_pages() and replaces
them with 'gup_flags' to make the use of FOLL_FORCE explicit in callers
as use of this flag can result in surprising behaviour (and hence bugs)
within the mm subsystem.
Signed-off-by: Lorenzo Stoakes <lstoakes@gmail.com>
Acked-by: Christian König <christian.koenig@amd.com>
Acked-by: Jesper Nilsson <jesper.nilsson@axis.com>
Acked-by: Michal Hocko <mhocko@suse.com>
Reviewed-by: Jan Kara <jack@suse.cz>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'arch/x86/mm')
-rw-r--r-- | arch/x86/mm/mpx.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/arch/x86/mm/mpx.c b/arch/x86/mm/mpx.c index 80476878eb4c..e4f800999b32 100644 --- a/arch/x86/mm/mpx.c +++ b/arch/x86/mm/mpx.c | |||
@@ -544,10 +544,9 @@ static int mpx_resolve_fault(long __user *addr, int write) | |||
544 | { | 544 | { |
545 | long gup_ret; | 545 | long gup_ret; |
546 | int nr_pages = 1; | 546 | int nr_pages = 1; |
547 | int force = 0; | ||
548 | 547 | ||
549 | gup_ret = get_user_pages((unsigned long)addr, nr_pages, write, | 548 | gup_ret = get_user_pages((unsigned long)addr, nr_pages, |
550 | force, NULL, NULL); | 549 | write ? FOLL_WRITE : 0, NULL, NULL); |
551 | /* | 550 | /* |
552 | * get_user_pages() returns number of pages gotten. | 551 | * get_user_pages() returns number of pages gotten. |
553 | * 0 means we failed to fault in and get anything, | 552 | * 0 means we failed to fault in and get anything, |