Quoting Mika Kuoppala (2019-11-07 08:32:47) > Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> writes: > > > The hidden aliasing-ppgtt's size is never revealed, as we only inspect > > the front GTT when engaged. However, we were "fixing" the hidden ppgtt > > to match, with the net result that we ended up leaking the unused > > portion (on Braswell were we preallocated the entire range). > > On here I think that the comment is a bit misleading. We preallocate > the pdps upfront, the va_ranges that the aliasing init does matches > in size. > > As there is two types of 'preallocation' at play in here, > the commit message should be amended to explicitly state that the top > pdps leak. You knew what I meant :-p Easiest way would to be use gen8_preallocate_top_level_pdp() instead. -Chris _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
