Skip to content

Commit

Permalink
ktest: Rename start_monitor_and_boot to start_monitor_and_install
Browse files Browse the repository at this point in the history
The function start_monitor_and_boot is a misnomer. It use to, but
now it starts the monitor and installs. It does not boot. Rename it
before I get confused by it again.

Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
  • Loading branch information
Steven Rostedt (Red Hat) committed Jan 29, 2015
1 parent 38fa3dc commit 64d9828
Showing 1 changed file with 5 additions and 5 deletions.
10 changes: 5 additions & 5 deletions tools/testing/ktest/ktest.pl
Original file line number Diff line number Diff line change
Expand Up @@ -2105,7 +2105,7 @@ sub get_version {
$have_version = 1;
}

sub start_monitor_and_boot {
sub start_monitor_and_install {
# Make sure the stable kernel has finished booting

# Install bisects, don't need console
Expand Down Expand Up @@ -2656,7 +2656,7 @@ sub run_bisect_test {
dodie "Failed on build" if $failed;

# Now boot the box
start_monitor_and_boot or $failed = 1;
start_monitor_and_install or $failed = 1;

if ($type ne "boot") {
if ($failed && $bisect_skip) {
Expand Down Expand Up @@ -3400,7 +3400,7 @@ sub patchcheck {

my $failed = 0;

start_monitor_and_boot or $failed = 1;
start_monitor_and_install or $failed = 1;

if (!$failed && $type ne "boot"){
do_run_test or $failed = 1;
Expand Down Expand Up @@ -3864,7 +3864,7 @@ sub make_min_config {
my $failed = 0;
build "oldconfig" or $failed = 1;
if (!$failed) {
start_monitor_and_boot or $failed = 1;
start_monitor_and_install or $failed = 1;

if ($type eq "test" && !$failed) {
do_run_test or $failed = 1;
Expand Down Expand Up @@ -4273,7 +4273,7 @@ sub set_test_option {

if ($test_type ne "build") {
my $failed = 0;
start_monitor_and_boot or $failed = 1;
start_monitor_and_install or $failed = 1;

if (!$failed && $test_type ne "boot" && defined($run_test)) {
do_run_test or $failed = 1;
Expand Down

0 comments on commit 64d9828

Please sign in to comment.