Re: [PATCH 6.12 182/189] drm/xe/oa: Separate batch submission from waiting for completion

On Wed, Jan 15, 2025 at 10:20:51AM -0800, Dixit, Ashutosh wrote:

On Wed, 15 Jan 2025 02:37:58 -0800, Greg Kroah-Hartman wrote:

...

Hi Greg,

...

6.12-stable review patch. If anyone has any objections, please let me know.

...

This is a note to let you know that I've just added the patch titled

drm/xe/oa: Separate batch submission from waiting for completion

to the 6.12-stable tree which can be found at: http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git%3Ba=su...

The filename of the patch is: drm-xe-oa-separate-batch-submission-from-waiting-for.patch and it can be found in the queue-6.12 subdirectory.

If you, or anyone else, feels it should not be added to the stable tree, please let stable@vger.kernel.org know about it.

I am writing about 3 emails I received (including this one) about 3 commits being added to stable. These are the 3 commits I am referring to (all commit SHA's refer to Linus' tree and first commit is at the bottom, everywhere in this email):

2fb4350a283af drm/xe/oa: Add input fence dependencies
c8507a25cebd1 drm/xe/oa/uapi: Define and parse OA sync properties
dddcb19ad4d4b drm/xe/oa: Separate batch submission from waiting for completion

Apparently these are added to stable to avoid conflicts with this commit:

f0ed39830e606 xe/oa: Fix query mode of operation for OAR/OAC

However, the 3 commits are part of a 7 commit series and are incomplete in themeselves and will break userspace. So we should add the remaining 4 commits of the series to stable too. Thes are the ones:

85d3f9e84e062 drm/xe/oa: Allow only certain property changes from config
9920c8b88c5cf drm/xe/oa: Add syncs support to OA config ioctl
cc4e6994d5a23 drm/xe/oa: Move functions up so they can be reused for config ioctl
343dd246fd9b5 drm/xe/oa: Signal output fences

The above list can be generated by running the following in Linus' tree:

git log --oneline -- drivers/gpu/drm/xe/xe_oa.c

For now I've just dropped all of these commits, can someone send a series in the correct order, or a properly backported fix of the original bugfix, and we can take them that way.

thanks,

greg k-h