virtio_pci_modern: correct sparse tags for notify
authorMichael S. Tsirkin <mst@redhat.com>
Tue, 4 May 2021 08:17:20 +0000 (04:17 -0400)
committerMichael S. Tsirkin <mst@redhat.com>
Tue, 4 May 2021 08:19:59 +0000 (04:19 -0400)
When switching virtio_pci_modern to use a helper for mappings we lost an
__iomem tag. Restore it.

Reported-by: kernel test robot <lkp@intel.com>
Fixes: 9e3bb9b79a71 ("virtio_pci_modern: introduce helper to map vq notify area")
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
drivers/virtio/virtio_pci_modern_dev.c
include/linux/virtio_pci_modern.h

index ae87b3f..54f2970 100644 (file)
@@ -605,8 +605,8 @@ static u16 vp_modern_get_queue_notify_off(struct virtio_pci_modern_device *mdev,
  *
  * Returns the address of the notification area
  */
-void *vp_modern_map_vq_notify(struct virtio_pci_modern_device *mdev,
-                             u16 index, resource_size_t *pa)
+void __iomem *vp_modern_map_vq_notify(struct virtio_pci_modern_device *mdev,
+                                     u16 index, resource_size_t *pa)
 {
        u16 off = vp_modern_get_queue_notify_off(mdev, index);
 
@@ -624,10 +624,9 @@ void *vp_modern_map_vq_notify(struct virtio_pci_modern_device *mdev,
                if (pa)
                        *pa = mdev->notify_pa +
                              off * mdev->notify_offset_multiplier;
-               return (void __force *)mdev->notify_base +
-                       off * mdev->notify_offset_multiplier;
+               return mdev->notify_base + off * mdev->notify_offset_multiplier;
        } else {
-               return (void __force *)vp_modern_map_capability(mdev,
+               return vp_modern_map_capability(mdev,
                                       mdev->notify_map_cap, 2, 2,
                                       off * mdev->notify_offset_multiplier, 2,
                                       NULL, pa);
index cdfabbe..6a95b58 100644 (file)
@@ -101,8 +101,8 @@ void vp_modern_set_queue_size(struct virtio_pci_modern_device *mdev,
 u16 vp_modern_get_queue_size(struct virtio_pci_modern_device *mdev,
                             u16 idx);
 u16 vp_modern_get_num_queues(struct virtio_pci_modern_device *mdev);
-void *vp_modern_map_vq_notify(struct virtio_pci_modern_device *mdev,
-                              u16 index, resource_size_t *pa);
+void __iomem * vp_modern_map_vq_notify(struct virtio_pci_modern_device *mdev,
+                                      u16 index, resource_size_t *pa);
 int vp_modern_probe(struct virtio_pci_modern_device *mdev);
 void vp_modern_remove(struct virtio_pci_modern_device *mdev);
 #endif