On Tue, Mar 29, 2022 at 02:12:40PM +0000, Jag Raman wrote:
> > On Mar 29, 2022, at 8:35 AM, Stefan Hajnoczi <stefa...@redhat.com> wrote:
> > On Fri, Mar 25, 2022 at 03:19:41PM -0400, Jagannathan Raman wrote:
> >> +void remote_iommu_del_device(PCIDevice *pci_dev)
> >> +{
> >> + int pci_bdf;
> >> +
> >> + if (!remote_iommu_table.elem_by_bdf || !pci_dev) {
> >> + return;
> >> + }
> >> +
> >> + pci_bdf = PCI_BUILD_BDF(pci_bus_num(pci_get_bus(pci_dev)),
> >> pci_dev->devfn);
> >> +
> >> + qemu_mutex_lock(&remote_iommu_table.lock);
> >> + g_hash_table_remove(remote_iommu_table.elem_by_bdf,
> >> INT2VOIDP(pci_bdf));
> >> + qemu_mutex_unlock(&remote_iommu_table.lock);
> >> +}
> >> +
> >> +void remote_configure_iommu(PCIBus *pci_bus)
> >> +{
> >> + if (!remote_iommu_table.elem_by_bdf) {
> >> + remote_iommu_table.elem_by_bdf =
> >> + g_hash_table_new_full(NULL, NULL, NULL,
> >> remote_iommu_del_elem);
> >> + qemu_mutex_init(&remote_iommu_table.lock);
> >> + }
> >> +
> >> + pci_setup_iommu(pci_bus, remote_iommu_find_add_as,
> >> &remote_iommu_table);
> >
> > Why is remote_iommu_table global? It could be per-PCIBus and indexed by
> > just devfn instead of the full BDF.
>
> It’s global because remote_iommu_del_device() needs it for cleanup.Can remote_iommu_del_device() use pci_get_bis(pci_dev)->irq_opaque to get the per-bus table? Stefan
signature.asc
Description: PGP signature
