[kbuild] Re: [PATCH] vdpa: bypass waking up vhost_woker for vdpa vq kick

2020-06-02 Thread Dan Carpenter
Hi Zhu,

url:
https://github.com/0day-ci/linux/commits/Zhu-Lingshan/vdpa-bypass-waking-up-vhost_woker-for-vdpa-vq-kick/20200526-133819
base:   https://git.kernel.org/pub/scm/linux/kernel/git/mst/vhost.git linux-next
config: x86_64-randconfig-m001-20200529 (attached as .config)
compiler: gcc-9 (Debian 9.3.0-13) 9.3.0

If you fix the issue, kindly add following tag as appropriate
Reported-by: kbuild test robot 
Reported-by: Dan Carpenter 

smatch warnings:
drivers/vhost/vdpa.c:348 vhost_vdpa_set_vring_kick() error: uninitialized 
symbol 'r'.

# 
https://github.com/0day-ci/linux/commit/a84ddbf1ef29f18aafb2bb8a93bcedd4a29a967d
git remote add linux-review https://github.com/0day-ci/linux
git remote update linux-review
git checkout a84ddbf1ef29f18aafb2bb8a93bcedd4a29a967d
vim +/r +348 drivers/vhost/vdpa.c

a84ddbf1ef29f1 Zhu Lingshan 2020-05-26  316  static long 
vhost_vdpa_set_vring_kick(struct vhost_virtqueue *vq,
a84ddbf1ef29f1 Zhu Lingshan 2020-05-26  317   
void __user *argp)
a84ddbf1ef29f1 Zhu Lingshan 2020-05-26  318  {
a84ddbf1ef29f1 Zhu Lingshan 2020-05-26  319 bool pollstart = false, 
pollstop = false;
a84ddbf1ef29f1 Zhu Lingshan 2020-05-26  320 struct file *eventfp, *filep = 
NULL;
a84ddbf1ef29f1 Zhu Lingshan 2020-05-26  321 struct vhost_vring_file f;
a84ddbf1ef29f1 Zhu Lingshan 2020-05-26  322 long r;
a84ddbf1ef29f1 Zhu Lingshan 2020-05-26  323  
a84ddbf1ef29f1 Zhu Lingshan 2020-05-26  324 if (copy_from_user(, argp, 
sizeof(f)))
a84ddbf1ef29f1 Zhu Lingshan 2020-05-26  325 return -EFAULT;
a84ddbf1ef29f1 Zhu Lingshan 2020-05-26  326  
a84ddbf1ef29f1 Zhu Lingshan 2020-05-26  327 eventfp = f.fd == -1 ? NULL : 
eventfd_fget(f.fd);
a84ddbf1ef29f1 Zhu Lingshan 2020-05-26  328 if (IS_ERR(eventfp)) {
a84ddbf1ef29f1 Zhu Lingshan 2020-05-26  329 r = PTR_ERR(eventfp);
a84ddbf1ef29f1 Zhu Lingshan 2020-05-26  330 return r;
a84ddbf1ef29f1 Zhu Lingshan 2020-05-26  331 }
a84ddbf1ef29f1 Zhu Lingshan 2020-05-26  332  
a84ddbf1ef29f1 Zhu Lingshan 2020-05-26  333 if (eventfp != vq->kick) {
a84ddbf1ef29f1 Zhu Lingshan 2020-05-26  334 pollstop = (filep = 
vq->kick) != NULL;
a84ddbf1ef29f1 Zhu Lingshan 2020-05-26  335 pollstart = (vq->kick = 
eventfp) != NULL;
a84ddbf1ef29f1 Zhu Lingshan 2020-05-26  336 } else
a84ddbf1ef29f1 Zhu Lingshan 2020-05-26  337 filep = eventfp;
a84ddbf1ef29f1 Zhu Lingshan 2020-05-26  338  
a84ddbf1ef29f1 Zhu Lingshan 2020-05-26  339 if (pollstop && vq->handle_kick)
a84ddbf1ef29f1 Zhu Lingshan 2020-05-26  340 
vhost_vdpa_poll_stop(vq);
a84ddbf1ef29f1 Zhu Lingshan 2020-05-26  341  
a84ddbf1ef29f1 Zhu Lingshan 2020-05-26  342 if (filep)
a84ddbf1ef29f1 Zhu Lingshan 2020-05-26  343 fput(filep);
a84ddbf1ef29f1 Zhu Lingshan 2020-05-26  344  
a84ddbf1ef29f1 Zhu Lingshan 2020-05-26  345 if (pollstart && 
vq->handle_kick)
a84ddbf1ef29f1 Zhu Lingshan 2020-05-26  346 r = 
vhost_vdpa_poll_start(vq);

"r" not initialized on else path.

a84ddbf1ef29f1 Zhu Lingshan 2020-05-26  347  
a84ddbf1ef29f1 Zhu Lingshan 2020-05-26 @348 return r;
a84ddbf1ef29f1 Zhu Lingshan 2020-05-26  349  }

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


.config.gz
Description: application/gzip
___
kbuild mailing list -- kbuild@lists.01.org
To unsubscribe send an email to kbuild-le...@lists.01.org


[kbuild] Re: [PATCH] vdpa: bypass waking up vhost_woker for vdpa vq kick

2020-05-29 Thread kbuild test robot
CC: kbuild-...@lists.01.org
In-Reply-To: <1590471145-4436-1-git-send-email-lingshan@intel.com>
References: <1590471145-4436-1-git-send-email-lingshan@intel.com>
TO: Zhu Lingshan 
TO: m...@redhat.com
TO: k...@vger.kernel.org
TO: virtualizat...@lists.linux-foundation.org
TO: linux-ker...@vger.kernel.org
TO: net...@vger.kernel.org
TO: jasow...@redhat.com
CC: l...@redhat.com
CC: dan.d...@intel.com
CC: cunming.li...@intel.com
CC: Zhu Lingshan 

Hi Zhu,

Thank you for the patch! Perhaps something to improve:

[auto build test WARNING on vhost/linux-next]
[also build test WARNING on v5.7-rc7 next-20200529]
[if your patch is applied to the wrong git tree, please drop us a note to help
improve the system. BTW, we also suggest to use '--base' option to specify the
base tree in git format-patch, please see https://stackoverflow.com/a/37406982]

url:
https://github.com/0day-ci/linux/commits/Zhu-Lingshan/vdpa-bypass-waking-up-vhost_woker-for-vdpa-vq-kick/20200526-133819
base:   https://git.kernel.org/pub/scm/linux/kernel/git/mst/vhost.git linux-next
:: branch date: 4 days ago
:: commit date: 4 days ago
config: x86_64-randconfig-m001-20200529 (attached as .config)
compiler: gcc-9 (Debian 9.3.0-13) 9.3.0

If you fix the issue, kindly add following tag as appropriate
Reported-by: kbuild test robot 
Reported-by: Dan Carpenter 

smatch warnings:
drivers/vhost/vdpa.c:348 vhost_vdpa_set_vring_kick() error: uninitialized 
symbol 'r'.

# 
https://github.com/0day-ci/linux/commit/a84ddbf1ef29f18aafb2bb8a93bcedd4a29a967d
git remote add linux-review https://github.com/0day-ci/linux
git remote update linux-review
git checkout a84ddbf1ef29f18aafb2bb8a93bcedd4a29a967d
vim +/r +348 drivers/vhost/vdpa.c

a84ddbf1ef29f1 Zhu Lingshan 2020-05-26  315  
a84ddbf1ef29f1 Zhu Lingshan 2020-05-26  316  static long 
vhost_vdpa_set_vring_kick(struct vhost_virtqueue *vq,
a84ddbf1ef29f1 Zhu Lingshan 2020-05-26  317   
void __user *argp)
a84ddbf1ef29f1 Zhu Lingshan 2020-05-26  318  {
a84ddbf1ef29f1 Zhu Lingshan 2020-05-26  319 bool pollstart = false, 
pollstop = false;
a84ddbf1ef29f1 Zhu Lingshan 2020-05-26  320 struct file *eventfp, *filep = 
NULL;
a84ddbf1ef29f1 Zhu Lingshan 2020-05-26  321 struct vhost_vring_file f;
a84ddbf1ef29f1 Zhu Lingshan 2020-05-26  322 long r;
a84ddbf1ef29f1 Zhu Lingshan 2020-05-26  323  
a84ddbf1ef29f1 Zhu Lingshan 2020-05-26  324 if (copy_from_user(, argp, 
sizeof(f)))
a84ddbf1ef29f1 Zhu Lingshan 2020-05-26  325 return -EFAULT;
a84ddbf1ef29f1 Zhu Lingshan 2020-05-26  326  
a84ddbf1ef29f1 Zhu Lingshan 2020-05-26  327 eventfp = f.fd == -1 ? NULL : 
eventfd_fget(f.fd);
a84ddbf1ef29f1 Zhu Lingshan 2020-05-26  328 if (IS_ERR(eventfp)) {
a84ddbf1ef29f1 Zhu Lingshan 2020-05-26  329 r = PTR_ERR(eventfp);
a84ddbf1ef29f1 Zhu Lingshan 2020-05-26  330 return r;
a84ddbf1ef29f1 Zhu Lingshan 2020-05-26  331 }
a84ddbf1ef29f1 Zhu Lingshan 2020-05-26  332  
a84ddbf1ef29f1 Zhu Lingshan 2020-05-26  333 if (eventfp != vq->kick) {
a84ddbf1ef29f1 Zhu Lingshan 2020-05-26  334 pollstop = (filep = 
vq->kick) != NULL;
a84ddbf1ef29f1 Zhu Lingshan 2020-05-26  335 pollstart = (vq->kick = 
eventfp) != NULL;
a84ddbf1ef29f1 Zhu Lingshan 2020-05-26  336 } else
a84ddbf1ef29f1 Zhu Lingshan 2020-05-26  337 filep = eventfp;
a84ddbf1ef29f1 Zhu Lingshan 2020-05-26  338  
a84ddbf1ef29f1 Zhu Lingshan 2020-05-26  339 if (pollstop && vq->handle_kick)
a84ddbf1ef29f1 Zhu Lingshan 2020-05-26  340 
vhost_vdpa_poll_stop(vq);
a84ddbf1ef29f1 Zhu Lingshan 2020-05-26  341  
a84ddbf1ef29f1 Zhu Lingshan 2020-05-26  342 if (filep)
a84ddbf1ef29f1 Zhu Lingshan 2020-05-26  343 fput(filep);
a84ddbf1ef29f1 Zhu Lingshan 2020-05-26  344  
a84ddbf1ef29f1 Zhu Lingshan 2020-05-26  345 if (pollstart && 
vq->handle_kick)
a84ddbf1ef29f1 Zhu Lingshan 2020-05-26  346 r = 
vhost_vdpa_poll_start(vq);
a84ddbf1ef29f1 Zhu Lingshan 2020-05-26  347  
a84ddbf1ef29f1 Zhu Lingshan 2020-05-26 @348 return r;
a84ddbf1ef29f1 Zhu Lingshan 2020-05-26  349  }
4c8cf31885f69e Tiwei Bie2020-03-26  350  

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


.config.gz
Description: application/gzip
___
kbuild mailing list -- kbuild@lists.01.org
To unsubscribe send an email to kbuild-le...@lists.01.org