Re: [kvm-devel] [PATCH]: Make migration handle errors a little better

2008-01-27 Thread Avi Kivity
Chris Lalancette wrote: > Uri Lublin wrote: > >> Patch looks good. >> Why did you define MIG_STAT_DIRTY_TRACK_FAIL and not >> MIG_STAT_KVM_SET_DIRTY_TRACKING_FAILED ? >> > > Oops, I slightly misunderstood this bit in my last e-mail. You were saying > that > there is already a "MIG_STAT_K

Re: [kvm-devel] [PATCH]: Make migration handle errors a little better

2008-01-24 Thread Chris Lalancette
Uri Lublin wrote: > Patch looks good. > Why did you define MIG_STAT_DIRTY_TRACK_FAIL and not > MIG_STAT_KVM_SET_DIRTY_TRACKING_FAILED ? Oops, I slightly misunderstood this bit in my last e-mail. You were saying that there is already a "MIG_STAT_KVM_SET_DIRTY_TRACKING_FAILED" error flag. In the

Re: [kvm-devel] [PATCH]: Make migration handle errors a little better

2008-01-24 Thread Chris Lalancette
Uri Lublin wrote: > > Chris Lalancette wrote: >> All, >> Attached is a fairly simple patch to the migration code to make it >> handle >> errors better. In particular, if the remote side wasn't set up properly >> (i.e. >> the memory size didn't match), the migration would fail, but the sour

Re: [kvm-devel] [PATCH]: Make migration handle errors a little better

2008-01-24 Thread Uri Lublin
Chris Lalancette wrote: > All, > Attached is a fairly simple patch to the migration code to make it handle > errors better. In particular, if the remote side wasn't set up properly (i.e. > the memory size didn't match), the migration would fail, but the source host > didn't actually react p

[kvm-devel] [PATCH]: Make migration handle errors a little better

2008-01-23 Thread Chris Lalancette
All, Attached is a fairly simple patch to the migration code to make it handle errors better. In particular, if the remote side wasn't set up properly (i.e. the memory size didn't match), the migration would fail, but the source host didn't actually react properly, so the migration would stil