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

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


  • To: Kieran Mansley <kmansley@xxxxxxxxxxxxxx>
  • From: Keir Fraser <keir@xxxxxxxxxxxxx>
  • Date: Fri, 13 Jul 2007 17:20:53 +0100
  • Cc: xen-devel@xxxxxxxxxxxxxxxxxxx
  • Delivery-date: Fri, 13 Jul 2007 09:18:49 -0700
  • List-id: Xen developer discussion <xen-devel.lists.xensource.com>
  • Thread-index: AcfFaciIB0SHDjFdEdy48QAX8io7RQ==
  • Thread-topic: [Xen-devel] [PATCH] Allow programatic iomem permissions

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

> Oops, sorry, forgot to refresh the patch before sending.  This one
> should compile better!

The put_page[_and_type] also needs to be on the far side of the TLB flush.
This is because if a refcount falls to zero then the page can change type
(e.g., to pagetable page) or be recycled to another domain! This is a safety
issue for Xen rather than for the granter. :-)

 -- Keir


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


 


Rackspace

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