Xen 
 
Home About Xen.org Xen Xen Summit Wiki Mailing List Bug Tracker Xen Downloads
 
   
 

xen-devel

[Xen-devel] Re: [PATCH] VT-d: IOTLB flush fixups

To: espen.skoglund@xxxxxxxxxxxxx
Subject: [Xen-devel] Re: [PATCH] VT-d: IOTLB flush fixups
From: "Yang, Xiaowei" <xiaowei.yang@xxxxxxxxx>
Date: Wed, 28 May 2008 17:01:18 +0800
Cc: xen-devel@xxxxxxxxxxxxxxxxxxx
Delivery-date: Wed, 28 May 2008 02:03:43 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
In-reply-to: <1211950777.17461.0.camel@ip6-localhost>
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: <1211950777.17461.0.camel@ip6-localhost>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
> On map: only flush when old PTE was valid or invalid PTE may be cached.
> On unmap: always flush old entry, but skip flush for unaffected IOMMUs.
> 
> Signed-off-by: Espen Skoglund <espen.skoglund@xxxxxxxxxxxxx>
> 
> --
>  iommu.c |   17 +++++++++++------
>  1 file changed, 11 insertions(+), 6 deletions(-)
> --

Seems my last mail sent to the xen-devel is lost and I have no local
copy so I have to write again...

Espen,
Thanks for the patch! I also noticed context/iotlb flush need a cleanup.
As flush of present/non-present entry are different, your change to
iommu_intel_map_page are not that correct. So I made up anther patch.
iommu_flush is also removed, as VTd table is not shared with p2m any
more.

Signed-off-by: Xiaowei Yang <xiaowei.yang@xxxxxxxxx>

Thanks,
Xiaowei

Attachment: flush.diff
Description: Text Data

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel
<Prev in Thread] Current Thread [Next in Thread>