public inbox for intel-gfx@lists.freedesktop.org
 help / color / mirror / Atom feed
* [PATCH v2 01/25] drm/i915/cmdparser: Remove stray intel_engine_cs *ring
@ 2016-07-22  9:03 Chris Wilson
  2016-07-22  9:03 ` [PATCH v2 02/25] drm/i915: Use engine to refer to the user's BSD intel_engine_cs Chris Wilson
                   ` (24 more replies)
  0 siblings, 25 replies; 27+ messages in thread
From: Chris Wilson @ 2016-07-22  9:03 UTC (permalink / raw)
  To: intel-gfx

When we refer to intel_engine_cs, we want to use engine so as not to
confuse ourselves about ringbuffers.

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
---
 drivers/gpu/drm/i915/i915_cmd_parser.c  |  4 ++--
 drivers/gpu/drm/i915/i915_drv.h         |  5 +++--
 drivers/gpu/drm/i915/intel_ringbuffer.h | 10 +++++-----
 3 files changed, 10 insertions(+), 9 deletions(-)

diff --git a/drivers/gpu/drm/i915/i915_cmd_parser.c b/drivers/gpu/drm/i915/i915_cmd_parser.c
index b0fd6a7b0603..b5b520176c59 100644
--- a/drivers/gpu/drm/i915/i915_cmd_parser.c
+++ b/drivers/gpu/drm/i915/i915_cmd_parser.c
@@ -636,7 +636,7 @@ static bool validate_cmds_sorted(struct intel_engine_cs *engine,
 	return ret;
 }
 
-static bool check_sorted(int ring_id,
+static bool check_sorted(int engine_id,
 			 const struct drm_i915_reg_descriptor *reg_table,
 			 int reg_count)
 {
@@ -649,7 +649,7 @@ static bool check_sorted(int ring_id,
 
 		if (curr < previous) {
 			DRM_ERROR("CMD: table not sorted ring=%d entry=%d reg=0x%08X prev=0x%08X\n",
-				  ring_id, i, curr, previous);
+				  engine_id, i, curr, previous);
 			ret = false;
 		}
 
diff --git a/drivers/gpu/drm/i915/i915_drv.h b/drivers/gpu/drm/i915/i915_drv.h
index c97724d380d3..31ddc5cb8216 100644
--- a/drivers/gpu/drm/i915/i915_drv.h
+++ b/drivers/gpu/drm/i915/i915_drv.h
@@ -2500,8 +2500,9 @@ struct drm_i915_cmd_descriptor {
 /*
  * A table of commands requiring special handling by the command parser.
  *
- * Each ring has an array of tables. Each table consists of an array of command
- * descriptors, which must be sorted with command opcodes in ascending order.
+ * Each engine has an array of tables. Each table consists of an array of
+ * command descriptors, which must be sorted with command opcodes in
+ * ascending order.
  */
 struct drm_i915_cmd_table {
 	const struct drm_i915_cmd_descriptor *table;
diff --git a/drivers/gpu/drm/i915/intel_ringbuffer.h b/drivers/gpu/drm/i915/intel_ringbuffer.h
index 0f8019488d33..0a830c116886 100644
--- a/drivers/gpu/drm/i915/intel_ringbuffer.h
+++ b/drivers/gpu/drm/i915/intel_ringbuffer.h
@@ -340,7 +340,7 @@ struct intel_engine_cs {
 
 	/*
 	 * Table of commands the command parser needs to know about
-	 * for this ring.
+	 * for this engine.
 	 */
 	DECLARE_HASHTABLE(cmd_hash, I915_CMD_HASH_ORDER);
 
@@ -354,11 +354,11 @@ struct intel_engine_cs {
 	 * Returns the bitmask for the length field of the specified command.
 	 * Return 0 for an unrecognized/invalid command.
 	 *
-	 * If the command parser finds an entry for a command in the ring's
+	 * If the command parser finds an entry for a command in the engines's
 	 * cmd_tables, it gets the command's length based on the table entry.
-	 * If not, it calls this function to determine the per-ring length field
-	 * encoding for the command (i.e. certain opcode ranges use certain bits
-	 * to encode the command length in the header).
+	 * If not, it calls this function to determine the per-engine length
+	 * field encoding for the command (i.e. different opcode ranges use
+	 * certain bits to encode the command length in the header).
 	 */
 	u32 (*get_cmd_length_mask)(u32 cmd_header);
 };
-- 
2.8.1

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

^ permalink raw reply related	[flat|nested] 27+ messages in thread

end of thread, other threads:[~2016-07-22  9:36 UTC | newest]

Thread overview: 27+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-07-22  9:03 [PATCH v2 01/25] drm/i915/cmdparser: Remove stray intel_engine_cs *ring Chris Wilson
2016-07-22  9:03 ` [PATCH v2 02/25] drm/i915: Use engine to refer to the user's BSD intel_engine_cs Chris Wilson
2016-07-22  9:03 ` [PATCH v2 03/25] drm/i915: Avoid using intel_engine_cs *ring for GPU error capture Chris Wilson
2016-07-22  9:03 ` [PATCH v2 04/25] drm/i915: Remove stray intel_engine_cs ring identifiers from i915_gem.c Chris Wilson
2016-07-22  9:03 ` [PATCH v2 05/25] drm/i915: Update a couple of hangcheck comments to talk about engines Chris Wilson
2016-07-22  9:03 ` [PATCH v2 06/25] drm/i915: Unify intel_logical_ring_emit and intel_ring_emit Chris Wilson
2016-07-22  9:03 ` [PATCH v2 07/25] drm/i915: Rename request->ringbuf to request->ring Chris Wilson
2016-07-22  9:03 ` [PATCH v2 08/25] drm/i915: Rename backpointer from intel_ringbuffer to intel_engine_cs Chris Wilson
2016-07-22  9:03 ` [PATCH v2 09/25] drm/i915: Rename intel_context[engine].ringbuf Chris Wilson
2016-07-22  9:03 ` [PATCH v2 10/25] drm/i915: Rename struct intel_ringbuffer to struct intel_ring Chris Wilson
2016-07-22  9:03 ` [PATCH v2 11/25] drm/i915: Rename residual ringbuf parameters Chris Wilson
2016-07-22  9:03 ` [PATCH v2 12/25] drm/i915: Rename intel_pin_and_map_ring() Chris Wilson
2016-07-22  9:03 ` [PATCH v2 13/25] drm/i915: Remove obsolete engine->gpu_caches_dirty Chris Wilson
2016-07-22  9:03 ` [PATCH v2 14/25] drm/i915: Simplify request_alloc by returning the allocated request Chris Wilson
2016-07-22  9:03 ` [PATCH v2 15/25] drm/i915: Unify legacy/execlists emission of MI_BATCHBUFFER_START Chris Wilson
2016-07-22  9:03 ` [PATCH v2 16/25] drm/i915: Remove intel_ring_get_tail() Chris Wilson
2016-07-22  9:03 ` [PATCH v2 17/25] drm/i915: Convert engine->write_tail to operate on a request Chris Wilson
2016-07-22  9:03 ` [PATCH v2 18/25] drm/i915: Unify request submission Chris Wilson
2016-07-22  9:03 ` [PATCH v2 19/25] drm/i915/lrc: Update function names to match request flow Chris Wilson
2016-07-22  9:23   ` Joonas Lahtinen
2016-07-22  9:03 ` [PATCH v2 20/25] drm/i915: Stop passing caller's num_dwords to engine->semaphore.signal() Chris Wilson
2016-07-22  9:03 ` [PATCH v2 21/25] drm/i915: Reuse legacy breadcrumbs + tail emission Chris Wilson
2016-07-22  9:03 ` [PATCH v2 22/25] drm/i915/ringbuffer: Specialise SNB+ request emission for semaphores Chris Wilson
2016-07-22  9:03 ` [PATCH v2 23/25] drm/i915: Remove duplicate golden render state init from execlists Chris Wilson
2016-07-22  9:03 ` [PATCH v2 24/25] drm/i915: Unify legacy/execlists submit_execbuf callbacks Chris Wilson
2016-07-22  9:03 ` [PATCH v2 25/25] drm/i915: Simplify calling engine->sync_to Chris Wilson
2016-07-22  9:36 ` ✗ Ro.CI.BAT: failure for series starting with [v2,01/25] drm/i915/cmdparser: Remove stray intel_engine_cs *ring Patchwork

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox