drm/i915/guc: re-optimise i915_guc_client layout
authorDave Gordon <david.s.gordon@intel.com>
Tue, 9 Aug 2016 14:19:23 +0000 (15:19 +0100)
committerTvrtko Ursulin <tvrtko.ursulin@intel.com>
Wed, 10 Aug 2016 09:40:05 +0000 (10:40 +0100)
As we're tweaking the GuC-related code in debugfs, we can
drop the no-longer-used 'q_fail' and repack the structure.

Signed-off-by: Dave Gordon <david.s.gordon@intel.com>
Reviewed-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
drivers/gpu/drm/i915/i915_debugfs.c
drivers/gpu/drm/i915/intel_guc.h

index b84fa93124ce3bb381bd12b74813c66bf2c3bbcc..ade2446d924a6ccf64ece92dd0b05df4f8d913d7 100644 (file)
@@ -2555,7 +2555,6 @@ static void i915_guc_client_info(struct seq_file *m,
                client->wq_size, client->wq_offset, client->wq_tail);
 
        seq_printf(m, "\tWork queue full: %u\n", client->no_wq_space);
-       seq_printf(m, "\tFailed to queue: %u\n", client->q_fail);
        seq_printf(m, "\tFailed doorbell: %u\n", client->b_fail);
        seq_printf(m, "\tLast submission result: %d\n", client->retcode);
 
index e57661ff00797126e8a46a50bf9616f90c43b119..26f3d9c0812055b38ddf41c37b7ad50e328bd60b 100644 (file)
@@ -71,19 +71,17 @@ struct i915_guc_client {
        uint32_t engines;               /* bitmap of (host) engine ids  */
        uint32_t priority;
        uint32_t ctx_index;
-
        uint32_t proc_desc_offset;
+
        uint32_t doorbell_offset;
        uint32_t cookie;
        uint16_t doorbell_id;
-       uint16_t padding;               /* Maintain alignment           */
+       uint16_t padding[3];            /* Maintain alignment           */
 
        uint32_t wq_offset;
        uint32_t wq_size;
        uint32_t wq_tail;
-
        uint32_t no_wq_space;
-       uint32_t q_fail;                /* No longer used               */
        uint32_t b_fail;
        int retcode;