Skip to content

Commit

Permalink
ivshmem: generalize ivshmem_setup_interrupts
Browse files Browse the repository at this point in the history
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>
  • Loading branch information
elmarco committed Feb 2, 2016
1 parent 00ffc3c commit fd47bfe
Showing 1 changed file with 12 additions and 11 deletions.
23 changes: 12 additions & 11 deletions hw/misc/ivshmem.c
Expand Up @@ -769,18 +769,20 @@ static void ivshmem_reset(DeviceState *d)
ivshmem_use_msix(s);
}

static int ivshmem_setup_msi(IVShmemState * s)
static int ivshmem_setup_interrupts(IVShmemState *s)
{
if (msix_init_exclusive_bar(PCI_DEVICE(s), s->vectors, 1)) {
return -1;
}
/* allocate QEMU callback data for receiving interrupts */
s->msi_vectors = g_malloc0(s->vectors * sizeof(MSIVector));

IVSHMEM_DPRINTF("msix initialized (%d vectors)\n", s->vectors);
if (ivshmem_has_feature(s, IVSHMEM_MSI)) {
if (msix_init_exclusive_bar(PCI_DEVICE(s), s->vectors, 1)) {
return -1;
}

/* allocate QEMU char devices for receiving interrupts */
s->msi_vectors = g_malloc0(s->vectors * sizeof(MSIVector));
IVSHMEM_DPRINTF("msix initialized (%d vectors)\n", s->vectors);
ivshmem_use_msix(s);
}

ivshmem_use_msix(s);
return 0;
}

Expand Down Expand Up @@ -947,9 +949,8 @@ static void pci_ivshmem_realize(PCIDevice *dev, Error **errp)
IVSHMEM_DPRINTF("using shared memory server (socket = %s)\n",
s->server_chr->filename);

if (ivshmem_has_feature(s, IVSHMEM_MSI) &&
ivshmem_setup_msi(s)) {
error_setg(errp, "msix initialization failed");
if (ivshmem_setup_interrupts(s) < 0) {
error_setg(errp, "failed to initialize interrupts");
return;
}

Expand Down

0 comments on commit fd47bfe

Please sign in to comment.