[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 01/11] ppc: Cleanup error handling in ppc_set_co
From: |
Eric Blake |
Subject: |
Re: [Qemu-devel] [PATCH 01/11] ppc: Cleanup error handling in ppc_set_compat() |
Date: |
Fri, 11 Dec 2015 06:58:12 -0700 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.3.0 |
On 12/10/2015 05:11 PM, David Gibson wrote:
> Current ppc_set_compat() returns -1 for errors, and also (unconditionally)
> reports an error message. The caller in h_client_architecture_support()
> may then report it again using an outdated fprintf().
>
> Clean this up by using the modern error reporting mechanisms.
>
> Signed-off-by: David Gibson <address@hidden>
> ---
> @@ -9208,12 +9208,13 @@ int ppc_set_compat(PowerPCCPU *cpu, uint32_t
> cpu_version)
> break;
> }
>
> - if (kvm_enabled() && kvmppc_set_compat(cpu, cpu->cpu_version) < 0) {
> - error_report("Unable to set compatibility mode in KVM");
> - ret = -1;
> + if (kvm_enabled()) {
> + ret = kvmppc_set_compat(cpu, cpu->cpu_version);
> + if (ret < 0) {
> + error_setg(errp, "Unable to set CPU compatibility mode in KVM:
> %s",
> + strerror(-ret));
> + }
Could use error_setg_errno() here instead of manually calling strerror().
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature
- [Qemu-devel] [PATCH 07/11] pseries: Cleanup error handling in spapr_kvm_type(), (continued)