Skip to content

Limit the grid size for the TBE forward kernel #4208

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

Closed
wants to merge 1 commit into from

Conversation

q10
Copy link
Contributor

@q10 q10 commented May 29, 2025

Summary:

  • Limit the grid size for the TBE forward kernel, as we are observing thread
    counts over 2^32 for AMD runs

Reviewed By: yoyoyocmu, r-barnes

Differential Revision: D75543767

Copy link

netlify bot commented May 29, 2025

Deploy Preview for pytorch-fbgemm-docs ready!

Name Link
🔨 Latest commit 8063381
🔍 Latest deploy log https://app.netlify.com/projects/pytorch-fbgemm-docs/deploys/683a0203688ff20008733acc
😎 Deploy Preview https://deploy-preview-4208--pytorch-fbgemm-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D75543767

Summary:
X-link: facebookresearch/FBGEMM#1283

- Limit the grid size for the TBE forward kernel, as we are observing thread
  counts over 2^32 for AMD runs

Reviewed By: yoyoyocmu, r-barnes

Differential Revision: D75543767
@q10 q10 force-pushed the export-D75543767 branch from c1323b7 to 8063381 Compare May 30, 2025 19:07
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D75543767

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in 5b75aec.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants