Re: [PATCH] drm/xe: Force write completion of MI_STORE_DATA_IMM

["Dixit, Ashutosh" <ashutosh.dixit@intel.com>]

On Thu, 19 Dec 2024 05:58:59 -0800, Souza, Jose wrote:
>
> On Wed, 2024-12-18 at 18:05 -0800, Dixit, Ashutosh wrote:
> > On Wed, 18 Dec 2024 09:27:36 -0800, Matthew Brost wrote:
> > >
> >
> > Folks,
> >
> > > On Wed, Dec 18, 2024 at 08:38:49AM -0700, Souza, Jose wrote:
> > > > On Tue, 2024-12-17 at 15:39 -0800, Matthew Brost wrote:
> > > > > On Tue, Dec 17, 2024 at 08:07:32AM -0800, José Roberto de Souza wrote:
> > > > > > With Force write completion unset there is no guarantees of when the
> > > > > > write will be globally visible what is not the behavior wanted.
> > > > > >
> > > > >
> > > > > Do we want this backported? If so, maybe add a fixes?
> > > >
> > > > Not sure, I don't have an actual issue that is fixed by this but I
> > > > think would be good to have it backported.  But what do you
> > > > suggest? Add a fixes tag to the patch removing force probe from
> > > > LNL?
> > > >
> > >
> > > Yea fixing LNL force probe removal sounds reasonable to me.
> >
> > Hmm the plan was add Fixes to this patch and also Cc: stable. Yet it was
> > merged without these:
> >
> > 1460bb1fef9c ("drm/xe: Force write completion of MI_STORE_DATA_IMM")
> >
> > Lucas/Thomas,
> >
> > Would it be possible to send it to -fixes with a Cc:stable. We have patches
> > depending on this one which we want to send to stable.
>
> Please correct me if I'm wrong but that is enough for it to be picked to
> stable, it will happen in the next Xe stable pull request.

Ah, ok, sorry I missed that there is a Fixes: in the merged patch, so maybe
that is sufficient for it to be picked to stable.