Hi Arnaldo, Rob, On Fri, May 25, 2018 at 12:27:13PM -0300, Arnaldo Carvalho de Melo wrote: > Em Fri, May 25, 2018 at 03:03:47PM +0100, Robert Walker escreveu: > > Hi Leo, > > > > Following the discussions from your reply to this with a simplified patch, > > this version of the patch works better as you also need to emit a branch > > sample when handling a CS_ETM_TRACE_ON packet to indicate the end of a block > > of trace.
I also will follow the suggestion as Rob mentioned in another email: "The deadbeefdeadbeef addresses are a bit ugly - these are just dummy values emitted in the decoder layer - maybe these should be changed to 0." > > This patch does not break the output from perf inject to generate > > instruction samples for AutoFDO, so I am happy with that. Thanks for confirmation. > > Regards > > > > Rob > > > > Reviewed-by: Robert Walker <robert.wal...@arm.com> > > So, Leo, can you please resubmit, bumping the v2 to v3 (or the latest > one, I haven't fully reread this thread) add this "Reviewed-by: Robert" > tag and any other that people may have provided, so that I can merge it? Sure! I will respin the v3 patch series by following up Rob's suggestion and add Rob's review tag. BTW, I'd like to get ack from Mathieu as well. Mathieu is working on CPU wide tracing, so I talked with Mathieu he will review the patch series if has conflict with CPU wide tracing. [...] Thanks, Leo Yan
