Hi While I am trying to put order in the atomic counters, I made in this series:
- convince and review code to see that everything is uint64_t. - f->shutdown is not needed. When we shutdown the file we put an error there if there is none. So remove it. - Make more clear how we use rate_limit. Please review. It is based on my previous series to the list: Subject: [PATCH 0/2] More migration stats Based-on: Message-Id: <20230504103357.22130-1-quint...@redhat.com> Juan Quintela (9): migration: max_postcopy_bandwidth is a size parameter migration: qemu_file_total_transferred() function is monotonic qemu-file: make qemu_file_[sg]et_rate_limit() use an uint64_t qemu-file: Make rate_limit_used an uint64_t qemu-file: No need to check for shutdown in qemu_file_rate_limit qemu-file: remove shutdown member qemu-file: Make total_transferred an uint64_t qemu-file: Make ram_control_save_page() use accessors for rate_limit qemu-file: Account for rate_limit usage on qemu_fflush() migration/block.c | 13 +++---------- migration/migration.c | 10 +++++----- migration/options.c | 2 +- migration/options.h | 2 +- migration/qemu-file.c | 42 +++++++++++++++++------------------------- migration/qemu-file.h | 10 +++++----- migration/savevm.c | 6 ++---- migration/vmstate.c | 2 +- 8 files changed, 35 insertions(+), 52 deletions(-) -- 2.40.0
