ivshmem: generalize ivshmem_setup_interrupts
Call ivshmem_setup_interrupts() with or without MSI, always allocate msi_vectors that is going to be used in all case in the following patch. Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com> Reviewed-by: Markus Armbruster <armbru@redhat.com>
This commit is contained in:
		
							parent
							
								
									00ffc3c166
								
							
						
					
					
						commit
						fd47bfe5ad
					
				@ -769,18 +769,20 @@ static void ivshmem_reset(DeviceState *d)
 | 
				
			|||||||
    ivshmem_use_msix(s);
 | 
					    ivshmem_use_msix(s);
 | 
				
			||||||
}
 | 
					}
 | 
				
			||||||
 | 
					
 | 
				
			||||||
static int ivshmem_setup_msi(IVShmemState * s)
 | 
					static int ivshmem_setup_interrupts(IVShmemState *s)
 | 
				
			||||||
{
 | 
					{
 | 
				
			||||||
 | 
					    /* allocate QEMU callback data for receiving interrupts */
 | 
				
			||||||
 | 
					    s->msi_vectors = g_malloc0(s->vectors * sizeof(MSIVector));
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					    if (ivshmem_has_feature(s, IVSHMEM_MSI)) {
 | 
				
			||||||
        if (msix_init_exclusive_bar(PCI_DEVICE(s), s->vectors, 1)) {
 | 
					        if (msix_init_exclusive_bar(PCI_DEVICE(s), s->vectors, 1)) {
 | 
				
			||||||
            return -1;
 | 
					            return -1;
 | 
				
			||||||
        }
 | 
					        }
 | 
				
			||||||
 | 
					
 | 
				
			||||||
        IVSHMEM_DPRINTF("msix initialized (%d vectors)\n", s->vectors);
 | 
					        IVSHMEM_DPRINTF("msix initialized (%d vectors)\n", s->vectors);
 | 
				
			||||||
 | 
					 | 
				
			||||||
    /* allocate QEMU char devices for receiving interrupts */
 | 
					 | 
				
			||||||
    s->msi_vectors = g_malloc0(s->vectors * sizeof(MSIVector));
 | 
					 | 
				
			||||||
 | 
					 | 
				
			||||||
        ivshmem_use_msix(s);
 | 
					        ivshmem_use_msix(s);
 | 
				
			||||||
 | 
					    }
 | 
				
			||||||
 | 
					
 | 
				
			||||||
    return 0;
 | 
					    return 0;
 | 
				
			||||||
}
 | 
					}
 | 
				
			||||||
 | 
					
 | 
				
			||||||
@ -947,9 +949,8 @@ static void pci_ivshmem_realize(PCIDevice *dev, Error **errp)
 | 
				
			|||||||
        IVSHMEM_DPRINTF("using shared memory server (socket = %s)\n",
 | 
					        IVSHMEM_DPRINTF("using shared memory server (socket = %s)\n",
 | 
				
			||||||
                        s->server_chr->filename);
 | 
					                        s->server_chr->filename);
 | 
				
			||||||
 | 
					
 | 
				
			||||||
        if (ivshmem_has_feature(s, IVSHMEM_MSI) &&
 | 
					        if (ivshmem_setup_interrupts(s) < 0) {
 | 
				
			||||||
            ivshmem_setup_msi(s)) {
 | 
					            error_setg(errp, "failed to initialize interrupts");
 | 
				
			||||||
            error_setg(errp, "msix initialization failed");
 | 
					 | 
				
			||||||
            return;
 | 
					            return;
 | 
				
			||||||
        }
 | 
					        }
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
				
			|||||||
		Loading…
	
	
			
			x
			
			
		
	
		Reference in New Issue
	
	Block a user