diff options
| author | Chris Wilson | 2019-03-21 15:07:11 +0100 |
|---|---|---|
| committer | Chris Wilson | 2019-03-21 16:59:31 +0100 |
| commit | 4daffb664a69532efdfee54f3eac5ce54e8c37dd (patch) | |
| tree | 0d688ae661c5e6ca036fca9dede5e9fc4e9fa5c9 /drivers/gpu/drm/i915/i915_gem_context.c | |
| parent | drm/i915: Stop storing ctx->user_handle (diff) | |
| download | kernel-qcow2-linux-4daffb664a69532efdfee54f3eac5ce54e8c37dd.tar.gz kernel-qcow2-linux-4daffb664a69532efdfee54f3eac5ce54e8c37dd.tar.xz kernel-qcow2-linux-4daffb664a69532efdfee54f3eac5ce54e8c37dd.zip | |
drm/i915: Stop storing the context name as the timeline name
The timeline->name is only used for convenience in pretty printing the
i915_request.fence->ops->get_timeline_name() and it is just as
convenient to pull it from the gem_context directly. The few instances
of its use inside GEM_TRACE() has proven more of a nuisance than
helpful, so not worth saving imo.
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Reviewed-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20190321140711.11190-4-chris@chris-wilson.co.uk
Diffstat (limited to 'drivers/gpu/drm/i915/i915_gem_context.c')
| -rw-r--r-- | drivers/gpu/drm/i915/i915_gem_context.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/drivers/gpu/drm/i915/i915_gem_context.c b/drivers/gpu/drm/i915/i915_gem_context.c index 9187910391d8..00dec72f6875 100644 --- a/drivers/gpu/drm/i915/i915_gem_context.c +++ b/drivers/gpu/drm/i915/i915_gem_context.c @@ -673,9 +673,8 @@ last_request_on_engine(struct i915_timeline *timeline, rq = i915_active_request_raw(&timeline->last_request, &engine->i915->drm.struct_mutex); if (rq && rq->engine == engine) { - GEM_TRACE("last request for %s on engine %s: %llx:%llu\n", - timeline->name, engine->name, - rq->fence.context, rq->fence.seqno); + GEM_TRACE("last request on engine %s: %llx:%llu\n", + engine->name, rq->fence.context, rq->fence.seqno); GEM_BUG_ON(rq->timeline != timeline); return rq; } |