Re: [PATCH] drm/i915/gt: Mask media GuC interrupts

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Daniele,

> > MTL features a dedicated media engine that operates on its
> > independent GT, requiring activation of its specific interrupt
> > set.
> > 
> > Enable the necessary interrupts in a single action when the media
> > engine is present, bypassing the need to iterate through all the
> > GTs.
> > 
> > Signed-off-by: Andi Shyti <andi.shyti@xxxxxxxxxxxxxxx>
> > ---
> > Hi,
> > 
> > I'm starting with v0 as this patch is very different from the
> > ones proposed recently.
> > 
> > After all the discussions on this patch, I took Matt's suggestion
> > since it seemed the most immediate.
> > 
> > However, in the long run, I agree that we should have a
> > specific mtl_ function for enabling interrupts.
> > 
> > Thank you Matt and Daniele for your input.
> > 
> > If this patch hasn't missed anything, is it too optimistic to
> > expect MTL to boot? :-)
> 
> The GSC engine also has interrupts tied to the media GT and those are
> conditional, so that engine won't work with just this patch. The system
> might boot because the GSC engine gets disabled if the FW is not there, but
> IMO if we want a single function to handle both GTs we need to do it proper
> support for the engines and not hack around just the GuC.

yeah... we are already having too many things to handle and at
this point I don't see any better way to handle this other than
using for_each_gt() as it was first sent.

Besides, they are different GT's, why not using for_each_gt?

Thank you, Daniele,
Andi



[Index of Archives]     [AMD Graphics]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux