xen-devel

[Top] [All Lists]

Re: [Xen-devel] Why call iommu_unmap_page() in set_p2m_entry()?

from [Wei Wang2]

[Permanent Link][Original]

To:	"Han, Weidong" <weidong.han@xxxxxxxxx>
Subject:	Re: [Xen-devel] Why call iommu_unmap_page() in set_p2m_entry()?
From:	Wei Wang2 <wei.wang2@xxxxxxx>
Date:	Tue, 18 Mar 2008 18:29:45 +0100
Cc:	xen-devel@xxxxxxxxxxxxxxxxxxx, Keir Fraser <keir.fraser@xxxxxxxxxxxxx>
Delivery-date:	Tue, 18 Mar 2008 10:30:28 -0700
Envelope-to:	www-data@xxxxxxxxxxxxxxxxxx
In-reply-to:	<08DF4D958216244799FC84F3514D70F00124FB6E@xxxxxxxxxxxxxxxxxxxxxxxxxxxx>
List-help:	<mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id:	Xen developer discussion <xen-devel.lists.xensource.com>
List-post:	<mailto:xen-devel@lists.xensource.com>
List-subscribe:	<http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe:	<http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
References:	<08DF4D958216244799FC84F3514D70F00124FB6E@xxxxxxxxxxxxxxxxxxxxxxxxxxxx>
Sender:	xen-devel-bounces@xxxxxxxxxxxxxxxxxxx

Hi Weidong,
Thanks for pointing it out.  iommu_unmap_page() is not intended for
flushing all iotlb entries. As AMD-IOMMU also supports domain-selective
unmapping and I seem to find related functions in vtd code,I hope this
operation can also be generic. If not, I plan to put this into amd
specific functions called by domain destroy routines.

Wei

On Thu, 2008-03-13 at 14:28 +0100, Han, Weidong wrote:
> As my understanding, AMD IOMMU doesn't share its page tables with p2m
> table. So it needs call iommu_unmap_page() to free pages when shut
> down
> or destroy domain. Now iommu_unmap_page() is only called in
> set_p2m_entry(), I think it won't be executed when shut down or
> destroy
> domain. I'm not familiar with AMD IOMMU code, maybe I missed
> something.
> Can you explain why do that? Thanks!
> 
> Randy (Weidong)
> 
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@xxxxxxxxxxxxxxxxxxx
> http://lists.xensource.com/xen-devel
> 
> 
> 

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel

[More with this subject...]

<Prev in Thread]	Current Thread	[Next in Thread>
[Xen-devel] Why call iommu_unmap_page() in set_p2m_entry()?, Han, Weidong Re: [Xen-devel] Why call iommu_unmap_page() in set_p2m_entry()?, Wei Wang2 <= RE: [Xen-devel] Why call iommu_unmap_page() in set_p2m_entry()?, Han, Weidong

Previous by Date:	[Xen-devel] Explanation of set_pte_at, Michael Abd-El-Malek
Next by Date:	[Xen-devel] [PATCH] permute with 2MB chunk, Jean Guyader
Previous by Thread:	[Xen-devel] Why call iommu_unmap_page() in set_p2m_entry()?, Han, Weidong
Next by Thread:	RE: [Xen-devel] Why call iommu_unmap_page() in set_p2m_entry()?, Han, Weidong
Indexes:	[Date] [Thread] [Top] [All Lists]