]> asedeno.scripts.mit.edu Git - linux.git/commit
vfio/spapr_tce: Skip unsetting already unset table
authorAlexey Kardashevskiy <aik@ozlabs.ru>
Mon, 11 Feb 2019 07:49:17 +0000 (18:49 +1100)
committerAlex Williamson <alex.williamson@redhat.com>
Wed, 13 Feb 2019 20:08:12 +0000 (13:08 -0700)
commita3906855890d94736d240f0f637585c1470d8d02
tree9b652b78d7482ef03b8d5a770190157ae02d933b
parent3e4835f7d7a84ab022682aaeecaae5c508aeb76c
vfio/spapr_tce: Skip unsetting already unset table

VFIO TCE IOMMU v2 owns IOMMU tables. When we detach an IOMMU group from
a container, we need to unset these tables from the group which we do by
calling unset_window(). We also unset tables when removing a DMA window
via the VFIO_IOMMU_SPAPR_TCE_REMOVE ioctl.

The window removal checks if the table actually exists (hidden inside
tce_iommu_find_table()) but the group detaching does not so the user
may see duplicating messages:
pci 0009:03     : [PE# fd] Removing DMA window #0
pci 0009:03     : [PE# fd] Removing DMA window #1
pci 0009:03     : [PE# fd] Removing DMA window #0
pci 0009:03     : [PE# fd] Removing DMA window #1

At the moment this is not a problem as the second invocation
of unset_window() writes zeroes to the HW registers again and exits early
as there is no table.

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