[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH v2 06/11] qemufile: Always return a verbose error
From: |
Peter Xu |
Subject: |
[PATCH v2 06/11] qemufile: Always return a verbose error |
Date: |
Tue, 12 Sep 2023 18:21:40 -0400 |
There're a lot of cases where we only have an errno set in last_error but
without a detailed error description. When this happens, try to generate
an error contains the errno as a descriptive error.
This will be helpful in cases where one relies on the Error*. E.g.,
migration state only caches Error* in MigrationState.error. With this,
we'll display correct error messages in e.g. query-migrate when the error
was only set by qemu_file_set_error().
Reviewed-by: Fabiano Rosas <farosas@suse.de>
Signed-off-by: Peter Xu <peterx@redhat.com>
---
migration/qemu-file.c | 15 ++++++++++++---
1 file changed, 12 insertions(+), 3 deletions(-)
diff --git a/migration/qemu-file.c b/migration/qemu-file.c
index eea7171192..3e64e900c9 100644
--- a/migration/qemu-file.c
+++ b/migration/qemu-file.c
@@ -142,15 +142,24 @@ void qemu_file_set_hooks(QEMUFile *f, const QEMUFileHooks
*hooks)
*
* Return negative error value if there has been an error on previous
* operations, return 0 if no error happened.
- * Optional, it returns Error* in errp, but it may be NULL even if return value
- * is not 0.
*
+ * If errp is specified, a verbose error message will be copied over.
*/
int qemu_file_get_error_obj(QEMUFile *f, Error **errp)
{
+ if (!f->last_error) {
+ return 0;
+ }
+
+ /* There is an error */
if (errp) {
- *errp = f->last_error_obj ? error_copy(f->last_error_obj) : NULL;
+ if (f->last_error_obj) {
+ *errp = error_copy(f->last_error_obj);
+ } else {
+ error_setg_errno(errp, -f->last_error, "Channel error");
+ }
}
+
return f->last_error;
}
--
2.41.0
- [PATCH v2 00/11] migration: Better error handling in rp thread, allow failures in recover, Peter Xu, 2023/09/12
- [PATCH v2 01/11] migration: Display error in query-migrate irrelevant of status, Peter Xu, 2023/09/12
- [PATCH v2 02/11] migration: Let migrate_set_error() take ownership, Peter Xu, 2023/09/12
- [PATCH v2 05/11] migration: Deliver return path file error to migrate state too, Peter Xu, 2023/09/12
- [PATCH v2 04/11] migration: Refactor error handling in source return path, Peter Xu, 2023/09/12
- [PATCH v2 08/11] migration: Add migration_rp_wait|kick(), Peter Xu, 2023/09/12
- [PATCH v2 03/11] migration: Introduce migrate_has_error(), Peter Xu, 2023/09/12
- [PATCH v2 06/11] qemufile: Always return a verbose error,
Peter Xu <=
- [PATCH v2 07/11] migration: Remember num of ramblocks to sync during recovery, Peter Xu, 2023/09/12
- [PATCH v2 09/11] migration: Allow network to fail even during recovery, Peter Xu, 2023/09/12
- [PATCH v2 11/11] tests/migration-test: Add a test for postcopy hangs during RECOVER, Peter Xu, 2023/09/12
- [PATCH v2 10/11] migration: Allow RECOVER->PAUSED convertion for dest qemu, Peter Xu, 2023/09/12