[libvirt] [PATCH] migration: Add more specific error code/message on migration abort

[Eric Blake]

On 12/16/2011 07:14 AM, Peter Krempa wrote:
> A generic error code was returned, if the user aborted a migration job.
> This made it hard to distinguish between a user requested abort and an
> error that might have occured. This patch introduces a new error code,

s/occured/occurred/

> which is returned if a migration API call fails. This makes it easier to

That wording didn't quite make sense to me - you made it sound like
arbitrary failure gets the new error value.  Maybe a better wording
would be:

This patch introduces a new error code, which is returned in the
specific case of a user abort, while leaving all other failures with
their existing code.

> distinguish between failure while mirgrating and an user requested
> abort.
> 
>  * include/libvirt/virterror.h: - add new error code
>  * src/util/virterror.c: - add message for the new error code
>  * src/qemu/qemu_migration.h: - Emit operation aborted error instead of
>                                 operation failed, on migration abort
> ---
>  include/libvirt/virterror.h |    2 ++
>  src/qemu/qemu_migration.c   |    2 +-
>  src/util/virterror.c        |    6 ++++++
>  3 files changed, 9 insertions(+), 1 deletions(-)

ACK.

-- 
Eric Blake   eblake at redhat.com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 620 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20111216/1a05610d/attachment-0001.sig>