diff options
Diffstat (limited to 'drivers/gpu/drm/i915/i915_request.c')
-rw-r--r-- | drivers/gpu/drm/i915/i915_request.c | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/drivers/gpu/drm/i915/i915_request.c b/drivers/gpu/drm/i915/i915_request.c index 5c2c93cbab12..09ed48833b54 100644 --- a/drivers/gpu/drm/i915/i915_request.c +++ b/drivers/gpu/drm/i915/i915_request.c | |||
@@ -527,7 +527,7 @@ void __i915_request_submit(struct i915_request *request) | |||
527 | 527 | ||
528 | seqno = timeline_get_seqno(&engine->timeline); | 528 | seqno = timeline_get_seqno(&engine->timeline); |
529 | GEM_BUG_ON(!seqno); | 529 | GEM_BUG_ON(!seqno); |
530 | GEM_BUG_ON(i915_seqno_passed(intel_engine_get_seqno(engine), seqno)); | 530 | GEM_BUG_ON(intel_engine_signaled(engine, seqno)); |
531 | 531 | ||
532 | /* We may be recursing from the signal callback of another i915 fence */ | 532 | /* We may be recursing from the signal callback of another i915 fence */ |
533 | spin_lock_nested(&request->lock, SINGLE_DEPTH_NESTING); | 533 | spin_lock_nested(&request->lock, SINGLE_DEPTH_NESTING); |
@@ -579,8 +579,7 @@ void __i915_request_unsubmit(struct i915_request *request) | |||
579 | */ | 579 | */ |
580 | GEM_BUG_ON(!request->global_seqno); | 580 | GEM_BUG_ON(!request->global_seqno); |
581 | GEM_BUG_ON(request->global_seqno != engine->timeline.seqno); | 581 | GEM_BUG_ON(request->global_seqno != engine->timeline.seqno); |
582 | GEM_BUG_ON(i915_seqno_passed(intel_engine_get_seqno(engine), | 582 | GEM_BUG_ON(intel_engine_has_completed(engine, request->global_seqno)); |
583 | request->global_seqno)); | ||
584 | engine->timeline.seqno--; | 583 | engine->timeline.seqno--; |
585 | 584 | ||
586 | /* We may be recursing from the signal callback of another i915 fence */ | 585 | /* We may be recursing from the signal callback of another i915 fence */ |
@@ -1205,7 +1204,7 @@ static bool __i915_spin_request(const struct i915_request *rq, | |||
1205 | * it is a fair assumption that it will not complete within our | 1204 | * it is a fair assumption that it will not complete within our |
1206 | * relatively short timeout. | 1205 | * relatively short timeout. |
1207 | */ | 1206 | */ |
1208 | if (!i915_seqno_passed(intel_engine_get_seqno(engine), seqno - 1)) | 1207 | if (!intel_engine_has_started(engine, seqno)) |
1209 | return false; | 1208 | return false; |
1210 | 1209 | ||
1211 | /* | 1210 | /* |
@@ -1222,7 +1221,7 @@ static bool __i915_spin_request(const struct i915_request *rq, | |||
1222 | irq = READ_ONCE(engine->breadcrumbs.irq_count); | 1221 | irq = READ_ONCE(engine->breadcrumbs.irq_count); |
1223 | timeout_us += local_clock_us(&cpu); | 1222 | timeout_us += local_clock_us(&cpu); |
1224 | do { | 1223 | do { |
1225 | if (i915_seqno_passed(intel_engine_get_seqno(engine), seqno)) | 1224 | if (intel_engine_has_completed(engine, seqno)) |
1226 | return seqno == i915_request_global_seqno(rq); | 1225 | return seqno == i915_request_global_seqno(rq); |
1227 | 1226 | ||
1228 | /* | 1227 | /* |