* David Woodhouse (dwmw2@xxxxxxxxxxxxx) wrote: > On Tue, 2011-06-07 at 08:10 -0700, Chris Wright wrote: > > * David Woodhouse (dwmw2@xxxxxxxxxxxxx) wrote: > > > On Tue, 2011-06-07 at 06:38 -0700, Chris Wright wrote: > > > > I think we still leak the list entry though. Bottom line is that we > > > > need to handle hotplug ADD_DEVICE and DEL_DEVICE notifications. We > > > > happen to pick up ADD_DEVICE by accident, but it's all pretty sloppy. > > > > > > Yeah, keeping a list of possible stale 'pci_dev' pointers is stupid. We > > > should figure out the matching DMAR unit directly from the ACPI table at > > > ADD_DEVICE time, and store it in pdev->archdata.iommu. > > > > > > I saw patches which were going in that direction... > > > > Cool, where are they? I'm working on something similar, and missed them. > > [PATCH] pci, dmar: Update dmar units devices list during hotplug Oh yeah, thanks for the reminder. thanks, -chris -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html