SWDEV-559166 - Fix data races in GetSubmissionBatch CaptureAndSet and…

[iassiour]

… SetQueueStatus

Motivation

Technical Details

Test Plan

Test Result

Submission Checklist

• Look over the contributing guidelines at https://github.com/ROCm/ROCm/blob/develop/CONTRIBUTING.md#pull-requests.

[Copilot]

Pull Request Overview

This PR addresses data races in the GetSubmissionBatch, CaptureAndSet, and SetQueueStatus functions by moving initialization logic and adding thread synchronization. The changes focus on resolving race conditions by relocating the execInfoOffset_ assignment and parameter descriptor modifications to thread-safe locations, and protecting access to the submission batch with a lock.

Key Changes

• Moved execInfoOffset_ initialization from captureAndSet() to the copy constructor to avoid race conditions
• Removed parameter descriptor modifications (rawPointer_ and defined_ flags) from captureAndSet() and moved defined_ flag initialization to InitParameters()
• Added thread synchronization to GetSubmissionBatch() and changed isActive_ to atomic type

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File	Description
projects/clr/rocclr/platform/kernel.hpp	Adds execInfoOffset_ initialization in copy constructor
projects/clr/rocclr/platform/kernel.cpp	Removes state-modifying operations from captureAndSet() method
projects/clr/rocclr/platform/commandqueue.hpp	Adds lock protection to GetSubmissionBatch() and makes isActive_ atomic
projects/clr/rocclr/device/devkernel.cpp	Moves defined_ flag initialization to InitParameters()

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}