diff options
author | Daniel Vetter <daniel.vetter@ffwll.ch> | 2015-05-21 08:21:25 -0400 |
---|---|---|
committer | Daniel Vetter <daniel.vetter@ffwll.ch> | 2015-05-21 09:10:48 -0400 |
commit | eed29a5b21557476bbd8b141946a8dfe5aacc4f3 (patch) | |
tree | 6a8103e3933141feb82bfa0e11f93c82895d0f26 /drivers/gpu/drm/i915/i915_debugfs.c | |
parent | 8ea6f8926b367725f0c13ff82f251074de9d95b4 (diff) |
drm/i915: s/\<rq\>/req/g
The merged seqno->request conversion from John called request
variables req, but some (not all) of Chris' recent patches changed
those to just rq. We've had a lenghty (and inconclusive) discussion on
irc which is the more meaningful name with maybe at most a slight bias
towards req.
Given that the "don't change names without good reason to avoid
conflicts" rule applies, so lets go back to a req everywhere for
consistency. I'll sed any patches for which this will cause conflicts
before applying.
Cc: Chris Wilson <chris@chris-wilson.co.uk>
Cc: John Harrison <John.C.Harrison@Intel.com>
[danvet: s/origina/merged/ as pointed out by Chris - the first
mass-conversion patch was from Chris, the merged one from John.]
Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
Diffstat (limited to 'drivers/gpu/drm/i915/i915_debugfs.c')
-rw-r--r-- | drivers/gpu/drm/i915/i915_debugfs.c | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/drivers/gpu/drm/i915/i915_debugfs.c b/drivers/gpu/drm/i915/i915_debugfs.c index a32b669bab89..f465af1b02f8 100644 --- a/drivers/gpu/drm/i915/i915_debugfs.c +++ b/drivers/gpu/drm/i915/i915_debugfs.c | |||
@@ -665,7 +665,7 @@ static int i915_gem_request_info(struct seq_file *m, void *data) | |||
665 | struct drm_device *dev = node->minor->dev; | 665 | struct drm_device *dev = node->minor->dev; |
666 | struct drm_i915_private *dev_priv = dev->dev_private; | 666 | struct drm_i915_private *dev_priv = dev->dev_private; |
667 | struct intel_engine_cs *ring; | 667 | struct intel_engine_cs *ring; |
668 | struct drm_i915_gem_request *rq; | 668 | struct drm_i915_gem_request *req; |
669 | int ret, any, i; | 669 | int ret, any, i; |
670 | 670 | ||
671 | ret = mutex_lock_interruptible(&dev->struct_mutex); | 671 | ret = mutex_lock_interruptible(&dev->struct_mutex); |
@@ -677,22 +677,22 @@ static int i915_gem_request_info(struct seq_file *m, void *data) | |||
677 | int count; | 677 | int count; |
678 | 678 | ||
679 | count = 0; | 679 | count = 0; |
680 | list_for_each_entry(rq, &ring->request_list, list) | 680 | list_for_each_entry(req, &ring->request_list, list) |
681 | count++; | 681 | count++; |
682 | if (count == 0) | 682 | if (count == 0) |
683 | continue; | 683 | continue; |
684 | 684 | ||
685 | seq_printf(m, "%s requests: %d\n", ring->name, count); | 685 | seq_printf(m, "%s requests: %d\n", ring->name, count); |
686 | list_for_each_entry(rq, &ring->request_list, list) { | 686 | list_for_each_entry(req, &ring->request_list, list) { |
687 | struct task_struct *task; | 687 | struct task_struct *task; |
688 | 688 | ||
689 | rcu_read_lock(); | 689 | rcu_read_lock(); |
690 | task = NULL; | 690 | task = NULL; |
691 | if (rq->pid) | 691 | if (req->pid) |
692 | task = pid_task(rq->pid, PIDTYPE_PID); | 692 | task = pid_task(req->pid, PIDTYPE_PID); |
693 | seq_printf(m, " %x @ %d: %s [%d]\n", | 693 | seq_printf(m, " %x @ %d: %s [%d]\n", |
694 | rq->seqno, | 694 | req->seqno, |
695 | (int) (jiffies - rq->emitted_jiffies), | 695 | (int) (jiffies - req->emitted_jiffies), |
696 | task ? task->comm : "<unknown>", | 696 | task ? task->comm : "<unknown>", |
697 | task ? task->pid : -1); | 697 | task ? task->pid : -1); |
698 | rcu_read_unlock(); | 698 | rcu_read_unlock(); |