Quoting Chris Wilson (2021-03-21 16:28:07) > Quoting Ville Syrjala (2021-03-21 16:10:38) > > diff --git a/drivers/gpu/drm/i915/gem/i915_gem_mman.c b/drivers/gpu/drm/i915/gem/i915_gem_mman.c > > index ec28a6cde49b..0b2434e29d00 100644 > > --- a/drivers/gpu/drm/i915/gem/i915_gem_mman.c > > +++ b/drivers/gpu/drm/i915/gem/i915_gem_mman.c > > @@ -189,7 +189,7 @@ compute_partial_view(const struct drm_i915_gem_object *obj, > > struct i915_ggtt_view view; > > > > if (i915_gem_object_is_tiled(obj)) > > - chunk = roundup(chunk, tile_row_pages(obj)); > > + chunk = roundup(chunk, tile_row_pages(obj) ?: 1); > > I was thinking the answer would be to align to the next page, and hey > presto! Wait, the tile row cannot be a single page. Something else is zero that should not be. -Chris _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
