Re: [RFC v1 2/6] virtio/vsock: add support for virtio datagram

[kernel test robot <lkp@intel.com>]

[-- Attachment #1: Type: text/plain, Size: 2908 bytes --]

Hi Jiang,

[FYI, it's a private test report for your RFC patch.]
[auto build test WARNING on vhost/linux-next]
[also build test WARNING on tip/perf/core linus/master v5.13-rc6]
[cannot apply to next-20210616]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]

url:    https://github.com/0day-ci/linux/commits/Jiang-Wang/virtio-vsock-introduce-SOCK_DGRAM-support/20210616-120056
base:   https://git.kernel.org/pub/scm/linux/kernel/git/mst/vhost.git linux-next
config: x86_64-randconfig-a002-20210616 (attached as .config)
compiler: clang version 13.0.0 (https://github.com/llvm/llvm-project 64720f57bea6a6bf033feef4a5751ab9c0c3b401)
reproduce (this is a W=1 build):
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # install x86_64 cross compiling tool for clang build
        # apt-get install binutils-x86-64-linux-gnu
        # https://github.com/0day-ci/linux/commit/b156a0ad587c43dbfc98397f01b34fad15054bf0
        git remote add linux-review https://github.com/0day-ci/linux
        git fetch --no-tags linux-review Jiang-Wang/virtio-vsock-introduce-SOCK_DGRAM-support/20210616-120056
        git checkout b156a0ad587c43dbfc98397f01b34fad15054bf0
        # save the attached .config to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=x86_64 

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>

All warnings (new ones prefixed by >>):

>> net/vmw_vsock/virtio_transport.c:450:7: warning: variable 'added' set but not used [-Wunused-but-set-variable]
           bool added = false;
                ^
   1 warning generated.


vim +/added +450 net/vmw_vsock/virtio_transport.c

   444	
   445	static void virtio_transport_dgram_tx_work(struct work_struct *work)
   446	{
   447		struct virtio_vsock *vsock =
   448			container_of(work, struct virtio_vsock, dgram_tx_work);
   449		struct virtqueue *vq;
 > 450		bool added = false;
   451	
   452		vq = vsock->vqs[VSOCK_VQ_DGRAM_TX];
   453		mutex_lock(&vsock->dgram_tx_lock);
   454	
   455		if (!vsock->dgram_tx_run)
   456			goto out;
   457	
   458		do {
   459			struct virtio_vsock_pkt *pkt;
   460			unsigned int len;
   461	
   462			virtqueue_disable_cb(vq);
   463			while ((pkt = virtqueue_get_buf(vq, &len)) != NULL) {
   464				virtio_transport_free_pkt(pkt);
   465				added = true;
   466			}
   467		} while (!virtqueue_enable_cb(vq));
   468	
   469	out:
   470		mutex_unlock(&vsock->dgram_tx_lock);
   471	}
   472	

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org

[-- Attachment #2: config.gz --]
[-- Type: application/gzip, Size: 38410 bytes --]