From: kernel test robot <lkp@intel.com> To: Tony Krowiak <akrowiak@linux.ibm.com>, linux-s390@vger.kernel.org, linux-kernel@vger.kernel.org, kvm@vger.kernel.org Cc: kbuild-all@lists.01.org, freude@linux.ibm.com, borntraeger@de.ibm.com, cohuck@redhat.com, mjrosato@linux.ibm.com, pasic@linux.ibm.com, alex.williamson@redhat.com, kwankhede@nvidia.com Subject: Re: [PATCH v11 12/14] s390/vfio-ap: handle host AP config change notification Date: Fri, 23 Oct 2020 05:17:34 +0800 [thread overview] Message-ID: <202010230508.bSSV3BvM-lkp@intel.com> (raw) In-Reply-To: <20201022171209.19494-13-akrowiak@linux.ibm.com> [-- Attachment #1: Type: text/plain, Size: 5265 bytes --] Hi Tony, I love your patch! Perhaps something to improve: [auto build test WARNING on s390/features] [also build test WARNING on linus/master next-20201022] [cannot apply to kvms390/next linux/master v5.9] [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/Tony-Krowiak/s390-vfio-ap-dynamic-configuration-support/20201023-011543 base: https://git.kernel.org/pub/scm/linux/kernel/git/s390/linux.git features config: s390-allyesconfig (attached as .config) compiler: s390-linux-gcc (GCC) 9.3.0 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 # https://github.com/0day-ci/linux/commit/32786ef6d4ba3703d993a8894ea1d763785fd3a4 git remote add linux-review https://github.com/0day-ci/linux git fetch --no-tags linux-review Tony-Krowiak/s390-vfio-ap-dynamic-configuration-support/20201023-011543 git checkout 32786ef6d4ba3703d993a8894ea1d763785fd3a4 # save the attached .config to linux build tree COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=s390 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 >>): drivers/s390/crypto/vfio_ap_ops.c:1316:5: warning: no previous prototype for 'vfio_ap_mdev_reset_queue' [-Wmissing-prototypes] 1316 | int vfio_ap_mdev_reset_queue(unsigned int apid, unsigned int apqi, | ^~~~~~~~~~~~~~~~~~~~~~~~ drivers/s390/crypto/vfio_ap_ops.c:1568:6: warning: no previous prototype for 'vfio_ap_mdev_hot_unplug_queue' [-Wmissing-prototypes] 1568 | void vfio_ap_mdev_hot_unplug_queue(struct vfio_ap_queue *q) | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~ drivers/s390/crypto/vfio_ap_ops.c: In function 'vfio_ap_mdev_on_cfg_remove': >> drivers/s390/crypto/vfio_ap_ops.c:1777:7: warning: variable 'unassigned' set but not used [-Wunused-but-set-variable] 1777 | bool unassigned = false; | ^~~~~~~~~~ drivers/s390/crypto/vfio_ap_ops.c: At top level: >> drivers/s390/crypto/vfio_ap_ops.c:1813:6: warning: no previous prototype for 'vfio_ap_mdev_on_cfg_add' [-Wmissing-prototypes] 1813 | void vfio_ap_mdev_on_cfg_add(void) | ^~~~~~~~~~~~~~~~~~~~~~~ In file included from drivers/s390/crypto/vfio_ap_ops.c:11: In function 'memcpy', inlined from 'vfio_ap_mdev_unassign_apids' at drivers/s390/crypto/vfio_ap_ops.c:1655:3, inlined from 'vfio_ap_mdev_on_cfg_remove' at drivers/s390/crypto/vfio_ap_ops.c:1800:8, inlined from 'vfio_ap_on_cfg_changed' at drivers/s390/crypto/vfio_ap_ops.c:1836:2: include/linux/string.h:402:4: error: call to '__read_overflow2' declared with attribute error: detected read beyond size of object passed as 2nd parameter 402 | __read_overflow2(); | ^~~~~~~~~~~~~~~~~~ vim +/unassigned +1777 drivers/s390/crypto/vfio_ap_ops.c 1774 1775 static void vfio_ap_mdev_on_cfg_remove(void) 1776 { > 1777 bool unassigned = false; 1778 int ap_remove, aq_remove; 1779 struct ap_matrix_mdev *matrix_mdev; 1780 DECLARE_BITMAP(apid_rem, AP_DEVICES); 1781 DECLARE_BITMAP(apqi_rem, AP_DOMAINS); 1782 unsigned long *cur_apm, *cur_aqm, *prev_apm, *prev_aqm; 1783 1784 cur_apm = (unsigned long *)matrix_dev->config_info.apm; 1785 cur_aqm = (unsigned long *)matrix_dev->config_info.aqm; 1786 prev_apm = (unsigned long *)matrix_dev->config_info_prev.apm; 1787 prev_aqm = (unsigned long *)matrix_dev->config_info_prev.aqm; 1788 1789 ap_remove = bitmap_andnot(apid_rem, prev_apm, cur_apm, AP_DEVICES); 1790 aq_remove = bitmap_andnot(apqi_rem, prev_aqm, cur_aqm, AP_DOMAINS); 1791 1792 if (!ap_remove && !aq_remove) 1793 return; 1794 1795 list_for_each_entry(matrix_mdev, &matrix_dev->mdev_list, node) { 1796 if (!vfio_ap_mdev_has_crycb(matrix_mdev)) 1797 continue; 1798 1799 if (ap_remove) { 1800 if (vfio_ap_mdev_unassign_apids(matrix_mdev, apid_rem)) 1801 unassigned = true; 1802 vfio_ap_mdev_unlink_apids(matrix_mdev, apid_rem); 1803 } 1804 1805 if (aq_remove) { 1806 if (vfio_ap_mdev_unassign_apqis(matrix_mdev, apqi_rem)) 1807 unassigned = true; 1808 vfio_ap_mdev_unlink_apqis(matrix_mdev, apqi_rem); 1809 } 1810 } 1811 } 1812 > 1813 void vfio_ap_mdev_on_cfg_add(void) 1814 { 1815 unsigned long *cur_apm, *cur_aqm, *prev_apm, *prev_aqm; 1816 1817 cur_apm = (unsigned long *)matrix_dev->config_info.apm; 1818 cur_aqm = (unsigned long *)matrix_dev->config_info.aqm; 1819 1820 prev_apm = (unsigned long *)matrix_dev->config_info_prev.apm; 1821 prev_aqm = (unsigned long *)matrix_dev->config_info_prev.aqm; 1822 1823 bitmap_andnot(matrix_dev->ap_add, cur_apm, prev_apm, AP_DEVICES); 1824 bitmap_andnot(matrix_dev->aq_add, cur_aqm, prev_aqm, AP_DOMAINS); 1825 } 1826 --- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org [-- Attachment #2: .config.gz --] [-- Type: application/gzip, Size: 63274 bytes --]
WARNING: multiple messages have this Message-ID (diff)
From: kernel test robot <lkp@intel.com> To: kbuild-all@lists.01.org Subject: Re: [PATCH v11 12/14] s390/vfio-ap: handle host AP config change notification Date: Fri, 23 Oct 2020 05:17:34 +0800 [thread overview] Message-ID: <202010230508.bSSV3BvM-lkp@intel.com> (raw) In-Reply-To: <20201022171209.19494-13-akrowiak@linux.ibm.com> [-- Attachment #1: Type: text/plain, Size: 5379 bytes --] Hi Tony, I love your patch! Perhaps something to improve: [auto build test WARNING on s390/features] [also build test WARNING on linus/master next-20201022] [cannot apply to kvms390/next linux/master v5.9] [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/Tony-Krowiak/s390-vfio-ap-dynamic-configuration-support/20201023-011543 base: https://git.kernel.org/pub/scm/linux/kernel/git/s390/linux.git features config: s390-allyesconfig (attached as .config) compiler: s390-linux-gcc (GCC) 9.3.0 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 # https://github.com/0day-ci/linux/commit/32786ef6d4ba3703d993a8894ea1d763785fd3a4 git remote add linux-review https://github.com/0day-ci/linux git fetch --no-tags linux-review Tony-Krowiak/s390-vfio-ap-dynamic-configuration-support/20201023-011543 git checkout 32786ef6d4ba3703d993a8894ea1d763785fd3a4 # save the attached .config to linux build tree COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=s390 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 >>): drivers/s390/crypto/vfio_ap_ops.c:1316:5: warning: no previous prototype for 'vfio_ap_mdev_reset_queue' [-Wmissing-prototypes] 1316 | int vfio_ap_mdev_reset_queue(unsigned int apid, unsigned int apqi, | ^~~~~~~~~~~~~~~~~~~~~~~~ drivers/s390/crypto/vfio_ap_ops.c:1568:6: warning: no previous prototype for 'vfio_ap_mdev_hot_unplug_queue' [-Wmissing-prototypes] 1568 | void vfio_ap_mdev_hot_unplug_queue(struct vfio_ap_queue *q) | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~ drivers/s390/crypto/vfio_ap_ops.c: In function 'vfio_ap_mdev_on_cfg_remove': >> drivers/s390/crypto/vfio_ap_ops.c:1777:7: warning: variable 'unassigned' set but not used [-Wunused-but-set-variable] 1777 | bool unassigned = false; | ^~~~~~~~~~ drivers/s390/crypto/vfio_ap_ops.c: At top level: >> drivers/s390/crypto/vfio_ap_ops.c:1813:6: warning: no previous prototype for 'vfio_ap_mdev_on_cfg_add' [-Wmissing-prototypes] 1813 | void vfio_ap_mdev_on_cfg_add(void) | ^~~~~~~~~~~~~~~~~~~~~~~ In file included from drivers/s390/crypto/vfio_ap_ops.c:11: In function 'memcpy', inlined from 'vfio_ap_mdev_unassign_apids' at drivers/s390/crypto/vfio_ap_ops.c:1655:3, inlined from 'vfio_ap_mdev_on_cfg_remove' at drivers/s390/crypto/vfio_ap_ops.c:1800:8, inlined from 'vfio_ap_on_cfg_changed' at drivers/s390/crypto/vfio_ap_ops.c:1836:2: include/linux/string.h:402:4: error: call to '__read_overflow2' declared with attribute error: detected read beyond size of object passed as 2nd parameter 402 | __read_overflow2(); | ^~~~~~~~~~~~~~~~~~ vim +/unassigned +1777 drivers/s390/crypto/vfio_ap_ops.c 1774 1775 static void vfio_ap_mdev_on_cfg_remove(void) 1776 { > 1777 bool unassigned = false; 1778 int ap_remove, aq_remove; 1779 struct ap_matrix_mdev *matrix_mdev; 1780 DECLARE_BITMAP(apid_rem, AP_DEVICES); 1781 DECLARE_BITMAP(apqi_rem, AP_DOMAINS); 1782 unsigned long *cur_apm, *cur_aqm, *prev_apm, *prev_aqm; 1783 1784 cur_apm = (unsigned long *)matrix_dev->config_info.apm; 1785 cur_aqm = (unsigned long *)matrix_dev->config_info.aqm; 1786 prev_apm = (unsigned long *)matrix_dev->config_info_prev.apm; 1787 prev_aqm = (unsigned long *)matrix_dev->config_info_prev.aqm; 1788 1789 ap_remove = bitmap_andnot(apid_rem, prev_apm, cur_apm, AP_DEVICES); 1790 aq_remove = bitmap_andnot(apqi_rem, prev_aqm, cur_aqm, AP_DOMAINS); 1791 1792 if (!ap_remove && !aq_remove) 1793 return; 1794 1795 list_for_each_entry(matrix_mdev, &matrix_dev->mdev_list, node) { 1796 if (!vfio_ap_mdev_has_crycb(matrix_mdev)) 1797 continue; 1798 1799 if (ap_remove) { 1800 if (vfio_ap_mdev_unassign_apids(matrix_mdev, apid_rem)) 1801 unassigned = true; 1802 vfio_ap_mdev_unlink_apids(matrix_mdev, apid_rem); 1803 } 1804 1805 if (aq_remove) { 1806 if (vfio_ap_mdev_unassign_apqis(matrix_mdev, apqi_rem)) 1807 unassigned = true; 1808 vfio_ap_mdev_unlink_apqis(matrix_mdev, apqi_rem); 1809 } 1810 } 1811 } 1812 > 1813 void vfio_ap_mdev_on_cfg_add(void) 1814 { 1815 unsigned long *cur_apm, *cur_aqm, *prev_apm, *prev_aqm; 1816 1817 cur_apm = (unsigned long *)matrix_dev->config_info.apm; 1818 cur_aqm = (unsigned long *)matrix_dev->config_info.aqm; 1819 1820 prev_apm = (unsigned long *)matrix_dev->config_info_prev.apm; 1821 prev_aqm = (unsigned long *)matrix_dev->config_info_prev.aqm; 1822 1823 bitmap_andnot(matrix_dev->ap_add, cur_apm, prev_apm, AP_DEVICES); 1824 bitmap_andnot(matrix_dev->aq_add, cur_aqm, prev_aqm, AP_DOMAINS); 1825 } 1826 --- 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: 63274 bytes --]
next prev parent reply other threads:[~2020-10-22 21:17 UTC|newest] Thread overview: 73+ messages / expand[flat|nested] mbox.gz Atom feed top 2020-10-22 17:11 [PATCH v11 00/14] s390/vfio-ap: dynamic configuration support Tony Krowiak 2020-10-22 17:11 ` [PATCH v11 01/14] s390/vfio-ap: No need to disable IRQ after queue reset Tony Krowiak 2020-10-22 19:44 ` kernel test robot 2020-10-22 19:44 ` kernel test robot 2020-10-26 16:57 ` Tony Krowiak 2020-10-27 6:48 ` Halil Pasic 2020-10-29 23:29 ` Tony Krowiak 2020-10-30 16:13 ` Tony Krowiak 2020-10-30 17:27 ` Halil Pasic 2020-10-30 20:45 ` Tony Krowiak 2020-10-30 17:42 ` Halil Pasic 2020-10-30 20:37 ` Tony Krowiak 2020-10-31 3:43 ` Halil Pasic 2020-11-02 14:35 ` Tony Krowiak 2020-10-30 17:54 ` Halil Pasic 2020-10-30 20:53 ` Tony Krowiak 2020-10-30 21:13 ` Tony Krowiak 2020-10-30 17:56 ` Halil Pasic 2020-10-30 21:17 ` Tony Krowiak 2020-10-22 17:11 ` [PATCH v11 02/14] 390/vfio-ap: use new AP bus interface to search for queue devices Tony Krowiak 2020-10-27 7:01 ` Halil Pasic 2020-11-02 21:57 ` Tony Krowiak 2020-10-22 17:11 ` [PATCH v11 03/14] s390/vfio-ap: manage link between queue struct and matrix mdev Tony Krowiak 2020-10-27 9:33 ` Halil Pasic 2020-10-22 17:11 ` [PATCH v11 04/14] s390/zcrypt: driver callback to indicate resource in use Tony Krowiak 2020-10-27 13:01 ` Halil Pasic 2020-10-27 16:55 ` Harald Freudenberger 2020-11-13 21:30 ` Tony Krowiak 2020-11-14 0:00 ` Halil Pasic 2020-11-16 16:23 ` Tony Krowiak 2020-10-22 17:12 ` [PATCH v11 05/14] s390/vfio-ap: implement in-use callback for vfio_ap driver Tony Krowiak 2020-10-27 13:27 ` Halil Pasic 2020-11-13 17:14 ` Tony Krowiak 2020-11-13 23:47 ` Halil Pasic 2020-11-16 16:58 ` Tony Krowiak 2020-11-23 17:03 ` Cornelia Huck 2020-11-23 19:23 ` Tony Krowiak 2020-10-22 17:12 ` [PATCH v11 06/14] s390/vfio-ap: introduce shadow APCB Tony Krowiak 2020-10-28 8:11 ` Halil Pasic 2020-11-13 17:18 ` Tony Krowiak 2020-10-22 17:12 ` [PATCH v11 07/14] s390/vfio-ap: sysfs attribute to display the guest's matrix Tony Krowiak 2020-10-28 8:17 ` Halil Pasic 2020-11-13 17:27 ` Tony Krowiak 2020-11-13 23:12 ` Halil Pasic 2020-11-19 18:15 ` Tony Krowiak 2020-10-22 17:12 ` [PATCH v11 08/14] s390/vfio-ap: hot plug/unplug queues on bind/unbind of queue device Tony Krowiak 2020-10-22 20:30 ` kernel test robot 2020-10-22 20:30 ` kernel test robot 2020-10-26 17:04 ` Tony Krowiak 2020-10-28 13:57 ` Halil Pasic 2020-11-03 22:49 ` Tony Krowiak 2020-11-04 12:52 ` Halil Pasic 2020-11-04 21:20 ` Tony Krowiak 2020-11-05 12:27 ` Halil Pasic 2020-11-13 20:36 ` Tony Krowiak 2020-11-04 13:23 ` Halil Pasic 2020-10-22 17:12 ` [PATCH v11 09/14] s390/vfio-ap: allow assignment of unavailable AP queues to mdev device Tony Krowiak 2020-10-28 15:03 ` Halil Pasic 2020-10-22 17:12 ` [PATCH v11 10/14] s390/vfio-ap: allow hot plug/unplug of AP resources using " Tony Krowiak 2020-10-22 17:12 ` [PATCH v11 11/14] s390/zcrypt: Notify driver on config changed and scan complete callbacks Tony Krowiak 2020-10-27 17:28 ` Harald Freudenberger 2020-11-13 20:58 ` Tony Krowiak 2020-10-22 17:12 ` [PATCH v11 12/14] s390/vfio-ap: handle host AP config change notification Tony Krowiak 2020-10-22 21:17 ` kernel test robot [this message] 2020-10-22 21:17 ` kernel test robot 2020-10-26 17:07 ` Tony Krowiak 2020-10-26 17:21 ` Tony Krowiak 2020-11-03 9:48 ` kernel test robot 2020-11-03 9:48 ` kernel test robot 2020-11-13 21:06 ` Tony Krowiak 2020-11-13 21:06 ` Tony Krowiak 2020-10-22 17:12 ` [PATCH v11 13/14] s390/vfio-ap: handle AP bus scan completed notification Tony Krowiak 2020-10-22 17:12 ` [PATCH v11 14/14] s390/vfio-ap: update docs to include dynamic config support Tony Krowiak
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=202010230508.bSSV3BvM-lkp@intel.com \ --to=lkp@intel.com \ --cc=akrowiak@linux.ibm.com \ --cc=alex.williamson@redhat.com \ --cc=borntraeger@de.ibm.com \ --cc=cohuck@redhat.com \ --cc=freude@linux.ibm.com \ --cc=kbuild-all@lists.01.org \ --cc=kvm@vger.kernel.org \ --cc=kwankhede@nvidia.com \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-s390@vger.kernel.org \ --cc=mjrosato@linux.ibm.com \ --cc=pasic@linux.ibm.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.