Re: [PATCH net-next v4 2/5] virtio_net: enable irq for the control vq
From: "Michael S. Tsirkin" <mst@redhat.com>
Date: 2024-06-19 21:19:23
Also in:
virtualization
On Thu, Jun 20, 2024 at 12:19:05AM +0800, Heng Qi wrote:
quoted hunk ↗ jump to hunk
@@ -5312,7 +5315,7 @@ static int virtnet_find_vqs(struct virtnet_info *vi) /* Parameters for control virtqueue, if any */ if (vi->has_cvq) { - callbacks[total_vqs - 1] = NULL; + callbacks[total_vqs - 1] = virtnet_cvq_done; names[total_vqs - 1] = "control"; }
If the # of MSIX vectors is exactly for data path VQs, this will cause irq sharing between VQs which will degrade performance significantly. So no, you can not just do it unconditionally. The correct fix probably requires virtio core/API extensions. -- MST