[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RFC v9 08/18] vfio: improve vfio_get_group to support
From: |
Alex Williamson |
Subject: |
Re: [Qemu-devel] [RFC v9 08/18] vfio: improve vfio_get_group to support adding group without devices |
Date: |
Tue, 09 Jun 2015 15:23:01 -0600 |
On Tue, 2015-06-09 at 11:37 +0800, Chen Fan wrote:
> Pre-adding all affected groups for aer devices, it could
> ensure the affected groups are owned in VM.
>
> Signed-off-by: Chen Fan <address@hidden>
> ---
> hw/vfio/common.c | 15 ++++++++++++---
> 1 file changed, 12 insertions(+), 3 deletions(-)
>
> diff --git a/hw/vfio/common.c b/hw/vfio/common.c
> index b1045da..4230f83 100644
> --- a/hw/vfio/common.c
> +++ b/hw/vfio/common.c
> @@ -793,8 +793,15 @@ VFIOGroup *vfio_get_group(int groupid, AddressSpace *as)
>
> QLIST_FOREACH(group, &vfio_group_list, next) {
> if (group->groupid == groupid) {
> + if (as && !group->container) {
> + if (vfio_connect_container(group, as)) {
> + error_report("vfio: failed to setup container for group
> %d",
> + groupid);
> + return NULL;
> + }
> + }
> /* Found it. Now is it already in the right context? */
> - if (group->container->space->as == as) {
> + if (!as || group->container->space->as == as) {
> return group;
> } else {
> error_report("vfio: group %d used in multiple address
> spaces",
> @@ -828,7 +835,7 @@ VFIOGroup *vfio_get_group(int groupid, AddressSpace *as)
> group->groupid = groupid;
> QLIST_INIT(&group->device_list);
>
> - if (vfio_connect_container(group, as)) {
> + if (as && vfio_connect_container(group, as)) {
> error_report("vfio: failed to setup container for group %d",
> groupid);
> goto close_fd_exit;
> }
> @@ -859,7 +866,9 @@ void vfio_put_group(VFIOGroup *group)
> }
>
> vfio_kvm_device_del_group(group);
> - vfio_disconnect_container(group);
> + if (group->container) {
> + vfio_disconnect_container(group);
> + }
> QLIST_REMOVE(group, next);
> trace_vfio_put_group(group->fd);
> close(group->fd);
It's an interesting idea, but should we instead separate the container
from vfio_get/put_group() to be done as a separate step? Using the
AddressSpace pointer is a bit obfuscated.
Also, this gets you the group, but the vfio_group_get_external_user()
interface used by vfio-pci to do a bus reset requires that the group is
active, with an iommu. If it didn't, we could potentially be resetting
a non-viable group, with devices still owned by the host or not isolated
by the iommu. So you're a step closer, but if we go back to the example
of a dual port NIC with isolated functions, QEMU may be able to get the
group for the second port, it may even be a viable group, but if the
second port doesn't eventually get connected to the VM, the bus reset
won't be allowed.
- [Qemu-devel] [RFC v9 00/18] vfio-pci: pass the aer error to guest, Chen Fan, 2015/06/08
- [Qemu-devel] [RFC v9 01/18] vfio: extract vfio_get_hot_reset_info as a single function, Chen Fan, 2015/06/08
- [Qemu-devel] [RFC v9 03/18] pcie: modify the capability size assert, Chen Fan, 2015/06/08
- [Qemu-devel] [RFC v9 04/18] vfio: make the 4 bytes aligned for capability size, Chen Fan, 2015/06/08
- [Qemu-devel] [RFC v9 02/18] vfio: squeeze out vfio_pci_do_hot_reset for support bus reset, Chen Fan, 2015/06/08
- [Qemu-devel] [RFC v9 06/18] aer: impove pcie_aer_init to support vfio device, Chen Fan, 2015/06/08
- [Qemu-devel] [RFC v9 05/18] vfio: add pcie extanded capability support, Chen Fan, 2015/06/08
- [Qemu-devel] [RFC v9 08/18] vfio: improve vfio_get_group to support adding group without devices, Chen Fan, 2015/06/08
- Re: [Qemu-devel] [RFC v9 08/18] vfio: improve vfio_get_group to support adding group without devices,
Alex Williamson <=
- [Qemu-devel] [RFC v9 09/18] vfio: add ref for group to support own affected groups, Chen Fan, 2015/06/08
- [Qemu-devel] [RFC v9 07/18] vfio: add aer support for vfio device, Chen Fan, 2015/06/08
- [Qemu-devel] [RFC v9 10/18] get all affected groups for each device support aer, Chen Fan, 2015/06/08
- [Qemu-devel] [RFC v9 12/18] pci: add bus reset_notifiers callbacks for host bus reset, Chen Fan, 2015/06/08
- [Qemu-devel] [RFC v9 14/18] vfio: improve vfio_pci_hot_reset to support more case, Chen Fan, 2015/06/08