Skip to content

Track updated rows in SSDTBE #4211

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

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

Conversation

chouxi
Copy link
Contributor

@chouxi chouxi commented May 29, 2025

Summary:
X-link: https://github.com/facebookresearch/FBGEMM/pull/1286

Tracking the updated rows inside the prefetch

  • track ids going to be changed in prefetch for L1
  • stream in the prefetch for the rows that are updated in previous iterations for L1
  • stream in sp evict for the rows in SP.

Reviewed By: q10

Differential Revision: D73819098

chouxi added 2 commits May 28, 2025 21:24
Summary:

X-link: facebookresearch/FBGEMM#1285

Added the functions `stream_cuda/stream`.
- When `blocking_tensor_copy=false` Create a new thread to execute the callback registered via the `stream_cuda` the reason as in comment. this is based on [the findings that callbacks are executed in serialized manner]( https://fb.workplace.com/permalink.php?story_fbid=pfbid02s7RWvRZ4g2nS5i42kkyApvLsCbiRpGrdAEPs2p5qr2MDnq5YbgfThQ6PXSB6y13Al&id=100026528794331)
- When `blocking_tensor_copy=true` just copy and enqueue in the callback thread.

Added profile to the callback functions.

Added the function  `stream_sync_cuda`
- To explicitly join the async copy thread to make sure the copy is happened before the buffer got overwritten again.

Reviewed By: q10

Differential Revision: D73819097
Summary:
X-link: facebookresearch/FBGEMM#1286

Tracking the updated rows inside the prefetch
- track ids going to be changed in prefetch for L1
- stream in the prefetch for the rows that are updated in previous iterations for L1
- stream in sp evict for the rows in SP.

Reviewed By: q10

Differential Revision: D73819098
Copy link

netlify bot commented May 29, 2025

Deploy Preview for pytorch-fbgemm-docs ready!

Name Link
🔨 Latest commit 538f5e3
🔍 Latest deploy log https://app.netlify.com/projects/pytorch-fbgemm-docs/deploys/6837e199311fa2000827ae06
😎 Deploy Preview https://deploy-preview-4211--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: D73819098

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