19.03.2014 21:24, Mario Smarduch wrote:
>
> Fix return condition check from kvm_vm_ioctl(s, KVM_GET_DIRTY_LOG, &d) to
> handle internal failures or no support for memory slot dirty bitmap.
> Otherwise the ioctl succeeds and continues with migration.
> Addresses BUG# 1294227
Thanks, applied to -trivial.
[]
> - if (kvm_vm_ioctl(s, KVM_GET_DIRTY_LOG, &d) == -1) {
> + if (kvm_vm_ioctl(s, KVM_GET_DIRTY_LOG, &d) < 0) {
Actually I'd make all error checks like this. Many checks compares
return value with -1 for some reason, while actually, any <0 value
should be treated as error in many many places... At the same time,
comparison with 0 is cheaper than comparison with -1 ;)
/mjt
