Re: [PATCH] migration: Fix qmp_query_migrate mbps value

2024-02-25 Thread Peter Xu
On Fri, Feb 23, 2024 at 09:39:12AM -0300, Fabiano Rosas wrote: > I've been planning to merge migration_completion() and > migration_iteration_finish(). It's too unintuitive to do the completion > routine deep inside migration_iteration_run(). AFAICS those are all tail > calls, so we could bring

Re: [PATCH] migration: Fix qmp_query_migrate mbps value

2024-02-23 Thread Fabiano Rosas
Peter Xu writes: > On Thu, Feb 22, 2024 at 10:49:12AM -0300, Fabiano Rosas wrote: >> Peter Xu writes: >> >> > On Thu, Feb 22, 2024 at 05:40:41PM +0800, Peter Xu wrote: >> >> On Wed, Feb 21, 2024 at 09:56:36AM -0300, Fabiano Rosas wrote: >> >> > Peter Xu writes: >> >> > >> >> > > On Mon, Feb

Re: [PATCH] migration: Fix qmp_query_migrate mbps value

2024-02-22 Thread Peter Xu
On Thu, Feb 22, 2024 at 10:49:12AM -0300, Fabiano Rosas wrote: > Peter Xu writes: > > > On Thu, Feb 22, 2024 at 05:40:41PM +0800, Peter Xu wrote: > >> On Wed, Feb 21, 2024 at 09:56:36AM -0300, Fabiano Rosas wrote: > >> > Peter Xu writes: > >> > > >> > > On Mon, Feb 19, 2024 at 04:44:57PM

Re: [PATCH] migration: Fix qmp_query_migrate mbps value

2024-02-22 Thread Fabiano Rosas
Peter Xu writes: > On Thu, Feb 22, 2024 at 05:40:41PM +0800, Peter Xu wrote: >> On Wed, Feb 21, 2024 at 09:56:36AM -0300, Fabiano Rosas wrote: >> > Peter Xu writes: >> > >> > > On Mon, Feb 19, 2024 at 04:44:57PM -0300, Fabiano Rosas wrote: >> > >> The QMP command query_migrate might see

Re: [PATCH] migration: Fix qmp_query_migrate mbps value

2024-02-22 Thread Peter Xu
On Thu, Feb 22, 2024 at 05:40:41PM +0800, Peter Xu wrote: > On Wed, Feb 21, 2024 at 09:56:36AM -0300, Fabiano Rosas wrote: > > Peter Xu writes: > > > > > On Mon, Feb 19, 2024 at 04:44:57PM -0300, Fabiano Rosas wrote: > > >> The QMP command query_migrate might see incorrect throughput numbers > >

Re: [PATCH] migration: Fix qmp_query_migrate mbps value

2024-02-22 Thread Peter Xu
On Wed, Feb 21, 2024 at 09:56:36AM -0300, Fabiano Rosas wrote: > Peter Xu writes: > > > On Mon, Feb 19, 2024 at 04:44:57PM -0300, Fabiano Rosas wrote: > >> The QMP command query_migrate might see incorrect throughput numbers > >> if it runs after we've set the migration completion status but

Re: [PATCH] migration: Fix qmp_query_migrate mbps value

2024-02-21 Thread Fabiano Rosas
Peter Xu writes: > On Mon, Feb 19, 2024 at 04:44:57PM -0300, Fabiano Rosas wrote: >> The QMP command query_migrate might see incorrect throughput numbers >> if it runs after we've set the migration completion status but before >> migration_calculate_complete() has updated s->total_time and

Re: [PATCH] migration: Fix qmp_query_migrate mbps value

2024-02-20 Thread Peter Xu
On Mon, Feb 19, 2024 at 04:44:57PM -0300, Fabiano Rosas wrote: > The QMP command query_migrate might see incorrect throughput numbers > if it runs after we've set the migration completion status but before > migration_calculate_complete() has updated s->total_time and s->mbps. > > The migration

[PATCH] migration: Fix qmp_query_migrate mbps value

2024-02-19 Thread Fabiano Rosas
The QMP command query_migrate might see incorrect throughput numbers if it runs after we've set the migration completion status but before migration_calculate_complete() has updated s->total_time and s->mbps. The migration status would show COMPLETED, but the throughput value would be the one