On Mon, 2013-01-21 at 19:48 +0100, Daniel Vetter wrote: > We already have the quirk entry for the mobile platform, but also > reports on some desktop versions. So be paranoid and set it > everywhere. > > References: http://www.mail-archive.com/dri-devel at lists.freedesktop.org/msg33138.html > Cc: stable at vger.kernel.org > Cc: David Woodhouse <dwmw2 at infradead.org> > Reported-and-tested-by: Mihai Moldovan <ionic at ionic.de> > Signed-off-by: Daniel Vetter <daniel.vetter at ffwll.ch> > --- > drivers/iommu/intel-iommu.c | 8 +++++++- > 1 file changed, 7 insertions(+), 1 deletion(-) > > diff --git a/drivers/iommu/intel-iommu.c b/drivers/iommu/intel-iommu.c > index 9743769..19854bf 100644 > --- a/drivers/iommu/intel-iommu.c > +++ b/drivers/iommu/intel-iommu.c > @@ -4215,13 +4215,19 @@ static void __devinit quirk_iommu_rwbf(struct pci_dev *dev) > { > /* > * Mobile 4 Series Chipset neglects to set RWBF capability, > - * but needs it: > + * but needs it. Same seems to hold for the desktop versions. > */ > printk(KERN_INFO "DMAR: Forcing write-buffer flush capability\n"); > rwbf_quirk = 1; > } > > DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_INTEL, 0x2a40, quirk_iommu_rwbf); > +DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_INTEL, 0x2e00, quirk_iommu_rwbf); > +DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_INTEL, 0x2e10, quirk_iommu_rwbf); > +DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_INTEL, 0x2e20, quirk_iommu_rwbf); > +DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_INTEL, 0x2e30, quirk_iommu_rwbf); > +DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_INTEL, 0x2e40, quirk_iommu_rwbf); > +DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_INTEL, 0x2e90, quirk_iommu_rwbf); > > #define GGC 0x52 > #define GGC_MEMORY_SIZE_MASK (0xf << 8) Again, I'm really unhappy about doing this kind of thing based on hearsay. This should have a specific reference (with URL) to a published erratum. Rajesh? -- dwmw2 -------------- next part -------------- A non-text attachment was scrubbed... Name: smime.p7s Type: application/x-pkcs7-signature Size: 6171 bytes Desc: not available URL: <http://lists.freedesktop.org/archives/intel-gfx/attachments/20130122/d10a2e92/attachment.bin>
