Re: [Qemu-devel] [PATCH] Avoid divide by zero when there is no block device to migrate

[Kevin Wolf]

Am 12.01.2011 12:01, schrieb Pierre Riteau:
> On 11 janv. 2011, at 12:54, Kevin Wolf wrote:
> 
>> Am 25.12.2010 21:52, schrieb Pierre Riteau:
>>> When block migration is requested and no read-write block device is
>>> present, a divide by zero exception is triggered because
>>> total_sector_sum equals zero.
>>>
>>> Signed-off-by: Pierre Riteau <address@hidden>
>>
>> Maybe in this case we should generate an error before actually starting
>> with block migration. If you bothered to request block migration you
>> certainly didn't have in mind to do nothing.
> 
> I had in mind a scenario where a management system would use block migration 
> all the time, regardless of what kind of devices were plugged in the VM.
> If we throw an error, should it be a fatal error? Or just a migration failed 
> result given to the monitor?

Okay, that makes sense.

We could still check for this condition before really starting block
migration and complete it immediately (successfully, though). I'm also
okay with your patch as it's a very simple solution to the problem. If
you resend it with the > 80 characters line fixed, I'll apply it.

Kevin