Skip to content

[TSAN][live speech translation] Fix A data race in caffe2#156378

Closed
Polyomino wants to merge 1 commit intopytorch:mainfrom
Polyomino:export-D76921416
Closed

[TSAN][live speech translation] Fix A data race in caffe2#156378
Polyomino wants to merge 1 commit intopytorch:mainfrom
Polyomino:export-D76921416

Conversation

@Polyomino
Copy link
Contributor

Summary: noticed that context quantized_engine is accessed and written from multiple threads

Test Plan:
➜ fbsource buck test --flagfile fbcode/mode/dev-tsan //xplat/assistant/integration_test/tests/supernova/speechtranslation:live_speech_translation_en_fr_tests -- --exact 'fbsource//xplat/assistant/integration_test/tests/supernova/speechtranslation:live_speech_translation_en_fr_tests - Translate/LiveSpeechTranslationTests.LiveSpeechTranslationEnFr/silence___fr_en'

Rollback Plan:

Differential Revision: D76921416

@pytorch-bot
Copy link

pytorch-bot bot commented Jun 18, 2025

This appears to be a diff that was exported from phabricator, but the PR author does not have sufficient permissions to run CI. @Polyomino, please do step 2 of internal wiki to get write access so you do not need to get CI approvals in the future. If you think this is a mistake, please contact the Pytorch Dev Infra team.

@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Jun 18, 2025

CLA Signed

The committers listed above are authorized under a signed CLA.

  • ✅ login: Polyomino / name: Patryk Ozga (49d23ab)

@pytorch-bot
Copy link

pytorch-bot bot commented Jun 18, 2025

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/156378

Note: Links to docs will display an error until the docs builds have been completed.

❌ 2 Cancelled Jobs

As of commit 49d23ab with merge base 9fe2d15 (image):

CANCELLED JOBS - The following jobs were cancelled. Please retry:

This comment was automatically generated by Dr. CI and updates every 15 minutes.

@facebook-github-bot
Copy link
Contributor

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

@Polyomino
Copy link
Contributor Author

/easycla

@jerryzh168
Copy link
Contributor

can you fix failed tests?

@facebook-github-bot
Copy link
Contributor

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

@facebook-github-bot
Copy link
Contributor

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

@facebook-github-bot
Copy link
Contributor

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

@Polyomino
Copy link
Contributor Author

/easycla

@Polyomino
Copy link
Contributor Author

@pytorchbot label "topic: not user facing"

@pytorch-bot pytorch-bot bot added the topic: not user facing topic category label Jun 20, 2025
@facebook-github-bot
Copy link
Contributor

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

@facebook-github-bot
Copy link
Contributor

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

@Polyomino Polyomino force-pushed the export-D76921416 branch 2 times, most recently from 33c790c to 54dd18a Compare June 26, 2025 05:48
@facebook-github-bot
Copy link
Contributor

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

@Polyomino Polyomino force-pushed the export-D76921416 branch 2 times, most recently from e6f6c84 to 1453803 Compare June 26, 2025 18:01
…6378)

Summary:
noticed that context quantized_engine is accessed and written from multiple threads

Pull Request resolved: pytorch#156378

Test Plan:
➜  fbsource buck test --flagfile fbcode/mode/dev-tsan //xplat/assistant/integration_test/tests/supernova/speechtranslation:live_speech_translation_en_fr_tests -- --exact 'fbsource//xplat/assistant/integration_test/tests/supernova/speechtranslation:live_speech_translation_en_fr_tests - Translate/LiveSpeechTranslationTests.LiveSpeechTranslationEnFr/silence___fr_en'

Rollback Plan:

Rollback Plan:

Reviewed By: jerryzh168

Differential Revision: D76921416

Pulled By: Polyomino
@facebook-github-bot
Copy link
Contributor

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

@facebook-github-bot
Copy link
Contributor

@pytorchbot merge

(Initiating merge automatically since Phabricator Diff has merged)

@pytorch-bot
Copy link

pytorch-bot bot commented Jun 27, 2025

Pull workflow has not been scheduled for the PR yet. It could be because author doesn't have permissions to run those or skip-checks keywords were added to PR/commits, aborting merge. Please get/give approval for the workflows and/or remove skip ci decorators before next merge attempt. If you think this is a mistake, please contact PyTorch Dev Infra.

@pytorch-bot pytorch-bot bot added the ciflow/trunk Trigger trunk jobs on your pull request label Jun 29, 2025
@cyyever
Copy link
Collaborator

cyyever commented Jun 29, 2025

@pytorchbot merge -i

@pytorchmergebot
Copy link
Collaborator

Merge started

Your change will be merged while ignoring the following 0 checks:

Learn more about merging in the wiki.

Questions? Feedback? Please reach out to the PyTorch DevX Team

Advanced Debugging
Check the merge workflow status
here

@pytorchmergebot
Copy link
Collaborator

Merge failed

Reason: 1 jobs have failed, first few of them are: trunk / cuda12.8-py3.10-gcc9-sm80 / build

Details for Dev Infra team Raised by workflow job

@cyyever
Copy link
Collaborator

cyyever commented Jun 29, 2025

@pytorchbot merge -i

@pytorchmergebot
Copy link
Collaborator

Merge started

Your change will be merged while ignoring the following 2 checks: trunk / cuda12.8-py3.10-gcc9-sm80 / build, trunk / linux-jammy-rocm-py3.10 / test (default, 2, 2, linux.rocm.gpu.2)

Learn more about merging in the wiki.

Questions? Feedback? Please reach out to the PyTorch DevX Team

Advanced Debugging
Check the merge workflow status
here

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

Labels

ciflow/trunk Trigger trunk jobs on your pull request fb-exported Merged topic: not user facing topic category

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants