Re: [PATCH 6.12 4/4] Revert "drm/xe/forcewake: Add a helper xe_force_wake_ref_has_domain()"

[Rodrigo Vivi <rodrigo.vivi@intel.com>]

On Tue, Jul 29, 2025 at 04:51:02PM +0200, Greg KH wrote:
> On Tue, Jul 29, 2025 at 07:05:25PM +0800, Tomita Moeko wrote:
> > This reverts commit deb05f8431f31e08fd6ab99a56069fc98014dbec.
> > 
> > The helper function introduced in the reverted commit is for handling
> > the "refcounted domain mask" introduced in commit a7ddcea1f5ac
> > ("drm/xe: Error handling in xe_force_wake_get()"). Since that API change
> > only exists in 6.13 and later, this helper is unnecessary in 6.12 stable
> > kernel.
> > 
> > Cc: Michal Wajdeczko <michal.wajdeczko@intel.com>
> > Cc: Badal Nilawar <badal.nilawar@intel.com>
> > Cc: Rodrigo Vivi <rodrigo.vivi@intel.com>
> > Cc: Himal Prasad Ghimiray <himal.prasad.ghimiray@intel.com>
> > Signed-off-by: Tomita Moeko <tomitamoeko@gmail.com>
> > ---
> >  drivers/gpu/drm/xe/xe_force_wake.h | 16 ----------------
> >  1 file changed, 16 deletions(-)
> 
> We need acks from the maintainers/developers for all of these before we
> can take the series.  Please work on getting that.

Acked-by: Rodrigo Vivi <rodrigo.vivi@intel.com>

Looking to the history here it looks these patches were ported as
dependency of a fix:

Stable-dep-of: 5dce85fecb87 ("drm/xe: Move the coredump registration to the
worker thread")

However it is definitely not needed and these reverts here end-up with
the right conflict resolution that should had been done in this
commit 5dce85fecb87 ("drm/xe: Move the coredump registration to the
worker thread").

Thanks for the clean-up,
Rodrigo.

> 
> thanks,
> 
> greg k-h