[PATCH v6 0/3] Migration: Make more ram_counters atomic

Juan Quintela posted 3 patches 1 year ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20230427095250.22350-1-quintela@redhat.com
Maintainers: Juan Quintela <quintela@redhat.com>
include/qemu/stats64.h |  6 ++++++
migration/migration.c  |  9 ++++++---
migration/ram.c        |  7 ++++---
migration/ram.h        |  4 ++--
util/stats64.c         | 11 +++++++++++
5 files changed, 29 insertions(+), 8 deletions(-)
[PATCH v6 0/3] Migration: Make more ram_counters atomic
Posted by Juan Quintela 1 year ago
Hi

Changes in v6:
- Create stat64_set() operation
  Impersonating Paolo in "Author" field, as he is the real author.
- Update to stat64_set()
- Everything is consistent and coherent under the sun.

Please review.

[v5]
Not only change the type of the counters, also use the __nocheck()
variants of the functions.

Please, review.

[v4]
- Change aligned_uint64_t to size_t to make (some) 32bit hosts happy.

Please review.

[v3]
- Addressed reviews
- All counters are now atomic, either Stat64 or atomic.
- Rename duplicated to zero_pages
- Rename normal to zero_pages.

Please review.

[v2]
- fix typos found by David Edmondson
- Add review-by tags.

Please review.

[v1]
On previous series we cerate ram_atomic_counters.  But we basically
need that all counters are atomic.  So move back to only have
ram_counters, just with a new type that allows the atomic counters.

Once there, move update of stats out of RAM mutex.
And make multifd_bytes atomic.

Later, Juan.

Juan Quintela (2):
  migration: Make dirty_pages_rate atomic
  migration: Make dirty_bytes_last_sync atomic

Paolo Bonzini (1):
  stat64: Add stat64_set() operation

 include/qemu/stats64.h |  6 ++++++
 migration/migration.c  |  9 ++++++---
 migration/ram.c        |  7 ++++---
 migration/ram.h        |  4 ++--
 util/stats64.c         | 11 +++++++++++
 5 files changed, 29 insertions(+), 8 deletions(-)

-- 
2.40.0