Quoting Matthew Auld (2017-12-06 18:17:30) > In i915_pages_create_for_stolen it probably makes more sense to check if > the range overflows the stolen_usable_size, since the size of dsm will also > include the reserved portion which we can't touch. > > Signed-off-by: Matthew Auld <matthew.auld@xxxxxxxxx> > Cc: Joonas Lahtinen <joonas.lahtinen@xxxxxxxxxxxxxxx> > Cc: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> > Cc: Paulo Zanoni <paulo.r.zanoni@xxxxxxxxx> > --- > drivers/gpu/drm/i915/i915_gem_stolen.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/i915/i915_gem_stolen.c b/drivers/gpu/drm/i915/i915_gem_stolen.c > index 2267af68c3e1..f8c9f0446712 100644 > --- a/drivers/gpu/drm/i915/i915_gem_stolen.c > +++ b/drivers/gpu/drm/i915/i915_gem_stolen.c > @@ -430,7 +430,7 @@ i915_pages_create_for_stolen(struct drm_device *dev, > struct sg_table *st; > struct scatterlist *sg; > > - GEM_BUG_ON(range_overflows(offset, size, resource_size(&dev_priv->dsm))); > + GEM_BUG_ON(range_overflows(offset, size, dev_priv->stolen_usable_size)); I'm not sold. The usable size is a restriction placed on the drm_mm range manager that not everyone may be privy to; certainly when coming from HW that knows only about the stolen region. -Chris _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
