diff options
author | Dave Airlie <airlied@redhat.com> | 2018-11-28 19:34:03 -0500 |
---|---|---|
committer | Dave Airlie <airlied@redhat.com> | 2018-11-28 19:34:03 -0500 |
commit | 1ec28f8b8ada4e4f77d1af006a3a474f4f83b8e3 (patch) | |
tree | 2e810e02a66cdec0bc82a8555796b7083ad03416 /tools/testing | |
parent | 61647c77cb15354a329cbb36fe7a2253b36b51b1 (diff) | |
parent | 2e6e902d185027f8e3cb8b7305238f7e35d6a436 (diff) |
Merge v4.20-rc4 into drm-next
Requested by Boris Brezillon for some vc4 fixes that are needed for future vc4 work.
Signed-off-by: Dave Airlie <airlied@redhat.com>
Diffstat (limited to 'tools/testing')
-rwxr-xr-x | tools/testing/selftests/tc-testing/tdc.py | 18 |
1 files changed, 13 insertions, 5 deletions
diff --git a/tools/testing/selftests/tc-testing/tdc.py b/tools/testing/selftests/tc-testing/tdc.py index 87a04a8a5945..7607ba3e3cbe 100755 --- a/tools/testing/selftests/tc-testing/tdc.py +++ b/tools/testing/selftests/tc-testing/tdc.py | |||
@@ -134,9 +134,9 @@ def exec_cmd(args, pm, stage, command): | |||
134 | (rawout, serr) = proc.communicate() | 134 | (rawout, serr) = proc.communicate() |
135 | 135 | ||
136 | if proc.returncode != 0 and len(serr) > 0: | 136 | if proc.returncode != 0 and len(serr) > 0: |
137 | foutput = serr.decode("utf-8") | 137 | foutput = serr.decode("utf-8", errors="ignore") |
138 | else: | 138 | else: |
139 | foutput = rawout.decode("utf-8") | 139 | foutput = rawout.decode("utf-8", errors="ignore") |
140 | 140 | ||
141 | proc.stdout.close() | 141 | proc.stdout.close() |
142 | proc.stderr.close() | 142 | proc.stderr.close() |
@@ -169,6 +169,8 @@ def prepare_env(args, pm, stage, prefix, cmdlist, output = None): | |||
169 | file=sys.stderr) | 169 | file=sys.stderr) |
170 | print("\n{} *** Error message: \"{}\"".format(prefix, foutput), | 170 | print("\n{} *** Error message: \"{}\"".format(prefix, foutput), |
171 | file=sys.stderr) | 171 | file=sys.stderr) |
172 | print("returncode {}; expected {}".format(proc.returncode, | ||
173 | exit_codes)) | ||
172 | print("\n{} *** Aborting test run.".format(prefix), file=sys.stderr) | 174 | print("\n{} *** Aborting test run.".format(prefix), file=sys.stderr) |
173 | print("\n\n{} *** stdout ***".format(proc.stdout), file=sys.stderr) | 175 | print("\n\n{} *** stdout ***".format(proc.stdout), file=sys.stderr) |
174 | print("\n\n{} *** stderr ***".format(proc.stderr), file=sys.stderr) | 176 | print("\n\n{} *** stderr ***".format(proc.stderr), file=sys.stderr) |
@@ -195,12 +197,18 @@ def run_one_test(pm, args, index, tidx): | |||
195 | print('-----> execute stage') | 197 | print('-----> execute stage') |
196 | pm.call_pre_execute() | 198 | pm.call_pre_execute() |
197 | (p, procout) = exec_cmd(args, pm, 'execute', tidx["cmdUnderTest"]) | 199 | (p, procout) = exec_cmd(args, pm, 'execute', tidx["cmdUnderTest"]) |
198 | exit_code = p.returncode | 200 | if p: |
201 | exit_code = p.returncode | ||
202 | else: | ||
203 | exit_code = None | ||
204 | |||
199 | pm.call_post_execute() | 205 | pm.call_post_execute() |
200 | 206 | ||
201 | if (exit_code != int(tidx["expExitCode"])): | 207 | if (exit_code is None or exit_code != int(tidx["expExitCode"])): |
202 | result = False | 208 | result = False |
203 | print("exit:", exit_code, int(tidx["expExitCode"])) | 209 | print("exit: {!r}".format(exit_code)) |
210 | print("exit: {}".format(int(tidx["expExitCode"]))) | ||
211 | #print("exit: {!r} {}".format(exit_code, int(tidx["expExitCode"]))) | ||
204 | print(procout) | 212 | print(procout) |
205 | else: | 213 | else: |
206 | if args.verbose > 0: | 214 | if args.verbose > 0: |