[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PULL 9/9] balloon: Eliminate silly QERR_ macros
From: |
Markus Armbruster |
Subject: |
[Qemu-devel] [PULL 9/9] balloon: Eliminate silly QERR_ macros |
Date: |
Thu, 5 Feb 2015 08:36:23 +0100 |
The QERR_ macros are leftovers from the days of "rich" error objects.
They're used with error_set() and qerror_report(), and expand into the
first *two* arguments. This trickiness has become pointless. Clean
up the balloon ones.
Signed-off-by: Markus Armbruster <address@hidden>
Reviewed-by: Eric Blake <address@hidden>
Signed-off-by: Markus Armbruster <address@hidden>
---
balloon.c | 6 ++++--
include/qapi/qmp/qerror.h | 6 ------
2 files changed, 4 insertions(+), 8 deletions(-)
diff --git a/balloon.c b/balloon.c
index 728bb70..dea19a4 100644
--- a/balloon.c
+++ b/balloon.c
@@ -39,11 +39,13 @@ static void *balloon_opaque;
static bool have_ballon(Error **errp)
{
if (kvm_enabled() && !kvm_has_sync_mmu()) {
- error_set(errp, QERR_KVM_MISSING_CAP, "synchronous MMU", "balloon");
+ error_set(errp, ERROR_CLASS_KVM_MISSING_CAP,
+ "Using KVM without synchronous MMU, balloon unavailable");
return false;
}
if (!balloon_event_fn) {
- error_set(errp, QERR_DEVICE_NOT_ACTIVE, "balloon");
+ error_set(errp, ERROR_CLASS_DEVICE_NOT_ACTIVE,
+ "No balloon device has been activated");
return false;
}
return true;
diff --git a/include/qapi/qmp/qerror.h b/include/qapi/qmp/qerror.h
index 28f980e..eeaf0cb 100644
--- a/include/qapi/qmp/qerror.h
+++ b/include/qapi/qmp/qerror.h
@@ -70,9 +70,6 @@ void qerror_report_err(Error *err);
#define QERR_DEVICE_NO_HOTPLUG \
ERROR_CLASS_GENERIC_ERROR, "Device '%s' does not support hotplugging"
-#define QERR_DEVICE_NOT_ACTIVE \
- ERROR_CLASS_DEVICE_NOT_ACTIVE, "No %s device has been activated"
-
#define QERR_DEVICE_NOT_ENCRYPTED \
ERROR_CLASS_GENERIC_ERROR, "Device '%s' is not encrypted"
@@ -109,9 +106,6 @@ void qerror_report_err(Error *err);
#define QERR_JSON_PARSING \
ERROR_CLASS_GENERIC_ERROR, "Invalid JSON syntax"
-#define QERR_KVM_MISSING_CAP \
- ERROR_CLASS_KVM_MISSING_CAP, "Using KVM without %s, %s unavailable"
-
#define QERR_MIGRATION_ACTIVE \
ERROR_CLASS_GENERIC_ERROR, "There's a migration process in progress"
--
1.9.3
- [Qemu-devel] [PULL 0/9] qmp hmp balloon: Cleanups around error reporting, Markus Armbruster, 2015/02/05
- [Qemu-devel] [PULL 3/9] hmp: Compile hmp_info_spice() only with CONFIG_SPICE, Markus Armbruster, 2015/02/05
- [Qemu-devel] [PULL 2/9] qmp hmp: Improve error messages when SPICE is not in use, Markus Armbruster, 2015/02/05
- [Qemu-devel] [PULL 1/9] qmp hmp: Factor out common "using spice" test, Markus Armbruster, 2015/02/05
- [Qemu-devel] [PULL 9/9] balloon: Eliminate silly QERR_ macros,
Markus Armbruster <=
- [Qemu-devel] [PULL 7/9] balloon: Inline qemu_balloon(), qemu_balloon_status(), Markus Armbruster, 2015/02/05
- [Qemu-devel] [PULL 6/9] qmp: Eliminate silly QERR_COMMAND_NOT_FOUND macro, Markus Armbruster, 2015/02/05
- [Qemu-devel] [PULL 5/9] qmp: Simplify recognition of capability negotiation command, Markus Armbruster, 2015/02/05
- [Qemu-devel] [PULL 8/9] balloon: Factor out common "is balloon active" test, Markus Armbruster, 2015/02/05
- [Qemu-devel] [PULL 4/9] qmp: Clean up qmp_query_spice() #ifndef !CONFIG_SPICE dummy, Markus Armbruster, 2015/02/05
- Re: [Qemu-devel] [PULL 0/9] qmp hmp balloon: Cleanups around error reporting, Peter Maydell, 2015/02/05