[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v4 44/47] ivshmem: remove EventfdEntry.vector
From: |
Claudio Fontana |
Subject: |
Re: [Qemu-devel] [PATCH v4 44/47] ivshmem: remove EventfdEntry.vector |
Date: |
Tue, 29 Sep 2015 16:32:02 +0200 |
User-agent: |
Mozilla/5.0 (Windows NT 6.1; rv:38.0) Gecko/20100101 Thunderbird/38.2.0 |
On 24.09.2015 13:37, address@hidden wrote:
> From: Marc-André Lureau <address@hidden>
>
> No need to store an extra int for the vector number when it can be
> computed easily by looking at the position in the array.
>
> Signed-off-by: Marc-André Lureau <address@hidden>
> ---
> hw/misc/ivshmem.c | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/hw/misc/ivshmem.c b/hw/misc/ivshmem.c
> index 9628e14..619947e 100644
> --- a/hw/misc/ivshmem.c
> +++ b/hw/misc/ivshmem.c
> @@ -68,7 +68,6 @@ typedef struct Peer {
>
> typedef struct EventfdEntry {
> PCIDevice *pdev;
> - int vector;
> } EventfdEntry;
>
> typedef struct IVShmemState {
> @@ -287,9 +286,11 @@ static void fake_irqfd(void *opaque, const uint8_t *buf,
> int size) {
>
> EventfdEntry *entry = opaque;
> PCIDevice *pdev = entry->pdev;
> + IVShmemState *s = IVSHMEM(pdev);
> + int vector = entry - s->eventfd_table;
>
> - IVSHMEM_DPRINTF("interrupt on vector %p %d\n", pdev, entry->vector);
> - msix_notify(pdev, entry->vector);
> + IVSHMEM_DPRINTF("interrupt on vector %p %d\n", pdev, vector);
> + msix_notify(pdev, vector);
> }
>
> static CharDriverState* create_eventfd_chr_device(void * opaque,
> EventNotifier *n,
> @@ -311,7 +312,6 @@ static CharDriverState* create_eventfd_chr_device(void *
> opaque, EventNotifier *
> /* if MSI is supported we need multiple interrupts */
> if (ivshmem_has_feature(s, IVSHMEM_MSI)) {
> s->eventfd_table[vector].pdev = PCI_DEVICE(s);
> - s->eventfd_table[vector].vector = vector;
>
> qemu_chr_add_handlers(chr, ivshmem_can_receive, fake_irqfd,
> ivshmem_event, &s->eventfd_table[vector]);
>
Reviewed-by: Claudio Fontana <address@hidden>
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: [Qemu-devel] [PATCH v4 44/47] ivshmem: remove EventfdEntry.vector,
Claudio Fontana <=