From: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> Make sure the dbuf slice mask for any individual pipe has no holes between the slices. Cc: Stanislav Lisovskiy <stanislav.lisovskiy@xxxxxxxxx> Signed-off-by: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> --- drivers/gpu/drm/i915/intel_pm.c | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/drivers/gpu/drm/i915/intel_pm.c b/drivers/gpu/drm/i915/intel_pm.c index 7edac506d343..fa39ab0b1223 100644 --- a/drivers/gpu/drm/i915/intel_pm.c +++ b/drivers/gpu/drm/i915/intel_pm.c @@ -3827,6 +3827,14 @@ static int intel_dbuf_slice_size(struct drm_i915_private *dev_priv) INTEL_INFO(dev_priv)->num_supported_dbuf_slices; } +static bool bitmask_is_contiguous(unsigned int bitmask) +{ + if (bitmask) + bitmask >>= ffs(bitmask) - 1; + + return is_power_of_2(bitmask + 1); +} + static void skl_ddb_entry_for_slices(struct drm_i915_private *dev_priv, u8 slice_mask, struct skl_ddb_entry *ddb) @@ -3844,6 +3852,7 @@ skl_ddb_entry_for_slices(struct drm_i915_private *dev_priv, u8 slice_mask, WARN_ON(ddb->start >= ddb->end); WARN_ON(ddb->end > intel_dbuf_size(dev_priv)); + WARN_ON(!bitmask_is_contiguous(slice_mask)); } static unsigned int intel_crtc_ddb_weight(const struct intel_crtc_state *crtc_state) -- 2.24.1 _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx