Skip to content

remove the rocprofiler early exit hack#1329

Closed
scotts wants to merge 1 commit intopytorch:mainfrom
scotts:export-D98163400
Closed

remove the rocprofiler early exit hack#1329
scotts wants to merge 1 commit intopytorch:mainfrom
scotts:export-D98163400

Conversation

@scotts
Copy link
Copy Markdown
Contributor

@scotts scotts commented Mar 25, 2026

Summary: Remove the early exit hack because it seems to be doing more harm than good.

Differential Revision: D98163400

Summary: Remove the early exit hack because it seems to be doing more harm than good.

Differential Revision: D98163400
@meta-cla meta-cla Bot added the cla signed label Mar 25, 2026
@meta-codesync
Copy link
Copy Markdown

meta-codesync Bot commented Mar 25, 2026

@scotts has exported this pull request. If you are a Meta employee, you can view the originating Diff in D98163400.

@meta-codesync
Copy link
Copy Markdown

meta-codesync Bot commented Mar 26, 2026

This pull request has been merged in 0c8ede0.

pytorchmergebot pushed a commit to pytorch/pytorch that referenced this pull request Apr 2, 2026
Includes the following commits:

* Expose occupany limiting factors (pytorch/kineto#1330)
* remove the rocprofiler early exit hack (pytorch/kineto#1329)
* Remove duplicate test ignore (pytorch/kineto#1328)
* Ensure that async doesn't loop while sync is active (pytorch/kineto#1327)
* Add host_name to OSS Kineto trace metadata via gethostname() (pytorch/kineto#1323)
* Revert D97166802 (pytorch/kineto#1326)
* Fix Lingering INT32 Overflow (pytorch/kineto#1324)
* Re-enabled some hardcoded tests (pytorch/kineto#1321)
* Expose occupany limiting factors (pytorch/kineto#1322)

Authored with Claude.
Pull Request resolved: #178918
Approved by: https://github.com/divyanshk
scotts added a commit to scotts/pytorch that referenced this pull request Apr 3, 2026
Includes the following commits:

- remove the rocprofiler early exit hack (pytorch/kineto#1329) 0c8ede0
- Remove duplicate test ignore (pytorch/kineto#1328) 4826a43
- Ensure that async doesn't loop while sync is active (pytorch/kineto#1327) 37fada9

Authored with Claude.
@scotts scotts deleted the export-D98163400 branch April 4, 2026 01:07
weifengpy pushed a commit to pytorch/pytorch that referenced this pull request Apr 7, 2026
Includes the following commits:

* Expose occupany limiting factors (pytorch/kineto#1330)
* remove the rocprofiler early exit hack (pytorch/kineto#1329)
* Remove duplicate test ignore (pytorch/kineto#1328)
* Ensure that async doesn't loop while sync is active (pytorch/kineto#1327)
* Add host_name to OSS Kineto trace metadata via gethostname() (pytorch/kineto#1323)
* Revert D97166802 (pytorch/kineto#1326)
* Fix Lingering INT32 Overflow (pytorch/kineto#1324)
* Re-enabled some hardcoded tests (pytorch/kineto#1321)
* Expose occupany limiting factors (pytorch/kineto#1322)

Authored with Claude.
Pull Request resolved: #178918
Approved by: https://github.com/divyanshk
scotts added a commit to scotts/pytorch that referenced this pull request Apr 7, 2026
Includes the following commits:

- remove the rocprofiler early exit hack (pytorch/kineto#1329) 0c8ede0

Authored with Claude.
nklshy-aws pushed a commit to nklshy-aws/pytorch that referenced this pull request Apr 7, 2026
Includes the following commits:

* Expose occupany limiting factors (pytorch/kineto#1330)
* remove the rocprofiler early exit hack (pytorch/kineto#1329)
* Remove duplicate test ignore (pytorch/kineto#1328)
* Ensure that async doesn't loop while sync is active (pytorch/kineto#1327)
* Add host_name to OSS Kineto trace metadata via gethostname() (pytorch/kineto#1323)
* Revert D97166802 (pytorch/kineto#1326)
* Fix Lingering INT32 Overflow (pytorch/kineto#1324)
* Re-enabled some hardcoded tests (pytorch/kineto#1321)
* Expose occupany limiting factors (pytorch/kineto#1322)

Authored with Claude.
Pull Request resolved: pytorch#178918
Approved by: https://github.com/divyanshk
pytorch-bot Bot pushed a commit to pytorch/pytorch that referenced this pull request Apr 8, 2026
Includes the following commits:

- remove the rocprofiler early exit hack (pytorch/kineto#1329) 0c8ede0

Authored with Claude.
scotts added a commit to scotts/pytorch that referenced this pull request Apr 9, 2026
Includes the following commits:

- Fix GPU trace collection on ROCm by ensuring early rocprofiler-sdk tool registration (pytorch/kineto#1338) 4ff89e8
- Remove tb_plugin CI workflows (pytorch/kineto#1341) e335a7d
- Remove INFO logs from kineto trace metadata to fix invalid JSON (pytorch/kineto#1336) 502c513
- Expose occupany limiting factors (pytorch/kineto#1330) 185fe9c
- remove the rocprofiler early exit hack (pytorch/kineto#1329) 0c8ede0

Authored with Claude.
pytorchmergebot pushed a commit to scotts/pytorch that referenced this pull request Apr 9, 2026
Includes the following commits:

- remove the rocprofiler early exit hack (pytorch/kineto#1329) 0c8ede0

Authored with Claude.
pytorchmergebot pushed a commit to pytorch/pytorch that referenced this pull request Apr 9, 2026
Includes the following commits:

- remove the rocprofiler early exit hack (pytorch/kineto#1329) 0c8ede0

Authored with Claude.
Pull Request resolved: #179579
Approved by: https://github.com/ryanzhang22
bobrenjc93 pushed a commit to bobrenjc93/pytorch that referenced this pull request Apr 10, 2026
Includes the following commits:

* Expose occupany limiting factors (pytorch/kineto#1330)
* remove the rocprofiler early exit hack (pytorch/kineto#1329)
* Remove duplicate test ignore (pytorch/kineto#1328)
* Ensure that async doesn't loop while sync is active (pytorch/kineto#1327)
* Add host_name to OSS Kineto trace metadata via gethostname() (pytorch/kineto#1323)
* Revert D97166802 (pytorch/kineto#1326)
* Fix Lingering INT32 Overflow (pytorch/kineto#1324)
* Re-enabled some hardcoded tests (pytorch/kineto#1321)
* Expose occupany limiting factors (pytorch/kineto#1322)

Authored with Claude.
Pull Request resolved: pytorch#178918
Approved by: https://github.com/divyanshk
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.

1 participant