linux-kernel - [PATCH][GIT PULL] ktest: Check parent options for iterated tests

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [day] [month] [year] [list]

Date:	Fri, 18 Nov 2011 11:36:36 -0500
From:	Steven Rostedt <rostedt@...dmis.org>
To:	LKML <linux-kernel@...r.kernel.org>
Cc:	Linus Torvalds <torvalds@...ux-foundation.org>,
	Rabin Vincent <rabin@....in>
Subject: [PATCH][GIT PULL] ktest: Check parent options for iterated tests


Linus,

This patch includes a bug fix for ktest. And also proves that ktest has
at least 3 users! 

Please pull the latest for-linus tree, which can be found at:

  git://git.kernel.org/pub/scm/linux/kernel/git/rostedt/linux-ktest.git
for-linus

Head SHA1: f9dfb65b055c56678afa03e13e942761459ea53d


Rabin Vincent (1):
      ktest: Check parent options for iterated tests

----
 tools/testing/ktest/ktest.pl |   16 ++++++++++++++++
 1 files changed, 16 insertions(+), 0 deletions(-)
---------------------------
commit f9dfb65b055c56678afa03e13e942761459ea53d
Author: Rabin Vincent <rabin@....in>
Date:   Fri Nov 18 17:05:30 2011 +0530

    ktest: Check parent options for iterated tests
    
    Let's say we have "OUTPUT_DIR = build/${TEST_NAME}", and we're iterating
    a test.  In the second iteration of a test, the TEST_NAME of the test
    we're repeating is not used.  Instead, ${TEST_NAME} appears literally:
    
       touch /home/rabin/kernel/test/build/${TEST_NAME}/.config ... SUCCESS
    
    Fix this by making __eval_option() check the parent test options
    for a repeated test.
    
    Link: http://lkml.kernel.org/r/1321616131-21352-2-git-send-email-rabin@rab.in
    
    Signed-off-by: Rabin Vincent <rabin@....in>
    Signed-off-by: Steven Rostedt <rostedt@...dmis.org>

diff --git a/tools/testing/ktest/ktest.pl b/tools/testing/ktest/ktest.pl
index 30e2bef..8b4c253 100755
--- a/tools/testing/ktest/ktest.pl
+++ b/tools/testing/ktest/ktest.pl
@@ -747,6 +747,18 @@ sub __eval_option {
     # Add space to evaluate the character before $
     $option = " $option";
     my $retval = "";
+    my $repeated = 0;
+    my $parent = 0;
+
+    foreach my $test (keys %repeat_tests) {
+	if ($i >= $test &&
+	    $i < $test + $repeat_tests{$test}) {
+
+	    $repeated = 1;
+	    $parent = $test;
+	    last;
+	}
+    }
 
     while ($option =~ /(.*?[^\\])\$\{(.*?)\}(.*)/) {
 	my $start = $1;
@@ -760,10 +772,14 @@ sub __eval_option {
 	# otherwise see if the default OPT (without [$i]) exists.
 
 	my $o = "$var\[$i\]";
+	my $parento = "$var\[$parent\]";
 
 	if (defined($opt{$o})) {
 	    $o = $opt{$o};
 	    $retval = "$retval$o";
+	} elsif ($repeated && defined($opt{$parento})) {
+	    $o = $opt{$parento};
+	    $retval = "$retval$o";
 	} elsif (defined($opt{$var})) {
 	    $o = $opt{$var};
 	    $retval = "$retval$o";



Download attachment "signature.asc" of type "application/pgp-signature" (837 bytes)