Displaying 3 results from an estimated 3 matches for "vp_modern_queue_vector".
2023 Jun 30
2
[PATCH] vp_vdpa: synchronize irq when free irq
...281287fae89f..d8ee3e68cd2d 100644
--- a/drivers/vdpa/virtio_pci/vp_vdpa.c
+++ b/drivers/vdpa/virtio_pci/vp_vdpa.c
@@ -116,6 +116,7 @@ static void vp_vdpa_free_irq(struct vp_vdpa *vp_vdpa)
 	for (i = 0; i < vp_vdpa->queues; i++) {
 		if (vp_vdpa->vring[i].irq != VIRTIO_MSI_NO_VECTOR) {
 			vp_modern_queue_vector(mdev, i, VIRTIO_MSI_NO_VECTOR);
+			synchronize_irq(vp_vdpa->vring[i].irq);
 			devm_free_irq(&pdev->dev, vp_vdpa->vring[i].irq,
 				      &vp_vdpa->vring[i]);
 			vp_vdpa->vring[i].irq = VIRTIO_MSI_NO_VECTOR;
@@ -124,6 +125,7 @@ static void vp_vdpa_free_irq(struct vp_vdpa *...
2023 Jun 30
1
[PATCH] vp_vdpa: synchronize irq when free irq
...0644
> --- a/drivers/vdpa/virtio_pci/vp_vdpa.c
> +++ b/drivers/vdpa/virtio_pci/vp_vdpa.c
> @@ -116,6 +116,7 @@ static void vp_vdpa_free_irq(struct vp_vdpa *vp_vdpa)
>  	for (i = 0; i < vp_vdpa->queues; i++) {
>  		if (vp_vdpa->vring[i].irq != VIRTIO_MSI_NO_VECTOR) {
>  			vp_modern_queue_vector(mdev, i, VIRTIO_MSI_NO_VECTOR);
> +			synchronize_irq(vp_vdpa->vring[i].irq);
>  			devm_free_irq(&pdev->dev, vp_vdpa->vring[i].irq,
>  				      &vp_vdpa->vring[i]);
>  			vp_vdpa->vring[i].irq = VIRTIO_MSI_NO_VECTOR;
> @@ -124,6 +125,7 @@ static void vp_vdpa_...
2023 Jun 30
1
[PATCH] vp_vdpa: synchronize irq when free irq
.../vp_vdpa.c
> +++ b/drivers/vdpa/virtio_pci/vp_vdpa.c
> @@ -116,6 +116,7 @@ static void vp_vdpa_free_irq(struct vp_vdpa *vp_vdpa)
>         for (i = 0; i < vp_vdpa->queues; i++) {
>                 if (vp_vdpa->vring[i].irq != VIRTIO_MSI_NO_VECTOR) {
>                         vp_modern_queue_vector(mdev, i, VIRTIO_MSI_NO_VECTOR);
> +                       synchronize_irq(vp_vdpa->vring[i].irq);
Interestingly, we don't do this in virtio_pci_common.c and we only
synchronize during reset there.
Any reason makes vp_vdpa different?
Thanks
>                         devm_free_irq(&a...