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

xense-devel

Re: [Xen-devel] [PATCH] Allow programatic iomem permissions

To: Kieran Mansley <kmansley@xxxxxxxxxxxxxx>, <xen-devel@xxxxxxxxxxxxxxxxxxx>
Subject: Re: [Xen-devel] [PATCH] Allow programatic iomem permissions
From: Keir Fraser <keir@xxxxxxxxxxxxx>
Date: Fri, 13 Jul 2007 13:29:58 +0100
Cc: derek.murray@xxxxxxxxxxxx, xense-devel@xxxxxxxxxxxxxxxxxxx, gscoker@xxxxxxxxxxxxxx
Delivery-date: Fri, 13 Jul 2007 05:27:53 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
In-reply-to: <1184324719.4144.10.camel@xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx>
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>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: AcfFSYZPxSJiODE8Edy48QAX8io7RQ==
Thread-topic: [Xen-devel] [PATCH] Allow programatic iomem permissions
User-agent: Microsoft-Entourage/11.3.3.061214


On 13/7/07 12:05, "Kieran Mansley" <kmansley@xxxxxxxxxxxxxx> wrote:

> Any thoughts on the above?
> 
> I'm happy to do some work to rectify this situation but don't want to
> expend effort on something that won't get accepted.  Any background on
> the reasons for the restriction (i.e. what the TLB flushing issues are)
> would also be of help.

The issue is that the granter is informed that the grant is released before
stale grantee TLB entries are flushed. If the grantee is multi-vcpu then he
could theoretically still access a granted page via a stale TLB entry after
the granter has recycled the page. The window is extremely tiny though! The
correct fix is to reorder the unmap operation to be unmap-list-of-grants
then TLB-flush then update-grant-entries-to-indicate-release. Then the whole
problem disappears.

 -- Keir


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

<Prev in Thread] Current Thread [Next in Thread>