> -----Original Message----- > From: David Hunt <david.h...@intel.com> > Sent: Wednesday, July 14, 2021 5:15 PM > To: Richael Zhuang <richael.zhu...@arm.com>; dev@dpdk.org > Cc: zhiminx.hu...@intel.com; sta...@dpdk.org > Subject: Re: [PATCH v1 1/1] test/power: check cpuinfo cur freq before scaling > cur freq > > > On 14/7/2021 9:44 AM, Richael Zhuang wrote: > > For acpi_cpufreq and cppc_cpufreq, both cpuinfo_cur_freq and > > scaling_cur_freq exist. For pstate, only scaling_cur_freq exists. > > And value in scaling_cur_freq and cpuinfo_cur_freq may not be the > > same. For acpi_cpufreq and cppc_cpufreq, we should check > > cpuinfo_cur_freq. So here checking cpuinfo_cur_freq before > > scaling_cur_freq to make sure it works for all cpufreq drivers. > > > > Fixes: ff6dfb8e492f ("test/power: fix CPU frequency check") > > Cc: david.h...@intel.com > > Cc: sta...@dpdk.org > > > > Signed-off-by: Richael Zhuang <richael.zhu...@arm.com> > > --- > > app/test/test_power_cpufreq.c | 4 ++-- > > 1 file changed, 2 insertions(+), 2 deletions(-) > > > > diff --git a/app/test/test_power_cpufreq.c > > b/app/test/test_power_cpufreq.c index b8fc53925c..f56abb6f86 100644 > > --- a/app/test/test_power_cpufreq.c > > +++ b/app/test/test_power_cpufreq.c > > @@ -62,13 +62,13 @@ check_cur_freq(unsigned int lcore_id, uint32_t idx, > bool turbo) > > int i; > > > > if (snprintf(fullpath, sizeof(fullpath), > > - TEST_POWER_SYSFILE_SCALING_FREQ, lcore_id) < 0) { > > + TEST_POWER_SYSFILE_CPUINFO_FREQ, lcore_id) < 0) { > > return 0; > > } > > f = fopen(fullpath, "r"); > > if (f == NULL) { > > if (snprintf(fullpath, sizeof(fullpath), > > - TEST_POWER_SYSFILE_CPUINFO_FREQ, lcore_id) < 0) > { > > + TEST_POWER_SYSFILE_SCALING_FREQ, lcore_id) < 0) > { > > return 0; > > } > > f = fopen(fullpath, "r"); > > > Hi Richael, I don't think this patch fixes anything. If the scaling file is > not > available, it will then attempt to open the cpuinfo file. > Changing the order does not address the underlying issue. > > It looks like the test is failing in check_cur_req, which is only rounding for > cppc driver. I think it also needs to round for the other drivers. I've just > checked intel_pstate driver now, and it needs the rounding. I would think > that acpi driver also needs it. I'll do a bit more investigation and see if I > canĀ change to acpi and attempt to confirm that all drivers need the rounding. > > Rgds, > Dave. > > Hi David, For acpi_cpufreq and cppc_cpufreq, both two files exist. So with the current code, it will check the scaling_cur_freq. But I think for this two drivers, it should check cpuinfo_cur_freq but not scaling_cur_freq. From my system, for acpi cpufreq, the value in cpuinfo_cur_freq and scaling_cur_freq are not the same. So if not changing the check sequence, the result is: ########## RTE>>power_cpufreq_autotest POWER: Env isn't set yet! POWER: Attempting to initialise ACPI cpufreq power management... POWER: Initialized successfully for lcore 2 power management POWER: Power management of lcore 2 has exited from 'userspace' mode and been set back to the original POWER: Lcore id 128 can not exceeds 127 POWER: Initialized successfully for lcore 2 power management POWER: Power management of lcore 2 is in use POWER: Invalid lcore ID POWER: NULL buffer supplied POWER: Buffer size is not enough POWER: Invalid lcore ID POWER: Power management of lcore 2 has exited from 'userspace' mode and been set back to the original Test Failed RTE>>quit #############
But after changing the check sequence, the result is OK from my test: ###### RTE>>power_cpufreq_autotest POWER: Env isn't set yet! POWER: Attempting to initialise ACPI cpufreq power management... POWER: Initialized successfully for lcore 2 power management POWER: Power management of lcore 2 has exited from 'userspace' mode and been set back to the original POWER: Lcore id 128 can not exceeds 127 POWER: Initialized successfully for lcore 2 power management POWER: Power management of lcore 2 is in use POWER: Invalid lcore ID POWER: NULL buffer supplied POWER: Buffer size is not enough POWER: Invalid lcore ID POWER: Invalid lcore ID POWER: Invalid frequency index 64, which should be less than 3 POWER: Invalid frequency index 3, which should be less than 3 POWER: Failed to enable turbo on lcore 2 POWER: Invalid lcore ID POWER: Invalid lcore ID POWER: Invalid lcore ID POWER: Invalid lcore ID Turbo not available on lcore 2, skipping test POWER: Power management of lcore 2 has exited from 'userspace' mode and been set back to the original POWER: Power management of lcore 2 is not used POWER: Lcore id 128 can not exceeds 127 Test OK ############