Re: [PATCH v5 1/2] power: introduce PM QoS API on CPU wide

["lihuisong (C)" <lihuisong@huawei.com>]

Hi

在 2024/7/3 9:32, zhoumin 写道:
> Hi Huisong,
>
> I knew that this patchset was based on the *dpdk-next-net* repository 
> in our previous communication. However, maybe it's better to rebase 
> this pactchset on the *dpdk* repository. Because the CI system is not 
> smart enough to recognize the patchset as a change for the 
> *dpdk-next-net* repository. I personally feel this patchset is more 
> likely a change for the *dpdk* repository because it modified the 
> `lib` directory which is the infrastructure of DPDK instead of a 
> feature for *dpdk-next-net*.
>
Generally, the 'dpdk-next-net' repo is ahead of 'dpdk' repo. And we make 
some development based on 'dpdk-next-net' repo.

I found that there are some patches also happen to the same issue that 
CI applt patch failed.
So I think the reason why this series trigger warning should be analyzed 
and resolved.
>
> On Tue, July 2, 2024 at 11:50AM, Huisong Li wrote:
>> The deeper the idle state, the lower the power consumption, but the 
>> longer
>> the resume time. Some service are delay sensitive and very except the 
>> low
>> resume time, like interrupt packet receiving mode.
>>
>> And the "/sys/devices/system/cpu/cpuX/power/pm_qos_resume_latency_us" 
>> sysfs
>> interface is used to set and get the resume latency limit on the cpuX 
>> for
>> userspace. Each cpuidle governor in Linux select which idle state to 
>> enter
>> based on this CPU resume latency in their idle task.
>>
>> The per-CPU PM QoS API can be used to control this CPU's idle state
>> selection and limit just enter the shallowest idle state to low the 
>> delay
>> after sleep by setting strict resume latency (zero value).
>>
>> Signed-off-by: Huisong Li <lihuisong@huawei.com>
>> Acked-by: Morten Brørup <mb@smartsharesystems.com>
>> ---
>>   doc/guides/prog_guide/power_man.rst    |  24 ++++++
>>   doc/guides/rel_notes/release_24_07.rst |   4 +
>>   lib/power/meson.build                  |   2 +
>>   lib/power/rte_power_qos.c              | 114 +++++++++++++++++++++++++
>>   lib/power/rte_power_qos.h              |  73 ++++++++++++++++
>>   lib/power/version.map                  |   2 +
>>   6 files changed, 219 insertions(+)
>>   create mode 100644 lib/power/rte_power_qos.c
>>   create mode 100644 lib/power/rte_power_qos.h
> <snip>
>
> .