On Sun, Sep 18, 2022 at 05:00:20PM +0800, Junbo wrote:

hi Michael

in virtio-net.c
    /* Parameters for control virtqueue, if any */
    if (vi->has_cvq) {
        callbacks[total_vqs - 1] = NULL;
        names[total_vqs - 1] = "control";
    }

I think the Author who write the code maybe want to name the control queue to
'virtioX-control',

That would be me I suspect ;)

but it never worked, we can see the name still be
'virtioX-config' in /proc/interrupts,

Nope, what you see in /proc/interrupts are the interrupts, not the queue
name.

for example 
 43:          0          0          0          0          0          0        
 0          0   PCI-MSI-edge      virtio0-config
 44:         64          0          0          0          0          0      
1845          0   PCI-MSI-edge      virtio0-input.0
 45:          1          0          0          0          0          0        
 0          0   PCI-MSI-edge      virtio0-output.0

Because in function vp_request_msix_vectors, it just allocate 'xxxx-config' to
every virtio devices, even the virtio device do not need it.

Oh yes, we can fix that. The result will be this line disappearing for
devices without a config interrupt. Not for net though, that
generally uses a config interrupt for things like link
state detection.

in /proc/
interrupts, we can see that each virtio device's first interrupt always named
'virtioX-config'.

So I think it's better to not explicitly give the "control" here, it's
useless...  

it's used for debugging.