[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PULL 07/36] multifd: Use proper maximum compression values
From: |
Juan Quintela |
Subject: |
[PULL 07/36] multifd: Use proper maximum compression values |
Date: |
Fri, 28 Jan 2022 19:30:04 +0100 |
It happens that there are functions to calculate the worst possible
compression size for a packet. Use them.
Suggested-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
Signed-off-by: Juan Quintela <quintela@redhat.com>
Reviewed-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
---
migration/multifd-zlib.c | 4 ++--
migration/multifd-zstd.c | 4 ++--
2 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/migration/multifd-zlib.c b/migration/multifd-zlib.c
index 9f6ebf1076..a2fec4d01d 100644
--- a/migration/multifd-zlib.c
+++ b/migration/multifd-zlib.c
@@ -54,8 +54,8 @@ static int zlib_send_setup(MultiFDSendParams *p, Error **errp)
error_setg(errp, "multifd %u: deflate init failed", p->id);
return -1;
}
- /* To be safe, we reserve twice the size of the packet */
- z->zbuff_len = MULTIFD_PACKET_SIZE * 2;
+ /* This is the maxium size of the compressed buffer */
+ z->zbuff_len = compressBound(MULTIFD_PACKET_SIZE);
z->zbuff = g_try_malloc(z->zbuff_len);
if (!z->zbuff) {
deflateEnd(&z->zs);
diff --git a/migration/multifd-zstd.c b/migration/multifd-zstd.c
index cc4e991724..97c08367d0 100644
--- a/migration/multifd-zstd.c
+++ b/migration/multifd-zstd.c
@@ -67,8 +67,8 @@ static int zstd_send_setup(MultiFDSendParams *p, Error **errp)
p->id, ZSTD_getErrorName(res));
return -1;
}
- /* To be safe, we reserve twice the size of the packet */
- z->zbuff_len = MULTIFD_PACKET_SIZE * 2;
+ /* This is the maxium size of the compressed buffer */
+ z->zbuff_len = ZSTD_compressBound(MULTIFD_PACKET_SIZE);
z->zbuff = g_try_malloc(z->zbuff_len);
if (!z->zbuff) {
ZSTD_freeCStream(z->zcs);
--
2.34.1
- [PULL 00/36] Migration 20220128 patches, Juan Quintela, 2022/01/28
- [PULL 01/36] migration: All this fields are unsigned, Juan Quintela, 2022/01/28
- [PULL 02/36] migration: We only need last_stage in two places, Juan Quintela, 2022/01/28
- [PULL 03/36] migration: ram_release_pages() always receive 1 page as argument, Juan Quintela, 2022/01/28
- [PULL 04/36] migration: Remove masking for compression, Juan Quintela, 2022/01/28
- [PULL 05/36] migration: simplify do_compress_ram_page, Juan Quintela, 2022/01/28
- [PULL 06/36] migration: Move ram_release_pages() call to save_zero_page_to_file(), Juan Quintela, 2022/01/28
- [PULL 07/36] multifd: Use proper maximum compression values,
Juan Quintela <=
- [PULL 08/36] multifd: Move iov from pages to params, Juan Quintela, 2022/01/28
- [PULL 09/36] multifd: Make zlib use iov's, Juan Quintela, 2022/01/28
- [PULL 11/36] multifd: Remove send_write() method, Juan Quintela, 2022/01/28
- [PULL 13/36] multifd: Unfold "used" variable by its value, Juan Quintela, 2022/01/28
- [PULL 10/36] multifd: Make zstd use iov's, Juan Quintela, 2022/01/28
- [PULL 12/36] multifd: Use a single writev on the send side, Juan Quintela, 2022/01/28
- [PULL 14/36] multifd: Use normal pages array on the send side, Juan Quintela, 2022/01/28
- [PULL 15/36] multifd: Use normal pages array on the recv side, Juan Quintela, 2022/01/28
- [PULL 16/36] multifd: recv side only needs the RAMBlock host address, Juan Quintela, 2022/01/28
- [PULL 17/36] multifd: Rename pages_used to normal_pages, Juan Quintela, 2022/01/28