Po-Hsu Lin

Merge ~cypressyew/+git/autotest-client-tests:ftrace-verbosity into ~canonical-kernel-team/+git/autotest-client-tests:master

  1. Git
  2. lp:~cypressyew/+git/autotest-client-tests
  3. ftrace-verbosity
  4. Merge into master
Proposed by Po-Hsu Lin
Status: Merged
Approved by: Francis Ginther
Approved revision: 15ec6966fdc39ca1dd969242fa8458eb8e16a6ff
Merged at revision: 751fe20a471cb643706dc1abcd77170ae6d5d817
Proposed branch: ~cypressyew/+git/autotest-client-tests:ftrace-verbosity
Merge into: ~canonical-kernel-team/+git/autotest-client-tests:master
Diff against target: 13 lines (+1/-1)
1 file modified
ubuntu_kselftests_ftrace/ubuntu_kselftests_ftrace.py (+1/-1)
Reviewer Review Type Date Requested Status
Francis Ginther Approve
Kevin Becker Approve
Review via email: mp+468489@code.launchpad.net

Commit message

KERNTT-1040
    Reduce the test output verbosity from -vvv to -v to reduce the size of
    the test log.

    Experiment shows on an ARM64 bare-metal node appleton-kernel with
    J-hwe-6.8 kernel, log size of these different verbosity level will be:
      * -vvv: 432M
      * -vv: 112M
      * -v: 108M

    Both -vv and -v can generate the identical error message when a test
    case fails. Therefore I think we can just use -v here.

To post a comment you must log in.
Revision history for this message
Kevin Becker (kevinbecker) wrote :

lgtm

review: Approve
Revision history for this message
Francis Ginther (fginther) wrote :

Let's do it!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/ubuntu_kselftests_ftrace/ubuntu_kselftests_ftrace.py b/ubuntu_kselftests_ftrace/ubuntu_kselftests_ftrace.py
2index 5ac87fc..fbd04a5 100644
3--- a/ubuntu_kselftests_ftrace/ubuntu_kselftests_ftrace.py
4+++ b/ubuntu_kselftests_ftrace/ubuntu_kselftests_ftrace.py
5@@ -108,7 +108,7 @@ class ubuntu_kselftests_ftrace(test.test):
6 os.chdir(dir_root)
7 # Run sub-tests with ftracetest script, convert test name back to path
8 test = sub_test.replace('--', '/')
9- cmd = './ftracetest -vvv {}'.format(test)
10+ cmd = './ftracetest -v {}'.format(test)
11 result = utils.system_output(cmd, retain_output=True)
12
13 # vi:set ts=4 sw=4 expandtab syntax=python:

Subscribers

People subscribed via source and target branches