[PATCH 1/3] futex: change signature of fetch_robust_entry()

From: Namhyung Kim
Date: Tue Sep 14 2010 - 08:44:15 EST


Make 3rd argument of fetch_robust_entry() 'unsigned int' in order to
remove following warnings from sparse:

kernel/futex.c:2495:59: warning: incorrect type in argument 3 (different signedness)
kernel/futex.c:2495:59: expected int *pi
kernel/futex.c:2495:59: got unsigned int *<noident>
kernel/futex.c:2506:67: warning: incorrect type in argument 3 (different signedness)
kernel/futex.c:2506:67: expected int *pi
kernel/futex.c:2506:67: got unsigned int *<noident>
kernel/futex.c:2515:69: warning: incorrect type in argument 3 (different signedness)
kernel/futex.c:2515:69: expected int *pi
kernel/futex.c:2515:69: got unsigned int *<noident>

Since all of its users were using 'unsigned int' already, it could
be changed safely.

Signed-off-by: Namhyung Kim <namhyung@xxxxxxxxx>
---
kernel/futex.c | 2 +-
kernel/futex_compat.c | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/kernel/futex.c b/kernel/futex.c
index 6a3a5fa..464de27 100644
--- a/kernel/futex.c
+++ b/kernel/futex.c
@@ -2458,7 +2458,7 @@ retry:
*/
static inline int fetch_robust_entry(struct robust_list __user **entry,
struct robust_list __user * __user *head,
- int *pi)
+ unsigned int *pi)
{
unsigned long uentry;

diff --git a/kernel/futex_compat.c b/kernel/futex_compat.c
index d49afb2..06da4df 100644
--- a/kernel/futex_compat.c
+++ b/kernel/futex_compat.c
@@ -19,7 +19,7 @@
*/
static inline int
fetch_robust_entry(compat_uptr_t *uentry, struct robust_list __user **entry,
- compat_uptr_t __user *head, int *pi)
+ compat_uptr_t __user *head, unsigned int *pi)
{
if (get_user(*uentry, head))
return -EFAULT;
--
1.7.2.2

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/