flowcache: make flow_cache_hash_size() return "unsigned int"
authorAlexey Dobriyan <adobriyan@gmail.com>
Sun, 2 Apr 2017 21:52:29 +0000 (00:52 +0300)
committerDavid S. Miller <davem@davemloft.net>
Tue, 4 Apr 2017 02:04:48 +0000 (19:04 -0700)
Hash size can't negative so "unsigned int" is logically correct.

Propagate "unsigned int" to loop counters.

Space savings:

add/remove: 0/0 grow/shrink: 2/2 up/down: 6/-18 (-12)
function                                     old     new   delta
flow_cache_flush_tasklet                     362     365      +3
__flow_cache_shrink                          333     336      +3
flow_cache_cpu_up_prep                       178     171      -7
flow_cache_lookup                           1159    1148     -11

Signed-off-by: Alexey Dobriyan <adobriyan@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/core/flow.c

index 98f977ec21b1ce1cc3efb38f576b11c0d89af465..923156d2a750ac571f634f2c2b00b4dd7181a76a 100644 (file)
@@ -47,7 +47,7 @@ struct flow_flush_info {
 
 static struct kmem_cache *flow_cachep __read_mostly;
 
-#define flow_cache_hash_size(cache)    (1 << (cache)->hash_shift)
+#define flow_cache_hash_size(cache)    (1U << (cache)->hash_shift)
 #define FLOW_HASH_RND_PERIOD           (10 * 60 * HZ)
 
 static void flow_cache_new_hashrnd(unsigned long arg)
@@ -119,9 +119,10 @@ static void __flow_cache_shrink(struct flow_cache *fc,
        struct flow_cache_entry *fle;
        struct hlist_node *tmp;
        LIST_HEAD(gc_list);
-       int i, deleted = 0;
+       int deleted = 0;
        struct netns_xfrm *xfrm = container_of(fc, struct netns_xfrm,
                                                flow_cache_global);
+       unsigned int i;
 
        for (i = 0; i < flow_cache_hash_size(fc); i++) {
                int saved = 0;
@@ -295,9 +296,10 @@ static void flow_cache_flush_tasklet(unsigned long data)
        struct flow_cache_entry *fle;
        struct hlist_node *tmp;
        LIST_HEAD(gc_list);
-       int i, deleted = 0;
+       int deleted = 0;
        struct netns_xfrm *xfrm = container_of(fc, struct netns_xfrm,
                                                flow_cache_global);
+       unsigned int i;
 
        fcp = this_cpu_ptr(fc->percpu);
        for (i = 0; i < flow_cache_hash_size(fc); i++) {
@@ -327,7 +329,7 @@ static void flow_cache_flush_tasklet(unsigned long data)
 static int flow_cache_percpu_empty(struct flow_cache *fc, int cpu)
 {
        struct flow_cache_percpu *fcp;
-       int i;
+       unsigned int i;
 
        fcp = per_cpu_ptr(fc->percpu, cpu);
        for (i = 0; i < flow_cache_hash_size(fc); i++)
@@ -402,12 +404,12 @@ void flow_cache_flush_deferred(struct net *net)
 static int flow_cache_cpu_prepare(struct flow_cache *fc, int cpu)
 {
        struct flow_cache_percpu *fcp = per_cpu_ptr(fc->percpu, cpu);
-       size_t sz = sizeof(struct hlist_head) * flow_cache_hash_size(fc);
+       unsigned int sz = sizeof(struct hlist_head) * flow_cache_hash_size(fc);
 
        if (!fcp->hash_table) {
                fcp->hash_table = kzalloc_node(sz, GFP_KERNEL, cpu_to_node(cpu));
                if (!fcp->hash_table) {
-                       pr_err("NET: failed to allocate flow cache sz %zu\n", sz);
+                       pr_err("NET: failed to allocate flow cache sz %u\n", sz);
                        return -ENOMEM;
                }
                fcp->hash_rnd_recalc = 1;