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] page ref/type count overflows

To: Tim Deegan <Tim.Deegan@xxxxxxxxxx>, Jan Beulich <jbeulich@xxxxxxxxxx>
Subject: Re: [Xen-devel] page ref/type count overflows
From: Keir Fraser <keir.fraser@xxxxxxxxxxxxx>
Date: Thu, 29 Jan 2009 11:26:24 +0000
Cc: Gianluca Guida <gianluca.guida@xxxxxxxxxxxxx>, xen-devel@xxxxxxxxxxxxxxxxxxx
Delivery-date: Thu, 29 Jan 2009 03:26:52 -0800
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <20090129110720.GC5300@xxxxxxxxxxxxxxxxxxxxx>
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/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: AcmCBGrLddvvQ9ePSEeJZ/JTDF37sg==
Thread-topic: [Xen-devel] page ref/type count overflows
User-agent: Microsoft-Entourage/12.14.0.081024
On 29/01/2009 11:07, "Tim Deegan" <Tim.Deegan@xxxxxxxxxx> wrote:

>> Does this look acceptable?
> 
> Seems fine, which is to say I don't see much advantage but have no
> objection. :)  Please try to make it clear in the comments which fields
> belong to a page which _has_ a shadow and which to a page that _is_ a
> shadow.

I'd agree with Tim. I perhaps have a slight preference for keeping it all in
one struct (i.e., the new approach) but I'm happy either way.

 -- Keir



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