[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v4 0/2] Migration: Make more ram_counters atomic
From: |
Juan Quintela |
Subject: |
Re: [PATCH v4 0/2] Migration: Make more ram_counters atomic |
Date: |
Wed, 26 Apr 2023 21:57:15 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/28.2 (gnu/linux) |
Juan Quintela <quintela@redhat.com> wrote:
> Hi
>
> In this v4:
> - Change aligned_uint64_t to size_t to make (some) 32bit hosts happy.
>
> Please review.
self-NACK
Still missing the removal of the __nocheck() functions.
>
> [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
>
> migration/migration.c | 10 +++++++---
> migration/ram.c | 8 +++++---
> migration/ram.h | 4 ++--
> 3 files changed, 14 insertions(+), 8 deletions(-)