Re: [libvirt] [PATCH] virterror: Add error message for unsupported operations.
On 08/09/2012 07:31 AM, Peter Krempa wrote:
This patch introduces a new error code VIR_ERR_OPERATION_UNSUPPORTED
to
mark error messages regarding operations that failed due to lack of
support on the hypervisor or other than libvirt issues.
The code is first used in reporting error if qemu does not support block
IO tuning variables yielding error message:
error: Unable to get block I/O throttle parameters
error: Operation not supported: block_io_throttle field
'total_bytes_sec' missing in qemu's output
instead of:
error: Unable to get block I/O throttle parameters
error: internal error cannot read total_bytes_sec
---
In the past, we have used VIR_ERR_CONFIG_UNSUPPORTED for messages about
a qemu binary that doesn't support something; would that be any better
than inventing a new error here? Or are all of those errors worth
switching over to this new code?
As written, your patch seems fine, but only if we agree that a new error
is the way to go.
--
Eric Blake eblake(a)redhat.com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
Attachments:
- signature.asc (application/pgp-signature — 620 bytes)