Re: [PATCH v2 02/17] migration/ram: Add load start trace event

[Maciej S. Szmigiero]

On 28.08.2024 20:44, Fabiano Rosas wrote:
> "Maciej S. Szmigiero" <mail@maciej.szmigiero.name> writes:
>
> > From: "Maciej S. Szmigiero" <maciej.szmigiero@oracle.com>
> >
> > There's a RAM load complete trace event but there wasn't its start equivalent.
> >
> > Signed-off-by: Maciej S. Szmigiero <maciej.szmigiero@oracle.com>
> > ---
> >   migration/ram.c        | 1 +
> >   migration/trace-events | 1 +
> >   2 files changed, 2 insertions(+)
> >
> > diff --git a/migration/ram.c b/migration/ram.c
> > index 67ca3d5d51a1..7997bd830b9c 100644
> > --- a/migration/ram.c
> > +++ b/migration/ram.c
> > @@ -4127,6 +4127,7 @@ static int ram_load_precopy(QEMUFile *f)
> >                             RAM_SAVE_FLAG_ZERO);
> >       }
> >   
> > +    trace_ram_load_start();
>
> This would fit better at ram_load() paired with trace_ram_load_complete(), no?

Right - will move it there in the next version of this patch set.

Thanks,
Maciej