Commit 250bae8b authored by Steven Rostedt's avatar Steven Rostedt Committed by Steven Rostedt

ktest: Fix bug when ADD_CONFIG is set but MIN_CONFIG is not

The MIN_CONFIG is a single config that is considered to have all the
configs that are required to boot the box.

ADD_CONFIG is a list of configs that we add that may contain configs
known to be broken (set off) or just configs that we want every box to
have and this can include shared configs.

If a config has no MIN_CONFIG defined, but has multiple files defined
for the ADD_CONFIG, the test will die, because the MIN_CONFIG will
default to ADD_CONFIG. The problem is the code to open MIN_CONFIG
expects a string of one file, not multiple, and the open will fail.

Since the real minconfig that is used is a concatination of MIN_CONFIG
and ADD_CONFIG files, we change the code to open that instead of
whatever MIN_CONFIG defaults to.
Signed-off-by: default avatarSteven Rostedt <rostedt@goodmis.org>
parent 43d1b651
...@@ -1970,12 +1970,6 @@ sub config_bisect { ...@@ -1970,12 +1970,6 @@ sub config_bisect {
unlink $tmpconfig; unlink $tmpconfig;
} }
# Add other configs
if (defined($addconfig)) {
run_command "cat $addconfig >> $tmpconfig" or
dodie "failed to append $addconfig";
}
if (-f $tmpconfig) { if (-f $tmpconfig) {
load_force_config($tmpconfig); load_force_config($tmpconfig);
process_config_ignore $tmpconfig; process_config_ignore $tmpconfig;
...@@ -1997,7 +1991,7 @@ sub config_bisect { ...@@ -1997,7 +1991,7 @@ sub config_bisect {
} }
close(IN); close(IN);
# Now run oldconfig with the minconfig (and addconfigs) # Now run oldconfig with the minconfig
make_oldconfig; make_oldconfig;
# check to see what we lost (or gained) # check to see what we lost (or gained)
...@@ -2901,11 +2895,12 @@ for (my $i = 1; $i <= $opt{"NUM_TESTS"}; $i++) { ...@@ -2901,11 +2895,12 @@ for (my $i = 1; $i <= $opt{"NUM_TESTS"}; $i++) {
unlink $dmesg; unlink $dmesg;
unlink $buildlog; unlink $buildlog;
if (!defined($minconfig)) { if (defined($addconfig)) {
$minconfig = $addconfig; my $min = $minconfig;
if (!defined($minconfig)) {
} elsif (defined($addconfig)) { $min = "";
run_command "cat $addconfig $minconfig > $tmpdir/add_config" or }
run_command "cat $addconfig $min > $tmpdir/add_config" or
dodie "Failed to create temp config"; dodie "Failed to create temp config";
$minconfig = "$tmpdir/add_config"; $minconfig = "$tmpdir/add_config";
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment