WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

Re: [Xen-devel] RE: [PATCH 1/2] Writable pagetable support for x86_64 xe

To: "Nakajima, Jun" <jun.nakajima@xxxxxxxxx>
Subject: Re: [Xen-devel] RE: [PATCH 1/2] Writable pagetable support for x86_64 xenlinux (xen part)
From: Keir Fraser <Keir.Fraser@xxxxxxxxxxxx>
Date: Tue, 9 Aug 2005 11:51:14 +0100
Cc: Chris Wright <chrisw@xxxxxxxx>, "Li, Xin B" <xin.b.li@xxxxxxxxx>, xen-devel@xxxxxxxxxxxxxxxxxxx
Delivery-date: Tue, 09 Aug 2005 10:45:08 +0000
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <7F740D512C7C1046AB53446D3720017304CD9AF3@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: <7F740D512C7C1046AB53446D3720017304CD9AF3@xxxxxxxxxxxxxxxxxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx

On 9 Aug 2005, at 08:59, Nakajima, Jun wrote:

One ugly code that I don't like is __not_mapped(l2_pgentry_t *pl2e). Any
idea for doing it better?

I used a patch from Chris/Xin for the x86_64 SMP kernel. Probably he has
more updates there.

i checked in a cleaned up version of your SMP patch, as part of some wider-ranging cleanups of your previous wrpt patch. A couple of notes on that:

1. I got rid of __not_mapped(). It should suffice to attempt the access through the linear mapping and handle the possible fault. 2. You had gross CONFIG_X86_64-specific tests in get_page_type(). I cannot see how any good could possible come of them so I removed them. Either you need a good justification why they're needed or, if they actually fixed observed problems, I think there must be some underlying bug that those conditional sections of code were masking. Either way, this needs looking at.

 -- Keir


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

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