Messages in this thread | | | Date | Thu, 19 Apr 2018 16:05:05 +0800 | From | kbuild test robot <> | Subject | Re: [PATCH net] virtio_net: split out ctrl buffer |
| |
Hi Michael,
I love your patch! Yet something to improve:
[auto build test ERROR on net/master]
url: https://github.com/0day-ci/linux/commits/Michael-S-Tsirkin/virtio_net-split-out-ctrl-buffer/20180419-145754 config: i386-randconfig-a0-201815 (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=i386
All errors (new ones prefixed by >>):
drivers/net/virtio_net.c: In function 'virtnet_free_queues': >> drivers/net/virtio_net.c:2365:10: error: 'struct virtnet_info' has no member named 'err_ctrl' kfree(vi->err_ctrl); ^ drivers/net/virtio_net.c: In function 'virtnet_alloc_queues': >> drivers/net/virtio_net.c:2589:8: error: 'vq' undeclared (first use in this function) kfree(vq->ctrl); ^ drivers/net/virtio_net.c:2589:8: note: each undeclared identifier is reported only once for each function it appears in
vim +2365 drivers/net/virtio_net.c
2347 2348 static void virtnet_free_queues(struct virtnet_info *vi) 2349 { 2350 int i; 2351 2352 for (i = 0; i < vi->max_queue_pairs; i++) { 2353 napi_hash_del(&vi->rq[i].napi); 2354 netif_napi_del(&vi->rq[i].napi); 2355 netif_napi_del(&vi->sq[i].napi); 2356 } 2357 2358 /* We called napi_hash_del() before netif_napi_del(), 2359 * we need to respect an RCU grace period before freeing vi->rq 2360 */ 2361 synchronize_net(); 2362 2363 kfree(vi->rq); 2364 kfree(vi->sq); > 2365 kfree(vi->err_ctrl); 2366 } 2367
--- 0-DAY kernel test infrastructure Open Source Technology Center https://lists.01.org/pipermail/kbuild-all Intel Corporation [unhandled content-type:application/gzip] | |