On Mon, 2017-03-20 at 01:24 +0300, Alexey Dobriyan wrote:
> Hash size can't negative so "unsigned int" is logically correct.
> 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);
I do not see any improvement here.
What is wrong with size_t exactly ?
