From mboxrd@z Thu Jan 1 00:00:00 1970
From: bugzilla-daemon@freedesktop.org
Subject: [Bug 108075] [CI][BAT] igt_runner: abort on TAINT_PAGE, TAINT_DIE,
and TAINT_OOPS
Date: Fri, 16 Nov 2018 14:33:32 +0000
Message-ID:
References:
Mime-Version: 1.0
Content-Type: multipart/mixed; boundary="===============1392220133=="
Return-path:
Received: from culpepper.freedesktop.org (culpepper.freedesktop.org
[131.252.210.165])
by gabe.freedesktop.org (Postfix) with ESMTP id DB3446E6C9
for ; Fri, 16 Nov 2018 14:33:31 +0000 (UTC)
In-Reply-To:
List-Unsubscribe: ,
List-Archive:
List-Post:
List-Help:
List-Subscribe: ,
Errors-To: dri-devel-bounces@lists.freedesktop.org
Sender: "dri-devel"
To: dri-devel@lists.freedesktop.org
List-Id: dri-devel@lists.freedesktop.org
--===============1392220133==
Content-Type: multipart/alternative; boundary="15423788110.4eC28Dd.15948"
Content-Transfer-Encoding: 7bit
--15423788110.4eC28Dd.15948
Date: Fri, 16 Nov 2018 14:33:31 +0000
MIME-Version: 1.0
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
X-Bugzilla-URL: http://bugs.freedesktop.org/
Auto-Submitted: auto-generated
https://bugs.freedesktop.org/show_bug.cgi?id=3D108075
Martin Peres changed:
What |Removed |Added
----------------------------------------------------------------------------
Resolution|--- |FIXED
Status|NEW |RESOLVED
--- Comment #3 from Martin Peres ---
author Petri Latvala 2018-11-09 13:13:03 +0200
committer Petri Latvala 2018-11-15 10:35:23
+0200
commit 111593c49d812a4f4ff9ab0ef053a3ab88a6f73f (patch)
tree 7f8a34859645b3972bc587af663c7530abdec24a
parent cab148ca3ec904a94d0cd43476cf7e1f8663f906 (diff)
runner: Implement --abort-on-monitored-error
Deviating a bit from the piglit command line flag, igt_runner takes an
optional comma-separated list as an argument to
--abort-on-monitored-error for the list of conditions to abort
on. Without a list all possible conditions will be checked.
Two conditions implemented:
- "taint" checks the kernel taint level for TAINT_PAGE, TAINT_DIE and
TAINT_OOPS
- "lockdep" checks the kernel lockdep status
Checking is done after every test binary execution, and if an abort
condition is met, the reason is printed to stderr (unless log level is
quiet) and the runner doesn't execute any further tests. Aborting
between subtests (when running in --multiple-mode) is not done.
v2:
- Remember to fclose
- Taints are unsigned long (Chris)
- Use getline instead of fgets (Chris)
v3:
- Fix brainfart with lockdep
v4:
- Rebase
- Refactor the abort condition checking to pass down strings
- Present the abort result in results.json as a pseudo test result
- Unit tests for the pseudo result
v5:
- Refactors (Chris)
- Don't claim lockdep was triggered if debug_locks is not on
anymore. Just say it's not active.
- Dump lockdep_stats when aborting due to lockdep (Chris)
- Use igt@runner@aborted instead for the pseudo result (Martin)
v6:
- If aborting after a test, generate results.json. Like was already
done for aborting at startup.
- Print the test that would be executed next as well when aborting,
as requested by Tomi.
v7:
- Remove the resolved TODO item from commit message
Signed-off-by: Petri Latvala
Cc: Arkadiusz Hiler
Cc: Tomi Sarvela
Cc: Martin Peres
Cc: Daniel Vetter
Cc: Chris Wilson
Reviewed-by: Arkadiusz Hiler
--=20
You are receiving this mail because:
You are the assignee for the bug.=
--15423788110.4eC28Dd.15948
Date: Fri, 16 Nov 2018 14:33:31 +0000
MIME-Version: 1.0
Content-Type: text/html; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
X-Bugzilla-URL: http://bugs.freedesktop.org/
Auto-Submitted: auto-generated
<=
span class=3D"fn">Martin Peres
changed
bug 10807=
5
What |
Removed |
Added |
Resolution |
---
|
FIXED
|
Status |
NEW
|
RESOLVED
|
Commen=
t # 3
on bug 10807=
5
from Martin Peres
author Petri Latvala <petri.latvala@intel.com> 2018-11-09 13:13:03 +0200
committer Petri Latvala <petri.latvala@intel.com> 2018-11-15 10:35:23
+0200
commit 111593c49d812a4f4ff9ab0ef053a3ab88a6f73f (patch)
tree 7f8a34859645b3972bc587af663c7530abdec24a
parent cab148ca3ec904a94d0cd43476cf7e1f8663f906 (diff)
runner: Implement --abort-on-monitored-error
Deviating a bit from the piglit command line flag, igt_runner takes an
optional comma-separated list as an argument to
--abort-on-monitored-error for the list of conditions to abort
on. Without a list all possible conditions will be checked.
Two conditions implemented:
- "taint" checks the kernel taint level for TAINT_PAGE, TAINT_DI=
E and
TAINT_OOPS
- "lockdep" checks the kernel lockdep status
Checking is done after every test binary execution, and if an abort
condition is met, the reason is printed to stderr (unless log level is
quiet) and the runner doesn't execute any further tests. Aborting
between subtests (when running in --multiple-mode) is not done.
v2:
- Remember to fclose
- Taints are unsigned long (Chris)
- Use getline instead of fgets (Chris)
v3:
- Fix brainfart with lockdep
v4:
- Rebase
- Refactor the abort condition checking to pass down strings
- Present the abort result in results.json as a pseudo test result
- Unit tests for the pseudo result
v5:
- Refactors (Chris)
- Don't claim lockdep was triggered if debug_locks is not on
anymore. Just say it's not active.
- Dump lockdep_stats when aborting due to lockdep (Chris)
- Use igt@runner@aborted instead for the pseudo result (Martin)
v6:
- If aborting after a test, generate results.json. Like was already
done for aborting at startup.
- Print the test that would be executed next as well when aborting,
as requested by Tomi.
v7:
- Remove the resolved TODO item from commit message
Signed-off-by: Petri Latvala <petri.latvala@intel.com>
Cc: Arkadiusz Hiler <ar=
kadiusz.hiler@intel.com>
Cc: Tomi Sarvela <tomi.p=
.sarvela@intel.com>
Cc: Martin Peres <ma=
rtin.peres@linux.intel.com>
Cc: Daniel Vetter <daniel.=
vetter@ffwll.ch>
Cc: Chris Wilson <chris&=
#64;chris-wilson.co.uk>
Reviewed-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>
You are receiving this mail because:
- You are the assignee for the bug.
=
--15423788110.4eC28Dd.15948--
--===============1392220133==
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: base64
Content-Disposition: inline
X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KZHJpLWRldmVs
IG1haWxpbmcgbGlzdApkcmktZGV2ZWxAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHBzOi8vbGlz
dHMuZnJlZWRlc2t0b3Aub3JnL21haWxtYW4vbGlzdGluZm8vZHJpLWRldmVsCg==
--===============1392220133==--