| 123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373374375376377378379380381382383384385386387388389390391392393394395396397398399400401402403404405406407408409410411412413414415416417418419420421422423424425426427428429 |
- // SPDX-License-Identifier: MIT
- /*
- * Copyright © 2019 Intel Corporation
- */
- #include <drm/drm_print.h>
- #include "i915_drv.h"
- #include "i915_jiffies.h"
- #include "i915_request.h"
- #include "intel_context.h"
- #include "intel_engine_heartbeat.h"
- #include "intel_engine_pm.h"
- #include "intel_engine.h"
- #include "intel_gt.h"
- #include "intel_reset.h"
- /*
- * While the engine is active, we send a periodic pulse along the engine
- * to check on its health and to flush any idle-barriers. If that request
- * is stuck, and we fail to preempt it, we declare the engine hung and
- * issue a reset -- in the hope that restores progress.
- */
- static bool next_heartbeat(struct intel_engine_cs *engine)
- {
- struct i915_request *rq;
- long delay;
- delay = READ_ONCE(engine->props.heartbeat_interval_ms);
- rq = engine->heartbeat.systole;
- /*
- * FIXME: The final period extension is disabled if the period has been
- * modified from the default. This is to prevent issues with certain
- * selftests which override the value and expect specific behaviour.
- * Once the selftests have been updated to either cope with variable
- * heartbeat periods (or to override the pre-emption timeout as well,
- * or just to add a selftest specific override of the extension), the
- * generic override can be removed.
- */
- if (rq && rq->sched.attr.priority >= I915_PRIORITY_BARRIER &&
- delay == engine->defaults.heartbeat_interval_ms) {
- long longer;
- /*
- * The final try is at the highest priority possible. Up until now
- * a pre-emption might not even have been attempted. So make sure
- * this last attempt allows enough time for a pre-emption to occur.
- */
- longer = READ_ONCE(engine->props.preempt_timeout_ms) * 2;
- longer = intel_clamp_heartbeat_interval_ms(engine, longer);
- if (longer > delay)
- delay = longer;
- }
- if (!delay)
- return false;
- delay = msecs_to_jiffies_timeout(delay);
- if (delay >= HZ)
- delay = round_jiffies_up_relative(delay);
- mod_delayed_work(system_highpri_wq, &engine->heartbeat.work, delay + 1);
- return true;
- }
- static struct i915_request *
- heartbeat_create(struct intel_context *ce, gfp_t gfp)
- {
- struct i915_request *rq;
- intel_context_enter(ce);
- rq = __i915_request_create(ce, gfp);
- intel_context_exit(ce);
- return rq;
- }
- static void idle_pulse(struct intel_engine_cs *engine, struct i915_request *rq)
- {
- engine->wakeref_serial = READ_ONCE(engine->serial) + 1;
- i915_request_add_active_barriers(rq);
- if (!engine->heartbeat.systole && intel_engine_has_heartbeat(engine))
- engine->heartbeat.systole = i915_request_get(rq);
- }
- static void heartbeat_commit(struct i915_request *rq,
- const struct i915_sched_attr *attr)
- {
- idle_pulse(rq->engine, rq);
- __i915_request_commit(rq);
- __i915_request_queue(rq, attr);
- }
- static void show_heartbeat(const struct i915_request *rq,
- struct intel_engine_cs *engine)
- {
- struct drm_printer p =
- drm_dbg_printer(&engine->i915->drm, DRM_UT_DRIVER, "heartbeat");
- if (!rq) {
- intel_engine_dump(engine, &p,
- "%s heartbeat not ticking\n",
- engine->name);
- } else {
- intel_engine_dump(engine, &p,
- "%s heartbeat {seqno:%llx:%lld, prio:%d} not ticking\n",
- engine->name,
- rq->fence.context,
- rq->fence.seqno,
- rq->sched.attr.priority);
- }
- }
- static void
- reset_engine(struct intel_engine_cs *engine, struct i915_request *rq)
- {
- if (IS_ENABLED(CONFIG_DRM_I915_DEBUG_GEM))
- show_heartbeat(rq, engine);
- if (intel_engine_uses_guc(engine))
- /*
- * GuC itself is toast or GuC's hang detection
- * is disabled. Either way, need to find the
- * hang culprit manually.
- */
- intel_guc_find_hung_context(engine);
- intel_gt_handle_error(engine->gt, engine->mask,
- I915_ERROR_CAPTURE,
- "stopped heartbeat on %s",
- engine->name);
- }
- static void heartbeat(struct work_struct *wrk)
- {
- struct i915_sched_attr attr = { .priority = I915_PRIORITY_MIN };
- struct intel_engine_cs *engine =
- container_of(wrk, typeof(*engine), heartbeat.work.work);
- struct intel_context *ce = engine->kernel_context;
- struct i915_request *rq;
- unsigned long serial;
- /* Just in case everything has gone horribly wrong, give it a kick */
- intel_engine_flush_submission(engine);
- rq = xchg(&engine->heartbeat.systole, NULL);
- if (rq) {
- if (i915_request_completed(rq))
- i915_request_put(rq);
- else
- engine->heartbeat.systole = rq;
- }
- if (!intel_engine_pm_get_if_awake(engine))
- return;
- if (intel_gt_is_wedged(engine->gt))
- goto out;
- if (i915_sched_engine_disabled(engine->sched_engine)) {
- reset_engine(engine, engine->heartbeat.systole);
- goto out;
- }
- if (engine->heartbeat.systole) {
- long delay = READ_ONCE(engine->props.heartbeat_interval_ms);
- /* Safeguard against too-fast worker invocations */
- if (!time_after(jiffies,
- rq->emitted_jiffies + msecs_to_jiffies(delay)))
- goto out;
- if (!i915_sw_fence_signaled(&rq->submit)) {
- /*
- * Not yet submitted, system is stalled.
- *
- * This more often happens for ring submission,
- * where all contexts are funnelled into a common
- * ringbuffer. If one context is blocked on an
- * external fence, not only is it not submitted,
- * but all other contexts, including the kernel
- * context are stuck waiting for the signal.
- */
- } else if (engine->sched_engine->schedule &&
- rq->sched.attr.priority < I915_PRIORITY_BARRIER) {
- /*
- * Gradually raise the priority of the heartbeat to
- * give high priority work [which presumably desires
- * low latency and no jitter] the chance to naturally
- * complete before being preempted.
- */
- attr.priority = I915_PRIORITY_NORMAL;
- if (rq->sched.attr.priority >= attr.priority)
- attr.priority = I915_PRIORITY_HEARTBEAT;
- if (rq->sched.attr.priority >= attr.priority)
- attr.priority = I915_PRIORITY_BARRIER;
- local_bh_disable();
- engine->sched_engine->schedule(rq, &attr);
- local_bh_enable();
- } else {
- reset_engine(engine, rq);
- }
- rq->emitted_jiffies = jiffies;
- goto out;
- }
- serial = READ_ONCE(engine->serial);
- if (engine->wakeref_serial == serial)
- goto out;
- if (!mutex_trylock(&ce->timeline->mutex)) {
- /* Unable to lock the kernel timeline, is the engine stuck? */
- if (xchg(&engine->heartbeat.blocked, serial) == serial)
- intel_gt_handle_error(engine->gt, engine->mask,
- I915_ERROR_CAPTURE,
- "no heartbeat on %s",
- engine->name);
- goto out;
- }
- rq = heartbeat_create(ce, GFP_NOWAIT | __GFP_NOWARN);
- if (IS_ERR(rq))
- goto unlock;
- heartbeat_commit(rq, &attr);
- unlock:
- mutex_unlock(&ce->timeline->mutex);
- out:
- if (!engine->i915->params.enable_hangcheck || !next_heartbeat(engine)) {
- rq = xchg(&engine->heartbeat.systole, NULL);
- if (rq)
- i915_request_put(rq);
- }
- intel_engine_pm_put(engine);
- }
- void intel_engine_unpark_heartbeat(struct intel_engine_cs *engine)
- {
- if (!CONFIG_DRM_I915_HEARTBEAT_INTERVAL)
- return;
- next_heartbeat(engine);
- }
- void intel_engine_park_heartbeat(struct intel_engine_cs *engine)
- {
- if (cancel_delayed_work(&engine->heartbeat.work)) {
- struct i915_request *rq;
- rq = xchg(&engine->heartbeat.systole, NULL);
- if (rq)
- i915_request_put(rq);
- }
- }
- void intel_gt_unpark_heartbeats(struct intel_gt *gt)
- {
- struct intel_engine_cs *engine;
- enum intel_engine_id id;
- for_each_engine(engine, gt, id)
- if (intel_engine_pm_is_awake(engine))
- intel_engine_unpark_heartbeat(engine);
- }
- void intel_gt_park_heartbeats(struct intel_gt *gt)
- {
- struct intel_engine_cs *engine;
- enum intel_engine_id id;
- for_each_engine(engine, gt, id)
- intel_engine_park_heartbeat(engine);
- }
- void intel_engine_init_heartbeat(struct intel_engine_cs *engine)
- {
- INIT_DELAYED_WORK(&engine->heartbeat.work, heartbeat);
- }
- static int __intel_engine_pulse(struct intel_engine_cs *engine)
- {
- struct i915_sched_attr attr = { .priority = I915_PRIORITY_BARRIER };
- struct intel_context *ce = engine->kernel_context;
- struct i915_request *rq;
- lockdep_assert_held(&ce->timeline->mutex);
- GEM_BUG_ON(!intel_engine_has_preemption(engine));
- GEM_BUG_ON(!intel_engine_pm_is_awake(engine));
- rq = heartbeat_create(ce, GFP_NOWAIT | __GFP_NOWARN);
- if (IS_ERR(rq))
- return PTR_ERR(rq);
- __set_bit(I915_FENCE_FLAG_SENTINEL, &rq->fence.flags);
- heartbeat_commit(rq, &attr);
- GEM_BUG_ON(rq->sched.attr.priority < I915_PRIORITY_BARRIER);
- /* Ensure the forced pulse gets a full period to execute */
- next_heartbeat(engine);
- return 0;
- }
- static unsigned long set_heartbeat(struct intel_engine_cs *engine,
- unsigned long delay)
- {
- unsigned long old;
- old = xchg(&engine->props.heartbeat_interval_ms, delay);
- if (delay)
- intel_engine_unpark_heartbeat(engine);
- else
- intel_engine_park_heartbeat(engine);
- return old;
- }
- int intel_engine_set_heartbeat(struct intel_engine_cs *engine,
- unsigned long delay)
- {
- struct intel_context *ce = engine->kernel_context;
- int err = 0;
- if (!delay && !intel_engine_has_preempt_reset(engine))
- return -ENODEV;
- /* FIXME: Remove together with equally marked hack in next_heartbeat. */
- if (delay != engine->defaults.heartbeat_interval_ms &&
- delay < 2 * engine->props.preempt_timeout_ms) {
- if (intel_engine_uses_guc(engine))
- drm_notice(&engine->i915->drm, "%s heartbeat interval adjusted to a non-default value which may downgrade individual engine resets to full GPU resets!\n",
- engine->name);
- else
- drm_notice(&engine->i915->drm, "%s heartbeat interval adjusted to a non-default value which may cause engine resets to target innocent contexts!\n",
- engine->name);
- }
- intel_engine_pm_get(engine);
- err = mutex_lock_interruptible(&ce->timeline->mutex);
- if (err)
- goto out_rpm;
- if (delay != engine->props.heartbeat_interval_ms) {
- unsigned long saved = set_heartbeat(engine, delay);
- /* recheck current execution */
- if (intel_engine_has_preemption(engine)) {
- err = __intel_engine_pulse(engine);
- if (err)
- set_heartbeat(engine, saved);
- }
- }
- mutex_unlock(&ce->timeline->mutex);
- out_rpm:
- intel_engine_pm_put(engine);
- return err;
- }
- int intel_engine_pulse(struct intel_engine_cs *engine)
- {
- struct intel_context *ce = engine->kernel_context;
- int err;
- if (!intel_engine_has_preemption(engine))
- return -ENODEV;
- if (!intel_engine_pm_get_if_awake(engine))
- return 0;
- err = -EINTR;
- if (!mutex_lock_interruptible(&ce->timeline->mutex)) {
- err = __intel_engine_pulse(engine);
- mutex_unlock(&ce->timeline->mutex);
- }
- intel_engine_flush_submission(engine);
- intel_engine_pm_put(engine);
- return err;
- }
- int intel_engine_flush_barriers(struct intel_engine_cs *engine)
- {
- struct i915_sched_attr attr = { .priority = I915_PRIORITY_MIN };
- struct intel_context *ce = engine->kernel_context;
- struct i915_request *rq;
- int err;
- if (llist_empty(&engine->barrier_tasks))
- return 0;
- if (!intel_engine_pm_get_if_awake(engine))
- return 0;
- if (mutex_lock_interruptible(&ce->timeline->mutex)) {
- err = -EINTR;
- goto out_rpm;
- }
- rq = heartbeat_create(ce, GFP_KERNEL);
- if (IS_ERR(rq)) {
- err = PTR_ERR(rq);
- goto out_unlock;
- }
- heartbeat_commit(rq, &attr);
- err = 0;
- out_unlock:
- mutex_unlock(&ce->timeline->mutex);
- out_rpm:
- intel_engine_pm_put(engine);
- return err;
- }
- #if IS_ENABLED(CONFIG_DRM_I915_SELFTEST)
- #include "selftest_engine_heartbeat.c"
- #endif
|