From: Dan Carpenter <dan.carpenter@oracle.com>
To: kbuild@lists.01.org, Jason Wang <jasowang@redhat.com>
Cc: lkp@intel.com, kbuild-all@lists.01.org,
linux-kernel@vger.kernel.org,
"Michael S. Tsirkin" <mst@redhat.com>,
Eli Cohen <elic@nvidia.com>
Subject: [kbuild] drivers/vdpa/virtio_pci/vp_vdpa.c:421 vp_vdpa_probe() warn: missing error code 'ret'
Date: Wed, 23 Jun 2021 12:21:36 +0300 [thread overview]
Message-ID: <202106231257.8kp6t8DF-lkp@intel.com> (raw)
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: 0c18f29aae7ce3dadd26d8ee3505d07cc982df75
commit: 11d8ffed00b231356008b35a3b0bc192e42333fa vp_vdpa: switch to use vp_modern_map_vq_notify()
config: microblaze-randconfig-m031-20210622 (attached as .config)
compiler: microblaze-linux-gcc (GCC) 9.3.0
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
Reported-by: Dan Carpenter <dan.carpenter@oracle.com>
New smatch warnings:
drivers/vdpa/virtio_pci/vp_vdpa.c:421 vp_vdpa_probe() warn: missing error code 'ret'
Old smatch warnings:
drivers/vdpa/virtio_pci/vp_vdpa.c:168 vp_vdpa_request_irq() warn: inconsistent indenting
vim +/ret +421 drivers/vdpa/virtio_pci/vp_vdpa.c
64b9f64f80a6f4 Jason Wang 2021-02-23 367 static int vp_vdpa_probe(struct pci_dev *pdev, const struct pci_device_id *id)
64b9f64f80a6f4 Jason Wang 2021-02-23 368 {
64b9f64f80a6f4 Jason Wang 2021-02-23 369 struct virtio_pci_modern_device *mdev;
64b9f64f80a6f4 Jason Wang 2021-02-23 370 struct device *dev = &pdev->dev;
64b9f64f80a6f4 Jason Wang 2021-02-23 371 struct vp_vdpa *vp_vdpa;
64b9f64f80a6f4 Jason Wang 2021-02-23 372 int ret, i;
64b9f64f80a6f4 Jason Wang 2021-02-23 373
64b9f64f80a6f4 Jason Wang 2021-02-23 374 ret = pcim_enable_device(pdev);
64b9f64f80a6f4 Jason Wang 2021-02-23 375 if (ret)
64b9f64f80a6f4 Jason Wang 2021-02-23 376 return ret;
64b9f64f80a6f4 Jason Wang 2021-02-23 377
64b9f64f80a6f4 Jason Wang 2021-02-23 378 vp_vdpa = vdpa_alloc_device(struct vp_vdpa, vdpa,
64b9f64f80a6f4 Jason Wang 2021-02-23 379 dev, &vp_vdpa_ops, NULL);
64b9f64f80a6f4 Jason Wang 2021-02-23 380 if (vp_vdpa == NULL) {
64b9f64f80a6f4 Jason Wang 2021-02-23 381 dev_err(dev, "vp_vdpa: Failed to allocate vDPA structure\n");
64b9f64f80a6f4 Jason Wang 2021-02-23 382 return -ENOMEM;
64b9f64f80a6f4 Jason Wang 2021-02-23 383 }
64b9f64f80a6f4 Jason Wang 2021-02-23 384
64b9f64f80a6f4 Jason Wang 2021-02-23 385 mdev = &vp_vdpa->mdev;
64b9f64f80a6f4 Jason Wang 2021-02-23 386 mdev->pci_dev = pdev;
64b9f64f80a6f4 Jason Wang 2021-02-23 387
64b9f64f80a6f4 Jason Wang 2021-02-23 388 ret = vp_modern_probe(mdev);
64b9f64f80a6f4 Jason Wang 2021-02-23 389 if (ret) {
64b9f64f80a6f4 Jason Wang 2021-02-23 390 dev_err(&pdev->dev, "Failed to probe modern PCI device\n");
64b9f64f80a6f4 Jason Wang 2021-02-23 391 goto err;
64b9f64f80a6f4 Jason Wang 2021-02-23 392 }
64b9f64f80a6f4 Jason Wang 2021-02-23 393
64b9f64f80a6f4 Jason Wang 2021-02-23 394 pci_set_master(pdev);
64b9f64f80a6f4 Jason Wang 2021-02-23 395 pci_set_drvdata(pdev, vp_vdpa);
64b9f64f80a6f4 Jason Wang 2021-02-23 396
64b9f64f80a6f4 Jason Wang 2021-02-23 397 vp_vdpa->vdpa.dma_dev = &pdev->dev;
64b9f64f80a6f4 Jason Wang 2021-02-23 398 vp_vdpa->queues = vp_modern_get_num_queues(mdev);
64b9f64f80a6f4 Jason Wang 2021-02-23 399
64b9f64f80a6f4 Jason Wang 2021-02-23 400 ret = devm_add_action_or_reset(dev, vp_vdpa_free_irq_vectors, pdev);
64b9f64f80a6f4 Jason Wang 2021-02-23 401 if (ret) {
64b9f64f80a6f4 Jason Wang 2021-02-23 402 dev_err(&pdev->dev,
64b9f64f80a6f4 Jason Wang 2021-02-23 403 "Failed for adding devres for freeing irq vectors\n");
64b9f64f80a6f4 Jason Wang 2021-02-23 404 goto err;
64b9f64f80a6f4 Jason Wang 2021-02-23 405 }
64b9f64f80a6f4 Jason Wang 2021-02-23 406
64b9f64f80a6f4 Jason Wang 2021-02-23 407 vp_vdpa->vring = devm_kcalloc(&pdev->dev, vp_vdpa->queues,
64b9f64f80a6f4 Jason Wang 2021-02-23 408 sizeof(*vp_vdpa->vring),
64b9f64f80a6f4 Jason Wang 2021-02-23 409 GFP_KERNEL);
64b9f64f80a6f4 Jason Wang 2021-02-23 410 if (!vp_vdpa->vring) {
64b9f64f80a6f4 Jason Wang 2021-02-23 411 ret = -ENOMEM;
64b9f64f80a6f4 Jason Wang 2021-02-23 412 dev_err(&pdev->dev, "Fail to allocate virtqueues\n");
64b9f64f80a6f4 Jason Wang 2021-02-23 413 goto err;
64b9f64f80a6f4 Jason Wang 2021-02-23 414 }
64b9f64f80a6f4 Jason Wang 2021-02-23 415
64b9f64f80a6f4 Jason Wang 2021-02-23 416 for (i = 0; i < vp_vdpa->queues; i++) {
64b9f64f80a6f4 Jason Wang 2021-02-23 417 vp_vdpa->vring[i].irq = VIRTIO_MSI_NO_VECTOR;
11d8ffed00b231 Jason Wang 2021-04-15 418 vp_vdpa->vring[i].notify = vp_modern_map_vq_notify(mdev, i);
11d8ffed00b231 Jason Wang 2021-04-15 419 if (!vp_vdpa->vring[i].notify) {
11d8ffed00b231 Jason Wang 2021-04-15 420 dev_warn(&pdev->dev, "Fail to map vq notify %d\n", i);
ret = -ENOMEM;?
11d8ffed00b231 Jason Wang 2021-04-15 @421 goto err;
11d8ffed00b231 Jason Wang 2021-04-15 422 }
64b9f64f80a6f4 Jason Wang 2021-02-23 423 }
64b9f64f80a6f4 Jason Wang 2021-02-23 424 vp_vdpa->config_irq = VIRTIO_MSI_NO_VECTOR;
64b9f64f80a6f4 Jason Wang 2021-02-23 425
64b9f64f80a6f4 Jason Wang 2021-02-23 426 ret = vdpa_register_device(&vp_vdpa->vdpa, vp_vdpa->queues);
64b9f64f80a6f4 Jason Wang 2021-02-23 427 if (ret) {
64b9f64f80a6f4 Jason Wang 2021-02-23 428 dev_err(&pdev->dev, "Failed to register to vdpa bus\n");
64b9f64f80a6f4 Jason Wang 2021-02-23 429 goto err;
64b9f64f80a6f4 Jason Wang 2021-02-23 430 }
64b9f64f80a6f4 Jason Wang 2021-02-23 431
64b9f64f80a6f4 Jason Wang 2021-02-23 432 return 0;
64b9f64f80a6f4 Jason Wang 2021-02-23 433
64b9f64f80a6f4 Jason Wang 2021-02-23 434 err:
64b9f64f80a6f4 Jason Wang 2021-02-23 435 put_device(&vp_vdpa->vdpa.dev);
64b9f64f80a6f4 Jason Wang 2021-02-23 436 return ret;
64b9f64f80a6f4 Jason Wang 2021-02-23 437 }
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
_______________________________________________
kbuild mailing list -- kbuild@lists.01.org
To unsubscribe send an email to kbuild-leave@lists.01.org
WARNING: multiple messages have this Message-ID (diff)
From: kernel test robot <lkp@intel.com>
To: kbuild@lists.01.org
Subject: drivers/vdpa/virtio_pci/vp_vdpa.c:421 vp_vdpa_probe() warn: missing error code 'ret'
Date: Wed, 23 Jun 2021 12:13:05 +0800 [thread overview]
Message-ID: <202106231257.8kp6t8DF-lkp@intel.com> (raw)
[-- Attachment #1: Type: text/plain, Size: 6090 bytes --]
CC: kbuild-all(a)lists.01.org
CC: linux-kernel(a)vger.kernel.org
TO: Jason Wang <jasowang@redhat.com>
CC: "Michael S. Tsirkin" <mst@redhat.com>
CC: Eli Cohen <elic@nvidia.com>
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: 0c18f29aae7ce3dadd26d8ee3505d07cc982df75
commit: 11d8ffed00b231356008b35a3b0bc192e42333fa vp_vdpa: switch to use vp_modern_map_vq_notify()
date: 7 weeks ago
:::::: branch date: 10 hours ago
:::::: commit date: 7 weeks ago
config: microblaze-randconfig-m031-20210622 (attached as .config)
compiler: microblaze-linux-gcc (GCC) 9.3.0
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
Reported-by: Dan Carpenter <dan.carpenter@oracle.com>
New smatch warnings:
drivers/vdpa/virtio_pci/vp_vdpa.c:421 vp_vdpa_probe() warn: missing error code 'ret'
Old smatch warnings:
drivers/vdpa/virtio_pci/vp_vdpa.c:168 vp_vdpa_request_irq() warn: inconsistent indenting
vim +/ret +421 drivers/vdpa/virtio_pci/vp_vdpa.c
64b9f64f80a6f4 Jason Wang 2021-02-23 366
64b9f64f80a6f4 Jason Wang 2021-02-23 367 static int vp_vdpa_probe(struct pci_dev *pdev, const struct pci_device_id *id)
64b9f64f80a6f4 Jason Wang 2021-02-23 368 {
64b9f64f80a6f4 Jason Wang 2021-02-23 369 struct virtio_pci_modern_device *mdev;
64b9f64f80a6f4 Jason Wang 2021-02-23 370 struct device *dev = &pdev->dev;
64b9f64f80a6f4 Jason Wang 2021-02-23 371 struct vp_vdpa *vp_vdpa;
64b9f64f80a6f4 Jason Wang 2021-02-23 372 int ret, i;
64b9f64f80a6f4 Jason Wang 2021-02-23 373
64b9f64f80a6f4 Jason Wang 2021-02-23 374 ret = pcim_enable_device(pdev);
64b9f64f80a6f4 Jason Wang 2021-02-23 375 if (ret)
64b9f64f80a6f4 Jason Wang 2021-02-23 376 return ret;
64b9f64f80a6f4 Jason Wang 2021-02-23 377
64b9f64f80a6f4 Jason Wang 2021-02-23 378 vp_vdpa = vdpa_alloc_device(struct vp_vdpa, vdpa,
64b9f64f80a6f4 Jason Wang 2021-02-23 379 dev, &vp_vdpa_ops, NULL);
64b9f64f80a6f4 Jason Wang 2021-02-23 380 if (vp_vdpa == NULL) {
64b9f64f80a6f4 Jason Wang 2021-02-23 381 dev_err(dev, "vp_vdpa: Failed to allocate vDPA structure\n");
64b9f64f80a6f4 Jason Wang 2021-02-23 382 return -ENOMEM;
64b9f64f80a6f4 Jason Wang 2021-02-23 383 }
64b9f64f80a6f4 Jason Wang 2021-02-23 384
64b9f64f80a6f4 Jason Wang 2021-02-23 385 mdev = &vp_vdpa->mdev;
64b9f64f80a6f4 Jason Wang 2021-02-23 386 mdev->pci_dev = pdev;
64b9f64f80a6f4 Jason Wang 2021-02-23 387
64b9f64f80a6f4 Jason Wang 2021-02-23 388 ret = vp_modern_probe(mdev);
64b9f64f80a6f4 Jason Wang 2021-02-23 389 if (ret) {
64b9f64f80a6f4 Jason Wang 2021-02-23 390 dev_err(&pdev->dev, "Failed to probe modern PCI device\n");
64b9f64f80a6f4 Jason Wang 2021-02-23 391 goto err;
64b9f64f80a6f4 Jason Wang 2021-02-23 392 }
64b9f64f80a6f4 Jason Wang 2021-02-23 393
64b9f64f80a6f4 Jason Wang 2021-02-23 394 pci_set_master(pdev);
64b9f64f80a6f4 Jason Wang 2021-02-23 395 pci_set_drvdata(pdev, vp_vdpa);
64b9f64f80a6f4 Jason Wang 2021-02-23 396
64b9f64f80a6f4 Jason Wang 2021-02-23 397 vp_vdpa->vdpa.dma_dev = &pdev->dev;
64b9f64f80a6f4 Jason Wang 2021-02-23 398 vp_vdpa->queues = vp_modern_get_num_queues(mdev);
64b9f64f80a6f4 Jason Wang 2021-02-23 399
64b9f64f80a6f4 Jason Wang 2021-02-23 400 ret = devm_add_action_or_reset(dev, vp_vdpa_free_irq_vectors, pdev);
64b9f64f80a6f4 Jason Wang 2021-02-23 401 if (ret) {
64b9f64f80a6f4 Jason Wang 2021-02-23 402 dev_err(&pdev->dev,
64b9f64f80a6f4 Jason Wang 2021-02-23 403 "Failed for adding devres for freeing irq vectors\n");
64b9f64f80a6f4 Jason Wang 2021-02-23 404 goto err;
64b9f64f80a6f4 Jason Wang 2021-02-23 405 }
64b9f64f80a6f4 Jason Wang 2021-02-23 406
64b9f64f80a6f4 Jason Wang 2021-02-23 407 vp_vdpa->vring = devm_kcalloc(&pdev->dev, vp_vdpa->queues,
64b9f64f80a6f4 Jason Wang 2021-02-23 408 sizeof(*vp_vdpa->vring),
64b9f64f80a6f4 Jason Wang 2021-02-23 409 GFP_KERNEL);
64b9f64f80a6f4 Jason Wang 2021-02-23 410 if (!vp_vdpa->vring) {
64b9f64f80a6f4 Jason Wang 2021-02-23 411 ret = -ENOMEM;
64b9f64f80a6f4 Jason Wang 2021-02-23 412 dev_err(&pdev->dev, "Fail to allocate virtqueues\n");
64b9f64f80a6f4 Jason Wang 2021-02-23 413 goto err;
64b9f64f80a6f4 Jason Wang 2021-02-23 414 }
64b9f64f80a6f4 Jason Wang 2021-02-23 415
64b9f64f80a6f4 Jason Wang 2021-02-23 416 for (i = 0; i < vp_vdpa->queues; i++) {
64b9f64f80a6f4 Jason Wang 2021-02-23 417 vp_vdpa->vring[i].irq = VIRTIO_MSI_NO_VECTOR;
11d8ffed00b231 Jason Wang 2021-04-15 418 vp_vdpa->vring[i].notify = vp_modern_map_vq_notify(mdev, i);
11d8ffed00b231 Jason Wang 2021-04-15 419 if (!vp_vdpa->vring[i].notify) {
11d8ffed00b231 Jason Wang 2021-04-15 420 dev_warn(&pdev->dev, "Fail to map vq notify %d\n", i);
11d8ffed00b231 Jason Wang 2021-04-15 @421 goto err;
11d8ffed00b231 Jason Wang 2021-04-15 422 }
64b9f64f80a6f4 Jason Wang 2021-02-23 423 }
64b9f64f80a6f4 Jason Wang 2021-02-23 424 vp_vdpa->config_irq = VIRTIO_MSI_NO_VECTOR;
64b9f64f80a6f4 Jason Wang 2021-02-23 425
64b9f64f80a6f4 Jason Wang 2021-02-23 426 ret = vdpa_register_device(&vp_vdpa->vdpa, vp_vdpa->queues);
64b9f64f80a6f4 Jason Wang 2021-02-23 427 if (ret) {
64b9f64f80a6f4 Jason Wang 2021-02-23 428 dev_err(&pdev->dev, "Failed to register to vdpa bus\n");
64b9f64f80a6f4 Jason Wang 2021-02-23 429 goto err;
64b9f64f80a6f4 Jason Wang 2021-02-23 430 }
64b9f64f80a6f4 Jason Wang 2021-02-23 431
64b9f64f80a6f4 Jason Wang 2021-02-23 432 return 0;
64b9f64f80a6f4 Jason Wang 2021-02-23 433
64b9f64f80a6f4 Jason Wang 2021-02-23 434 err:
64b9f64f80a6f4 Jason Wang 2021-02-23 435 put_device(&vp_vdpa->vdpa.dev);
64b9f64f80a6f4 Jason Wang 2021-02-23 436 return ret;
64b9f64f80a6f4 Jason Wang 2021-02-23 437 }
64b9f64f80a6f4 Jason Wang 2021-02-23 438
---
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: 25753 bytes --]
WARNING: multiple messages have this Message-ID (diff)
From: Dan Carpenter <dan.carpenter@oracle.com>
To: kbuild-all@lists.01.org
Subject: [kbuild] drivers/vdpa/virtio_pci/vp_vdpa.c:421 vp_vdpa_probe() warn: missing error code 'ret'
Date: Wed, 23 Jun 2021 12:21:36 +0300 [thread overview]
Message-ID: <202106231257.8kp6t8DF-lkp@intel.com> (raw)
[-- Attachment #1: Type: text/plain, Size: 5910 bytes --]
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: 0c18f29aae7ce3dadd26d8ee3505d07cc982df75
commit: 11d8ffed00b231356008b35a3b0bc192e42333fa vp_vdpa: switch to use vp_modern_map_vq_notify()
config: microblaze-randconfig-m031-20210622 (attached as .config)
compiler: microblaze-linux-gcc (GCC) 9.3.0
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
Reported-by: Dan Carpenter <dan.carpenter@oracle.com>
New smatch warnings:
drivers/vdpa/virtio_pci/vp_vdpa.c:421 vp_vdpa_probe() warn: missing error code 'ret'
Old smatch warnings:
drivers/vdpa/virtio_pci/vp_vdpa.c:168 vp_vdpa_request_irq() warn: inconsistent indenting
vim +/ret +421 drivers/vdpa/virtio_pci/vp_vdpa.c
64b9f64f80a6f4 Jason Wang 2021-02-23 367 static int vp_vdpa_probe(struct pci_dev *pdev, const struct pci_device_id *id)
64b9f64f80a6f4 Jason Wang 2021-02-23 368 {
64b9f64f80a6f4 Jason Wang 2021-02-23 369 struct virtio_pci_modern_device *mdev;
64b9f64f80a6f4 Jason Wang 2021-02-23 370 struct device *dev = &pdev->dev;
64b9f64f80a6f4 Jason Wang 2021-02-23 371 struct vp_vdpa *vp_vdpa;
64b9f64f80a6f4 Jason Wang 2021-02-23 372 int ret, i;
64b9f64f80a6f4 Jason Wang 2021-02-23 373
64b9f64f80a6f4 Jason Wang 2021-02-23 374 ret = pcim_enable_device(pdev);
64b9f64f80a6f4 Jason Wang 2021-02-23 375 if (ret)
64b9f64f80a6f4 Jason Wang 2021-02-23 376 return ret;
64b9f64f80a6f4 Jason Wang 2021-02-23 377
64b9f64f80a6f4 Jason Wang 2021-02-23 378 vp_vdpa = vdpa_alloc_device(struct vp_vdpa, vdpa,
64b9f64f80a6f4 Jason Wang 2021-02-23 379 dev, &vp_vdpa_ops, NULL);
64b9f64f80a6f4 Jason Wang 2021-02-23 380 if (vp_vdpa == NULL) {
64b9f64f80a6f4 Jason Wang 2021-02-23 381 dev_err(dev, "vp_vdpa: Failed to allocate vDPA structure\n");
64b9f64f80a6f4 Jason Wang 2021-02-23 382 return -ENOMEM;
64b9f64f80a6f4 Jason Wang 2021-02-23 383 }
64b9f64f80a6f4 Jason Wang 2021-02-23 384
64b9f64f80a6f4 Jason Wang 2021-02-23 385 mdev = &vp_vdpa->mdev;
64b9f64f80a6f4 Jason Wang 2021-02-23 386 mdev->pci_dev = pdev;
64b9f64f80a6f4 Jason Wang 2021-02-23 387
64b9f64f80a6f4 Jason Wang 2021-02-23 388 ret = vp_modern_probe(mdev);
64b9f64f80a6f4 Jason Wang 2021-02-23 389 if (ret) {
64b9f64f80a6f4 Jason Wang 2021-02-23 390 dev_err(&pdev->dev, "Failed to probe modern PCI device\n");
64b9f64f80a6f4 Jason Wang 2021-02-23 391 goto err;
64b9f64f80a6f4 Jason Wang 2021-02-23 392 }
64b9f64f80a6f4 Jason Wang 2021-02-23 393
64b9f64f80a6f4 Jason Wang 2021-02-23 394 pci_set_master(pdev);
64b9f64f80a6f4 Jason Wang 2021-02-23 395 pci_set_drvdata(pdev, vp_vdpa);
64b9f64f80a6f4 Jason Wang 2021-02-23 396
64b9f64f80a6f4 Jason Wang 2021-02-23 397 vp_vdpa->vdpa.dma_dev = &pdev->dev;
64b9f64f80a6f4 Jason Wang 2021-02-23 398 vp_vdpa->queues = vp_modern_get_num_queues(mdev);
64b9f64f80a6f4 Jason Wang 2021-02-23 399
64b9f64f80a6f4 Jason Wang 2021-02-23 400 ret = devm_add_action_or_reset(dev, vp_vdpa_free_irq_vectors, pdev);
64b9f64f80a6f4 Jason Wang 2021-02-23 401 if (ret) {
64b9f64f80a6f4 Jason Wang 2021-02-23 402 dev_err(&pdev->dev,
64b9f64f80a6f4 Jason Wang 2021-02-23 403 "Failed for adding devres for freeing irq vectors\n");
64b9f64f80a6f4 Jason Wang 2021-02-23 404 goto err;
64b9f64f80a6f4 Jason Wang 2021-02-23 405 }
64b9f64f80a6f4 Jason Wang 2021-02-23 406
64b9f64f80a6f4 Jason Wang 2021-02-23 407 vp_vdpa->vring = devm_kcalloc(&pdev->dev, vp_vdpa->queues,
64b9f64f80a6f4 Jason Wang 2021-02-23 408 sizeof(*vp_vdpa->vring),
64b9f64f80a6f4 Jason Wang 2021-02-23 409 GFP_KERNEL);
64b9f64f80a6f4 Jason Wang 2021-02-23 410 if (!vp_vdpa->vring) {
64b9f64f80a6f4 Jason Wang 2021-02-23 411 ret = -ENOMEM;
64b9f64f80a6f4 Jason Wang 2021-02-23 412 dev_err(&pdev->dev, "Fail to allocate virtqueues\n");
64b9f64f80a6f4 Jason Wang 2021-02-23 413 goto err;
64b9f64f80a6f4 Jason Wang 2021-02-23 414 }
64b9f64f80a6f4 Jason Wang 2021-02-23 415
64b9f64f80a6f4 Jason Wang 2021-02-23 416 for (i = 0; i < vp_vdpa->queues; i++) {
64b9f64f80a6f4 Jason Wang 2021-02-23 417 vp_vdpa->vring[i].irq = VIRTIO_MSI_NO_VECTOR;
11d8ffed00b231 Jason Wang 2021-04-15 418 vp_vdpa->vring[i].notify = vp_modern_map_vq_notify(mdev, i);
11d8ffed00b231 Jason Wang 2021-04-15 419 if (!vp_vdpa->vring[i].notify) {
11d8ffed00b231 Jason Wang 2021-04-15 420 dev_warn(&pdev->dev, "Fail to map vq notify %d\n", i);
ret = -ENOMEM;?
11d8ffed00b231 Jason Wang 2021-04-15 @421 goto err;
11d8ffed00b231 Jason Wang 2021-04-15 422 }
64b9f64f80a6f4 Jason Wang 2021-02-23 423 }
64b9f64f80a6f4 Jason Wang 2021-02-23 424 vp_vdpa->config_irq = VIRTIO_MSI_NO_VECTOR;
64b9f64f80a6f4 Jason Wang 2021-02-23 425
64b9f64f80a6f4 Jason Wang 2021-02-23 426 ret = vdpa_register_device(&vp_vdpa->vdpa, vp_vdpa->queues);
64b9f64f80a6f4 Jason Wang 2021-02-23 427 if (ret) {
64b9f64f80a6f4 Jason Wang 2021-02-23 428 dev_err(&pdev->dev, "Failed to register to vdpa bus\n");
64b9f64f80a6f4 Jason Wang 2021-02-23 429 goto err;
64b9f64f80a6f4 Jason Wang 2021-02-23 430 }
64b9f64f80a6f4 Jason Wang 2021-02-23 431
64b9f64f80a6f4 Jason Wang 2021-02-23 432 return 0;
64b9f64f80a6f4 Jason Wang 2021-02-23 433
64b9f64f80a6f4 Jason Wang 2021-02-23 434 err:
64b9f64f80a6f4 Jason Wang 2021-02-23 435 put_device(&vp_vdpa->vdpa.dev);
64b9f64f80a6f4 Jason Wang 2021-02-23 436 return ret;
64b9f64f80a6f4 Jason Wang 2021-02-23 437 }
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org
_______________________________________________
kbuild mailing list -- kbuild(a)lists.01.org
To unsubscribe send an email to kbuild-leave(a)lists.01.org
next reply other threads:[~2021-06-23 9:21 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-06-23 4:13 kernel test robot [this message]
2021-06-23 9:21 ` [kbuild] drivers/vdpa/virtio_pci/vp_vdpa.c:421 vp_vdpa_probe() warn: missing error code 'ret' Dan Carpenter
2021-06-23 9:21 ` Dan Carpenter
2021-06-23 19:57 ` Michael S. Tsirkin
2021-06-23 19:57 ` Michael S. Tsirkin
2021-06-24 4:00 ` Jason Wang
2021-06-24 4:00 ` Jason Wang
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=202106231257.8kp6t8DF-lkp@intel.com \
--to=dan.carpenter@oracle.com \
--cc=elic@nvidia.com \
--cc=jasowang@redhat.com \
--cc=kbuild-all@lists.01.org \
--cc=kbuild@lists.01.org \
--cc=linux-kernel@vger.kernel.org \
--cc=lkp@intel.com \
--cc=mst@redhat.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.