[pve-devel] [PATCH 3/3] add downtime && expected_downtime query-migrate info

[Alexandre DERUMIER]

I have just done some little tests with qemu 1.3 + migration patches set,

now it's working fine, no more crash ! (playing video hd).

setting migrate_set_downtime to 1sec, give me:

for no memory activity : 36ms downtime

for high memory activity : 650ms downtime


I'll do more extensive tests monday.

Thanks for the patches !



----- Mail original ----- 

De: "Stefan Priebe" <s.priebe at profihost.ag> 
À: "Alexandre DERUMIER" <aderumier at odiso.com> 
Cc: pve-devel at pve.proxmox.com, "Dietmar Maurer" <dietmar at proxmox.com> 
Envoyé: Samedi 29 Décembre 2012 15:07:30 
Objet: Re: [pve-devel] [PATCH 3/3] add downtime && expected_downtime query-migrate info 

Hi, 

Am 29.12.2012 15:02, schrieb Alexandre DERUMIER: 
> oh, ok. 
> 
> I'll tests the patchs mondays. 
Thanks! 

> Does migration works fine for you with them ? 
Yes - it works fine / perfectly. But i've no VM with a desktop to play 
HD videos ;-) 

Stefan 

> ----- Mail original ----- 
> 
> De: "Stefan Priebe" <s.priebe at profihost.ag> 
> À: "Alexandre DERUMIER" <aderumier at odiso.com> 
> Cc: pve-devel at pve.proxmox.com, "Dietmar Maurer" <dietmar at proxmox.com> 
> Envoyé: Samedi 29 Décembre 2012 14:55:33 
> Objet: Re: [pve-devel] [PATCH 3/3] add downtime && expected_downtime query-migrate info 
> 
> Hi Alexandre, 
> Am 29.12.2012 14:51, schrieb Alexandre DERUMIER: 
>> Great to known that you finally found it !. 
> Not found ;-) just know that the rework of the migration code fixes it. 
> May be it might be a deadlock - they've changed the locking and 
> threading handling. 
> 
>> (Do you have respond to Paolo Bonzini, because the last response I see was that it was not working). 
> No - i just tried to apply parts of the patchset on qemu 1.3 instead of 
> using whole 1.4 git trunk. 
> 
>>>> Alexandre might you try my rebased patches on top of pve-qemu 1.3? 
>> So, Do you want to apply the first patch only (to fix you bug), or apply the whole patches set ? (equal to current git code ?). 
> All patches i had attached ;-) 
> 
>> Not sure that Dietmar want to apply the big patches set ;) 
> sure but just wanted to know if they're stable for you too. So if 
> they're i will care about them in my own branch. So I've at least a 
> working migration. 
> 
>> I'll retest the whole patches set, because I was having migration problem (vm paused with error) with last qemu git. 
> That's why i've ported not all patches and on top of qemu 1.3 instead of 
> qemu 1.4. 
> 
> Stefan 
>