whole series is Reviewed-by: José Roberto de Souza <[email protected]>
On Thu, 2021-06-10 at 21:32 +0300, Ville Syrjala wrote: > From: Ville Syrjälä <[email protected]> > > Since the llb allocation has a fixed size, let's grab it before > the potentially variable sized cfb. That should avoid some allocation > failure cases once we allow different compression ratios for FBC1. > > Signed-off-by: Ville Syrjälä <[email protected]> > --- > drivers/gpu/drm/i915/display/intel_fbc.c | 18 +++++++++--------- > 1 file changed, 9 insertions(+), 9 deletions(-) > > diff --git a/drivers/gpu/drm/i915/display/intel_fbc.c > b/drivers/gpu/drm/i915/display/intel_fbc.c > index b6bfb4439a8b..7dc72e4a4656 100644 > --- a/drivers/gpu/drm/i915/display/intel_fbc.c > +++ b/drivers/gpu/drm/i915/display/intel_fbc.c > @@ -492,6 +492,13 @@ static int intel_fbc_alloc_cfb(struct drm_i915_private > *dev_priv, > drm_WARN_ON(&dev_priv->drm, > drm_mm_node_allocated(&fbc->compressed_llb)); > > + if (DISPLAY_VER(dev_priv) < 5 && !IS_G4X(dev_priv)) { > + ret = i915_gem_stolen_insert_node(dev_priv, > &fbc->compressed_llb, > + 4096, 4096); > + if (ret) > + goto err; > + } > + > ret = find_compression_limit(dev_priv, size, fb_cpp); > if (!ret) > goto err_llb; > @@ -502,22 +509,15 @@ static int intel_fbc_alloc_cfb(struct drm_i915_private > *dev_priv, > > fbc->limit = ret; > > - if (DISPLAY_VER(dev_priv) < 5 && !IS_G4X(dev_priv)) { > - ret = i915_gem_stolen_insert_node(dev_priv, > &fbc->compressed_llb, > - 4096, 4096); > - if (ret) > - goto err_fb; > - } > - > drm_dbg_kms(&dev_priv->drm, > "reserved %llu bytes of contiguous stolen space for FBC, > limit: %d\n", > fbc->compressed_fb.size, fbc->limit); > > return 0; > > -err_fb: > - i915_gem_stolen_remove_node(dev_priv, &fbc->compressed_fb); > err_llb: > + i915_gem_stolen_remove_node(dev_priv, &fbc->compressed_llb); > +err: > if (drm_mm_initialized(&dev_priv->mm.stolen)) > drm_info_once(&dev_priv->drm, "not enough stolen space for > compressed buffer (need %d more bytes), disabling. Hint: you may be able to > increase stolen memory size in the BIOS to avoid this.\n", size); > return -ENOSPC; _______________________________________________ Intel-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/intel-gfx
