On 22/02/2021 10:54, vincent.donnef...@arm.com wrote:
> From: Vincent Donnefort <vincent.donnef...@arm.com>

[...]

> Also, replace sub_positive with lsub_positive which saves one explicit
> load-store.

IMHO, in case you're going to fix this now in compute_energy(), this
optimization could still be done. Maybe in an extra patch?
cpu_util_without() is using lsub_positive() to remove task util from cpu
util as well.

[...]

Reply via email to