Quoting Lionel Landwerlin (2019-07-09 13:33:48)
> We want to set this flag in the next commit on requests containing
> perf queries so that the result of the perf query can just be a delta
> of global counters, rather than doing post processing of the OA
> buffer.
>
> Signed-off-by: Lionel Landwerlin <lionel.g.landwerlin@xxxxxxxxx>
> ---
> drivers/gpu/drm/i915/gt/intel_lrc.c | 7 ++++++-
> drivers/gpu/drm/i915/i915_priolist_types.h | 7 +++++++
> drivers/gpu/drm/i915/i915_request.c | 4 ++--
> drivers/gpu/drm/i915/i915_request.h | 14 +++++++++++++-
> drivers/gpu/drm/i915/intel_guc_submission.c | 10 +++++++++-
> drivers/gpu/drm/i915/intel_pm.c | 5 +++--
> 6 files changed, 40 insertions(+), 7 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/gt/intel_lrc.c b/drivers/gpu/drm/i915/gt/intel_lrc.c
> index 30782af8f4bc..6c35d33f9647 100644
> --- a/drivers/gpu/drm/i915/gt/intel_lrc.c
> +++ b/drivers/gpu/drm/i915/gt/intel_lrc.c
> @@ -256,7 +256,12 @@ static inline int rq_prio(const struct i915_request *rq)
>
> static int effective_prio(const struct i915_request *rq)
> {
> - int prio = rq_prio(rq);
> + int prio;
> +
> + if (i915_request_has_perf(rq))
> + prio = I915_USER_PRIORITY(I915_PRIORITY_PERF);
Ok, I may just sneak in a change here to embed the I915_USER_PRIORITY()
into the I915_PRIORITY_FOO itself so that it looks distinct from the set
of user priorities.
Reviewed-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx>
I'll work up a selftest and chase up some review for that. Thanks,
-Chris
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
