[sashal-linux-stable:queue-4.14 37/79] Warning: ffffffff819d751e: 0f ff 48 8d ud0 -0x73(%rax),%ecx
by kbuild test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/sashal/linux-stable.git queue-4.14
head: 7a7ddea1b419e20931152e388252c661b6fca95c
commit: 05d5cdee151724e77ea0953accd654a2a4eb0ee4 [37/79] net: ipv6: add net argument to ip6_dst_lookup_flow
config: x86_64-randconfig-a013-20200517 (attached as .config)
compiler: clang version 11.0.0 (https://github.com/llvm/llvm-project 135b877874fae96b4372c8a3fbfaa8ff44ff86e3)
reproduce:
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
git checkout 05d5cdee151724e77ea0953accd654a2a4eb0ee4
# 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: kbuild test robot <lkp(a)intel.com>
All warnings (new ones prefixed by >>, old ones prefixed by <<):
>> Warning: ffffffff819d751e: 0f ff 48 8d ud0 -0x73(%rax),%ecx
Warning: objdump says 4 bytes, but insn_get_length() says 2
Warning: arch/x86/tools/test_get_len found difference at <memcpy_page_flushcache>:ffffffff819d6c30
--
>> Warning: ffffffff819d76a2: 0f ff 48 8d ud0 -0x73(%rax),%ecx
Warning: objdump says 4 bytes, but insn_get_length() says 2
Warning: arch/x86/tools/test_get_len found difference at <memcpy_page_flushcache>:ffffffff819d6c30
--
>> Warning: ffffffff819d77d3: 0f ff 48 8d ud0 -0x73(%rax),%ecx
Warning: objdump says 4 bytes, but insn_get_length() says 2
Warning: arch/x86/tools/test_get_len found difference at <memcpy_page_flushcache>:ffffffff819d6c30
--
>> Warning: ffffffff819d78cc: 0f ff 48 8d ud0 -0x73(%rax),%ecx
Warning: objdump says 4 bytes, but insn_get_length() says 2
Warning: arch/x86/tools/test_get_len found difference at <memcpy_page_flushcache>:ffffffff819d6c30
--
>> Warning: ffffffff819d7aa0: 0f ff 49 8d ud0 -0x73(%rcx),%ecx
Warning: objdump says 4 bytes, but insn_get_length() says 2
Warning: arch/x86/tools/test_get_len found difference at <memcpy_page_flushcache>:ffffffff819d6c30
--
>> Warning: ffffffff819d6efd: 0f ff 48 8d ud0 -0x73(%rax),%ecx
Warning: objdump says 4 bytes, but insn_get_length() says 2
Warning: arch/x86/tools/test_get_len found difference at <memcpy_page_flushcache>:ffffffff819d6c30
--
>> Warning: ffffffff819d823e: 0f ff 48 8d ud0 -0x73(%rax),%ecx
Warning: objdump says 4 bytes, but insn_get_length() says 2
Warning: arch/x86/tools/test_get_len found difference at <memcpy_page_flushcache>:ffffffff819d6c30
--
>> Warning: ffffffff819d8522: 0f ff 49 8d ud0 -0x73(%rcx),%ecx
Warning: objdump says 4 bytes, but insn_get_length() says 2
Warning: arch/x86/tools/test_get_len found difference at <memcpy_page_flushcache>:ffffffff819d6c30
--
>> Warning: ffffffff819d8731: 0f ff 48 8d ud0 -0x73(%rax),%ecx
Warning: objdump says 4 bytes, but insn_get_length() says 2
Warning: arch/x86/tools/test_get_len found difference at <memcpy_page_flushcache>:ffffffff819d6c30
--
>> Warning: ffffffff819d73fe: 0f ff 48 8d ud0 -0x73(%rax),%ecx
Warning: objdump says 4 bytes, but insn_get_length() says 2
Warning: arch/x86/tools/test_get_len found difference at <memcpy_page_flushcache>:ffffffff819d6c30
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
2 years, 4 months
Re: [PATCH 2/2] mfd: mfd_cell: constify platform_data
by kbuild test robot
Hi Tomas,
I love your patch! Yet something to improve:
[auto build test ERROR on ljones-mfd/for-mfd-next]
[also build test ERROR on v5.7-rc5 next-20200515]
[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/Tomas-Winkler/mfd-constify-point...
base: https://git.kernel.org/pub/scm/linux/kernel/git/lee/mfd.git for-mfd-next
config: arm-randconfig-r006-20200515 (attached as .config)
compiler: clang version 11.0.0 (https://github.com/llvm/llvm-project 13d44b2a0c7ef404b13b16644765977cd5310fe2)
reproduce:
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# install arm cross compiling tool for clang build
# apt-get install binutils-arm-linux-gnueabi
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=arm
If you fix the issue, kindly add following tag as appropriate
Reported-by: kbuild test robot <lkp(a)intel.com>
All errors (new ones prefixed by >>, old ones prefixed by <<):
>> drivers/hid/hid-sensor-hub.c:738:33: error: initializing 'struct hid_sensor_hub_device *' with an expression of type 'const void *' discards qualifiers [-Werror,-Wincompatible-pointer-types-discards-qualifiers]
struct hid_sensor_hub_device *hsdev =
^
1 error generated.
vim +738 drivers/hid/hid-sensor-hub.c
401ca24fb34aee0 srinivas pandruvada 2012-09-05 726
401ca24fb34aee0 srinivas pandruvada 2012-09-05 727 static void sensor_hub_remove(struct hid_device *hdev)
401ca24fb34aee0 srinivas pandruvada 2012-09-05 728 {
401ca24fb34aee0 srinivas pandruvada 2012-09-05 729 struct sensor_hub_data *data = hid_get_drvdata(hdev);
401ca24fb34aee0 srinivas pandruvada 2012-09-05 730 unsigned long flags;
e651a1da442ae02 Srinivas Pandruvada 2015-02-19 731 int i;
401ca24fb34aee0 srinivas pandruvada 2012-09-05 732
401ca24fb34aee0 srinivas pandruvada 2012-09-05 733 hid_dbg(hdev, " hardware removed\n");
401ca24fb34aee0 srinivas pandruvada 2012-09-05 734 hid_hw_close(hdev);
f2f13a68c37c13a Axel Lin 2012-09-19 735 hid_hw_stop(hdev);
401ca24fb34aee0 srinivas pandruvada 2012-09-05 736 spin_lock_irqsave(&data->lock, flags);
e651a1da442ae02 Srinivas Pandruvada 2015-02-19 737 for (i = 0; i < data->hid_sensor_client_cnt; ++i) {
e651a1da442ae02 Srinivas Pandruvada 2015-02-19 @738 struct hid_sensor_hub_device *hsdev =
e651a1da442ae02 Srinivas Pandruvada 2015-02-19 739 data->hid_sensor_hub_client_devs[i].platform_data;
e651a1da442ae02 Srinivas Pandruvada 2015-02-19 740 if (hsdev->pending.status)
e651a1da442ae02 Srinivas Pandruvada 2015-02-19 741 complete(&hsdev->pending.ready);
e651a1da442ae02 Srinivas Pandruvada 2015-02-19 742 }
401ca24fb34aee0 srinivas pandruvada 2012-09-05 743 spin_unlock_irqrestore(&data->lock, flags);
401ca24fb34aee0 srinivas pandruvada 2012-09-05 744 mfd_remove_devices(&hdev->dev);
401ca24fb34aee0 srinivas pandruvada 2012-09-05 745 mutex_destroy(&data->mutex);
401ca24fb34aee0 srinivas pandruvada 2012-09-05 746 }
401ca24fb34aee0 srinivas pandruvada 2012-09-05 747
:::::: The code at line 738 was first introduced by commit
:::::: e651a1da442ae02a50081e38309dea5e89da2d41 HID: hid-sensor-hub: Allow parallel synchronous reads
:::::: TO: Srinivas Pandruvada <srinivas.pandruvada(a)linux.intel.com>
:::::: CC: Jiri Kosina <jkosina(a)suse.cz>
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
2 years, 4 months
[leon-rdma:rdma-next 9/36] drivers/infiniband/ulp/rtrs/rtrs-srv.c:1579:8: error: too few arguments to function 'rdma_reject'
by kbuild test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/leon/linux-rdma.git rdma-next
head: b108f2e57cf8856283e0f6600ae5365d18c460f8
commit: bf1591a580210cdbfbba57f2dd5bbd1adb5e16f6 [9/36] RDMA/cma: Provide ECE reject reason
config: arm-allyesconfig (attached as .config)
compiler: arm-linux-gnueabi-gcc (GCC) 9.3.0
reproduce:
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
git checkout bf1591a580210cdbfbba57f2dd5bbd1adb5e16f6
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=arm
If you fix the issue, kindly add following tag as appropriate
Reported-by: kbuild test robot <lkp(a)intel.com>
All errors (new ones prefixed by >>, old ones prefixed by <<):
drivers/infiniband/ulp/rtrs/rtrs-srv.c: In function 'rtrs_rdma_do_reject':
>> drivers/infiniband/ulp/rtrs/rtrs-srv.c:1579:8: error: too few arguments to function 'rdma_reject'
1579 | err = rdma_reject(cm_id, &msg, sizeof(msg));
| ^~~~~~~~~~~
In file included from drivers/infiniband/ulp/rtrs/rtrs-pri.h:14,
from drivers/infiniband/ulp/rtrs/rtrs-srv.h:15,
from drivers/infiniband/ulp/rtrs/rtrs-srv.c:16:
include/rdma/rdma_cm.h:331:5: note: declared here
331 | int rdma_reject(struct rdma_cm_id *id, const void *private_data,
| ^~~~~~~~~~~
vim +/rdma_reject +1579 drivers/infiniband/ulp/rtrs/rtrs-srv.c
9cb837480424e7 Jack Wang 2020-05-11 1567
9cb837480424e7 Jack Wang 2020-05-11 1568 static int rtrs_rdma_do_reject(struct rdma_cm_id *cm_id, int errno)
9cb837480424e7 Jack Wang 2020-05-11 1569 {
9cb837480424e7 Jack Wang 2020-05-11 1570 struct rtrs_msg_conn_rsp msg;
9cb837480424e7 Jack Wang 2020-05-11 1571 int err;
9cb837480424e7 Jack Wang 2020-05-11 1572
9cb837480424e7 Jack Wang 2020-05-11 1573 msg = (struct rtrs_msg_conn_rsp) {
9cb837480424e7 Jack Wang 2020-05-11 1574 .magic = cpu_to_le16(RTRS_MAGIC),
9cb837480424e7 Jack Wang 2020-05-11 1575 .version = cpu_to_le16(RTRS_PROTO_VER),
9cb837480424e7 Jack Wang 2020-05-11 1576 .errno = cpu_to_le16(errno),
9cb837480424e7 Jack Wang 2020-05-11 1577 };
9cb837480424e7 Jack Wang 2020-05-11 1578
9cb837480424e7 Jack Wang 2020-05-11 @1579 err = rdma_reject(cm_id, &msg, sizeof(msg));
9cb837480424e7 Jack Wang 2020-05-11 1580 if (err)
9cb837480424e7 Jack Wang 2020-05-11 1581 pr_err("rdma_reject(), err: %d\n", err);
9cb837480424e7 Jack Wang 2020-05-11 1582
9cb837480424e7 Jack Wang 2020-05-11 1583 /* Bounce errno back */
9cb837480424e7 Jack Wang 2020-05-11 1584 return errno;
9cb837480424e7 Jack Wang 2020-05-11 1585 }
9cb837480424e7 Jack Wang 2020-05-11 1586
:::::: The code at line 1579 was first introduced by commit
:::::: 9cb837480424e78ed585376f944088246685aec3 RDMA/rtrs: server: main functionality
:::::: TO: Jack Wang <jinpu.wang(a)cloud.ionos.com>
:::::: CC: Jason Gunthorpe <jgg(a)mellanox.com>
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
2 years, 4 months
[jkirsher-next-queue:dev-queue 7/115] drivers/net/ethernet/intel/ixgbe/ixgbe_main.c:2271:15: error: redefinition of 'truesize'
by kbuild test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/jkirsher/next-queue.git dev-queue
head: 1ae51690bc317eb011c6fcc8f5549b151b347167
commit: cc703d543a3b97138318bfe7d3c86cbf78136395 [7/115] ixgbe: fix XDP redirect on archs with PAGE_SIZE above 4K
config: powerpc-defconfig (attached as .config)
compiler: powerpc64-linux-gcc (GCC) 9.3.0
reproduce:
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
git checkout cc703d543a3b97138318bfe7d3c86cbf78136395
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day GCC_VERSION=9.3.0 make.cross ARCH=powerpc
If you fix the issue, kindly add following tag as appropriate
Reported-by: kbuild test robot <lkp(a)intel.com>
All errors (new ones prefixed by >>, old ones prefixed by <<):
drivers/net/ethernet/intel/ixgbe/ixgbe_main.c: In function 'ixgbe_rx_buffer_flip':
>> drivers/net/ethernet/intel/ixgbe/ixgbe_main.c:2271:15: error: redefinition of 'truesize'
2271 | unsigned int truesize = ring_uses_build_skb(rx_ring) ?
| ^~~~~~~~
drivers/net/ethernet/intel/ixgbe/ixgbe_main.c:2267:15: note: previous definition of 'truesize' was here
2267 | unsigned int truesize = ixgbe_rx_frame_truesize(rx_ring, size);
| ^~~~~~~~
vim +/truesize +2271 drivers/net/ethernet/intel/ixgbe/ixgbe_main.c
2262
2263 static void ixgbe_rx_buffer_flip(struct ixgbe_ring *rx_ring,
2264 struct ixgbe_rx_buffer *rx_buffer,
2265 unsigned int size)
2266 {
2267 unsigned int truesize = ixgbe_rx_frame_truesize(rx_ring, size);
2268 #if (PAGE_SIZE < 8192)
2269 rx_buffer->page_offset ^= truesize;
2270 #else
> 2271 unsigned int truesize = ring_uses_build_skb(rx_ring) ?
2272 SKB_DATA_ALIGN(IXGBE_SKB_PAD + size) +
2273 SKB_DATA_ALIGN(sizeof(struct skb_shared_info)) :
2274 SKB_DATA_ALIGN(size);
2275
2276 rx_buffer->page_offset += truesize;
2277 #endif
2278 }
2279
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
2 years, 4 months
[linux-next:master 6172/9662] drivers/video/fbdev/atmel_lcdfb.c:362:20: sparse: sparse: incorrect type in argument 1 (different address spaces)
by kbuild test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master
head: bdecf38f228bcca73b31ada98b5b7ba1215eb9c9
commit: bbd7ffdbef6888459f301c5889f3b14ada38b913 [6172/9662] clk: Allow the common clk framework to be selectable
config: ia64-randconfig-s001-20200518 (attached as .config)
reproduce:
# apt-get install sparse
# sparse version: v0.6.1-193-gb8fad4bc-dirty
git checkout bbd7ffdbef6888459f301c5889f3b14ada38b913
# save the attached .config to linux build tree
make C=1 ARCH=ia64 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__'
If you fix the issue, kindly add following tag as appropriate
Reported-by: kbuild test robot <lkp(a)intel.com>
sparse warnings: (new ones prefixed by >>)
drivers/video/fbdev/atmel_lcdfb.c:354:27: sparse: sparse: incorrect type in assignment (different address spaces) @@ expected char [noderef] <asn:2> *screen_base @@ got n:2> *screen_base @@
drivers/video/fbdev/atmel_lcdfb.c:354:27: sparse: expected char [noderef] <asn:2> *screen_base
drivers/video/fbdev/atmel_lcdfb.c:354:27: sparse: got void *
>> drivers/video/fbdev/atmel_lcdfb.c:362:20: sparse: sparse: incorrect type in argument 1 (different address spaces) @@ expected void * @@ got char [noderef] <asvoid * @@
>> drivers/video/fbdev/atmel_lcdfb.c:362:20: sparse: expected void *
drivers/video/fbdev/atmel_lcdfb.c:362:20: sparse: got char [noderef] <asn:2> *screen_base
drivers/video/fbdev/atmel_lcdfb.c:333:59: sparse: sparse: incorrect type in argument 3 (different address spaces) @@ expected void *cpu_addr @@ got char [noderef] <asvoid *cpu_addr @@
drivers/video/fbdev/atmel_lcdfb.c:333:59: sparse: expected void *cpu_addr
drivers/video/fbdev/atmel_lcdfb.c:333:59: sparse: got char [noderef] <asn:2> *screen_base
drivers/video/fbdev/atmel_lcdfb.c:333:59: sparse: sparse: incorrect type in argument 3 (different address spaces) @@ expected void *cpu_addr @@ got char [noderef] <asvoid *cpu_addr @@
drivers/video/fbdev/atmel_lcdfb.c:333:59: sparse: expected void *cpu_addr
drivers/video/fbdev/atmel_lcdfb.c:333:59: sparse: got char [noderef] <asn:2> *screen_base
vim +362 drivers/video/fbdev/atmel_lcdfb.c
14340586148e7c drivers/video/atmel_lcdfb.c Nicolas Ferre 2007-05-10 336
14340586148e7c drivers/video/atmel_lcdfb.c Nicolas Ferre 2007-05-10 337 /**
14340586148e7c drivers/video/atmel_lcdfb.c Nicolas Ferre 2007-05-10 338 * atmel_lcdfb_alloc_video_memory - Allocate framebuffer memory
14340586148e7c drivers/video/atmel_lcdfb.c Nicolas Ferre 2007-05-10 339 * @sinfo: the frame buffer to allocate memory for
1d01e83557105e drivers/video/atmel_lcdfb.c Krzysztof Helt 2009-07-08 340 *
1d01e83557105e drivers/video/atmel_lcdfb.c Krzysztof Helt 2009-07-08 341 * This function is called only from the atmel_lcdfb_probe()
1d01e83557105e drivers/video/atmel_lcdfb.c Krzysztof Helt 2009-07-08 342 * so no locking by fb_info->mm_lock around smem_len setting is needed.
14340586148e7c drivers/video/atmel_lcdfb.c Nicolas Ferre 2007-05-10 343 */
14340586148e7c drivers/video/atmel_lcdfb.c Nicolas Ferre 2007-05-10 344 static int atmel_lcdfb_alloc_video_memory(struct atmel_lcdfb_info *sinfo)
14340586148e7c drivers/video/atmel_lcdfb.c Nicolas Ferre 2007-05-10 345 {
14340586148e7c drivers/video/atmel_lcdfb.c Nicolas Ferre 2007-05-10 346 struct fb_info *info = sinfo->info;
14340586148e7c drivers/video/atmel_lcdfb.c Nicolas Ferre 2007-05-10 347 struct fb_var_screeninfo *var = &info->var;
ea757acad5a518 drivers/video/atmel_lcdfb.c Haavard Skinnemoen 2008-08-12 348 unsigned int smem_len;
14340586148e7c drivers/video/atmel_lcdfb.c Nicolas Ferre 2007-05-10 349
ea757acad5a518 drivers/video/atmel_lcdfb.c Haavard Skinnemoen 2008-08-12 350 smem_len = (var->xres_virtual * var->yres_virtual
14340586148e7c drivers/video/atmel_lcdfb.c Nicolas Ferre 2007-05-10 351 * ((var->bits_per_pixel + 7) / 8));
ea757acad5a518 drivers/video/atmel_lcdfb.c Haavard Skinnemoen 2008-08-12 352 info->fix.smem_len = max(smem_len, sinfo->smem_len);
14340586148e7c drivers/video/atmel_lcdfb.c Nicolas Ferre 2007-05-10 353
f6e45661f9be54 drivers/video/fbdev/atmel_lcdfb.c Luis R. Rodriguez 2016-01-22 354 info->screen_base = dma_alloc_wc(info->device, info->fix.smem_len,
f6e45661f9be54 drivers/video/fbdev/atmel_lcdfb.c Luis R. Rodriguez 2016-01-22 355 (dma_addr_t *)&info->fix.smem_start,
f6e45661f9be54 drivers/video/fbdev/atmel_lcdfb.c Luis R. Rodriguez 2016-01-22 356 GFP_KERNEL);
14340586148e7c drivers/video/atmel_lcdfb.c Nicolas Ferre 2007-05-10 357
14340586148e7c drivers/video/atmel_lcdfb.c Nicolas Ferre 2007-05-10 358 if (!info->screen_base) {
14340586148e7c drivers/video/atmel_lcdfb.c Nicolas Ferre 2007-05-10 359 return -ENOMEM;
14340586148e7c drivers/video/atmel_lcdfb.c Nicolas Ferre 2007-05-10 360 }
14340586148e7c drivers/video/atmel_lcdfb.c Nicolas Ferre 2007-05-10 361
01d3a5e7fab773 drivers/video/atmel_lcdfb.c Haavard Skinnemoen 2008-04-28 @362 memset(info->screen_base, 0, info->fix.smem_len);
01d3a5e7fab773 drivers/video/atmel_lcdfb.c Haavard Skinnemoen 2008-04-28 363
14340586148e7c drivers/video/atmel_lcdfb.c Nicolas Ferre 2007-05-10 364 return 0;
14340586148e7c drivers/video/atmel_lcdfb.c Nicolas Ferre 2007-05-10 365 }
14340586148e7c drivers/video/atmel_lcdfb.c Nicolas Ferre 2007-05-10 366
:::::: The code at line 362 was first introduced by commit
:::::: 01d3a5e7fab7732cfc5d5d4533e9378ea435295a atmel_lcdfb: don't initialize a pre-allocated framebuffer
:::::: TO: Haavard Skinnemoen <hskinnemoen(a)atmel.com>
:::::: CC: Linus Torvalds <torvalds(a)linux-foundation.org>
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
2 years, 4 months
Re: [PATCH 1/2] opp: Reorder the code for !target_freq case
by kbuild test robot
Hi Viresh,
I love your patch! Perhaps something to improve:
[auto build test WARNING on linus/master]
[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/Viresh-Kumar/opp-core-add-regula...
base: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git 1ae7efb388540adc1653a51a3bc3b2c9cef5ec1a
If you fix the issue, kindly add following tag as appropriate
Reported-by: kbuild test robot <lkp(a)intel.com>
cppcheck warnings: (new ones prefixed by >>)
>> drivers/opp/core.c:833:7: warning: Variable 'ret' is reassigned a value before the old one has been used. [redundantAssignment]
ret = _set_required_opps(dev, opp_table, NULL);
^
drivers/opp/core.c:830:8: note: ret is assigned
ret = -EINVAL;
^
drivers/opp/core.c:833:7: note: ret is overwritten
ret = _set_required_opps(dev, opp_table, NULL);
^
vim +/ret +833 drivers/opp/core.c
793
794 /**
795 * dev_pm_opp_set_rate() - Configure new OPP based on frequency
796 * @dev: device for which we do this operation
797 * @target_freq: frequency to achieve
798 *
799 * This configures the power-supplies to the levels specified by the OPP
800 * corresponding to the target_freq, and programs the clock to a value <=
801 * target_freq, as rounded by clk_round_rate(). Device wanting to run at fmax
802 * provided by the opp, should have already rounded to the target OPP's
803 * frequency.
804 */
805 int dev_pm_opp_set_rate(struct device *dev, unsigned long target_freq)
806 {
807 struct opp_table *opp_table;
808 unsigned long freq, old_freq, temp_freq;
809 struct dev_pm_opp *old_opp, *opp;
810 struct clk *clk;
811 int ret;
812
813 opp_table = _find_opp_table(dev);
814 if (IS_ERR(opp_table)) {
815 dev_err(dev, "%s: device opp doesn't exist\n", __func__);
816 return PTR_ERR(opp_table);
817 }
818
819 if (unlikely(!target_freq)) {
820 /*
821 * Some drivers need to support cases where some platforms may
822 * have OPP table for the device, while others don't and
823 * opp_set_rate() just needs to behave like clk_set_rate().
824 */
825 if (!_get_opp_count(opp_table))
826 return 0;
827
828 if (!opp_table->required_opp_tables) {
829 dev_err(dev, "target frequency can't be 0\n");
830 ret = -EINVAL;
831 }
832
> 833 ret = _set_required_opps(dev, opp_table, NULL);
834 goto put_opp_table;
835 }
836
837 clk = opp_table->clk;
838 if (IS_ERR(clk)) {
839 dev_err(dev, "%s: No clock available for the device\n",
840 __func__);
841 ret = PTR_ERR(clk);
842 goto put_opp_table;
843 }
844
845 freq = clk_round_rate(clk, target_freq);
846 if ((long)freq <= 0)
847 freq = target_freq;
848
849 old_freq = clk_get_rate(clk);
850
851 /* Return early if nothing to do */
852 if (old_freq == freq) {
853 dev_dbg(dev, "%s: old/new frequencies (%lu Hz) are same, nothing to do\n",
854 __func__, freq);
855 ret = 0;
856 goto put_opp_table;
857 }
858
859 /*
860 * For IO devices which require an OPP on some platforms/SoCs
861 * while just needing to scale the clock on some others
862 * we look for empty OPP tables with just a clock handle and
863 * scale only the clk. This makes dev_pm_opp_set_rate()
864 * equivalent to a clk_set_rate()
865 */
866 if (!_get_opp_count(opp_table)) {
867 ret = _generic_set_opp_clk_only(dev, clk, freq);
868 goto put_opp_table;
869 }
870
871 temp_freq = old_freq;
872 old_opp = _find_freq_ceil(opp_table, &temp_freq);
873 if (IS_ERR(old_opp)) {
874 dev_err(dev, "%s: failed to find current OPP for freq %lu (%ld)\n",
875 __func__, old_freq, PTR_ERR(old_opp));
876 }
877
878 temp_freq = freq;
879 opp = _find_freq_ceil(opp_table, &temp_freq);
880 if (IS_ERR(opp)) {
881 ret = PTR_ERR(opp);
882 dev_err(dev, "%s: failed to find OPP for freq %lu (%d)\n",
883 __func__, freq, ret);
884 goto put_old_opp;
885 }
886
887 dev_dbg(dev, "%s: switching OPP: %lu Hz --> %lu Hz\n", __func__,
888 old_freq, freq);
889
890 /* Scaling up? Configure required OPPs before frequency */
891 if (freq >= old_freq) {
892 ret = _set_required_opps(dev, opp_table, opp);
893 if (ret)
894 goto put_opp;
895 }
896
897 if (opp_table->set_opp) {
898 ret = _set_opp_custom(opp_table, dev, old_freq, freq,
899 IS_ERR(old_opp) ? NULL : old_opp->supplies,
900 opp->supplies);
901 } else if (opp_table->regulators) {
902 ret = _generic_set_opp_regulator(opp_table, dev, old_freq, freq,
903 IS_ERR(old_opp) ? NULL : old_opp->supplies,
904 opp->supplies);
905 } else {
906 /* Only frequency scaling */
907 ret = _generic_set_opp_clk_only(dev, clk, freq);
908 }
909
910 /* Scaling down? Configure required OPPs after frequency */
911 if (!ret && freq < old_freq) {
912 ret = _set_required_opps(dev, opp_table, opp);
913 if (ret)
914 dev_err(dev, "Failed to set required opps: %d\n", ret);
915 }
916
917 put_opp:
918 dev_pm_opp_put(opp);
919 put_old_opp:
920 if (!IS_ERR(old_opp))
921 dev_pm_opp_put(old_opp);
922 put_opp_table:
923 dev_pm_opp_put_opp_table(opp_table);
924 return ret;
925 }
926 EXPORT_SYMBOL_GPL(dev_pm_opp_set_rate);
927
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
2 years, 4 months
Re: [RFC PATCH v4 09/10] i40e/vf_migration: register a migration vendor region
by kbuild test robot
Hi Yan,
[FYI, it's a private test report for your RFC patch.]
[auto build test ERROR on vfio/next]
[also build test ERROR on jkirsher-next-queue/dev-queue linus/master v5.7-rc6 next-20200515]
[cannot apply to linux/master]
[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/Yan-Zhao/Introduce-vendor-ops-in...
base: https://github.com/awilliam/linux-vfio.git next
config: i386-allyesconfig (attached as .config)
compiler: gcc-7 (Ubuntu 7.5.0-6ubuntu2) 7.5.0
reproduce:
# save the attached .config to linux build tree
make ARCH=i386
If you fix the issue, kindly add following tag as appropriate
Reported-by: kbuild test robot <lkp(a)intel.com>
All error/warnings (new ones prefixed by >>, old ones prefixed by <<):
drivers/net/ethernet/intel/i40e/i40e_vf_migration.c: In function 'i40e_vf_set_device_state':
>> drivers/net/ethernet/intel/i40e/i40e_vf_migration.c:132:22: error: dereferencing pointer to incomplete type 'struct vfio_device_migration_info'
if (state == mig_ctl->device_state)
^~
>> drivers/net/ethernet/intel/i40e/i40e_vf_migration.c:136:7: error: 'VFIO_DEVICE_STATE_RUNNING' undeclared (first use in this function); did you mean 'VFIO_EEH_PE_STATE_UNAVAIL'?
case VFIO_DEVICE_STATE_RUNNING:
^~~~~~~~~~~~~~~~~~~~~~~~~
VFIO_EEH_PE_STATE_UNAVAIL
drivers/net/ethernet/intel/i40e/i40e_vf_migration.c:136:7: note: each undeclared identifier is reported only once for each function it appears in
>> drivers/net/ethernet/intel/i40e/i40e_vf_migration.c:138:7: error: 'VFIO_DEVICE_STATE_SAVING' undeclared (first use in this function); did you mean 'VFIO_DEVICE_STATE_RUNNING'?
case VFIO_DEVICE_STATE_SAVING | VFIO_DEVICE_STATE_RUNNING:
^~~~~~~~~~~~~~~~~~~~~~~~
VFIO_DEVICE_STATE_RUNNING
>> drivers/net/ethernet/intel/i40e/i40e_vf_migration.c:144:7: error: 'VFIO_DEVICE_STATE_STOP' undeclared (first use in this function); did you mean 'VFIO_DEVICE_STATE_SAVING'?
case VFIO_DEVICE_STATE_STOP:
^~~~~~~~~~~~~~~~~~~~~~
VFIO_DEVICE_STATE_SAVING
>> drivers/net/ethernet/intel/i40e/i40e_vf_migration.c:149:7: error: 'VFIO_DEVICE_STATE_RESUMING' undeclared (first use in this function); did you mean 'VFIO_DEVICE_STATE_SAVING'?
case VFIO_DEVICE_STATE_RESUMING:
^~~~~~~~~~~~~~~~~~~~~~~~~~
VFIO_DEVICE_STATE_SAVING
In file included from <command-line>:0:0:
drivers/net/ethernet/intel/i40e/i40e_vf_migration.c: In function 'i40e_vf_region_migration_rw':
>> include/linux/compiler_types.h:129:35: error: invalid use of undefined type 'struct vfio_device_migration_info'
#define __compiler_offsetof(a, b) __builtin_offsetof(a, b)
^
include/linux/stddef.h:17:32: note: in expansion of macro '__compiler_offsetof'
#define offsetof(TYPE, MEMBER) __compiler_offsetof(TYPE, MEMBER)
^~~~~~~~~~~~~~~~~~~
>> drivers/net/ethernet/intel/i40e/i40e_vf_migration.c:166:23: note: in expansion of macro 'offsetof'
#define VDM_OFFSET(x) offsetof(struct vfio_device_migration_info, x)
^~~~~~~~
>> drivers/net/ethernet/intel/i40e/i40e_vf_migration.c:172:7: note: in expansion of macro 'VDM_OFFSET'
case VDM_OFFSET(device_state):
^~~~~~~~~~
>> include/linux/compiler_types.h:129:35: error: invalid use of undefined type 'struct vfio_device_migration_info'
#define __compiler_offsetof(a, b) __builtin_offsetof(a, b)
^
include/linux/stddef.h:17:32: note: in expansion of macro '__compiler_offsetof'
#define offsetof(TYPE, MEMBER) __compiler_offsetof(TYPE, MEMBER)
^~~~~~~~~~~~~~~~~~~
>> drivers/net/ethernet/intel/i40e/i40e_vf_migration.c:166:23: note: in expansion of macro 'offsetof'
#define VDM_OFFSET(x) offsetof(struct vfio_device_migration_info, x)
^~~~~~~~
drivers/net/ethernet/intel/i40e/i40e_vf_migration.c:195:7: note: in expansion of macro 'VDM_OFFSET'
case VDM_OFFSET(reserved):
^~~~~~~~~~
>> include/linux/compiler_types.h:129:35: error: invalid use of undefined type 'struct vfio_device_migration_info'
#define __compiler_offsetof(a, b) __builtin_offsetof(a, b)
^
include/linux/stddef.h:17:32: note: in expansion of macro '__compiler_offsetof'
#define offsetof(TYPE, MEMBER) __compiler_offsetof(TYPE, MEMBER)
^~~~~~~~~~~~~~~~~~~
>> drivers/net/ethernet/intel/i40e/i40e_vf_migration.c:166:23: note: in expansion of macro 'offsetof'
#define VDM_OFFSET(x) offsetof(struct vfio_device_migration_info, x)
^~~~~~~~
drivers/net/ethernet/intel/i40e/i40e_vf_migration.c:199:7: note: in expansion of macro 'VDM_OFFSET'
case VDM_OFFSET(pending_bytes):
^~~~~~~~~~
>> include/linux/compiler_types.h:129:35: error: invalid use of undefined type 'struct vfio_device_migration_info'
#define __compiler_offsetof(a, b) __builtin_offsetof(a, b)
^
include/linux/stddef.h:17:32: note: in expansion of macro '__compiler_offsetof'
#define offsetof(TYPE, MEMBER) __compiler_offsetof(TYPE, MEMBER)
^~~~~~~~~~~~~~~~~~~
>> drivers/net/ethernet/intel/i40e/i40e_vf_migration.c:166:23: note: in expansion of macro 'offsetof'
#define VDM_OFFSET(x) offsetof(struct vfio_device_migration_info, x)
^~~~~~~~
drivers/net/ethernet/intel/i40e/i40e_vf_migration.c:216:7: note: in expansion of macro 'VDM_OFFSET'
case VDM_OFFSET(data_offset):
^~~~~~~~~~
>> include/linux/compiler_types.h:129:35: error: invalid use of undefined type 'struct vfio_device_migration_info'
#define __compiler_offsetof(a, b) __builtin_offsetof(a, b)
^
include/linux/stddef.h:17:32: note: in expansion of macro '__compiler_offsetof'
#define offsetof(TYPE, MEMBER) __compiler_offsetof(TYPE, MEMBER)
^~~~~~~~~~~~~~~~~~~
>> drivers/net/ethernet/intel/i40e/i40e_vf_migration.c:166:23: note: in expansion of macro 'offsetof'
#define VDM_OFFSET(x) offsetof(struct vfio_device_migration_info, x)
^~~~~~~~
drivers/net/ethernet/intel/i40e/i40e_vf_migration.c:225:7: note: in expansion of macro 'VDM_OFFSET'
case VDM_OFFSET(data_size):
^~~~~~~~~~
drivers/net/ethernet/intel/i40e/i40e_vf_migration.c: In function 'i40e_vf_open':
>> drivers/net/ethernet/intel/i40e/i40e_vf_migration.c:385:12: error: 'VFIO_REGION_TYPE_MIGRATION' undeclared (first use in this function); did you mean 'VFIO_REGION_TYPE_GFX'?
VFIO_REGION_TYPE_MIGRATION,
^~~~~~~~~~~~~~~~~~~~~~~~~~
VFIO_REGION_TYPE_GFX
>> drivers/net/ethernet/intel/i40e/i40e_vf_migration.c:386:12: error: 'VFIO_REGION_SUBTYPE_MIGRATION' undeclared (first use in this function); did you mean 'VFIO_REGION_TYPE_MIGRATION'?
VFIO_REGION_SUBTYPE_MIGRATION,
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
VFIO_REGION_TYPE_MIGRATION
In file included from drivers/net/ethernet/intel/i40e/i40e_vf_migration.c:16:0:
>> drivers/net/ethernet/intel/i40e/i40e_vf_migration.h:20:37: error: invalid application of 'sizeof' to incomplete type 'struct vfio_device_migration_info'
#define MIGRATION_REGION_SZ (sizeof(struct vfio_device_migration_info))
^
>> drivers/net/ethernet/intel/i40e/i40e_vf_migration.c:388:12: note: in expansion of macro 'MIGRATION_REGION_SZ'
MIGRATION_REGION_SZ,
^~~~~~~~~~~~~~~~~~~
vim +132 drivers/net/ethernet/intel/i40e/i40e_vf_migration.c
125
126 static size_t i40e_vf_set_device_state(struct i40e_vf_migration *i40e_vf_dev,
127 u32 state)
128 {
129 int ret = 0;
130 struct vfio_device_migration_info *mig_ctl = i40e_vf_dev->mig_ctl;
131
> 132 if (state == mig_ctl->device_state)
133 return 0;
134
135 switch (state) {
> 136 case VFIO_DEVICE_STATE_RUNNING:
137 break;
> 138 case VFIO_DEVICE_STATE_SAVING | VFIO_DEVICE_STATE_RUNNING:
139 ret = i40e_vf_prepare_dirty_track(i40e_vf_dev);
140 break;
141 case VFIO_DEVICE_STATE_SAVING:
142 // do the last round of dirty page scanning
143 break;
> 144 case VFIO_DEVICE_STATE_STOP:
145 // release dirty page tracking resources
146 if (mig_ctl->device_state == VFIO_DEVICE_STATE_SAVING)
147 i40e_vf_stop_dirty_track(i40e_vf_dev);
148 break;
> 149 case VFIO_DEVICE_STATE_RESUMING:
150 break;
151 default:
152 ret = -EFAULT;
153 }
154
155 if (!ret)
156 mig_ctl->device_state = state;
157
158 return ret;
159 }
160
161 static
162 ssize_t i40e_vf_region_migration_rw(struct i40e_vf_migration *i40e_vf_dev,
163 char __user *buf, size_t count,
164 loff_t *ppos, bool iswrite)
165 {
> 166 #define VDM_OFFSET(x) offsetof(struct vfio_device_migration_info, x)
167 struct vfio_device_migration_info *mig_ctl = i40e_vf_dev->mig_ctl;
168 u64 pos = *ppos & VFIO_PCI_OFFSET_MASK;
169 int ret = 0;
170
171 switch (pos) {
> 172 case VDM_OFFSET(device_state):
173 if (count != sizeof(mig_ctl->device_state)) {
174 ret = -EINVAL;
175 break;
176 }
177
178 if (iswrite) {
179 u32 device_state;
180
181 if (copy_from_user(&device_state, buf, count)) {
182 ret = -EFAULT;
183 break;
184 }
185
186 ret = i40e_vf_set_device_state(i40e_vf_dev,
187 device_state) ?
188 ret : count;
189 } else {
190 ret = copy_to_user(buf, &mig_ctl->device_state,
191 count) ? -EFAULT : count;
192 }
193 break;
194
195 case VDM_OFFSET(reserved):
196 ret = -EFAULT;
197 break;
198
199 case VDM_OFFSET(pending_bytes):
200 {
201 if (count != sizeof(mig_ctl->pending_bytes)) {
202 ret = -EINVAL;
203 break;
204 }
205
206 if (iswrite)
207 ret = -EFAULT;
208 else
209 ret = copy_to_user(buf,
210 &mig_ctl->pending_bytes,
211 count) ? -EFAULT : count;
212
213 break;
214 }
215
216 case VDM_OFFSET(data_offset):
217 {
218 /* as we don't support device internal dirty data
219 * and our pending_bytes is always 0,
220 * return error here.
221 */
222 ret = -EFAULT;
223 break;
224 }
225 case VDM_OFFSET(data_size):
226 if (count != sizeof(mig_ctl->data_size)) {
227 ret = -EINVAL;
228 break;
229 }
230
231 if (iswrite)
232 ret = copy_from_user(&mig_ctl->data_size, buf, count) ?
233 -EFAULT : count;
234 else
235 ret = copy_to_user(buf, &mig_ctl->data_size, count) ?
236 -EFAULT : count;
237 break;
238
239 default:
240 ret = -EFAULT;
241 break;
242 }
243 return ret;
244 }
245
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
2 years, 4 months
[arnd-playground:randconfig-20200504 199/355] include/linux/time32.h:75:5: error: incompatible types when assigning to type 'struct __kernel_old_timeval' from type 'int'
by kbuild test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/arnd/playground.git randconfig-20200504
head: 5867adb94a59773ef2e3262b70538803fe896906
commit: c0d4007762ea821f644e7c87e338a5d3bb24b896 [199/355] y2038: move ns_to_old_timeval32 definition
config: i386-tinyconfig (attached as .config)
compiler: gcc-7 (Ubuntu 7.5.0-6ubuntu2) 7.5.0
reproduce:
git checkout c0d4007762ea821f644e7c87e338a5d3bb24b896
# save the attached .config to linux build tree
make ARCH=i386
If you fix the issue, kindly add following tag as appropriate
Reported-by: kbuild test robot <lkp(a)intel.com>
Note: the arnd-playground/randconfig-20200504 HEAD 5867adb94a59773ef2e3262b70538803fe896906 builds fine.
It only hurts bisectibility.
All error/warnings (new ones prefixed by >>, old ones prefixed by <<):
In file included from include/linux/time.h:74:0,
from include/linux/ktime.h:24,
from include/linux/timer.h:6,
from include/linux/workqueue.h:9,
from include/linux/mm_types.h:16,
from include/linux/mmzone.h:21,
from include/linux/gfp.h:6,
from include/linux/slab.h:15,
from include/linux/crypto.h:19,
from arch/x86/kernel/asm-offsets.c:9:
include/linux/time32.h: In function 'ns_to_old_timeval32':
>> include/linux/time32.h:75:7: warning: implicit declaration of function 'ns_to_kernel_old_timeval'; did you mean 'ns_to_old_timeval32'? [-Wimplicit-function-declaration]
tv = ns_to_kernel_old_timeval(nsec);
^~~~~~~~~~~~~~~~~~~~~~~~
ns_to_old_timeval32
>> include/linux/time32.h:75:5: error: incompatible types when assigning to type 'struct __kernel_old_timeval' from type 'int'
tv = ns_to_kernel_old_timeval(nsec);
^
In file included from include/linux/ktime.h:24:0,
from include/linux/timer.h:6,
from include/linux/workqueue.h:9,
from include/linux/mm_types.h:16,
from include/linux/mmzone.h:21,
from include/linux/gfp.h:6,
from include/linux/slab.h:15,
from include/linux/crypto.h:19,
from arch/x86/kernel/asm-offsets.c:9:
include/linux/time.h: At top level:
>> include/linux/time.h:91:36: error: conflicting types for 'ns_to_kernel_old_timeval'
extern struct __kernel_old_timeval ns_to_kernel_old_timeval(s64 nsec);
^~~~~~~~~~~~~~~~~~~~~~~~
In file included from include/linux/time.h:74:0,
from include/linux/ktime.h:24,
from include/linux/timer.h:6,
from include/linux/workqueue.h:9,
from include/linux/mm_types.h:16,
from include/linux/mmzone.h:21,
from include/linux/gfp.h:6,
from include/linux/slab.h:15,
from include/linux/crypto.h:19,
from arch/x86/kernel/asm-offsets.c:9:
include/linux/time32.h:75:7: note: previous implicit declaration of 'ns_to_kernel_old_timeval' was here
tv = ns_to_kernel_old_timeval(nsec);
^~~~~~~~~~~~~~~~~~~~~~~~
make[2]: *** [scripts/Makefile.build:106: arch/x86/kernel/asm-offsets.s] Error 1
make[2]: Target '__build' not remade because of errors.
make[1]: *** [Makefile:1121: prepare0] Error 2
make[1]: Target 'prepare' not remade because of errors.
make: *** [Makefile:180: sub-make] Error 2
vim +75 include/linux/time32.h
63
64 /**
65 * ns_to_old_timeval32 - Compat version of ns_to_timeval
66 * @nsec: the nanoseconds value to be converted
67 *
68 * Returns the old_timeval32 representation of the nsec parameter.
69 */
70 static inline struct old_timeval32 ns_to_old_timeval32(s64 nsec)
71 {
72 struct __kernel_old_timeval tv;
73 struct old_timeval32 ctv;
74
> 75 tv = ns_to_kernel_old_timeval(nsec);
76 ctv.tv_sec = tv.tv_sec;
77 ctv.tv_usec = tv.tv_usec;
78
79 return ctv;
80 }
81
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
2 years, 4 months
[sashal-linux-stable:queue-4.14 33/79] Warning: ffffffff819d7465: 0f ff 49 8d ud0 -0x73(%rcx),%ecx
by kbuild test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/sashal/linux-stable.git queue-4.14
head: 7a7ddea1b419e20931152e388252c661b6fca95c
commit: aff1cb67424d5fa9b2c5c23b052b4206ad94bf98 [33/79] x86/unwind/orc: Fix error path for bad ORC entry type
config: x86_64-randconfig-a013-20200517 (attached as .config)
compiler: clang version 11.0.0 (https://github.com/llvm/llvm-project 135b877874fae96b4372c8a3fbfaa8ff44ff86e3)
reproduce:
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
git checkout aff1cb67424d5fa9b2c5c23b052b4206ad94bf98
# 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: kbuild test robot <lkp(a)intel.com>
All warnings (new ones prefixed by >>, old ones prefixed by <<):
>> Warning: ffffffff819d7465: 0f ff 49 8d ud0 -0x73(%rcx),%ecx
Warning: objdump says 4 bytes, but insn_get_length() says 2
Warning: arch/x86/tools/test_get_len found difference at <memcpy_page_flushcache>:ffffffff819d6bf0
--
>> Warning: ffffffff819d7573: 0f ff 48 8d ud0 -0x73(%rax),%ecx
Warning: objdump says 4 bytes, but insn_get_length() says 2
Warning: arch/x86/tools/test_get_len found difference at <memcpy_page_flushcache>:ffffffff819d6bf0
--
>> Warning: ffffffff819d77a5: 0f ff 48 8d ud0 -0x73(%rax),%ecx
Warning: objdump says 4 bytes, but insn_get_length() says 2
Warning: arch/x86/tools/test_get_len found difference at <memcpy_page_flushcache>:ffffffff819d6bf0
--
>> Warning: ffffffff819d79a5: 0f ff 48 8d ud0 -0x73(%rax),%ecx
Warning: objdump says 4 bytes, but insn_get_length() says 2
Warning: arch/x86/tools/test_get_len found difference at <memcpy_page_flushcache>:ffffffff819d6bf0
--
>> Warning: ffffffff819d79b7: 0f ff 48 8d ud0 -0x73(%rax),%ecx
Warning: objdump says 4 bytes, but insn_get_length() says 2
Warning: arch/x86/tools/test_get_len found difference at <memcpy_page_flushcache>:ffffffff819d6bf0
--
>> Warning: ffffffff819d7a11: 0f ff 48 8d ud0 -0x73(%rax),%ecx
Warning: objdump says 4 bytes, but insn_get_length() says 2
Warning: arch/x86/tools/test_get_len found difference at <memcpy_page_flushcache>:ffffffff819d6bf0
--
>> Warning: ffffffff819d72c8: 0f ff 48 8d ud0 -0x73(%rax),%ecx
Warning: objdump says 4 bytes, but insn_get_length() says 2
Warning: arch/x86/tools/test_get_len found difference at <memcpy_page_flushcache>:ffffffff819d6bf0
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
2 years, 4 months
Re: [RFC PATCH v4 05/10] vfio/pci: export vfio_pci_get_barmap
by kbuild test robot
Hi Yan,
[FYI, it's a private test report for your RFC patch.]
[auto build test WARNING on vfio/next]
[also build test WARNING on jkirsher-next-queue/dev-queue linus/master v5.7-rc6 next-20200515]
[cannot apply to linux/master]
[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/Yan-Zhao/Introduce-vendor-ops-in...
base: https://github.com/awilliam/linux-vfio.git next
config: x86_64-allyesconfig (attached as .config)
reproduce:
# apt-get install sparse
# sparse version: v0.6.1-193-gb8fad4bc-dirty
# save the attached .config to linux build tree
make C=1 ARCH=x86_64 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__'
If you fix the issue, kindly add following tag as appropriate
Reported-by: kbuild test robot <lkp(a)intel.com>
sparse warnings: (new ones prefixed by >>)
>> drivers/vfio/pci/vfio_pci_rdwr.c:162:20: sparse: sparse: incompatible types in conditional expression (different address spaces):
>> drivers/vfio/pci/vfio_pci_rdwr.c:162:20: sparse: void *
>> drivers/vfio/pci/vfio_pci_rdwr.c:162:20: sparse: void [noderef] <asn:2> *
vim +162 drivers/vfio/pci/vfio_pci_rdwr.c
155
156 void __iomem *vfio_pci_get_barmap(void *device_data, int bar)
157 {
158 int ret;
159 struct vfio_pci_device *vdev = device_data;
160
161 ret = vfio_pci_setup_barmap(vdev, bar);
> 162 return ret ? ERR_PTR(ret) : vdev->barmap[bar];
163 }
164 EXPORT_SYMBOL_GPL(vfio_pci_get_barmap);
165
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
2 years, 4 months