vfio-ccw: use vfio_set_irq_signaling
Use the new helper. Reviewed-by: Eric Auger <eric.auger@redhat.com> Reviewed-by: Eric Farman <farman@linux.ibm.com> Message-Id: <20190617101036.4087-1-cohuck@redhat.com> Signed-off-by: Cornelia Huck <cohuck@redhat.com>
This commit is contained in:
parent
474f3938d7
commit
b5e89f044d
|
@ -198,9 +198,8 @@ static void vfio_ccw_register_io_notifier(VFIOCCWDevice *vcdev, Error **errp)
|
||||||
{
|
{
|
||||||
VFIODevice *vdev = &vcdev->vdev;
|
VFIODevice *vdev = &vcdev->vdev;
|
||||||
struct vfio_irq_info *irq_info;
|
struct vfio_irq_info *irq_info;
|
||||||
struct vfio_irq_set *irq_set;
|
|
||||||
size_t argsz;
|
size_t argsz;
|
||||||
int32_t *pfd;
|
int fd;
|
||||||
|
|
||||||
if (vdev->num_irqs < VFIO_CCW_IO_IRQ_INDEX + 1) {
|
if (vdev->num_irqs < VFIO_CCW_IO_IRQ_INDEX + 1) {
|
||||||
error_setg(errp, "vfio: unexpected number of io irqs %u",
|
error_setg(errp, "vfio: unexpected number of io irqs %u",
|
||||||
|
@ -224,56 +223,32 @@ static void vfio_ccw_register_io_notifier(VFIOCCWDevice *vcdev, Error **errp)
|
||||||
goto out_free_info;
|
goto out_free_info;
|
||||||
}
|
}
|
||||||
|
|
||||||
argsz = sizeof(*irq_set) + sizeof(*pfd);
|
fd = event_notifier_get_fd(&vcdev->io_notifier);
|
||||||
irq_set = g_malloc0(argsz);
|
qemu_set_fd_handler(fd, vfio_ccw_io_notifier_handler, NULL, vcdev);
|
||||||
irq_set->argsz = argsz;
|
|
||||||
irq_set->flags = VFIO_IRQ_SET_DATA_EVENTFD |
|
|
||||||
VFIO_IRQ_SET_ACTION_TRIGGER;
|
|
||||||
irq_set->index = VFIO_CCW_IO_IRQ_INDEX;
|
|
||||||
irq_set->start = 0;
|
|
||||||
irq_set->count = 1;
|
|
||||||
pfd = (int32_t *) &irq_set->data;
|
|
||||||
|
|
||||||
*pfd = event_notifier_get_fd(&vcdev->io_notifier);
|
if (vfio_set_irq_signaling(vdev, VFIO_CCW_IO_IRQ_INDEX, 0,
|
||||||
qemu_set_fd_handler(*pfd, vfio_ccw_io_notifier_handler, NULL, vcdev);
|
VFIO_IRQ_SET_ACTION_TRIGGER, fd, errp)) {
|
||||||
if (ioctl(vdev->fd, VFIO_DEVICE_SET_IRQS, irq_set)) {
|
qemu_set_fd_handler(fd, NULL, NULL, vcdev);
|
||||||
error_setg(errp, "vfio: Failed to set up io notification");
|
|
||||||
qemu_set_fd_handler(*pfd, NULL, NULL, vcdev);
|
|
||||||
event_notifier_cleanup(&vcdev->io_notifier);
|
event_notifier_cleanup(&vcdev->io_notifier);
|
||||||
}
|
}
|
||||||
|
|
||||||
g_free(irq_set);
|
|
||||||
|
|
||||||
out_free_info:
|
out_free_info:
|
||||||
g_free(irq_info);
|
g_free(irq_info);
|
||||||
}
|
}
|
||||||
|
|
||||||
static void vfio_ccw_unregister_io_notifier(VFIOCCWDevice *vcdev)
|
static void vfio_ccw_unregister_io_notifier(VFIOCCWDevice *vcdev)
|
||||||
{
|
{
|
||||||
struct vfio_irq_set *irq_set;
|
Error *err = NULL;
|
||||||
size_t argsz;
|
|
||||||
int32_t *pfd;
|
|
||||||
|
|
||||||
argsz = sizeof(*irq_set) + sizeof(*pfd);
|
vfio_set_irq_signaling(&vcdev->vdev, VFIO_CCW_IO_IRQ_INDEX, 0,
|
||||||
irq_set = g_malloc0(argsz);
|
VFIO_IRQ_SET_ACTION_TRIGGER, -1, &err);
|
||||||
irq_set->argsz = argsz;
|
if (err) {
|
||||||
irq_set->flags = VFIO_IRQ_SET_DATA_EVENTFD |
|
error_reportf_err(err, VFIO_MSG_PREFIX, vcdev->vdev.name);
|
||||||
VFIO_IRQ_SET_ACTION_TRIGGER;
|
|
||||||
irq_set->index = VFIO_CCW_IO_IRQ_INDEX;
|
|
||||||
irq_set->start = 0;
|
|
||||||
irq_set->count = 1;
|
|
||||||
pfd = (int32_t *) &irq_set->data;
|
|
||||||
*pfd = -1;
|
|
||||||
|
|
||||||
if (ioctl(vcdev->vdev.fd, VFIO_DEVICE_SET_IRQS, irq_set)) {
|
|
||||||
error_report("vfio: Failed to de-assign device io fd: %m");
|
|
||||||
}
|
}
|
||||||
|
|
||||||
qemu_set_fd_handler(event_notifier_get_fd(&vcdev->io_notifier),
|
qemu_set_fd_handler(event_notifier_get_fd(&vcdev->io_notifier),
|
||||||
NULL, NULL, vcdev);
|
NULL, NULL, vcdev);
|
||||||
event_notifier_cleanup(&vcdev->io_notifier);
|
event_notifier_cleanup(&vcdev->io_notifier);
|
||||||
|
|
||||||
g_free(irq_set);
|
|
||||||
}
|
}
|
||||||
|
|
||||||
static void vfio_ccw_get_region(VFIOCCWDevice *vcdev, Error **errp)
|
static void vfio_ccw_get_region(VFIOCCWDevice *vcdev, Error **errp)
|
||||||
|
|
Loading…
Reference in a new issue