On Mon, 19 Apr 2021 08:10:57 +0300
Tzvetomir Stoyanov <tz.stoyanov@xxxxxxxxx> wrote:
> > --- a/lib/trace-cmd/trace-input.c
> > +++ b/lib/trace-cmd/trace-input.c
> > @@ -1234,9 +1234,11 @@ static unsigned long long timestamp_calc(unsigned long long ts, int cpu,
> > ts -= handle->tsc_calc.offset;
> > ts = mul_u64_u32_shr(ts, handle->tsc_calc.mult, handle->tsc_calc.shift);
> > } else {
> > - tracecmd_warning("Timestamp $llu is before the initial offset %llu, set it to 0",
> > + tracecmd_warning("Timestamp %llu is before the initial offset %llu\n"
> > + "\tSetting offset to 0",
> > ts, handle->tsc_calc.offset);
> > - ts = 0;
> > + handle->tsc_calc.offset = 0;
> > + ts = mul_u64_u32_shr(ts, handle->tsc_calc.mult, handle->tsc_calc.shift);
>
> That looks like from a different patch? I'll send v3 from my patch set
> with these fixes.
>
> > }
> > }
Yes, that was leftover from another change that accidentally got committed
into this one. Thanks for noticing. I'll be sending out a v2.
-- Steve
|