From mboxrd@z Thu Jan 1 00:00:00 1970 From: Keir Fraser Subject: Re: Illegal PV kernel pfm/pfn translations on PROT_NONE ioremaps Date: Wed, 19 Mar 2008 18:52:13 +0000 Message-ID: References: <1205946765.7277.40.camel@sisko.scot.redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset="US-ASCII" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1205946765.7277.40.camel@sisko.scot.redhat.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xensource.com Errors-To: xen-devel-bounces@lists.xensource.com To: "Stephen C. Tweedie" Cc: "xen-devel@lists.xensource.com" , Jan Beulich List-Id: xen-devel@lists.xenproject.org On 19/3/08 17:12, "Stephen C. Tweedie" wrote: > On Wed, 2008-03-19 at 16:42 +0000, Keir Fraser wrote: > >>> It might be easier to do this at the pte_machine_to_phys level instead, >>> where we can potentially take advantage of other bits of the pte to >>> encode the special casing. >> >> Oh yes, the PAGE_IO type of trick I mentioned in my other email just now. > > Yep. There are a number of bits that could be used: for example, I > don't think PROT_WRITE is ever going to be set on a PROT_NONE page, > either. My point is the flag would be set even when the protection is not PROT_NONE. It has other uses beyond fixing this bug. I might look into doing this myself as I think it is probably the best fix for this issue. -- Keir