Re: [PATCH 8/8] HVM save restore: PV driver support

["Zhai, Edwin" <edwin.zhai@intel.com>]

On Fri, Jan 19, 2007 at 11:11:51AM +0000, Tim Deegan wrote:
> Hi Edwin, 
> 
> At 13:14 +0800 on 19 Jan (1169212491), Zhai, Edwin wrote:
> > thanks for push in save/restore patches, but this one for save/restore
> > PV driver on HVM seems to be missing.
> > 
> > if any concerns about this patch, pls. let me know.
> 
> The immediate reason I didn't take this patch is that it changes the
> struct page_info definition.  It also breaks PV save/restore (though I
> haven't yet investigated why).

i have met the issue that PV guest restore fail with "can't map 
p2m_frame_list_..".  but after i rebuild everything and try to find root cause,
this bug disappeared.

i'm not sure if it's a padding issue or something else.

> 
> Since we could take the basic HVM save/restore without this, and we were
> keen to code into the tree, I left this out for now.  I plan to tidy up
> basic HVM save/restore first and then return to PV-on-HVM.

I agree

> 
> Cheers,
> 
> Tim.
> 

-- 
best rgds,
edwin