Hi This series do several cleanups: - Dave found that I was using "%d" for unsigned, fix all uses. - We pass last_stage left and right, but we only use it in two places Just move it to RAMState. - do_compress_page() used a goto when not needed. - ram_release_pages() was always used with one page - And put it when we detect zero pages, instead of everywhere we have find a zero page.
Please, review. Juan Quintela (5): migration: All this fields are unsigned migration: We only need last_stage in two places migration: ram_release_pages() always receive 1 page as argument migration: simplify do_compress_ram_page migration: Move ram_release_pages() call to save_zero_page_to_file() migration/multifd-zlib.c | 20 +++++------ migration/multifd-zstd.c | 24 +++++++------- migration/multifd.c | 16 ++++----- migration/ram.c | 71 +++++++++++++++++----------------------- migration/trace-events | 26 +++++++-------- 5 files changed, 73 insertions(+), 84 deletions(-) -- 2.33.1
