From: kernel test robot <lkp@intel.com>
To: kbuild-all@lists.01.org
Subject: Re: [RFC v2 09/11] HID: hyperv: Make ringbuffer at least take two pages
Date: Wed, 02 Sep 2020 13:43:17 +0800 [thread overview]
Message-ID: <202009021313.uj71pltF%lkp@intel.com> (raw)
In-Reply-To: <20200902030107.33380-10-boqun.feng@gmail.com>
[-- Attachment #1: Type: text/plain, Size: 9015 bytes --]
Hi Boqun,
[FYI, it's a private test report for your RFC patch.]
[auto build test WARNING on mkp-scsi/for-next]
[also build test WARNING on scsi/for-next linus/master v5.9-rc3 next-20200828]
[cannot apply to input/next]
[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/Boqun-Feng/Hyper-V-Support-PAGE_SIZE-larger-than-4K/20200902-110406
base: https://git.kernel.org/pub/scm/linux/kernel/git/mkp/scsi.git for-next
config: i386-randconfig-s002-20200901 (attached as .config)
compiler: gcc-9 (Debian 9.3.0-15) 9.3.0
reproduce:
# apt-get install sparse
# sparse version: v0.6.2-191-g10164920-dirty
# save the attached .config to linux build tree
make W=1 C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__' ARCH=i386
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
sparse warnings: (new ones prefixed by >>)
drivers/hid/hid-hyperv.c:202:40: sparse: sparse: incorrect type in assignment (different base types) @@ expected unsigned int [usertype] report_desc_size @@ got restricted __le16 [usertype] wDescriptorLength @@
drivers/hid/hid-hyperv.c:202:40: sparse: expected unsigned int [usertype] report_desc_size
drivers/hid/hid-hyperv.c:202:40: sparse: got restricted __le16 [usertype] wDescriptorLength
drivers/hid/hid-hyperv.c:218:9: sparse: sparse: incorrect type in argument 3 (different base types) @@ expected unsigned int @@ got restricted __le16 [usertype] wDescriptorLength @@
drivers/hid/hid-hyperv.c:218:9: sparse: expected unsigned int
drivers/hid/hid-hyperv.c:218:9: sparse: got restricted __le16 [usertype] wDescriptorLength
>> drivers/hid/hid-hyperv.c:461:17: sparse: sparse: incompatible types in comparison expression (different type sizes):
>> drivers/hid/hid-hyperv.c:461:17: sparse: int *
>> drivers/hid/hid-hyperv.c:461:17: sparse: unsigned long *
drivers/hid/hid-hyperv.c:562:26: sparse: sparse: incompatible types in comparison expression (different type sizes):
drivers/hid/hid-hyperv.c:562:26: sparse: int *
drivers/hid/hid-hyperv.c:562:26: sparse: unsigned long *
# https://github.com/0day-ci/linux/commit/ec2aac3d927fd18d45bb493d854ddbb3b8620cae
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Boqun-Feng/Hyper-V-Support-PAGE_SIZE-larger-than-4K/20200902-110406
git checkout ec2aac3d927fd18d45bb493d854ddbb3b8620cae
vim +461 drivers/hid/hid-hyperv.c
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 447
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 448 static int mousevsc_probe(struct hv_device *device,
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 449 const struct hv_vmbus_device_id *dev_id)
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 450 {
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 451 int ret;
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 452 struct mousevsc_dev *input_dev;
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 453 struct hid_device *hid_dev;
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 454
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 455 input_dev = mousevsc_alloc_device(device);
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 456
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 457 if (!input_dev)
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 458 return -ENOMEM;
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 459
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 460 ret = vmbus_open(device->channel,
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 @461 INPUTVSC_SEND_RING_BUFFER_SIZE,
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 462 INPUTVSC_RECV_RING_BUFFER_SIZE,
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 463 NULL,
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 464 0,
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 465 mousevsc_on_channel_callback,
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 466 device
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 467 );
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 468
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 469 if (ret)
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 470 goto probe_err0;
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 471
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 472 ret = mousevsc_connect_to_vsp(device);
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 473
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 474 if (ret)
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 475 goto probe_err1;
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 476
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 477 /* workaround SA-167 */
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 478 if (input_dev->report_desc[14] == 0x25)
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 479 input_dev->report_desc[14] = 0x29;
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 480
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 481 hid_dev = hid_allocate_device();
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 482 if (IS_ERR(hid_dev)) {
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 483 ret = PTR_ERR(hid_dev);
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 484 goto probe_err1;
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 485 }
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 486
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 487 hid_dev->ll_driver = &mousevsc_ll_driver;
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 488 hid_dev->driver = &mousevsc_hid_driver;
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 489 hid_dev->bus = BUS_VIRTUAL;
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 490 hid_dev->vendor = input_dev->hid_dev_info.vendor;
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 491 hid_dev->product = input_dev->hid_dev_info.product;
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 492 hid_dev->version = input_dev->hid_dev_info.version;
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 493 input_dev->hid_device = hid_dev;
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 494
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 495 sprintf(hid_dev->name, "%s", "Microsoft Vmbus HID-compliant Mouse");
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 496
b565a3906a92fe K. Y. Srinivasan 2012-05-16 497 hid_set_drvdata(hid_dev, device);
b565a3906a92fe K. Y. Srinivasan 2012-05-16 498
74c4fb058083b4 K. Y. Srinivasan 2011-11-30 499 ret = hid_add_device(hid_dev);
74c4fb058083b4 K. Y. Srinivasan 2011-11-30 500 if (ret)
74c4fb058083b4 K. Y. Srinivasan 2011-11-30 501 goto probe_err1;
74c4fb058083b4 K. Y. Srinivasan 2011-11-30 502
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 503
07d9ab4f0e52cb Henrik Rydberg 2012-04-22 504 ret = hid_parse(hid_dev);
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 505 if (ret) {
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 506 hid_err(hid_dev, "parse failed\n");
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 507 goto probe_err2;
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 508 }
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 509
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 510 ret = hid_hw_start(hid_dev, HID_CONNECT_HIDINPUT | HID_CONNECT_HIDDEV);
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 511
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 512 if (ret) {
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 513 hid_err(hid_dev, "hw start failed\n");
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 514 goto probe_err2;
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 515 }
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 516
f1210455e78a61 Dexuan Cui 2014-08-01 517 device_init_wakeup(&device->device, true);
f1210455e78a61 Dexuan Cui 2014-08-01 518
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 519 input_dev->connected = true;
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 520 input_dev->init_complete = true;
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 521
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 522 return ret;
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 523
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 524 probe_err2:
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 525 hid_destroy_device(hid_dev);
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 526
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 527 probe_err1:
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 528 vmbus_close(device->channel);
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 529
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 530 probe_err0:
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 531 mousevsc_free_device(input_dev);
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 532
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 533 return ret;
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 534 }
b95f5bcb811e39 K. Y. Srinivasan 2011-11-17 535
---
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: 31586 bytes --]
next prev parent reply other threads:[~2020-09-02 5:43 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-09-02 3:00 [RFC v2 00/11] Hyper-V: Support PAGE_SIZE larger than 4K Boqun Feng
2020-09-02 3:00 ` [RFC v2 01/11] Drivers: hv: vmbus: Always use HV_HYP_PAGE_SIZE for gpadl Boqun Feng
2020-09-02 3:00 ` [RFC v2 02/11] Drivers: hv: vmbus: Move __vmbus_open() Boqun Feng
2020-09-05 0:04 ` Michael Kelley
2020-09-02 3:00 ` [RFC v2 03/11] Drivers: hv: vmbus: Introduce types of GPADL Boqun Feng
2020-09-05 0:19 ` Michael Kelley
2020-09-06 4:51 ` Boqun Feng
2020-09-02 3:01 ` [RFC v2 04/11] Drivers: hv: Use HV_HYP_PAGE in hv_synic_enable_regs() Boqun Feng
2020-09-02 3:01 ` [RFC v2 05/11] Drivers: hv: vmbus: Move virt_to_hvpfn() to hyperv header Boqun Feng
2020-09-02 3:01 ` [RFC v2 06/11] hv: hyperv.h: Introduce some hvpfn helper functions Boqun Feng
2020-09-02 3:01 ` [RFC v2 07/11] hv_netvsc: Use HV_HYP_PAGE_SIZE for Hyper-V communication Boqun Feng
2020-09-05 0:30 ` Michael Kelley
2020-09-07 0:09 ` Boqun Feng
2020-09-02 3:01 ` [RFC v2 08/11] Input: hyperv-keyboard: Make ringbuffer at least take two pages Boqun Feng
2020-09-02 3:01 ` [RFC v2 09/11] HID: hyperv: " Boqun Feng
2020-09-02 5:43 ` kernel test robot [this message]
2020-09-02 7:54 ` kernel test robot
2020-09-02 8:18 ` Jiri Kosina
2020-09-02 3:01 ` [RFC v2 10/11] Driver: hv: util: " Boqun Feng
2020-09-02 3:01 ` [RFC v2 11/11] scsi: storvsc: Support PAGE_SIZE larger than 4K Boqun Feng
2020-09-05 2:55 ` Michael Kelley
2020-09-05 14:15 ` Boqun Feng
2020-09-05 15:37 ` Michael Kelley
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=202009021313.uj71pltF%lkp@intel.com \
--to=lkp@intel.com \
--cc=kbuild-all@lists.01.org \
/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.