On Sat, Feb 23, 2013 at 3:29 AM, Stijn Tintel <stijn@xxxxxxxxxxxxx> wrote: > On 19-12-12 11:58, Andrew Cooks wrote: >> This is my second attempt to make Marvell 88SE91xx SATA controllers work when IOMMU is enabled.[1][2] >> As suggested, it no longer tries to add support for phantom functions. >> >> What's missing: >> * No AMD support. I need some help with this. >> * Table of affected chip IDs is incomplete. I think 0x9123, 0x9125, 0x9128 are also affected. > This one is also affected: 08:00.0 0106: 1b4b:9130 (rev 11), this is in > dmesg: > > [ 1.914086] dmar: DMAR:[DMA Read] Request device [08:00.1] fault addr > fff00000 Handling specific chip revisions is another issue I'm not sure how to handle. We could use another bitmap field, but we need more vendor cooperation to know exactly which chip model and revision combinations are affected. >> >> Patch is against 3.7.1 >> >> Review and feedback would be appreciated. > I changed your patch to check for my chip ID, and it solves my problem: > no more hang during boot, and the HDD connected to this controller is > now detected with IOMMU enabled. > > Also see 1 minor comment inline. Thanks for the feedback! I'll include your PCI_VENDOR_ID change in the next iteration of the patch. -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
