From: Thomas Gleixner <tglx@linutronix.de>
Date: Thu, 25 Jun 2009 12:27:58 +0000 (+0200)
Subject: futex: request only one page from get_user_pages()
X-Git-Tag: accepted/tizen/common/20141203.182822~15417^2
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=aa715284b4d28cabde6c25c568d769a6be712bc8;p=platform%2Fkernel%2Flinux-arm64.git

futex: request only one page from get_user_pages()

Yanmin noticed that fault_in_user_writeable() requests 4 pages instead
of one.

That's the result of blindly trusting Linus' proposal :) I even looked
up the prototype to verify the correctness: the argument in question
is confusingly enough named "len" while in reality it means number of
pages.

Pointed-out-by: Yanmin Zhang <yanmin_zhang@linux.intel.com>
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
---

diff --git a/kernel/futex.c b/kernel/futex.c
index 1c33711..794c862 100644
--- a/kernel/futex.c
+++ b/kernel/futex.c
@@ -299,7 +299,7 @@ void put_futex_key(int fshared, union futex_key *key)
 static int fault_in_user_writeable(u32 __user *uaddr)
 {
 	int ret = get_user_pages(current, current->mm, (unsigned long)uaddr,
-				 sizeof(*uaddr), 1, 0, NULL, NULL);
+				 1, 1, 0, NULL, NULL);
 	return ret < 0 ? ret : 0;
 }