Re: [PATCH] intel-iommu: Manage iommu_coherency globally

From: Chris Wright
Date: Wed Nov 16 2011 - 05:28:50 EST


* Alex Williamson (alex.williamson@xxxxxxxxxx) wrote:
> We currently manage iommu_coherency on a per domain basis,
> choosing the safest setting across the iommus attached to a
> particular domain. This unfortunately has a bug that when
> no iommus are attached, the domain defaults to coherent.
> If we fall into this mode, then later add a device behind a
> non-coherent iommu to that domain, the context entry is
> updated using the wrong coherency setting, and we get dmar
> faults.
>
> Since we expect chipsets to be consistent in their coherency
> setting, we can instead determine the coherency once and use
> it globally.
>
> Signed-off-by: Alex Williamson <alex.williamson@xxxxxxxxxx>

Looks good to me.

Acked-by: Chris Wright <chrisw@xxxxxxxxxxxx>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/