From mboxrd@z Thu Jan 1 00:00:00 1970 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 EFEF9C433EF for ; Thu, 27 Jan 2022 08:22:54 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id 814F310E319; Thu, 27 Jan 2022 08:22:54 +0000 (UTC) Received: from mga18.intel.com (mga18.intel.com [134.134.136.126]) by gabe.freedesktop.org (Postfix) with ESMTPS id 01C9E10E319 for ; Thu, 27 Jan 2022 08:22:52 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1643271773; x=1674807773; h=date:from:to:cc:subject:message-id:references: mime-version:content-transfer-encoding:in-reply-to; bh=HNyo5y0pZd7esG/LXYCZgX07q/wyhmxlaPO0G4We1oo=; b=oFjGF86ZE6BP/D5rYId5jadqYJUkbHrnKtlfdBFfr6clRqrzZt/9DKA8 AVMBu9sc2XyMz0J3o9VW0JdXkvIh0inTOOvNCi4uIRSsuc5olTdwSp+BW FDXy00HLJ8HGQdWYhxHFM58FxsnUWMSj2hqov8psZaEahESCIm67rgfzA pbt/BD0Zx9uPdMqGVZ1Qmc9//g/3SI5y/rBc0Q7G2k0L2Jl7TE/oQNe2m 66uePppdfB+2NsJiMTrQq0l7jJ5V3J3lQukufQ5QWXt6JfcSyqKqb/i+/ Z8RCtuEoZ4O/wHqMy8GHmVptjfdw/gsfgWH51JvAP/yIg24qCprg9RL/7 Q==; X-IronPort-AV: E=McAfee;i="6200,9189,10239"; a="230360217" X-IronPort-AV: E=Sophos;i="5.88,320,1635231600"; d="scan'208";a="230360217" Received: from orsmga003.jf.intel.com ([10.7.209.27]) by orsmga106.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 27 Jan 2022 00:22:52 -0800 X-IronPort-AV: E=Sophos;i="5.88,320,1635231600"; d="scan'208";a="477779937" Received: from unknown (HELO intel.com) ([10.237.72.65]) by orsmga003-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 27 Jan 2022 00:22:50 -0800 Date: Thu, 27 Jan 2022 10:22:58 +0200 From: "Lisovskiy, Stanislav" To: Ville Syrjala Message-ID: <20220127082258.GD31846@intel.com> References: <20220118092354.11631-1-ville.syrjala@linux.intel.com> <20220118092354.11631-5-ville.syrjala@linux.intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20220118092354.11631-5-ville.syrjala@linux.intel.com> User-Agent: Mutt/1.9.4 (2018-02-28) Subject: Re: [Intel-gfx] [PATCH 04/15] drm/i915: Introduce skl_plane_ddb_iter 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: intel-gfx@lists.freedesktop.org Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" On Tue, Jan 18, 2022 at 11:23:43AM +0200, Ville Syrjala wrote: > From: Ville Syrjälä > > Collect a bit of the stuff used during the plane ddb > allocation into a struct we can pass around. > > Signed-off-by: Ville Syrjälä Reviewed-by: Stanislav Lisovskiy > --- > drivers/gpu/drm/i915/intel_pm.c | 97 +++++++++++++++++---------------- > 1 file changed, 49 insertions(+), 48 deletions(-) > > diff --git a/drivers/gpu/drm/i915/intel_pm.c b/drivers/gpu/drm/i915/intel_pm.c > index e8fb56f288b4..cd1b5f09f241 100644 > --- a/drivers/gpu/drm/i915/intel_pm.c > +++ b/drivers/gpu/drm/i915/intel_pm.c > @@ -5066,6 +5066,13 @@ static bool icl_need_wm1_wa(struct drm_i915_private *i915, > (IS_DISPLAY_VER(i915, 12, 13) && plane_id == PLANE_CURSOR); > } > > +struct skl_plane_ddb_iter { > + u64 data_rate; > + u16 total[I915_MAX_PLANES]; > + u16 uv_total[I915_MAX_PLANES]; > + u16 start, size; > +}; > + > static int > skl_allocate_plane_ddb(struct intel_atomic_state *state, > struct intel_crtc *crtc) > @@ -5077,10 +5084,7 @@ skl_allocate_plane_ddb(struct intel_atomic_state *state, > intel_atomic_get_new_dbuf_state(state); > const struct skl_ddb_entry *alloc = &dbuf_state->ddb[crtc->pipe]; > int num_active = hweight8(dbuf_state->active_pipes); > - u16 alloc_size, start = 0; > - u16 total[I915_MAX_PLANES] = {}; > - u16 uv_total[I915_MAX_PLANES] = {}; > - u64 total_data_rate; > + struct skl_plane_ddb_iter iter = {}; > enum plane_id plane_id; > u32 blocks; > int level; > @@ -5093,23 +5097,21 @@ skl_allocate_plane_ddb(struct intel_atomic_state *state, > return 0; > > if (DISPLAY_VER(dev_priv) >= 11) > - total_data_rate = > - icl_get_total_relative_data_rate(state, crtc); > + iter.data_rate = icl_get_total_relative_data_rate(state, crtc); > else > - total_data_rate = > - skl_get_total_relative_data_rate(state, crtc); > + iter.data_rate = skl_get_total_relative_data_rate(state, crtc); > > - alloc_size = skl_ddb_entry_size(alloc); > - if (alloc_size == 0) > + iter.size = skl_ddb_entry_size(alloc); > + if (iter.size == 0) > return 0; > > /* Allocate fixed number of blocks for cursor. */ > - total[PLANE_CURSOR] = skl_cursor_allocation(crtc_state, num_active); > - alloc_size -= total[PLANE_CURSOR]; > + iter.total[PLANE_CURSOR] = skl_cursor_allocation(crtc_state, num_active); > + iter.size -= iter.total[PLANE_CURSOR]; > skl_ddb_entry_init(&crtc_state->wm.skl.plane_ddb_y[PLANE_CURSOR], > - alloc->end - total[PLANE_CURSOR], alloc->end); > + alloc->end - iter.total[PLANE_CURSOR], alloc->end); > > - if (total_data_rate == 0) > + if (iter.data_rate == 0) > return 0; > > /* > @@ -5123,7 +5125,7 @@ skl_allocate_plane_ddb(struct intel_atomic_state *state, > &crtc_state->wm.skl.optimal.planes[plane_id]; > > if (plane_id == PLANE_CURSOR) { > - if (wm->wm[level].min_ddb_alloc > total[PLANE_CURSOR]) { > + if (wm->wm[level].min_ddb_alloc > iter.total[PLANE_CURSOR]) { > drm_WARN_ON(&dev_priv->drm, > wm->wm[level].min_ddb_alloc != U16_MAX); > blocks = U32_MAX; > @@ -5136,8 +5138,8 @@ skl_allocate_plane_ddb(struct intel_atomic_state *state, > blocks += wm->uv_wm[level].min_ddb_alloc; > } > > - if (blocks <= alloc_size) { > - alloc_size -= blocks; > + if (blocks <= iter.size) { > + iter.size -= blocks; > break; > } > } > @@ -5146,7 +5148,7 @@ skl_allocate_plane_ddb(struct intel_atomic_state *state, > drm_dbg_kms(&dev_priv->drm, > "Requested display configuration exceeds system DDB limitations"); > drm_dbg_kms(&dev_priv->drm, "minimum required %d/%d\n", > - blocks, alloc_size); > + blocks, iter.size); > return -EINVAL; > } > > @@ -5158,7 +5160,7 @@ skl_allocate_plane_ddb(struct intel_atomic_state *state, > for_each_plane_id_on_crtc(crtc, plane_id) { > const struct skl_plane_wm *wm = > &crtc_state->wm.skl.optimal.planes[plane_id]; > - u64 rate; > + u64 data_rate; > u16 extra; > > if (plane_id == PLANE_CURSOR) > @@ -5168,32 +5170,30 @@ skl_allocate_plane_ddb(struct intel_atomic_state *state, > * We've accounted for all active planes; remaining planes are > * all disabled. > */ > - if (total_data_rate == 0) > + if (iter.data_rate == 0) > break; > > - rate = crtc_state->plane_data_rate[plane_id]; > - extra = min_t(u16, alloc_size, > - DIV64_U64_ROUND_UP(alloc_size * rate, > - total_data_rate)); > - total[plane_id] = wm->wm[level].min_ddb_alloc + extra; > - alloc_size -= extra; > - total_data_rate -= rate; > + data_rate = crtc_state->plane_data_rate[plane_id]; > + extra = min_t(u16, iter.size, > + DIV64_U64_ROUND_UP(iter.size * data_rate, iter.data_rate)); > + iter.total[plane_id] = wm->wm[level].min_ddb_alloc + extra; > + iter.size -= extra; > + iter.data_rate -= data_rate; > > - if (total_data_rate == 0) > + if (iter.data_rate == 0) > break; > > - rate = crtc_state->uv_plane_data_rate[plane_id]; > - extra = min_t(u16, alloc_size, > - DIV64_U64_ROUND_UP(alloc_size * rate, > - total_data_rate)); > - uv_total[plane_id] = wm->uv_wm[level].min_ddb_alloc + extra; > - alloc_size -= extra; > - total_data_rate -= rate; > + data_rate = crtc_state->uv_plane_data_rate[plane_id]; > + extra = min_t(u16, iter.size, > + DIV64_U64_ROUND_UP(iter.size * data_rate, iter.data_rate)); > + iter.uv_total[plane_id] = wm->uv_wm[level].min_ddb_alloc + extra; > + iter.size -= extra; > + iter.data_rate -= data_rate; > } > - drm_WARN_ON(&dev_priv->drm, alloc_size != 0 || total_data_rate != 0); > + drm_WARN_ON(&dev_priv->drm, iter.size != 0 || iter.data_rate != 0); > > /* Set the actual DDB start/end points for each plane */ > - start = alloc->start; > + iter.start = alloc->start; > for_each_plane_id_on_crtc(crtc, plane_id) { > struct skl_ddb_entry *plane_alloc = > &crtc_state->wm.skl.plane_ddb_y[plane_id]; > @@ -5205,16 +5205,16 @@ skl_allocate_plane_ddb(struct intel_atomic_state *state, > > /* Gen11+ uses a separate plane for UV watermarks */ > drm_WARN_ON(&dev_priv->drm, > - DISPLAY_VER(dev_priv) >= 11 && uv_total[plane_id]); > + DISPLAY_VER(dev_priv) >= 11 && iter.uv_total[plane_id]); > > /* Leave disabled planes at (0,0) */ > - if (total[plane_id]) > - start = skl_ddb_entry_init(plane_alloc, start, > - start + total[plane_id]); > + if (iter.total[plane_id]) > + iter.start = skl_ddb_entry_init(plane_alloc, iter.start, > + iter.start + iter.total[plane_id]); > > - if (uv_total[plane_id]) > - start = skl_ddb_entry_init(uv_plane_alloc, start, > - start + uv_total[plane_id]); > + if (iter.uv_total[plane_id]) > + iter.start = skl_ddb_entry_init(uv_plane_alloc, iter.start, > + iter.start + iter.uv_total[plane_id]); > } > > /* > @@ -5229,7 +5229,8 @@ skl_allocate_plane_ddb(struct intel_atomic_state *state, > &crtc_state->wm.skl.optimal.planes[plane_id]; > > skl_check_nv12_wm_level(&wm->wm[level], &wm->uv_wm[level], > - total[plane_id], uv_total[plane_id]); > + iter.total[plane_id], > + iter.uv_total[plane_id]); > > if (icl_need_wm1_wa(dev_priv, plane_id) && > level == 1 && wm->wm[0].enable) { > @@ -5248,9 +5249,9 @@ skl_allocate_plane_ddb(struct intel_atomic_state *state, > struct skl_plane_wm *wm = > &crtc_state->wm.skl.optimal.planes[plane_id]; > > - skl_check_wm_level(&wm->trans_wm, total[plane_id]); > - skl_check_wm_level(&wm->sagv.wm0, total[plane_id]); > - skl_check_wm_level(&wm->sagv.trans_wm, total[plane_id]); > + skl_check_wm_level(&wm->trans_wm, iter.total[plane_id]); > + skl_check_wm_level(&wm->sagv.wm0, iter.total[plane_id]); > + skl_check_wm_level(&wm->sagv.trans_wm, iter.total[plane_id]); > } > > return 0; > -- > 2.32.0 >