[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 1/2] vfio-pci: Introduce vfio_register_event_not
From: |
Cornelia Huck |
Subject: |
Re: [Qemu-devel] [PATCH 1/2] vfio-pci: Introduce vfio_register_event_notifier helper |
Date: |
Thu, 17 Jan 2019 09:59:56 +0100 |
On Thu, 17 Jan 2019 14:46:42 +1100
Alexey Kardashevskiy <address@hidden> wrote:
> On 12/01/2019 03:58, Eric Auger wrote:
> > The code used to attach the eventfd handler for the ERR and
> > REQ irq indices can be factorized into a helper. In subsequent
> > patches we will extend this helper to support other irq indices.
> >
> > We test the notification is allowed outside of the helper:
> > respectively check vdev->pci_aer and VFIO_FEATURE_ENABLE_REQ.
> > Depending on the returned value we set vdev->pci_aer and
> > vdev->req_enabled. An error handle is introduced for future usage
> > although not strictly useful here.>
> > Signed-off-by: Eric Auger <address@hidden>
> > ---
> > hw/vfio/pci.c | 291 ++++++++++++++++++++++----------------------------
> > 1 file changed, 127 insertions(+), 164 deletions(-)
> >
> > diff --git a/hw/vfio/pci.c b/hw/vfio/pci.c
> > index c0cb1ec289..c589a4e666 100644
> > --- a/hw/vfio/pci.c
> > +++ b/hw/vfio/pci.c
> > @@ -105,6 +105,95 @@ static void vfio_intx_eoi(VFIODevice *vbasedev)
> > vfio_unmask_single_irqindex(vbasedev, VFIO_PCI_INTX_IRQ_INDEX);
> > }
> >
> > +/*
> > + * vfio_register_event_notifier - setup/tear down eventfd
> > + * notification and handling for IRQ indices that span over
> > + * a single IRQ
> > + *
> > + * @vdev: VFIO device handle
> > + * @index: IRQ index the eventfd/handler is associated to
> > + * @target_state: true means notifier needs to be set up
> > + * @handler to attach if @target_state is true
> > + * @errp error handle
> > + */
> > +static int vfio_register_event_notifier(VFIOPCIDevice *vdev,
> > + int index,
> > + bool target_state,
> > + void (*handler)(void *opaque),
> > + Error **errp)
> > +{
> > + struct vfio_irq_info irq_info = { .argsz = sizeof(irq_info),
> > + .index = index };
> > + struct vfio_irq_set *irq_set;
> > + EventNotifier *notifier;
> > + int argsz, ret = 0;
> > + int32_t *pfd, fd;
> > +
> > + switch (index) {
>
> I'd pass the notifier as a parameter as well so index/handler/notifier
> would walk together.
I first had the same thought, but I actually like that everything is
together in this switch statement.
Alternatively, pack it together in an indexed array? But that gets a
bit awkward with the intx stuff.
>
>
> > + case VFIO_PCI_REQ_IRQ_INDEX:
> > + notifier = &vdev->req_notifier;
> > + break;
> > + case VFIO_PCI_ERR_IRQ_INDEX:
> > + notifier = &vdev->err_notifier;
> > + break;
> > + default:
> > + return -EINVAL;
> > + }
[Qemu-devel] [PATCH 2/2] vfio-pci: Use vfio_register_event_notifier in vfio_intx_enable_kvm, Eric Auger, 2019/01/11