]> git.proxmox.com Git - mirror_qemu.git/commit
vfio: Start improving VFIO/EEH interface
authorDavid Gibson <david@gibson.dropbear.id.au>
Wed, 9 Mar 2016 00:56:06 +0000 (11:56 +1100)
committerDavid Gibson <david@gibson.dropbear.id.au>
Tue, 15 Mar 2016 22:55:10 +0000 (09:55 +1100)
commit3153119e9bb79ebd82e08d2fde80d1c3a039b6b2
treeb7874a5faa5fd67c65fe3fa1e92aebf127fb686b
parentf1a6cf3ef734aab142d5f7ce52e219474ababf6b
vfio: Start improving VFIO/EEH interface

At present the code handling IBM's Enhanced Error Handling (EEH) interface
on VFIO devices operates by bypassing the usual VFIO logic with
vfio_container_ioctl().  That's a poorly designed interface with unclear
semantics about exactly what can be operated on.

In particular it operates on a single vfio container internally (hence the
name), but takes an address space and group id, from which it deduces the
container in a rather roundabout way.  groupids are something that code
outside vfio shouldn't even be aware of.

This patch creates new interfaces for EEH operations.  Internally we
have vfio_eeh_container_op() which takes a VFIOContainer object
directly.  For external use we have vfio_eeh_as_ok() which determines
if an AddressSpace is usable for EEH (at present this means it has a
single container with exactly one group attached), and vfio_eeh_as_op()
which will perform an operation on an AddressSpace in the unambiguous case,
and otherwise returns an error.

This interface still isn't great, but it's enough of an improvement to
allow a number of cleanups in other places.

Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
Reviewed-by: Alexey Kardashevskiy <aik@ozlabs.ru>
Acked-by: Alex Williamson <alex.williamson@redhat.com>
hw/vfio/common.c
include/hw/vfio/vfio.h