Merge lp:~mwasilew/lava-test/lava-test-perf into lp:lava-test/0.0

[Milosz Wasilewski]

Fixed regular expression to match perf test output starting with two digit number.

[Tyler Baker (tyler-baker)]

Approved.

review: Approve

[Antonio Terceiro (terceiro)]

Hi Milosz,

I'm sorry, but I can't understand this merge proposal.

Compara lava-test (the branch you are proposing to merge to):
http://bazaar.launchpad.net/~linaro-validation/lava-test/trunk/files

To the branch you are proposing to merge from:
http://bazaar.launchpad.net/~mwasilew/lava-test/lava-test-perf/files

lp:lava-test does not look like the right target for this, maybe you
were looking for some other repository?

[Tyler Baker (tyler-baker)]

I did not realize this was the lava-test repository. It should be the
linaro-maintainers repo, good catch Antonio.

lp:~linaro-maintainers/lava-test/lava-test-perf

On 10 June 2013 12:19, Antonio Terceiro <email address hidden> wrote:

> Hi Milosz,
>
> I'm sorry, but I can't understand this merge proposal.
>
> Compara lava-test (the branch you are proposing to merge to):
> http://bazaar.launchpad.net/~linaro-validation/lava-test/trunk/files
>
> To the branch you are proposing to merge from:
> http://bazaar.launchpad.net/~mwasilew/lava-test/lava-test-perf/files
>
> lp:lava-test does not look like the right target for this, maybe you
> were looking for some other repository?
>
> --
> https://code.launchpad.net/~mwasilew/lava-test/lava-test-perf/+merge/168132
> You are reviewing the proposed merge of
> lp:~mwasilew/lava-test/lava-test-perf into lp:lava-test.
>

--
Tyler Baker
Technical Architect, LAVA
Linaro.org | Open source software for ARM SoCs
Follow Linaro: http://www.facebook.com/pages/Linaro
http://twitter.com/#!/linaroorg - http://www.linaro.org/linaro-blog

[Milosz Wasilewski (mwasilew)]

You're absolutely right. This should go to the branch Tyler pointed.
Now I posted it for the review in the right place. Sorry for the
hassle.

milosz

On 10 June 2013 20:30, Tyler Baker <email address hidden> wrote:
> I did not realize this was the lava-test repository. It should be the
> linaro-maintainers repo, good catch Antonio.
>
> lp:~linaro-maintainers/lava-test/lava-test-perf
>
>
>
> On 10 June 2013 12:19, Antonio Terceiro <email address hidden> wrote:
>>
>> Hi Milosz,
>>
>> I'm sorry, but I can't understand this merge proposal.
>>
>> Compara lava-test (the branch you are proposing to merge to):
>> http://bazaar.launchpad.net/~linaro-validation/lava-test/trunk/files
>>
>> To the branch you are proposing to merge from:
>> http://bazaar.launchpad.net/~mwasilew/lava-test/lava-test-perf/files
>>
>> lp:lava-test does not look like the right target for this, maybe you
>> were looking for some other repository?
>>
>> --
>>
>> https://code.launchpad.net/~mwasilew/lava-test/lava-test-perf/+merge/168132
>> You are reviewing the proposed merge of
>> lp:~mwasilew/lava-test/lava-test-perf into lp:lava-test.
>
>
>
>
> --
> Tyler Baker
> Technical Architect, LAVA
> Linaro.org | Open source software for ARM SoCs
> Follow Linaro: http://www.facebook.com/pages/Linaro
> http://twitter.com/#!/linaroorg - http://www.linaro.org/linaro-blog

Unmerged revisions

9. By Milosz Wasilewski on 2013-06-07

fixed regular expression

8. By Marcin Juszkiewicz on 2013-03-25

let's assume that if /usr/bin/apt-get exists then we have Ubuntu

7. By Fathi Boudra on 2012-06-26

Test that the script is run as root and drop sudo calls.
Use gt operand for comparison (fixes incorrect test results).
Fix bashisms.

6. By Avik Sil on 2012-04-12

Enhance test output log

5. By Avik Sil on 2012-04-11

Non-interactive install of linux-linaro-tools-* package

4. By Avik Sil on 2012-04-11

Install proper linux-linaro-tools-* package

3. By Avik Sil on 2012-04-10

Add perf record data file

2. By Avik Sil on 2012-04-10

Fix grep option

1. By Avik Sil on 2012-04-10

perf test definition for lava-test

=== added file 'run-perf-test.sh'
--- run-perf-test.sh	1970-01-01 00:00:00 +0000
+++ run-perf-test.sh	2013-06-07 17:22:25 +0000
@@ -0,0 +1,84 @@
+#!/bin/sh
+# Copyright (C) 2012, Linaro Limited.
+#
+# This program is free software; you can redistribute it and/or
+# modify it under the terms of the GNU General Public License
+# as published by the Free Software Foundation; either version 2
+# of the License, or (at your option) any later version.
+#
+# This program is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+# GNU General Public License for more details.
+#
+# You should have received a copy of the GNU General Public License
+# along with this program; if not, write to the Free Software
+# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301, USA.
+#
+# Author: Avik Sil <[e-mail]>
+#
+
+set -x
+
+# Test user id
+if [ `whoami` != 'root' ] ; then
+    echo "You must be the superuser to run this script" >&2
+    exit 1
+fi
+
+# Let's assume that only Ubuntu has apt-get...
+if [ -e /usr/bin/apt-get ]; then
+	# Install most appropriate linux-linaro-tools-* package
+	KERNELVER=`uname -r | cut -f 1 -d'-'`
+	PKGNAME=`apt-cache search "linux-linaro-tools-$KERNELVER" | head -1 | cut -f 1 -d' '`
+	PERFBIN_PREFIX="/usr/bin/perf_"
+	PERFBIN_VER=`uname -r | awk -F '-' '{print $1"-"$2}'`
+	if [ ! -e $PERFBIN_PREFIX$PERFBIN_VER ]; then
+		apt-get install --yes $PKGNAME
+		PERFBIN=`dpkg -L $PKGNAME | grep perf_`
+		if [ "$PERFBIN" != "$PERFBIN_PREFIX$PERFBIN_VER" ]; then
+			ln -s $PERFBIN $PERFBIN_PREFIX$PERFBIN_VER
+		fi
+	fi
+fi
+
+# Test 'perf record'
+echo "Performing perf record test..."
+TCID="perf record test"
+perf record -e cycles -o perf-lava-test.data stress -c 4 -t 10  2>&1 | tee perf-record.log
+samples=`grep -ao "[0-9]\+[ ]\+samples" perf-record.log| cut -f 1 -d' '`
+if [ $samples -gt 1 ]; then
+    echo "$TCID : PASS"
+else
+    echo "$TCID : FAIL"
+fi
+rm perf-record.log
+
+# Test 'perf report'
+echo "Performing perf report test..."
+TCID="perf report test"
+perf report -i perf-lava-test.data 2>&1 | tee perf-report.log
+pcnt_samples=`grep -c -e "^[ ]\+[0-9]\+.[0-9]\+%" perf-report.log`
+if [ $pcnt_samples -gt 1 ]; then
+    echo "$TCID : PASS"
+else 
+    echo "$TCID : FAIL"
+fi
+rm perf-report.log perf-lava-test.data
+
+# Test 'perf stat'
+echo "Performing perf stat test..."
+TCID="perf stat test"
+perf stat -e cycles stress -c 4 -t 10 2>&1 | tee perf-stat.log
+cycles=`grep -o "[0-9,]\+[ ]\+cycles" perf-stat.log | sed 's/,//g' | cut -f 1 -d' '`
+if [ $cycles -gt 1 ]; then
+    echo "$TCID : PASS"
+else
+    echo "$TCID : FAIL"
+fi
+rm perf-stat.log
+
+# Test 'perf test'
+echo "Performing 'perf test'..."
+TCID="perf test"
+perf test 2>&1 | sed -e 's/FAILED!/FAIL/g' -e 's/Ok/PASS/g' -e "s/[ ]\?[0-9]\+:/$TCID -/g" -e 's/:/ :/g'