[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH 6/9] qemu-file: remove shutdown member
From: |
Juan Quintela |
Subject: |
[PATCH 6/9] qemu-file: remove shutdown member |
Date: |
Thu, 4 May 2023 13:38:38 +0200 |
The first thing that we do after setting the shutdown value is set the
error as -EIO if there is not a previous error.
So this value is reduntant. Just remove it and use
qemu_file_get_error() in the places that it was tested.
Signed-off-by: Juan Quintela <quintela@redhat.com>
---
migration/qemu-file.c | 10 +++-------
1 file changed, 3 insertions(+), 7 deletions(-)
diff --git a/migration/qemu-file.c b/migration/qemu-file.c
index 771aba7369..7b32ef45a9 100644
--- a/migration/qemu-file.c
+++ b/migration/qemu-file.c
@@ -63,8 +63,6 @@ struct QEMUFile {
int last_error;
Error *last_error_obj;
- /* has the file has been shutdown */
- bool shutdown;
};
/*
@@ -78,8 +76,6 @@ int qemu_file_shutdown(QEMUFile *f)
{
int ret = 0;
- f->shutdown = true;
-
/*
* We must set qemufile error before the real shutdown(), otherwise
* there can be a race window where we thought IO all went though
@@ -294,7 +290,7 @@ void qemu_fflush(QEMUFile *f)
return;
}
- if (f->shutdown) {
+ if (qemu_file_get_error(f)) {
return;
}
if (f->iovcnt > 0) {
@@ -407,7 +403,7 @@ static ssize_t coroutine_mixed_fn qemu_fill_buffer(QEMUFile
*f)
f->buf_index = 0;
f->buf_size = pending;
- if (f->shutdown) {
+ if (qemu_file_get_error(f)) {
return 0;
}
@@ -496,7 +492,7 @@ static int add_to_iovec(QEMUFile *f, const uint8_t *buf,
size_t size,
} else {
if (f->iovcnt >= MAX_IOV_SIZE) {
/* Should only happen if a previous fflush failed */
- assert(f->shutdown || !qemu_file_is_writable(f));
+ assert(qemu_file_get_error(f) || !qemu_file_is_writable(f));
return 1;
}
if (may_free) {
--
2.40.0
- [PATCH 0/9] QEMU file cleanups, Juan Quintela, 2023/05/04
- [PATCH 1/9] migration: max_postcopy_bandwidth is a size parameter, Juan Quintela, 2023/05/04
- [PATCH 2/9] migration: qemu_file_total_transferred() function is monotonic, Juan Quintela, 2023/05/04
- [PATCH 4/9] qemu-file: Make rate_limit_used an uint64_t, Juan Quintela, 2023/05/04
- [PATCH 5/9] qemu-file: No need to check for shutdown in qemu_file_rate_limit, Juan Quintela, 2023/05/04
- [PATCH 6/9] qemu-file: remove shutdown member,
Juan Quintela <=
- [PATCH 8/9] qemu-file: Make ram_control_save_page() use accessors for rate_limit, Juan Quintela, 2023/05/04
- [PATCH 7/9] qemu-file: Make total_transferred an uint64_t, Juan Quintela, 2023/05/04
- [PATCH 9/9] qemu-file: Account for rate_limit usage on qemu_fflush(), Juan Quintela, 2023/05/04