2021-08-31 10:22:37

by Li Zhijian

Subject: [PATCH v3 0/2] kselftests: clean configs

0Day will check if all configs listing under selftests are able
to be enabled properly.

For the missing configs, it will report something like:
LKP WARN miss config CONFIG_SYNC= of sync/config

CC: "Rafael J. Wysocki" <[email protected]>
CC: Viresh Kumar <[email protected]>
CC: [email protected]
Reported-by: kernel test robot <[email protected]>

Li Zhijian (2):
selftests/sync: Remove the deprecated config SYNC
selftests/cpufreq: Rename DEBUG_PI_LIST to DEBUG_PLIST

tools/testing/selftests/cpufreq/config | 2 +-
tools/testing/selftests/sync/config | 1 -
2 files changed, 1 insertion(+), 2 deletions(-)

--
2.31.1

2021-08-31 10:22:37

by Li Zhijian

Subject: [PATCH v3 2/2] selftests/cpufreq: Rename DEBUG_PI_LIST to DEBUG_PLIST

DEBUG_PI_LIST was renamed to DEBUG_PLIST since
8e18faeac3 ("lib/plist: rename DEBUG_PI_LIST to DEBUG_PLIST")

- It's not reasonable to keep the deprecated configs.
- configs under kselftests are recommended by corresponding tests.
So if some configs are missing, it will impact the testing results

CC: "Rafael J. Wysocki" <[email protected]>
CC: Viresh Kumar <[email protected]>
CC: [email protected]
Signed-off-by: Li Zhijian <[email protected]>

---
v3: update commit logs # Shuah
---
tools/testing/selftests/cpufreq/config | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tools/testing/selftests/cpufreq/config b/tools/testing/selftests/cpufreq/config
index 27ff72ebd0f5..75e900793e8a 100644
--- a/tools/testing/selftests/cpufreq/config
+++ b/tools/testing/selftests/cpufreq/config
@@ -6,7 +6,7 @@ CONFIG_CPU_FREQ_GOV_ONDEMAND=y
CONFIG_CPU_FREQ_GOV_CONSERVATIVE=y
CONFIG_CPU_FREQ_GOV_SCHEDUTIL=y
CONFIG_DEBUG_RT_MUTEXES=y
-CONFIG_DEBUG_PI_LIST=y
+CONFIG_DEBUG_PLIST=y
CONFIG_DEBUG_SPINLOCK=y
CONFIG_DEBUG_MUTEXES=y
CONFIG_DEBUG_LOCK_ALLOC=y
--
2.31.1

2021-08-31 12:15:26

by Viresh Kumar

Subject: Re: [PATCH v3 2/2] selftests/cpufreq: Rename DEBUG_PI_LIST to DEBUG_PLIST

On 31-08-21, 18:23, Li Zhijian wrote:
> DEBUG_PI_LIST was renamed to DEBUG_PLIST since
> 8e18faeac3 ("lib/plist: rename DEBUG_PI_LIST to DEBUG_PLIST")
>
> - It's not reasonable to keep the deprecated configs.
> - configs under kselftests are recommended by corresponding tests.
> So if some configs are missing, it will impact the testing results
>
> CC: "Rafael J. Wysocki" <[email protected]>
> CC: Viresh Kumar <[email protected]>
> CC: [email protected]
> Signed-off-by: Li Zhijian <[email protected]>
>
> ---
> v3: update commit logs # Shuah
> ---
> tools/testing/selftests/cpufreq/config | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tools/testing/selftests/cpufreq/config b/tools/testing/selftests/cpufreq/config
> index 27ff72ebd0f5..75e900793e8a 100644
> --- a/tools/testing/selftests/cpufreq/config
> +++ b/tools/testing/selftests/cpufreq/config
> @@ -6,7 +6,7 @@ CONFIG_CPU_FREQ_GOV_ONDEMAND=y
> CONFIG_CPU_FREQ_GOV_CONSERVATIVE=y
> CONFIG_CPU_FREQ_GOV_SCHEDUTIL=y
> CONFIG_DEBUG_RT_MUTEXES=y
> -CONFIG_DEBUG_PI_LIST=y
> +CONFIG_DEBUG_PLIST=y
> CONFIG_DEBUG_SPINLOCK=y
> CONFIG_DEBUG_MUTEXES=y
> CONFIG_DEBUG_LOCK_ALLOC=y

Acked-by: Viresh Kumar <[email protected]>

--
viresh

2021-08-31 16:51:50

by Shuah Khan

Subject: Re: [PATCH v3 0/2] kselftests: clean configs

On 8/31/21 4:23 AM, Li Zhijian wrote:
> 0Day will check if all configs listing under selftests are able
> to be enabled properly.
>
> For the missing configs, it will report something like:
> LKP WARN miss config CONFIG_SYNC= of sync/config
>
> CC: "Rafael J. Wysocki" <[email protected]>
> CC: Viresh Kumar <[email protected]>
> CC: [email protected]
> Reported-by: kernel test robot <[email protected]>
>
> Li Zhijian (2):
> selftests/sync: Remove the deprecated config SYNC
> selftests/cpufreq: Rename DEBUG_PI_LIST to DEBUG_PLIST
>
> tools/testing/selftests/cpufreq/config | 2 +-
> tools/testing/selftests/sync/config | 1 -
> 2 files changed, 1 insertion(+), 2 deletions(-)
>

Thank you. These two patches will be queued up for 5.15

thanks,
-- Shuah