Will Deacon <will.dea...@arm.com> writes: > On Fri, Feb 17, 2017 at 04:06:59PM +0200, Alexander Shishkin wrote: >> Will Deacon <will.dea...@arm.com> writes: >> >> > On Fri, Feb 17, 2017 at 03:40:23PM +0200, Alexander Shishkin wrote: >> >> Will Deacon <will.dea...@arm.com> writes: >> >> >> >> > @@ -485,7 +485,8 @@ int intel_bts_interrupt(void) >> >> > return handled; >> >> > >> >> > perf_aux_output_end(&bts->handle, local_xchg(&buf->data_size, >> >> > 0), >> >> > - !!local_xchg(&buf->lost, 0)); >> >> > + local_xchg(&buf->lost, 0) ? >> >> > + PERF_AUX_FLAG_OVERWRITE : 0); >> >> >> >> Heh, this one would have taken some time to debug. :) >> > >> > Don't worry, this isn't a bug fix! This patch changes the prototype for >> > perf_aux_output_end so that it takes the flag instead of a "bool truncated" >> > parameter, so this is just fixing up the callers at the same time. >> >> Yeah, I got that, what I'm saying is that the above should be >> PERF_AUX_FLAG_TRUNCATED, not OVERWRITE. I only spotted it by accident. > > D'oh, quite right, I'll fix that now. Thanks for having a look.
Wait a bit with the fixing, I'm about to post an amended version. Regards, -- Alex
