lkml.org 
[lkml]   [2022]   [Dec]   [2]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: [PATCH] ktest.pl minconfig: Unset configs instead of just removing them
From
On 12/2/22 08:59, Steven Rostedt wrote:
> From: Steven Rostedt <rostedt@goodmis.org>
>
> After a full run of a make_min_config test, I noticed there were a lot of
> CONFIGs still enabled that really should not be. Looking at them, I
> noticed they were all defined as "default y". The issue is that the test
> simple removes the config and re-runs make oldconfig, which enables it
> again because it is set to default 'y'. Instead, explicitly disable the
> config with writing "# CONFIG_FOO is not set" to the file to keep it from
> being set again.
>
> With this change, one of my box's minconfigs went from 768 configs set,
> down to 521 configs set.
>
> Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
> ---
> tools/testing/ktest/ktest.pl | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/tools/testing/ktest/ktest.pl b/tools/testing/ktest/ktest.pl
> index 590518144167..56135e7d2295 100755
> --- a/tools/testing/ktest/ktest.pl
> +++ b/tools/testing/ktest/ktest.pl
> @@ -3775,9 +3775,10 @@ sub test_this_config {
> # .config to make sure it is missing the config that
> # we had before
> my %configs = %min_configs;
> - delete $configs{$config};
> + $configs{$config} = "# $config is not set";
> make_new_config ((values %configs), (values %keep_configs));
> make_oldconfig;
> + delete $configs{$config};
> undef %configs;
> assign_configs \%configs, $output_config;
>

Reviewed-by: John 'Warthog9' Hawley (VMware) <warthog9@eaglescrag.net>

\
 
 \ /
  Last update: 2022-12-02 22:41    [W:0.087 / U:0.040 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site