[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 10/12] hw/core: Improve the query-hotpluggable-cpus error mes
From: |
Markus Armbruster |
Subject: |
Re: [PATCH 10/12] hw/core: Improve the query-hotpluggable-cpus error message |
Date: |
Tue, 07 Feb 2023 13:53:39 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/28.2 (gnu/linux) |
Philippe Mathieu-Daudé <philmd@linaro.org> writes:
> On 7/2/23 08:51, Markus Armbruster wrote:
>> The QERR_ macros are leftovers from the days of "rich" error objects.
>> We've been trying to reduce their remaining use.
>> Get rid of a use of QERR_FEATURE_DISABLED, and improve the slightly
>> awkward error message
>> (qemu) info hotpluggable-cpus
>> Error: The feature 'query-hotpluggable-cpus' is not enabled
>> to
>> Error: machine does not support hot-plugging CPUs
>> Signed-off-by: Markus Armbruster <armbru@redhat.com>
>> ---
>> hw/core/machine-qmp-cmds.c | 3 +--
>> 1 file changed, 1 insertion(+), 2 deletions(-)
>> diff --git a/hw/core/machine-qmp-cmds.c b/hw/core/machine-qmp-cmds.c
>> index a6ed3a63c3..2d904747c0 100644
>> --- a/hw/core/machine-qmp-cmds.c
>> +++ b/hw/core/machine-qmp-cmds.c
>> @@ -16,7 +16,6 @@
>> #include "qapi/error.h"
>> #include "qapi/qapi-builtin-visit.h"
>> #include "qapi/qapi-commands-machine.h"
>> -#include "qapi/qmp/qerror.h"
>> #include "qapi/qmp/qobject.h"
>> #include "qapi/qobject-input-visitor.h"
>> #include "qapi/type-helpers.h"
>> @@ -141,7 +140,7 @@ HotpluggableCPUList *qmp_query_hotpluggable_cpus(Error
>> **errp)
>> MachineClass *mc = MACHINE_GET_CLASS(ms);
>> if (!mc->has_hotpluggable_cpus) {
>> - error_setg(errp, QERR_FEATURE_DISABLED, "query-hotpluggable-cpus");
>> + error_setg(errp, "machine does not support hot-plugging CPUs");
>
> "hotplug CPUs" or "hot-pluggable CPUs"?
I think "hotplug CPUs" is strictly worse than "hotpluggable CPUs",
because "hotpluggable" is an adjective, while "hotplug" is not.
I slightly prefer "does not support hot-plugging CPUs" over "does not
support hot-pluggable CPUs".
As always, I'm happy to take advice from native speakers :)
>
>> return NULL;
>> }
>>
- [PATCH 12/12] rocker: Tweak stubbed out monitor commands' error messages, (continued)
- [PATCH 12/12] rocker: Tweak stubbed out monitor commands' error messages, Markus Armbruster, 2023/02/07
- [PATCH 11/12] migration/colo: Improve an x-colo-lost-heartbeat error message, Markus Armbruster, 2023/02/07
- [PATCH 09/12] replay: Simplify setting replay blockers, Markus Armbruster, 2023/02/07
- [PATCH 10/12] hw/core: Improve the query-hotpluggable-cpus error message, Markus Armbruster, 2023/02/07
- [PATCH 06/12] hw/acpi: Dumb down acpi_table_add() stub, Markus Armbruster, 2023/02/07
- [PATCH 08/12] qga: Drop dangling reference to QERR_QGA_LOGGING_DISABLED, Markus Armbruster, 2023/02/07
- [PATCH 01/12] error: Drop superfluous #include "qapi/qmp/qerror.h", Markus Armbruster, 2023/02/07
- [PATCH 03/12] dump: Assert cpu_get_note_size() can't fail, Markus Armbruster, 2023/02/07