ktest: Fix result of rebooting the kernel
authorSteven Rostedt <srostedt@redhat.com>
Thu, 2 Jun 2011 03:25:13 +0000 (23:25 -0400)
committerSteven Rostedt <rostedt@goodmis.org>
Thu, 2 Jun 2011 15:57:56 +0000 (11:57 -0400)
The command that is called that reboots the kernel may fail
but the return code is not passed back to the ktest.pl script.
This is because a ';' is used between the two commands and
if the second command fails, only the first command's return
code is returned. Using a '&&' between the two commands fixes
this.

Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
tools/testing/ktest/ktest.pl

index 8dc8c3cf3acf700a2bd661feba3c4f3373942f2d..6c68259e730d461feb794ed8031bc58f79227ec9 100755 (executable)
@@ -788,7 +788,7 @@ sub wait_for_input
 
 sub reboot_to {
     if ($reboot_type eq "grub") {
-       run_ssh "'(echo \"savedefault --default=$grub_number --once\" | grub --batch; reboot)'";
+       run_ssh "'(echo \"savedefault --default=$grub_number --once\" | grub --batch && reboot)'";
        return;
     }