[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH v2 08/10] migration: No need to return the size of t
From: |
Juan Quintela |
Subject: |
[Qemu-devel] [PATCH v2 08/10] migration: No need to return the size of the cache |
Date: |
Wed, 18 Oct 2017 12:36:31 +0200 |
After the previous commits, we make sure that the value passed is
right, or we just drop an error. So now we return if there is one
error or we have setup correctly the value passed.
Signed-off-by: Juan Quintela <address@hidden>
---
migration/migration.c | 6 ++----
migration/ram.c | 8 +++-----
migration/ram.h | 2 +-
3 files changed, 6 insertions(+), 10 deletions(-)
diff --git a/migration/migration.c b/migration/migration.c
index 3feffb5e26..f3d4503ce2 100644
--- a/migration/migration.c
+++ b/migration/migration.c
@@ -1371,14 +1371,12 @@ void qmp_migrate_cancel(Error **errp)
void qmp_migrate_set_cache_size(int64_t value, Error **errp)
{
MigrationState *s = migrate_get_current();
- int64_t new_size;
- new_size = xbzrle_cache_resize(value, errp);
- if (new_size < 0) {
+ if (xbzrle_cache_resize(value, errp) < 0) {
return;
}
- s->xbzrle_cache_size = new_size;
+ s->xbzrle_cache_size = value;
}
int64_t qmp_query_migrate_cache_size(Error **errp)
diff --git a/migration/ram.c b/migration/ram.c
index c84f22d759..ed4d3c6295 100644
--- a/migration/ram.c
+++ b/migration/ram.c
@@ -111,15 +111,15 @@ static void XBZRLE_cache_unlock(void)
* migration may be using the cache and might finish during this call,
* hence changes to the cache are protected by XBZRLE.lock().
*
- * Returns the new_size or negative in case of error.
+ * Returns the 0 or negative in case of error.
*
* @new_size: new cache size
* @errp: set *errp if the check failed, with reason
*/
-int64_t xbzrle_cache_resize(int64_t new_size, Error **errp)
+int xbzrle_cache_resize(int64_t new_size, Error **errp)
{
PageCache *new_cache;
- int64_t ret;
+ int64_t ret = 0;
/* Check for truncation */
if (new_size != (size_t)new_size) {
@@ -152,8 +152,6 @@ int64_t xbzrle_cache_resize(int64_t new_size, Error **errp)
cache_fini(XBZRLE.cache);
XBZRLE.cache = new_cache;
}
-
- ret = new_size;
out:
XBZRLE_cache_unlock();
return ret;
diff --git a/migration/ram.h b/migration/ram.h
index 511b3dc582..c8ae382b5b 100644
--- a/migration/ram.h
+++ b/migration/ram.h
@@ -35,7 +35,7 @@
extern MigrationStats ram_counters;
extern XBZRLECacheStats xbzrle_counters;
-int64_t xbzrle_cache_resize(int64_t new_size, Error **errp);
+int xbzrle_cache_resize(int64_t new_size, Error **errp);
uint64_t ram_bytes_remaining(void);
uint64_t ram_bytes_total(void);
--
2.13.6
- [Qemu-devel] [PATCH v2 00/10] Make xbzrle_cache_size a migration parameter, Juan Quintela, 2017/10/18
- [Qemu-devel] [PATCH v2 01/10] migration: Fix migrate_test_apply for multifd parameters, Juan Quintela, 2017/10/18
- [Qemu-devel] [PATCH v2 03/10] migration: Make cache size elements use the right types, Juan Quintela, 2017/10/18
- [Qemu-devel] [PATCH v2 02/10] migratiom: Remove max_item_age parameter, Juan Quintela, 2017/10/18
- [Qemu-devel] [PATCH v2 04/10] migration: Move xbzrle cache resize error handling to xbzrle_cache_resize, Juan Quintela, 2017/10/18
- [Qemu-devel] [PATCH v2 06/10] migration: Make sure that we pass the right cache size, Juan Quintela, 2017/10/18
- [Qemu-devel] [PATCH v2 05/10] migration: Make cache_init() take an error parameter, Juan Quintela, 2017/10/18
- [Qemu-devel] [PATCH v2 07/10] migration: Don't play games with the requested cache size, Juan Quintela, 2017/10/18
- [Qemu-devel] [PATCH v2 08/10] migration: No need to return the size of the cache,
Juan Quintela <=
- [Qemu-devel] [PATCH v2 09/10] migration: Make xbzrle_cache_size a migration parameter, Juan Quintela, 2017/10/18
- [Qemu-devel] [PATCH v2 10/10] migration: [RFC] Use proper types in json, Juan Quintela, 2017/10/18