On 14.03.2023 11:24, Jan Beulich wrote: > On 14.03.2023 02:50, Tian, Kevin wrote: >>> From: Marek Marczykowski-Górecki <[email protected]> >>> Sent: Tuesday, March 14, 2023 9:32 AM >>> >>> If the scope for IGD's IOMMU contains additional device that doesn't >>> actually exist, iommu=no-igfx would not disable that IOMMU. In this >>> particular case (Thinkpad x230) it included >>> 00:02.1, but there is no such device on this platform. >>> Consider only existing devices for "gfx only" check. >>> >>> Fixes: 2d7f191b392e ("VT-d: generalize and correct "iommu=no-igfx" >>> handling") >>> Signed-off-by: Marek Marczykowski-Górecki >>> <[email protected]> >> >> this should be rebased on top of Jan's patch. > > Right - I guess I could take care of that while applying. But I wonder > whether the description wouldn't then want adjusting some as well. Or > wait, with the v2 change it should actually have been adjusted already, > as the igd_drhd_address determination is now (intentionally) also > affected.
Since it wasn't clear if/when a v3 would appear, I've done the adjustments myself even if originally I didn't mean to. I've also adjusted the title a little, first and foremost to replace "phantom device" (which has an entirely different meaning in PCI). Jan
