Reviewed-by: Mike Leach <redacted>


On Fri, 3 Sept 2021 at 12:16, Arnaldo Carvalho de Melo [off-list ref] wrote:

Em Fri, Sep 03, 2021 at 10:15:07AM +0100, Suzuki K Poulose escreveu:

quoted

On 06/08/2021 14:41, James Clark wrote:

quoted

If the magic number indicates ETE instantiate a OCSD_BUILTIN_DCD_ETE
decoder instead of OCSD_BUILTIN_DCD_ETMV4I. ETE is the new trace feature
for Armv9.

Testing performed
=================

* Old files with v0 and v1 headers for ETMv4 still open correctly
* New files with new magic number open on new versions of perf
* New files with new magic number fail to open on old versions of perf
* Decoding with the ETE decoder results in the same output as the ETMv4
   decoder as long as there are no new ETE packet types

Signed-off-by: James Clark <redacted>

Changes look good to me with my limited knowledge about the decoder
facing code.

FWIW,

Acked-by: Suzuki K Poulose <suzuki.poulose@arm.com>

Thanks, collected the Acked-by.

- Arnaldo

--
Mike Leach
Principal Engineer, ARM Ltd.
Manchester Design Centre. UK