motorhead/qemu
1
0
Fork 0
mirror of https://github.com/Motorhead1991/qemu.git synced 2025-08-05 16:53:55 -06:00
Code Issues Projects Releases Packages Wiki Activity Actions

vfio/container: Rename vfio_init_container to vfio_set_iommu

Browse source 
vfio_container_init() and vfio_init_container() names are confusing
especially when we see vfio_init_container() calls vfio_container_init().

vfio_container_init() operates on base container which is consistent
with all routines handling 'VFIOContainerBase *' ops.

vfio_init_container() operates on legacy container and setup IOMMU
context with ioctl(VFIO_SET_IOMMU).

So choose to rename vfio_init_container to vfio_set_iommu to avoid
the confusion.

No functional change intended.

Suggested-by: Cédric Le Goater <clg@redhat.com>
Signed-off-by: Zhenzhong Duan <zhenzhong.duan@intel.com>
Reviewed-by: Cédric Le Goater <clg@redhat.com>
This commit is contained in:
Zhenzhong Duan 2023-12-21 10:45:17 +08:00 • committed by Cédric Le Goater
parent 6ba254801f
commit 9f734a117c
1 changed files with 3 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

6
hw/vfio/container.c
View file

@ -392,8 +392,8 @@ static const VFIOIOMMUClass *vfio_get_iommu_class(int iommu_type, Error **errp)
return VFIO_IOMMU_CLASS(klass); return VFIO_IOMMU_CLASS(klass);
} }
static int vfio_init_container(VFIOContainer *container, int group_fd, static int vfio_set_iommu(VFIOContainer *container, int group_fd,
VFIOAddressSpace *space, Error **errp) VFIOAddressSpace *space, Error **errp)
{ {
int iommu_type, ret; int iommu_type, ret;
const VFIOIOMMUClass *vioc; const VFIOIOMMUClass *vioc;
@ -616,7 +616,7 @@ static int vfio_connect_container(VFIOGroup *group, AddressSpace *as,
container->fd = fd; container->fd = fd;
bcontainer = &container->bcontainer; bcontainer = &container->bcontainer;
ret = vfio_init_container(container, group->fd, space, errp); ret = vfio_set_iommu(container, group->fd, space, errp);
if (ret) { if (ret) {
goto free_container_exit; goto free_container_exit;
} }