Quoting Matthew Auld (2021-01-26 10:38:09)
> On 26/01/2021 10:09, Tvrtko Ursulin wrote:
> >
> > On 26/01/2021 09:46, Matthew Auld wrote:
> >> +struct intel_memory_region *
> >> +i915_gem_setup_lmem(struct drm_i915_private *i915)
> >> +{
> >> + return setup_lmem(i915);
> >> +}
> >
> > Was it ever discussed if there was an easy way (and if it makes sense
> > actually) to move this from GEM to GT (in name and input parameter?
>
> Yeah, makes sense, especially since this is now in gt/
>
> So just:
> intel_gt_setup_lmem(gt)
The implication is that the memory region should be added during the GT
probe, so some jiggling around required. (So as each GT comes online,
we add it's available memory to the pool.)
Shouldn't be too bad as conceptually it's just moving the memprobe down
a couple of lines in i915_driver_hw_probe :)
-Chris
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
