Hi Matt, ... > > Cc: <stable@xxxxxxxxxxxxxxx> # v6.3+ > > No stable kernels support MTL (even if they have some of the commits, > it's all dead code). We don't want to tag things for stable if they > don't meet the stable kernel requirements. yes, right... how could I have missed that :-D > > When the access to the hardware was added. > > > > BTW, given that currently we hold the forcewake already, is this > > really a fix or is this more looking at the future? Is the Fixes > > tag necessary? > > I'm not 100% sure we hold forcewake in all the cases where we work with > MCR registers. For example, some of the big ones like wa_list_apply() > don't grab forcewake until after we've already acquired the MCR > semaphore. yeah... OK! > > Reviewed-by: Andi Shyti <andi.shyti@xxxxxxxxxxxxxxx> This stands. Thanks, Matt! Andi