search

ARM-software / trappy

This repository has moved to https://gitlab.arm.com/tooling/trappy
Apache License 2.0
60 stars 39 forks source link

ftrace: Only add dynamic classes on non-custom scopes #300

Closed douglas-raillard-arm closed 4 years ago

douglas-raillard-arm commented 4 years ago

Custom scope is supposed to give only the events that are explicitly listed in events. Therefore, avoid adding dynamic events in this case, to speed up parsing and avoid consuming memory.

douglas-raillard-arm commented 4 years ago

Could you fix the tests and doc/Dynamic traces.ipynb

Do you mean converting Python 3 code to Python 2 ? Other than that the way the Ftrace is used should stay the same, as scope defaults to "all", which is != 'custom' and will therefore get the dynamic parsers.

EDIT: maybe you meant removing the register_dynamic_parser as it is now somewhat deprecated ?

douglas-raillard-arm commented 4 years ago

@JaviMerino Updated with unit test fix