[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

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



>>> 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


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.