Re: Re: [PATCH 0/4] stubdom-dm: Support live migration of HVM domain with stubdom-dm

[Yosuke Iwamatsu <y-iwamatsu@ab.jp.nec.com>]

[-- Attachment #1: Type: text/plain, Size: 1605 bytes --]

Ian Jackson wrote:
> Samuel Thibault writes ("[Xen-devel] Re: [PATCH 0/4] stubdom-dm: Support live migration of HVM domain with stubdom-dm"):
>> Yosuke Iwamatsu, le Tue 16 Dec 2008 18:22:20 +0900, a écrit :
>>> This series of patches is an attempt to support live migration of hvm
>>> domains with stubdom-dm.
>> Err, I'm a bit surprised. I haven't tested it since then, but I'm sure
>> live migration was working with stubdoms a few months ago, without
>> the need for a logdirty shared buffer.  This works thanks to the few
>> lines at the end of the cpu_physical_memory_rw() function that tell the
>> hypervisor about modified memory.  Actually, the plan was even to use
>> that bit of code in all cases (not only non-stubdom), so as to get rid
>> of the logdirty shared buffer completely.
> 
> I take it you're referring to this:
> 
> #ifdef CONFIG_STUBDOM
>     if (logdirty_bitmap != NULL)
>         xc_hvm_modified_memory(xc_handle, domid, _addr >> TARGET_PAGE_BITS,
>                 ((_addr + _len + TARGET_PAGE_SIZE - 1) >> TARGET_PAGE_BITS)
>                     - (_addr >> TARGET_PAGE_BITS));
> #endif

Ah, I didn't notice this.

> Yosuke Iwamatsu: is this not working for you then ?  It's possible
> that I've broken it somehow.

Live-migration fails because stubdom-dm doesn't respond to the request
from xc_save for the logdirty bitmap segment. The attached patch
simply generated from old ioemu tree will fix the issue, then.

Derived from old xen internal tools/ioemu tree.
Signed-off-by: Yosuke Iwamatsu <y-iwamatsu@ab.jp.nec.com>

Thanks,
-- Yosuke

[-- Attachment #2: stubdom_respond_to_logdirty_event.patch --]
[-- Type: all/allfiles, Size: 1360 bytes --]

[-- Attachment #3: Type: text/plain, Size: 138 bytes --]

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