Hi Phil,

On 24/9/2020 6:39 AM, Phil Yang wrote:
Since rte_atomicXX APIs are not allowed to be used, use C11 atomic
builtins for power in use state update.

Signed-off-by: Phil Yang <phil.y...@arm.com>
Reviewed-by: Ruifeng Wang <ruifeng.w...@arm.com>
Reviewed-by: Honnappa Nagarahalli <honnappa.nagaraha...@arm.com>
---
  lib/librte_power/power_acpi_cpufreq.c   | 45 +++++++++++++++++++++++++--------
  lib/librte_power/power_pstate_cpufreq.c | 45 +++++++++++++++++++++++++--------
  2 files changed, 70 insertions(+), 20 deletions(-)


Looks good to me. Code looks good, and I applied it locally to have a test, and it's entering and exiting power manangemnt state fine on my systems here.
Thanks.

Acked-by: David Hunt <david.h...@intel.com>


Reply via email to