| Message ID | 1618886913-6594-1-git-send-email-bingbu.cao@intel.com |
|---|---|
| State | New |
| Headers | show |
| Series | [RESEND,v2] iommu/vt-d: Use passthrough mode for the Intel IPUs | expand |
Hi Bingbu, Thanks for the patch. On Tue, Apr 20, 2021 at 10:48:33AM +0800, Bingbu Cao wrote: > Intel IPU(Image Processing Unit) has its own (IO)MMU hardware, > The IPU driver allocates its own page table that is not mapped > via the DMA, and thus the Intel IOMMU driver blocks access giving > this error: The page table should be mapped to the possible IOMMU using the DMA API. > > DMAR: DRHD: handling fault status reg 3 > DMAR: [DMA Read] Request device [00:05.0] PASID ffffffff > fault addr 76406000 [fault reason 06] PTE Read access is not set > > As IPU is not an external facing device which is not risky, so use > IOMMU passthrough mode for Intel IPUs. I think a factor here is that the page tables aren't accessible by the IPU firmware. > > Fixes: 26f5689592e2 ("media: staging/intel-ipu3: mmu: Implement driver") > Signed-off-by: Bingbu Cao <bingbu.cao@intel.com> > --- > drivers/iommu/intel/iommu.c | 29 +++++++++++++++++++++++++++++ > 1 file changed, 29 insertions(+) > > diff --git a/drivers/iommu/intel/iommu.c b/drivers/iommu/intel/iommu.c > index ee0932307d64..7e2fbdae467e 100644 > --- a/drivers/iommu/intel/iommu.c > +++ b/drivers/iommu/intel/iommu.c > @@ -55,6 +55,12 @@ > #define IS_GFX_DEVICE(pdev) ((pdev->class >> 16) == PCI_BASE_CLASS_DISPLAY) > #define IS_USB_DEVICE(pdev) ((pdev->class >> 8) == PCI_CLASS_SERIAL_USB) > #define IS_ISA_DEVICE(pdev) ((pdev->class >> 8) == PCI_CLASS_BRIDGE_ISA) > +#define IS_INTEL_IPU(pdev) ((pdev)->vendor == PCI_VENDOR_ID_INTEL && \ > + ((pdev)->device == 0x9a19 || \ > + (pdev)->device == 0x9a39 || \ > + (pdev)->device == 0x4e19 || \ > + (pdev)->device == 0x465d || \ > + (pdev)->device == 0x1919)) > #define IS_AZALIA(pdev) ((pdev)->vendor == 0x8086 && (pdev)->device == 0x3a3e) > > #define IOAPIC_RANGE_START (0xfee00000) > @@ -360,6 +366,7 @@ int intel_iommu_enabled = 0; > EXPORT_SYMBOL_GPL(intel_iommu_enabled); > > static int dmar_map_gfx = 1; > +static int dmar_map_ipu = 1; This works as long as there's only one IPU. Same for graphics. But I guess this can be reworked in the future if the presumption changes. > static int dmar_forcedac; > static int intel_iommu_strict; > static int intel_iommu_superpage = 1; > @@ -368,6 +375,7 @@ static int iommu_skip_te_disable; > > #define IDENTMAP_GFX 2 > #define IDENTMAP_AZALIA 4 > +#define IDENTMAP_IPU 8 > > int intel_iommu_gfx_mapped; > EXPORT_SYMBOL_GPL(intel_iommu_gfx_mapped); > @@ -2839,6 +2847,9 @@ static int device_def_domain_type(struct device *dev) > > if ((iommu_identity_mapping & IDENTMAP_GFX) && IS_GFX_DEVICE(pdev)) > return IOMMU_DOMAIN_IDENTITY; > + > + if ((iommu_identity_mapping & IDENTMAP_IPU) && IS_INTEL_IPU(pdev)) > + return IOMMU_DOMAIN_IDENTITY; > } > > return 0; > @@ -3278,6 +3289,9 @@ static int __init init_dmars(void) > if (!dmar_map_gfx) > iommu_identity_mapping |= IDENTMAP_GFX; > > + if (!dmar_map_ipu) > + iommu_identity_mapping |= IDENTMAP_IPU; > + > check_tylersburg_isoch(); > > ret = si_domain_init(hw_pass_through); > @@ -5622,6 +5636,18 @@ static void quirk_iommu_igfx(struct pci_dev *dev) > dmar_map_gfx = 0; > } > > +static void quirk_iommu_ipu(struct pci_dev *dev) > +{ > + if (!IS_INTEL_IPU(dev)) > + return; > + > + if (risky_device(dev)) > + return; > + > + pci_info(dev, "Passthrough IOMMU for integrated Intel IPU\n"); > + dmar_map_ipu = 0; > +} > + > /* G4x/GM45 integrated gfx dmar support is totally busted. */ > DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_INTEL, 0x2a40, quirk_iommu_igfx); > DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_INTEL, 0x2e00, quirk_iommu_igfx); > @@ -5657,6 +5683,9 @@ DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_INTEL, 0x1632, quirk_iommu_igfx); > DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_INTEL, 0x163A, quirk_iommu_igfx); > DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_INTEL, 0x163D, quirk_iommu_igfx); > > +/* disable IPU dmar support */ > +DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_INTEL, PCI_ANY_ID, quirk_iommu_ipu); > + > static void quirk_iommu_rwbf(struct pci_dev *dev) > { > if (risky_device(dev))
On Tue, Apr 20, 2021 at 10:48:33AM +0800, Bingbu Cao wrote: > Intel IPU(Image Processing Unit) has its own (IO)MMU hardware, > The IPU driver allocates its own page table that is not mapped > via the DMA, and thus the Intel IOMMU driver blocks access giving > this error: > > DMAR: DRHD: handling fault status reg 3 > DMAR: [DMA Read] Request device [00:05.0] PASID ffffffff > fault addr 76406000 [fault reason 06] PTE Read access is not set > > As IPU is not an external facing device which is not risky, so use > IOMMU passthrough mode for Intel IPUs. > > Fixes: 26f5689592e2 ("media: staging/intel-ipu3: mmu: Implement driver") > Signed-off-by: Bingbu Cao <bingbu.cao@intel.com> > --- > drivers/iommu/intel/iommu.c | 29 +++++++++++++++++++++++++++++ This misses the changelog from v1 followed by the explanation why resent.
On Tue, Apr 20, 2021 at 10:48:33AM +0800, Bingbu Cao wrote: > Intel IPU(Image Processing Unit) has its own (IO)MMU hardware, > The IPU driver allocates its own page table that is not mapped > via the DMA, and thus the Intel IOMMU driver blocks access giving > this error: > > DMAR: DRHD: handling fault status reg 3 > DMAR: [DMA Read] Request device [00:05.0] PASID ffffffff > fault addr 76406000 [fault reason 06] PTE Read access is not set > > As IPU is not an external facing device which is not risky, so use > IOMMU passthrough mode for Intel IPUs. I'm wondering if IPU MMU should be described properly in the DMAR table.
Andy, On 4/20/21 6:20 PM, Andy Shevchenko wrote: > On Tue, Apr 20, 2021 at 10:48:33AM +0800, Bingbu Cao wrote: >> Intel IPU(Image Processing Unit) has its own (IO)MMU hardware, >> The IPU driver allocates its own page table that is not mapped >> via the DMA, and thus the Intel IOMMU driver blocks access giving >> this error: >> >> DMAR: DRHD: handling fault status reg 3 >> DMAR: [DMA Read] Request device [00:05.0] PASID ffffffff >> fault addr 76406000 [fault reason 06] PTE Read access is not set >> >> As IPU is not an external facing device which is not risky, so use >> IOMMU passthrough mode for Intel IPUs. >> >> Fixes: 26f5689592e2 ("media: staging/intel-ipu3: mmu: Implement driver") >> Signed-off-by: Bingbu Cao <bingbu.cao@intel.com> >> --- >> drivers/iommu/intel/iommu.c | 29 +++++++++++++++++++++++++++++ > > This misses the changelog from v1 followed by the explanation why resent. > I noticed there was a typo in the recipient list: stable.vger.kernel.org -> stable@vger.kernel.org no code change for resent.
Hi Bingbu, On Tue, Apr 20, 2021 at 06:34:26PM +0800, Bingbu Cao wrote: > Andy, > > On 4/20/21 6:20 PM, Andy Shevchenko wrote: > > On Tue, Apr 20, 2021 at 10:48:33AM +0800, Bingbu Cao wrote: > >> Intel IPU(Image Processing Unit) has its own (IO)MMU hardware, > >> The IPU driver allocates its own page table that is not mapped > >> via the DMA, and thus the Intel IOMMU driver blocks access giving > >> this error: > >> > >> DMAR: DRHD: handling fault status reg 3 > >> DMAR: [DMA Read] Request device [00:05.0] PASID ffffffff > >> fault addr 76406000 [fault reason 06] PTE Read access is not set > >> > >> As IPU is not an external facing device which is not risky, so use > >> IOMMU passthrough mode for Intel IPUs. > >> > >> Fixes: 26f5689592e2 ("media: staging/intel-ipu3: mmu: Implement driver") > >> Signed-off-by: Bingbu Cao <bingbu.cao@intel.com> > >> --- > >> drivers/iommu/intel/iommu.c | 29 +++++++++++++++++++++++++++++ > > > > This misses the changelog from v1 followed by the explanation why resent. > > > I noticed there was a typo in the recipient list: > stable.vger.kernel.org -> stable@vger.kernel.org > > no code change for resent. When you're submitting a patch and want it reach the stable kernels, you'll need to add a Cc tag: Cc: stable@vger.kernel.org But not actually add the address to cc. I dropped stable@vger address from distribution. Please change this for v3.
On Tue, Apr 20, 2021 at 01:56:40PM +0300, Sakari Ailus wrote: > On Tue, Apr 20, 2021 at 06:34:26PM +0800, Bingbu Cao wrote: > > On 4/20/21 6:20 PM, Andy Shevchenko wrote: > > > On Tue, Apr 20, 2021 at 10:48:33AM +0800, Bingbu Cao wrote: ... > > > This misses the changelog from v1 followed by the explanation why resent. > > > > > I noticed there was a typo in the recipient list: > > stable.vger.kernel.org -> stable@vger.kernel.org > > > > no code change for resent. > > When you're submitting a patch and want it reach the stable kernels, you'll > need to add a Cc tag: > > Cc: stable@vger.kernel.org > > But not actually add the address to cc. I dropped stable@vger address from > distribution. Does it really matter?
On Tue, Apr 20, 2021 at 02:55:33PM +0300, Andy Shevchenko wrote: > On Tue, Apr 20, 2021 at 01:56:40PM +0300, Sakari Ailus wrote: > > On Tue, Apr 20, 2021 at 06:34:26PM +0800, Bingbu Cao wrote: > > > On 4/20/21 6:20 PM, Andy Shevchenko wrote: > > > > On Tue, Apr 20, 2021 at 10:48:33AM +0800, Bingbu Cao wrote: > > ... > > > > > This misses the changelog from v1 followed by the explanation why resent. > > > > > > > I noticed there was a typo in the recipient list: > > > stable.vger.kernel.org -> stable@vger.kernel.org > > > > > > no code change for resent. > > > > When you're submitting a patch and want it reach the stable kernels, you'll > > need to add a Cc tag: > > > > Cc: stable@vger.kernel.org > > > > But not actually add the address to cc. I dropped stable@vger address from > > distribution. > > Does it really matter? Usually aligning what you're doing with Documentation/process/submitting-patches.rst is not a bad idea.
On Tue, Apr 20, 2021 at 05:37:27PM +0300, Sakari Ailus wrote: > On Tue, Apr 20, 2021 at 02:55:33PM +0300, Andy Shevchenko wrote: > > On Tue, Apr 20, 2021 at 01:56:40PM +0300, Sakari Ailus wrote: > > > On Tue, Apr 20, 2021 at 06:34:26PM +0800, Bingbu Cao wrote: > > > > On 4/20/21 6:20 PM, Andy Shevchenko wrote: > > > > > On Tue, Apr 20, 2021 at 10:48:33AM +0800, Bingbu Cao wrote: > > > > ... > > > > > > > This misses the changelog from v1 followed by the explanation why resent. > > > > > > > > > I noticed there was a typo in the recipient list: > > > > stable.vger.kernel.org -> stable@vger.kernel.org > > > > > > > > no code change for resent. > > > > > > When you're submitting a patch and want it reach the stable kernels, you'll > > > need to add a Cc tag: > > > > > > Cc: stable@vger.kernel.org > > > > > > But not actually add the address to cc. I dropped stable@vger address from > > > distribution. > > > > Does it really matter? > > Usually aligning what you're doing with > Documentation/process/submitting-patches.rst is not a bad idea. True, my point is that technically both ways will give the same result, no?
On Tue, Apr 20, 2021 at 05:54:57PM +0300, Andy Shevchenko wrote: > On Tue, Apr 20, 2021 at 05:37:27PM +0300, Sakari Ailus wrote: > > On Tue, Apr 20, 2021 at 02:55:33PM +0300, Andy Shevchenko wrote: > > > On Tue, Apr 20, 2021 at 01:56:40PM +0300, Sakari Ailus wrote: > > > > On Tue, Apr 20, 2021 at 06:34:26PM +0800, Bingbu Cao wrote: > > > > > On 4/20/21 6:20 PM, Andy Shevchenko wrote: > > > > > > On Tue, Apr 20, 2021 at 10:48:33AM +0800, Bingbu Cao wrote: > > > > > > ... > > > > > > > > > This misses the changelog from v1 followed by the explanation why resent. > > > > > > > > > > > I noticed there was a typo in the recipient list: > > > > > stable.vger.kernel.org -> stable@vger.kernel.org > > > > > > > > > > no code change for resent. > > > > > > > > When you're submitting a patch and want it reach the stable kernels, you'll > > > > need to add a Cc tag: > > > > > > > > Cc: stable@vger.kernel.org > > > > > > > > But not actually add the address to cc. I dropped stable@vger address from > > > > distribution. > > > > > > Does it really matter? > > > > Usually aligning what you're doing with > > Documentation/process/submitting-patches.rst is not a bad idea. > > True, my point is that technically both ways will give the same result, no? If you get your patch merged this way, then yes. :-)
On Tue, Apr 20, 2021 at 11:02 AM Sakari Ailus <sakari.ailus@linux.intel.com> wrote: > > Hi Bingbu, > > Thanks for the patch. > > On Tue, Apr 20, 2021 at 10:48:33AM +0800, Bingbu Cao wrote: > > Intel IPU(Image Processing Unit) has its own (IO)MMU hardware, > > The IPU driver allocates its own page table that is not mapped > > via the DMA, and thus the Intel IOMMU driver blocks access giving > > this error: > > The page table should be mapped to the possible IOMMU using the DMA API. I've made the same "observation": this patch is intentionally enables using "intel_iommu=on" (IIRC) to strictly enforce "all" DMA transactions (except the ones we explicitly allow to identity map). The question is: Is the security of IPU MMU the same as the system IOMMU for the few devices behind the IPU MMU? If not, then we (Chrome OS) require child devices to be "mapped" twice: once in IPU MMU and again in the system IOMMU. I believe dma_ops can be nested though I can't confidently point at examples (IDE drivers maybe?) This adds some latency to each DMA transaction - decades ago I've measured roughly 5% on Itanium and PA-RISC systems from HP. Perhaps Intel can measure this penatly on current HW they are shipping. If yes, then I think the IPU driver just needs to be consistent about it's use of DMA API for it's own house keeping: Either use DMA API for all IPU DMA operations or use it for none. This is the current plan for Chrome OS (I didn't make this decision and wasn't party to the discussion). The IPU driver requires it's child devices to use DMA API and provides the dma_ops table for those devices - this use of dma_ops is seperate from IPU page tables and other host memory transactions to manage the IPU MMU page tables. CAVEAT: I'm not an expert in IPU driver - I've been reviewing Intel IPU code for chromium.org inclusion here: https://chromium-review.googlesource.com/c/chromiumos/third_party/kernel/+/2787723 I have no illusions that I'm going to be an expert after staring at 28k lines of code less than 10h. > > DMAR: DRHD: handling fault status reg 3 > > DMAR: [DMA Read] Request device [00:05.0] PASID ffffffff > > fault addr 76406000 [fault reason 06] PTE Read access is not set > > > > As IPU is not an external facing device which is not risky, so use > > IOMMU passthrough mode for Intel IPUs. > > I think a factor here is that the page tables aren't accessible by the IPU > firmware. Correct. At least not accessible through the system IOMMU. This is why Intel prefers the IPU to bypass the system IOMMU. cheers, grant > > > > > Fixes: 26f5689592e2 ("media: staging/intel-ipu3: mmu: Implement driver") > > Signed-off-by: Bingbu Cao <bingbu.cao@intel.com> > > --- > > drivers/iommu/intel/iommu.c | 29 +++++++++++++++++++++++++++++ > > 1 file changed, 29 insertions(+) > > > > diff --git a/drivers/iommu/intel/iommu.c b/drivers/iommu/intel/iommu.c > > index ee0932307d64..7e2fbdae467e 100644 > > --- a/drivers/iommu/intel/iommu.c > > +++ b/drivers/iommu/intel/iommu.c > > @@ -55,6 +55,12 @@ > > #define IS_GFX_DEVICE(pdev) ((pdev->class >> 16) == PCI_BASE_CLASS_DISPLAY) > > #define IS_USB_DEVICE(pdev) ((pdev->class >> 8) == PCI_CLASS_SERIAL_USB) > > #define IS_ISA_DEVICE(pdev) ((pdev->class >> 8) == PCI_CLASS_BRIDGE_ISA) > > +#define IS_INTEL_IPU(pdev) ((pdev)->vendor == PCI_VENDOR_ID_INTEL && \ > > + ((pdev)->device == 0x9a19 || \ > > + (pdev)->device == 0x9a39 || \ > > + (pdev)->device == 0x4e19 || \ > > + (pdev)->device == 0x465d || \ > > + (pdev)->device == 0x1919)) > > #define IS_AZALIA(pdev) ((pdev)->vendor == 0x8086 && (pdev)->device == 0x3a3e) > > > > #define IOAPIC_RANGE_START (0xfee00000) > > @@ -360,6 +366,7 @@ int intel_iommu_enabled = 0; > > EXPORT_SYMBOL_GPL(intel_iommu_enabled); > > > > static int dmar_map_gfx = 1; > > +static int dmar_map_ipu = 1; > > This works as long as there's only one IPU. Same for graphics. But I guess > this can be reworked in the future if the presumption changes. > > > static int dmar_forcedac; > > static int intel_iommu_strict; > > static int intel_iommu_superpage = 1; > > @@ -368,6 +375,7 @@ static int iommu_skip_te_disable; > > > > #define IDENTMAP_GFX 2 > > #define IDENTMAP_AZALIA 4 > > +#define IDENTMAP_IPU 8 > > > > int intel_iommu_gfx_mapped; > > EXPORT_SYMBOL_GPL(intel_iommu_gfx_mapped); > > @@ -2839,6 +2847,9 @@ static int device_def_domain_type(struct device *dev) > > > > if ((iommu_identity_mapping & IDENTMAP_GFX) && IS_GFX_DEVICE(pdev)) > > return IOMMU_DOMAIN_IDENTITY; > > + > > + if ((iommu_identity_mapping & IDENTMAP_IPU) && IS_INTEL_IPU(pdev)) > > + return IOMMU_DOMAIN_IDENTITY; > > } > > > > return 0; > > @@ -3278,6 +3289,9 @@ static int __init init_dmars(void) > > if (!dmar_map_gfx) > > iommu_identity_mapping |= IDENTMAP_GFX; > > > > + if (!dmar_map_ipu) > > + iommu_identity_mapping |= IDENTMAP_IPU; > > + > > check_tylersburg_isoch(); > > > > ret = si_domain_init(hw_pass_through); > > @@ -5622,6 +5636,18 @@ static void quirk_iommu_igfx(struct pci_dev *dev) > > dmar_map_gfx = 0; > > } > > > > +static void quirk_iommu_ipu(struct pci_dev *dev) > > +{ > > + if (!IS_INTEL_IPU(dev)) > > + return; > > + > > + if (risky_device(dev)) > > + return; > > + > > + pci_info(dev, "Passthrough IOMMU for integrated Intel IPU\n"); > > + dmar_map_ipu = 0; > > +} > > + > > /* G4x/GM45 integrated gfx dmar support is totally busted. */ > > DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_INTEL, 0x2a40, quirk_iommu_igfx); > > DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_INTEL, 0x2e00, quirk_iommu_igfx); > > @@ -5657,6 +5683,9 @@ DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_INTEL, 0x1632, quirk_iommu_igfx); > > DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_INTEL, 0x163A, quirk_iommu_igfx); > > DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_INTEL, 0x163D, quirk_iommu_igfx); > > > > +/* disable IPU dmar support */ > > +DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_INTEL, PCI_ANY_ID, quirk_iommu_ipu); > > + > > static void quirk_iommu_rwbf(struct pci_dev *dev) > > { > > if (risky_device(dev)) > > -- > Kind regards, > > Sakari Ailus
diff --git a/drivers/iommu/intel/iommu.c b/drivers/iommu/intel/iommu.c index ee0932307d64..7e2fbdae467e 100644 --- a/drivers/iommu/intel/iommu.c +++ b/drivers/iommu/intel/iommu.c @@ -55,6 +55,12 @@ #define IS_GFX_DEVICE(pdev) ((pdev->class >> 16) == PCI_BASE_CLASS_DISPLAY) #define IS_USB_DEVICE(pdev) ((pdev->class >> 8) == PCI_CLASS_SERIAL_USB) #define IS_ISA_DEVICE(pdev) ((pdev->class >> 8) == PCI_CLASS_BRIDGE_ISA) +#define IS_INTEL_IPU(pdev) ((pdev)->vendor == PCI_VENDOR_ID_INTEL && \ + ((pdev)->device == 0x9a19 || \ + (pdev)->device == 0x9a39 || \ + (pdev)->device == 0x4e19 || \ + (pdev)->device == 0x465d || \ + (pdev)->device == 0x1919)) #define IS_AZALIA(pdev) ((pdev)->vendor == 0x8086 && (pdev)->device == 0x3a3e) #define IOAPIC_RANGE_START (0xfee00000) @@ -360,6 +366,7 @@ int intel_iommu_enabled = 0; EXPORT_SYMBOL_GPL(intel_iommu_enabled); static int dmar_map_gfx = 1; +static int dmar_map_ipu = 1; static int dmar_forcedac; static int intel_iommu_strict; static int intel_iommu_superpage = 1; @@ -368,6 +375,7 @@ static int iommu_skip_te_disable; #define IDENTMAP_GFX 2 #define IDENTMAP_AZALIA 4 +#define IDENTMAP_IPU 8 int intel_iommu_gfx_mapped; EXPORT_SYMBOL_GPL(intel_iommu_gfx_mapped); @@ -2839,6 +2847,9 @@ static int device_def_domain_type(struct device *dev) if ((iommu_identity_mapping & IDENTMAP_GFX) && IS_GFX_DEVICE(pdev)) return IOMMU_DOMAIN_IDENTITY; + + if ((iommu_identity_mapping & IDENTMAP_IPU) && IS_INTEL_IPU(pdev)) + return IOMMU_DOMAIN_IDENTITY; } return 0; @@ -3278,6 +3289,9 @@ static int __init init_dmars(void) if (!dmar_map_gfx) iommu_identity_mapping |= IDENTMAP_GFX; + if (!dmar_map_ipu) + iommu_identity_mapping |= IDENTMAP_IPU; + check_tylersburg_isoch(); ret = si_domain_init(hw_pass_through); @@ -5622,6 +5636,18 @@ static void quirk_iommu_igfx(struct pci_dev *dev) dmar_map_gfx = 0; } +static void quirk_iommu_ipu(struct pci_dev *dev) +{ + if (!IS_INTEL_IPU(dev)) + return; + + if (risky_device(dev)) + return; + + pci_info(dev, "Passthrough IOMMU for integrated Intel IPU\n"); + dmar_map_ipu = 0; +} + /* G4x/GM45 integrated gfx dmar support is totally busted. */ DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_INTEL, 0x2a40, quirk_iommu_igfx); DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_INTEL, 0x2e00, quirk_iommu_igfx); @@ -5657,6 +5683,9 @@ DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_INTEL, 0x1632, quirk_iommu_igfx); DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_INTEL, 0x163A, quirk_iommu_igfx); DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_INTEL, 0x163D, quirk_iommu_igfx); +/* disable IPU dmar support */ +DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_INTEL, PCI_ANY_ID, quirk_iommu_ipu); + static void quirk_iommu_rwbf(struct pci_dev *dev) { if (risky_device(dev))
Intel IPU(Image Processing Unit) has its own (IO)MMU hardware, The IPU driver allocates its own page table that is not mapped via the DMA, and thus the Intel IOMMU driver blocks access giving this error: DMAR: DRHD: handling fault status reg 3 DMAR: [DMA Read] Request device [00:05.0] PASID ffffffff fault addr 76406000 [fault reason 06] PTE Read access is not set As IPU is not an external facing device which is not risky, so use IOMMU passthrough mode for Intel IPUs. Fixes: 26f5689592e2 ("media: staging/intel-ipu3: mmu: Implement driver") Signed-off-by: Bingbu Cao <bingbu.cao@intel.com> --- drivers/iommu/intel/iommu.c | 29 +++++++++++++++++++++++++++++ 1 file changed, 29 insertions(+)