[PATCH 0/5] migration: misc cleanups

Juan Quintela posted 5 patches 2 years, 4 months ago
Test checkpatch passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20211216091332.25681-1-quintela@redhat.com
Maintainers: "Dr. David Alan Gilbert" <dgilbert@redhat.com>, Juan Quintela <quintela@redhat.com>
There is a newer version of this series
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(-)
[PATCH 0/5] migration: misc cleanups
Posted by Juan Quintela 2 years, 4 months ago
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