Re: [Xen-devel] [PATCH 1/3] iommu: set correct IOMMU entries when iommu_hap_pt_share == 0

["Jan Beulich" <JBeulich@xxxxxxxx>]

>>> On 07.04.14 at 18:02, <roger.pau@xxxxxxxxxx> wrote:
> If the memory map is not shared between HAP and IOMMU we fails to set
> correct IOMMU mappings for memory types different than p2m_ram_rw.
> 
> This patchs adds IOMMU support for the following memory types:
> p2m_grant_map_rw, p2m_map_foreign, p2m_ram_ro and p2m_grant_map_ro.

I'm curious about the justification for p2m_map_foreign; the others
I agree with.

I also wonder whether p2m_ram_logdirty shouldn't be treated
equally to p2m_ram_rw: It's clearly better to have some video
corruption than to kill the guest due to excessive IOMMU faults,
should its kernel choose to DMA into the frame buffer. But at the
very minimum this needs to be included alongside p2m_ram_ro.

Jan


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel