Re: [PATCH v1 3/7] qapi/migration: Introduce the iteration-count

2024-09-18 Thread Fabiano Rosas
Yong Huang writes: > On Tue, Sep 17, 2024 at 4:35 AM Fabiano Rosas wrote: > >> Hyman Huang writes: >> >> > The original migration information dirty-sync-count could >> > no longer reflect iteration count due to the introduction >> > of background synchronization in the next commit; >> > add the

Re: [PATCH v1 3/7] qapi/migration: Introduce the iteration-count

2024-09-18 Thread Yong Huang
On Tue, Sep 17, 2024 at 4:35 AM Fabiano Rosas wrote: > Hyman Huang writes: > > > The original migration information dirty-sync-count could > > no longer reflect iteration count due to the introduction > > of background synchronization in the next commit; > > add the iteration count to compensate

Re: [PATCH v1 3/7] qapi/migration: Introduce the iteration-count

2024-09-16 Thread Yong Huang
On Tue, Sep 17, 2024 at 4:35 AM Fabiano Rosas wrote: > Hyman Huang writes: > > > The original migration information dirty-sync-count could > > no longer reflect iteration count due to the introduction > > of background synchronization in the next commit; > > add the iteration count to compensate

Re: [PATCH v1 3/7] qapi/migration: Introduce the iteration-count

2024-09-16 Thread Fabiano Rosas
Hyman Huang writes: > The original migration information dirty-sync-count could > no longer reflect iteration count due to the introduction > of background synchronization in the next commit; > add the iteration count to compensate. I agree with the overall idea, but I feel we're lacking some in

[PATCH v1 3/7] qapi/migration: Introduce the iteration-count

2024-09-15 Thread Hyman Huang
The original migration information dirty-sync-count could no longer reflect iteration count due to the introduction of background synchronization in the next commit; add the iteration count to compensate. Signed-off-by: Hyman Huang --- migration/migration-stats.h | 4 migration/migration.