Commit 19384452 authored by Zhang Xiaoxu's avatar Zhang Xiaoxu Committed by Chris Wilson

drm/i915/selftests: Fix wrong return value of perf_request_latency()

If intel context create failed, the perf_request_latency() will return 0
rather than error, because we doesn't initialize the return value.

Fixes: 25c26f18 ("drm/i915/selftests: Measure dispatch latency")
Reported-by: default avatarHulk Robot <hulkci@huawei.com>
Signed-off-by: default avatarZhang Xiaoxu <zhangxiaoxu5@huawei.com>
Reviewed-by: default avatarMika Kuoppala <mika.kuoppala@linux.intel.com>
Signed-off-by: default avatarChris Wilson <chris@chris-wilson.co.uk>
Link: https://patchwork.freedesktop.org/patch/msgid/20201116143540.3648870-1-zhangxiaoxu5@huawei.com
parent 01d70884
...@@ -2293,8 +2293,10 @@ static int perf_request_latency(void *arg) ...@@ -2293,8 +2293,10 @@ static int perf_request_latency(void *arg)
struct intel_context *ce; struct intel_context *ce;
ce = intel_context_create(engine); ce = intel_context_create(engine);
if (IS_ERR(ce)) if (IS_ERR(ce)) {
err = PTR_ERR(ce);
goto out; goto out;
}
err = intel_context_pin(ce); err = intel_context_pin(ce);
if (err) { if (err) {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment