From patchwork Thu Sep 1 19:32:27 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Rodrigo Vivi X-Patchwork-Id: 12963180 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id A1F04ECAAD3 for ; Thu, 1 Sep 2022 19:32:40 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id E0E6610E1A8; Thu, 1 Sep 2022 19:32:38 +0000 (UTC) Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) by gabe.freedesktop.org (Postfix) with ESMTPS id 1E5A910E1A8 for ; Thu, 1 Sep 2022 19:32:33 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1662060753; x=1693596753; h=from:to:cc:subject:date:message-id:mime-version: content-transfer-encoding; bh=izPlVFtX0lE216niE0zh2Tn7jVKhQLKFSthFzMF3AV8=; b=L15yfVd33pNBLRxh/GOjMiuHttUVzrxdQK9FnSiImZpqGN2YAy5FrohN U567so/G3m9vyc9GroCKW99hLK/DVDe8BXqvSnvI74XYA7oOhZdCg1OuC i6guFRDZiP/ipSNFkakb32qZhiDABN5t5bQVJRMn/qOCb7wMZjgCBkbwm qs7kz1Hwq2N0bKg2EeGohpwIZhVLuBmNEaAZmeZUFfFnO8HiuhfdA3zXb BjK4X4YBfEFh5lsArWVGb+ICMX/te6ZXZn0DwJbkTkQzQffjFiYaxoj5P CFPJCMdAVZBmseC94Uc3epF4a6lfXsNtYmIRMwIkLOvzuwVKck9pS9HIN Q==; X-IronPort-AV: E=McAfee;i="6500,9779,10457"; a="321964982" X-IronPort-AV: E=Sophos;i="5.93,281,1654585200"; d="scan'208";a="321964982" Received: from orsmga003.jf.intel.com ([10.7.209.27]) by fmsmga101.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 01 Sep 2022 12:32:32 -0700 X-IronPort-AV: E=Sophos;i="5.93,281,1654585200"; d="scan'208";a="563592317" Received: from rishabhj-mobl1.amr.corp.intel.com (HELO rdvivi-mobl4.intel.com) ([10.255.36.114]) by orsmga003-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 01 Sep 2022 12:32:30 -0700 From: Rodrigo Vivi To: intel-gfx@lists.freedesktop.org Date: Thu, 1 Sep 2022 15:32:27 -0400 Message-Id: <20220901193228.255098-1-rodrigo.vivi@intel.com> X-Mailer: git-send-email 2.37.2 MIME-Version: 1.0 Subject: [Intel-gfx] [PATCH 1/2] drm/i915: Move some of the request decisions out of rps_boost function. X-BeenThere: intel-gfx@lists.freedesktop.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Intel graphics driver community testing & development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Rodrigo Vivi Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" Ideally all the decisions should be made before calling the boost function. And rps functions only receiving the rps struct. At least lets move most of the decisions to the request component, but still leave the test and set of the fence flag boost inside the rps because that might be time sensitive. Cc: Ashutosh Dixit Signed-off-by: Rodrigo Vivi --- drivers/gpu/drm/i915/display/intel_atomic_plane.c | 2 +- drivers/gpu/drm/i915/gem/i915_gem_wait.c | 3 ++- drivers/gpu/drm/i915/gt/intel_rps.c | 3 --- drivers/gpu/drm/i915/gt/intel_rps.h | 1 + drivers/gpu/drm/i915/i915_request.h | 5 +++-- 5 files changed, 7 insertions(+), 7 deletions(-) diff --git a/drivers/gpu/drm/i915/display/intel_atomic_plane.c b/drivers/gpu/drm/i915/display/intel_atomic_plane.c index dd876dbbaa39..6967c47c7ba0 100644 --- a/drivers/gpu/drm/i915/display/intel_atomic_plane.c +++ b/drivers/gpu/drm/i915/display/intel_atomic_plane.c @@ -918,7 +918,7 @@ static int do_rps_boost(struct wait_queue_entry *_wait, * is reasonable to assume that it will complete before the next * vblank without our intervention, so leave RPS alone. */ - if (!i915_request_started(rq)) + if (!i915_request_started(rq) && i915_request_needs_boost(rq)) intel_rps_boost(rq); i915_request_put(rq); diff --git a/drivers/gpu/drm/i915/gem/i915_gem_wait.c b/drivers/gpu/drm/i915/gem/i915_gem_wait.c index e6e01c2a74a6..2f2ca5e27248 100644 --- a/drivers/gpu/drm/i915/gem/i915_gem_wait.c +++ b/drivers/gpu/drm/i915/gem/i915_gem_wait.c @@ -58,7 +58,8 @@ i915_gem_object_boost(struct dma_resv *resv, unsigned int flags) dma_resv_usage_rw(flags & I915_WAIT_ALL)); dma_resv_for_each_fence_unlocked(&cursor, fence) if (dma_fence_is_i915(fence) && - !i915_request_started(to_request(fence))) + !i915_request_started(to_request(fence)) && + i915_request_needs_boost(to_request(fence))) intel_rps_boost(to_request(fence)); dma_resv_iter_end(&cursor); } diff --git a/drivers/gpu/drm/i915/gt/intel_rps.c b/drivers/gpu/drm/i915/gt/intel_rps.c index 579ae9ac089c..2c8d9eeb7e7e 100644 --- a/drivers/gpu/drm/i915/gt/intel_rps.c +++ b/drivers/gpu/drm/i915/gt/intel_rps.c @@ -1006,9 +1006,6 @@ void intel_rps_boost(struct i915_request *rq) { struct intel_guc_slpc *slpc; - if (i915_request_signaled(rq) || i915_request_has_waitboost(rq)) - 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; diff --git a/drivers/gpu/drm/i915/gt/intel_rps.h b/drivers/gpu/drm/i915/gt/intel_rps.h index 4509dfdc52e0..9a053f1b04e8 100644 --- a/drivers/gpu/drm/i915/gt/intel_rps.h +++ b/drivers/gpu/drm/i915/gt/intel_rps.h @@ -23,6 +23,7 @@ void intel_rps_disable(struct intel_rps *rps); void intel_rps_park(struct intel_rps *rps); void intel_rps_unpark(struct intel_rps *rps); +bool intel_rps_request_needs_boost(struct i915_request *rq); void intel_rps_boost(struct i915_request *rq); void intel_rps_dec_waiters(struct intel_rps *rps); u32 intel_rps_get_boost_frequency(struct intel_rps *rps); diff --git a/drivers/gpu/drm/i915/i915_request.h b/drivers/gpu/drm/i915/i915_request.h index 47041ec68df8..4f5049ef1ab9 100644 --- a/drivers/gpu/drm/i915/i915_request.h +++ b/drivers/gpu/drm/i915/i915_request.h @@ -625,9 +625,10 @@ static inline void i915_request_mark_complete(struct i915_request *rq) (u32 *)&rq->fence.seqno); } -static inline bool i915_request_has_waitboost(const struct i915_request *rq) +static inline bool i915_request_needs_boost(const struct i915_request *rq) { - return test_bit(I915_FENCE_FLAG_BOOST, &rq->fence.flags); + return i915_request_signaled(rq) + && test_bit(I915_FENCE_FLAG_BOOST, &rq->fence.flags); } static inline bool i915_request_has_nopreempt(const struct i915_request *rq)