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] [PATCH] libxc: failure to save/restore 32bit HVM guest

To: Keir Fraser <keir.fraser@xxxxxxxxxxxxx>
Subject: Re: [Xen-devel] [PATCH] libxc: failure to save/restore 32bit HVM guest
From: Mukesh Rathor <mukesh.rathor@xxxxxxxxxx>
Date: Tue, 24 Nov 2009 11:23:48 -0800
Cc: "Xen-devel@xxxxxxxxxxxxxxxxxxx" <Xen-devel@xxxxxxxxxxxxxxxxxxx>
Delivery-date: Tue, 24 Nov 2009 11:26:00 -0800
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <C7314E50.16A1%keir.fraser@xxxxxxxxxxxxx>
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>
References: <20091123151406.36e032c8@xxxxxxxxxxxxxxxxxxxx> <C7314E50.16A1%keir.fraser@xxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
On Tue, 24 Nov 2009 08:48:16 +0000
Keir Fraser <keir.fraser@xxxxxxxxxxxxx> wrote:

> On 23/11/2009 23:14, "Mukesh Rathor" <mukesh.rathor@xxxxxxxxxx> wrote:
> 
> > Following appears to be an obvious bug. Save/restore of HVM guests
> > works fine after the following fix. Including the patch, please let
> > me know if you agree. I'll make parallel fix in our previous
> > versions here.
> 
> Save/restore of HVM guests should work fine before this fix too, since
> nothing in xc_core.c is used for guest save/restore. It's used for
> core dumps, as the name implies, and which is a rather different
> thing.
> 
> Furthermore I think the code in xc_core.c is correct as is.
> 
>  -- Keir
> 

Sorry, my bad, I meant dump core. I was simultaneously looking at
save/restore, and accidentally said that.

Anyways, since 32bit HVM returns 64bit contexts, it appears the way it
is, is buggy, since the sizeof(ctxt[0].c) will return 32bit context
size in 32 bit libxc. Infact, after making the fix, user is able to get
proper core of a Java HVM guest. Can you please quickly explain why you
think it's correct in such case?

thanks,
Mukesh


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