Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[DeviceASAN][bugfix] Not allow concurrent kernel launches across different queue #2679

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

yingcong-wu
Copy link
Contributor

@yingcong-wu yingcong-wu commented Feb 8, 2025

Not allow concurrent kernel launches across different queues as the local/private shadow memory implementation does not support it.

intel/llvm PR: intel/llvm#16935

@yingcong-wu yingcong-wu requested a review from a team as a code owner February 8, 2025 02:22
@yingcong-wu yingcong-wu added the v0.11.x Include in the v0.11.x release label Feb 8, 2025
@github-actions github-actions bot added the loader Loader related feature/bug label Feb 8, 2025
@yingcong-wu yingcong-wu changed the title [DeviceASAN][bugfix] Not allow concurrent kernel launches across different queue. [DeviceASAN][bugfix] Not allow concurrent kernel launches across different queue Feb 8, 2025
Copy link
Contributor

@zhaomaosu zhaomaosu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@yingcong-wu yingcong-wu force-pushed the yc/0208-ur-dasan-bugfix-main branch from efe628f to 029ef10 Compare February 8, 2025 03:25
@yingcong-wu yingcong-wu requested review from a team as code owners February 8, 2025 03:25
@yingcong-wu yingcong-wu marked this pull request as draft February 8, 2025 03:26
@yingcong-wu yingcong-wu force-pushed the yc/0208-ur-dasan-bugfix-main branch from 029ef10 to fd60baf Compare February 8, 2025 03:27
@yingcong-wu yingcong-wu marked this pull request as ready for review February 8, 2025 03:27
@yingcong-wu
Copy link
Contributor Author

yingcong-wu commented Feb 8, 2025

Current UR head will have CI failures because #2606 is merged but its intel/llvm part intel/llvm#16747 is not merged.
Rebased to pervious commit to run the CI properly.
Sorry for any review request sent by my pervious mistake git operation.

@yingcong-wu
Copy link
Contributor Author

From the CI steps, it seems my PR will be merged to latest UR head before building (instead of directly using my branch). Therefore there is nothing I can do here.

Copy link
Contributor

@pbalcer pbalcer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm, just one nit.

@pbalcer pbalcer removed request for a team February 10, 2025 08:02
@pbalcer pbalcer removed request for a team, frasercrmck and reble February 10, 2025 08:02
@pbalcer
Copy link
Contributor

pbalcer commented Feb 10, 2025

I removed all other required reviewers since this only touches the sanitizer layer.

@pbalcer pbalcer added the ready to merge Added to PR's which are ready to merge label Feb 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
loader Loader related feature/bug ready to merge Added to PR's which are ready to merge v0.11.x Include in the v0.11.x release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants