From mboxrd@z Thu Jan  1 00:00:00 1970
From: Mukesh Rathor <mukesh.rathor@oracle.com>
Subject: Re: Error restoring DomU when using GPLPV
Date: Fri, 04 Sep 2009 21:02:44 -0700
Message-ID: <4AA1E2E4.4070000@oracle.com>
References: <C69DF260.9ECD%keir.fraser@eu.citrix.com>
Reply-To: mukesh.rathor@oracle.com
Mime-Version: 1.0
Content-Type: text/plain; charset=US-ASCII; format=flowed
Content-Transfer-Encoding: 7bit
Return-path: <xen-devel-bounces@lists.xensource.com>
In-Reply-To: <C69DF260.9ECD%keir.fraser@eu.citrix.com>
List-Unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>,
	<mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
List-Post: <mailto:xen-devel@lists.xensource.com>
List-Help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-Subscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>,
	<mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
Sender: xen-devel-bounces@lists.xensource.com
Errors-To: xen-devel-bounces@lists.xensource.com
To: Keir Fraser <keir.fraser@eu.citrix.com>
Cc: Joshua West <jwest@brandeis.edu>, James Harper <james.harper@bendigoit.com.au>, "xen-devel@lists.xensource.com" <xen-devel@lists.xensource.com>
List-Id: xen-devel@lists.xenproject.org



Keir Fraser wrote:
> On 04/08/2009 12:34, "James Harper" <james.harper@bendigoit.com.au> wrote:
> 
>>> Like I said before -- unmapping the gnttab pages I think will not help
>> you
>>> for live migration, but I suppose it is a reasonable thing to do
>> anyway. For
>>> live migration I think xc_domain_save needs t get a bit smarter about
>>> Xenheap pages in HVM guests.
>> Understood. Do you have any idea about why it worked fine under 3.3.x
>> but not 3.4.x?
> 
> The bit of code in 3.3's xc_domain_save.c that is commented "Skip PFNs that
> aren't really there" is removed in 3.4. That will be the reason.
> 
>  -- Keir

Hi,

I started looking at this couple days ago, and finally understand
what's going on. In our case, win migration/save-restore just fails, as
Annie/Wayne had posted.

In the short run, since frames for vga etc are skipped anyways, can we
just put the above change back in libxc (xen 3.4) and be ok?

thanks,
Mukesh


changeset:   18383:dade7f0bdc8d
user:        Keir Fraser <keir.fraser@citrix.com>
date:        Wed Aug 27 14:53:39 2008 +0100
summary:     hvm: Use main memory for video memory.

diff -r 2397555ebcc2 -r dade7f0bdc8d tools/libxc/xc_domain_save.c
--- a/tools/libxc/xc_domain_save.c      Wed Aug 27 13:31:01 2008 +0100
+++ b/tools/libxc/xc_domain_save.c      Wed Aug 27 14:53:39 2008 +0100
@@ -1111,12 +1111,6 @@
                         (test_bit(n, to_fix)  && last_iter)) )
                      continue;

-                /* Skip PFNs that aren't really there */
-                if ( hvm && ((n >= 0xa0 && n < 0xc0) /* VGA hole */
-                             || (n >= (HVM_BELOW_4G_MMIO_START >> PAGE_SHIFT)
-                                 && n < (1ULL<<32) >> PAGE_SHIFT)) /* MMIO */ )
-                    continue;
-