search

ARM-software / lisa

Linux Integrated System Analysis
Apache License 2.0
203 stars 119 forks source link

lisa._assets.kmodules.lisa: follow latest upstream #2161

Closed honxia02 closed 9 months ago

honxia02 commented 10 months ago

Upstream has completely removed struct util_est. Now util_est is just an unsigned int field.

douglas-raillard-arm commented 10 months ago

I see that this change also renames event fields. Do we really want to do that ? If so this needs to come with the the fixes everywhere it's required in LISA, and ensure it goes through an analysis method that will deal with backward compat of event format.

douglas-raillard-arm commented 10 months ago

The commit message will need to be updated to add the FIX tag (see history).

honxia02 commented 9 months ago

Closed, because so far the conclusion is that the behavior of the trace event has changed and it's not a good idea to modify the old ones. Instead we should treat this as a new signal.