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

xen-devel

Re: [Xen-devel] Question on save/restore mfn canonicalization

To: Andres Lagar-Cavilla <andreslc@xxxxxxxxxxxxxx>
Subject: Re: [Xen-devel] Question on save/restore mfn canonicalization
From: John Levon <levon@xxxxxxxxxxxxxxxxx>
Date: Fri, 28 Sep 2007 18:54:29 +0100
Cc: xen-devel@xxxxxxxxxxxxxxxxxxx
Delivery-date: Fri, 28 Sep 2007 10:55:19 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
In-reply-to: <46FD2AF8.30107@xxxxxxxxxxxxxx>
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: <46FD2AF8.30107@xxxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
User-agent: Mutt/1.5.9i
On Fri, Sep 28, 2007 at 12:25:28PM -0400, Andres Lagar-Cavilla wrote:

> While exploring the paravirt save/restore code path, I noted a 
> (seemingly) lack of consistency between where/who canonicalizes 
> something and who un-canonicalizes. For example, the guest kernel 
> canonicalizes the store and console mfn's in pre_suspend, but 
> xc_domain_restore uncanonicalizes them before scheduling back the 
> restored guest.
> The question is if there is a mandatory reason for this, or is just the 
> way the code was written. Can I, e.g, fill the 
> pfn_to_mfn_frame_list(_list) entries from "outside" and remove that code 
> from post_suspend, or will something break?

You'll break the ABI. I don't know of a reason why it's so inconsistent.

regards
john

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

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