Re: [PATCH] net: use kvmalloc rather than open coded variant

From: kbuild test robot
Date: Fri Jan 06 2017 - 22:33:36 EST


Hi Michal,

[auto build test ERROR on net-next/master]
[also build test ERROR on v4.10-rc2 next-20170106]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]

url: https://github.com/0day-ci/linux/commits/Michal-Hocko/net-use-kvmalloc-rather-than-open-coded-variant/20170107-104105
config: x86_64-randconfig-i0-201701 (attached as .config)
compiler: gcc-4.9 (Debian 4.9.4-2) 4.9.4
reproduce:
# save the attached .config to linux build tree
make ARCH=x86_64

Note: the linux-review/Michal-Hocko/net-use-kvmalloc-rather-than-open-coded-variant/20170107-104105 HEAD 29df6a817f53555953b47c6f8d09397f9f7b598c builds fine.
It only hurts bisectibility.

All error/warnings (new ones prefixed by >>):

net/core/dev.c: In function 'netif_alloc_rx_queues':
>> net/core/dev.c:7114:2: error: implicit declaration of function 'kvzalloc' [-Werror=implicit-function-declaration]
rx = kvzalloc(sz, GFP_KERNEL | __GFP_REPEAT);
^
>> net/core/dev.c:7114:5: warning: assignment makes pointer from integer without a cast
rx = kvzalloc(sz, GFP_KERNEL | __GFP_REPEAT);
^
net/core/dev.c: In function 'netif_alloc_netdev_queues':
net/core/dev.c:7154:5: warning: assignment makes pointer from integer without a cast
tx = kvzalloc(sz, GFP_KERNEL | __GFP_REPEAT);
^
net/core/dev.c: In function 'alloc_netdev_mqs':
net/core/dev.c:7690:4: warning: assignment makes pointer from integer without a cast
p = kvzalloc(alloc_size, GFP_KERNEL | __GFP_REPEAT);
^
cc1: some warnings being treated as errors

vim +/kvzalloc +7114 net/core/dev.c

7108 unsigned int i, count = dev->num_rx_queues;
7109 struct netdev_rx_queue *rx;
7110 size_t sz = count * sizeof(*rx);
7111
7112 BUG_ON(count < 1);
7113
> 7114 rx = kvzalloc(sz, GFP_KERNEL | __GFP_REPEAT);
7115 if (!rx)
7116 return -ENOMEM;
7117

---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation

Attachment: .config.gz
Description: application/gzip