netfilter: xtables: fix warnings on 32bit platforms
authorFlorian Westphal <fw@strlen.de>
Wed, 17 Jun 2015 21:58:28 +0000 (23:58 +0200)
committerPablo Neira Ayuso <pablo@netfilter.org>
Thu, 18 Jun 2015 19:14:33 +0000 (21:14 +0200)
On 32bit archs gcc complains due to cast from void* to u64.
Add intermediate casts to long to silence these warnings.

include/linux/netfilter/x_tables.h:376:10: warning: cast from pointer to integer of different size [-Wpointer-to-int-cast]
include/linux/netfilter/x_tables.h:384:15: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast]
include/linux/netfilter/x_tables.h:391:23: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast]
include/linux/netfilter/x_tables.h:400:22: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast]

Fixes: 71ae0dff02d756e ("netfilter: xtables: use percpu rule counters")
Reported-by: kbuild test robot <fengguang.wu@intel.com>
Signed-off-by: Florian Westphal <fw@strlen.de>
Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
include/linux/netfilter/x_tables.h

index 1c97a2204379e14676ee33f0b15c5174b7afe680..286098a5667f5f40852aacefce423ac5e3721ae8 100644 (file)
@@ -372,7 +372,7 @@ static inline u64 xt_percpu_counter_alloc(void)
                if (res == NULL)
                        return (u64) -ENOMEM;
 
-               return (__force u64) res;
+               return (u64) (__force unsigned long) res;
        }
 
        return 0;
@@ -380,14 +380,14 @@ static inline u64 xt_percpu_counter_alloc(void)
 static inline void xt_percpu_counter_free(u64 pcnt)
 {
        if (nr_cpu_ids > 1)
-               free_percpu((void __percpu *) pcnt);
+               free_percpu((void __percpu *) (unsigned long) pcnt);
 }
 
 static inline struct xt_counters *
 xt_get_this_cpu_counter(struct xt_counters *cnt)
 {
        if (nr_cpu_ids > 1)
-               return this_cpu_ptr((void __percpu *) cnt->pcnt);
+               return this_cpu_ptr((void __percpu *) (unsigned long) cnt->pcnt);
 
        return cnt;
 }
@@ -396,7 +396,7 @@ static inline struct xt_counters *
 xt_get_per_cpu_counter(struct xt_counters *cnt, unsigned int cpu)
 {
        if (nr_cpu_ids > 1)
-               return per_cpu_ptr((void __percpu *) cnt->pcnt, cpu);
+               return per_cpu_ptr((void __percpu *) (unsigned long) cnt->pcnt, cpu);
 
        return cnt;
 }