[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 44/46] ivshmem: remove EventfdEntry.vector
From: |
Claudio Fontana |
Subject: |
Re: [Qemu-devel] [PATCH v3 44/46] ivshmem: remove EventfdEntry.vector |
Date: |
Tue, 22 Sep 2015 16:59:37 +0200 |
User-agent: |
Mozilla/5.0 (Windows NT 6.1; rv:38.0) Gecko/20100101 Thunderbird/38.2.0 |
On 15.09.2015 18:08, 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>
This one does not seem particularly valuable to me.. btw isn't it slightly
easier to debug with the vector in the structure?
> ---
> 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 ac90f0a..976fbea 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]);
>
- [Qemu-devel] [PATCH v3 41/46] ivshmem: do not keep shm_fd open, (continued)
- [Qemu-devel] [PATCH v3 41/46] ivshmem: do not keep shm_fd open, marcandre . lureau, 2015/09/15
- [Qemu-devel] [PATCH v3 42/46] ivshmem: make ivshmem_get_size() more generic, marcandre . lureau, 2015/09/15
- [Qemu-devel] [PATCH v3 43/46] ivshmem: add hostmem backend, marcandre . lureau, 2015/09/15
- [Qemu-devel] [PATCH v3 44/46] ivshmem: remove EventfdEntry.vector, marcandre . lureau, 2015/09/15
- Re: [Qemu-devel] [PATCH v3 44/46] ivshmem: remove EventfdEntry.vector,
Claudio Fontana <=
- [Qemu-devel] [PATCH v3 45/46] ivshmem: rename MSI eventfd_table, marcandre . lureau, 2015/09/15
- [Qemu-devel] [PATCH v3 46/46] ivshmem: use kvm irqfd for msi notifications, marcandre . lureau, 2015/09/15
- Re: [Qemu-devel] [PATCH v3 00/46] ivshmem improvements (please review), Claudio Fontana, 2015/09/16
- Re: [Qemu-devel] [PATCH v3 00/46] ivshmem improvements (please review), Vladimir Sementsov-Ogievskiy, 2015/09/16