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: link
Be 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.