Hi Mattias, On 14 Nov 15:10, Mattias Rönnblom wrote: > Calling pstate's or acpi's rte_power_freq_up() when on the highest > non-turbo frequency results in an error, if turbo is disabled. The > error is in the form of a return code and a RTE_LOG() entry on the ERR > level. > > According to the API documentation, the frequency is scaled up > "according to the available frequencies". In case turbo is disabled, > that frequency is not available. This patch's rte_power_freq_up() > behaviour is also consistent with how rte_power_freq_max() is > implemented (i.e. the highest non-turbo frequency is set, in case > turbo is disabled). > > Fixes: 445c6528b55f ("power: common interface for guest and host") > Fixes: e6c6dc0f96c8 ("power: add p-state driver compatibility") > Cc: sta...@dpdk.org > > Signed-off-by: Mattias Rönnblom <mattias.ronnb...@ericsson.com> > ---
Thanks for your patches. I reviewed it and I'm OK with that changes. Reviewed-by: Liang Ma <liang.j...@intel.com>