Quoting Chris Wilson (2021-03-21 16:30:32) > 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.
Which fortunately does not matter here, as we start with a 2MiB chunk and want to align that to a multiple of tile rows. Still, as you said, something stinks. -Chris _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx
