[PATCH] drm/i915/slpc: Optmize waitboost for SLPC

[Vinay Belgaumkar]

Waitboost (when SLPC is enabled) results in a H2G message. This can result
in thousands of messages during a stress test and fill up an already full
CTB. There is no need to request for RP0 if GuC is already requesting the
same.

Signed-off-by: Vinay Belgaumkar <vinay.belgaum...@intel.com>
---
 drivers/gpu/drm/i915/gt/intel_rps.c | 9 ++++++++-
 1 file changed, 8 insertions(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/i915/gt/intel_rps.c 
b/drivers/gpu/drm/i915/gt/intel_rps.c
index fc23c562d9b2..a20ae4fceac8 100644
--- a/drivers/gpu/drm/i915/gt/intel_rps.c
+++ b/drivers/gpu/drm/i915/gt/intel_rps.c
@@ -1005,13 +1005,20 @@ void intel_rps_dec_waiters(struct intel_rps *rps)
 void intel_rps_boost(struct i915_request *rq)
 {
        struct intel_guc_slpc *slpc;
+       struct intel_rps *rps = &READ_ONCE(rq->engine)->gt->rps;
 
        if (i915_request_signaled(rq) || i915_request_has_waitboost(rq))
                return;
 
+       /* If GuC is already requesting RP0, skip */
+       if (rps_uses_slpc(rps)) {
+               slpc = rps_to_slpc(rps);
+               if (intel_rps_get_requested_frequency(rps) == slpc->rp0_freq)
+                       return;
+       }
+
        /* Serializes with i915_request_retire() */
        if (!test_and_set_bit(I915_FENCE_FLAG_BOOST, &rq->fence.flags)) {
-               struct intel_rps *rps = &READ_ONCE(rq->engine)->gt->rps;
 
                if (rps_uses_slpc(rps)) {
                        slpc = rps_to_slpc(rps);
-- 
2.35.1