Commit 9acc99d8 authored by Tvrtko Ursulin's avatar Tvrtko Ursulin Committed by Chris Wilson

drm/i915/query: Align flavour of engine data lookup

Commit 750e76b4 ("drm/i915/gt: Move the [class][inst] lookup for
engines onto the GT") changed the engine query to iterate over uabi
engines but left the buffer size calculation look at the physical engine
count. Difference has no practical consequence but it is nicer to align
both queries.
Signed-off-by: default avatarTvrtko Ursulin <tvrtko.ursulin@intel.com>
Fixes: 750e76b4 ("drm/i915/gt: Move the [class][inst] lookup for engines onto the GT")
Cc: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Daniele Ceraolo Spurio <daniele.ceraolospurio@intel.com>
Reviewed-by: default avatarChris Wilson <chris@chris-wilson.co.uk>
Signed-off-by: default avatarChris Wilson <chris@chris-wilson.co.uk>
Link: https://patchwork.freedesktop.org/patch/msgid/20191122104115.29610-1-tvrtko.ursulin@linux.intel.com
parent 6025ba12
...@@ -103,15 +103,18 @@ query_engine_info(struct drm_i915_private *i915, ...@@ -103,15 +103,18 @@ query_engine_info(struct drm_i915_private *i915,
struct drm_i915_engine_info __user *info_ptr; struct drm_i915_engine_info __user *info_ptr;
struct drm_i915_query_engine_info query; struct drm_i915_query_engine_info query;
struct drm_i915_engine_info info = { }; struct drm_i915_engine_info info = { };
unsigned int num_uabi_engines = 0;
struct intel_engine_cs *engine; struct intel_engine_cs *engine;
int len, ret; int len, ret;
if (query_item->flags) if (query_item->flags)
return -EINVAL; return -EINVAL;
for_each_uabi_engine(engine, i915)
num_uabi_engines++;
len = sizeof(struct drm_i915_query_engine_info) + len = sizeof(struct drm_i915_query_engine_info) +
RUNTIME_INFO(i915)->num_engines * num_uabi_engines * sizeof(struct drm_i915_engine_info);
sizeof(struct drm_i915_engine_info);
ret = copy_query_item(&query, sizeof(query), len, query_item); ret = copy_query_item(&query, sizeof(query), len, query_item);
if (ret != 0) if (ret != 0)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment