Skip to content

Conversation

trivedivivek
Copy link
Contributor

@trivedivivek trivedivivek commented Aug 4, 2025

Stack from ghstack (oldest at bottom):

This diff introduces a new configuration option, execute_max_cmds, to limit the maximum number of command buffers created when splitting execution between multiple command buffers. This feature allows for more efficient management of command buffers, particularly in scenarios where the number of nodes in the graph is large.

Differential Revision: D79575908

…iff adds a config to limit the maximum number of command buffers created when splitting execution between multiple command buffers.

This diff introduces a new configuration option, `execute_max_cmds`, to limit the maximum number of command buffers created when splitting execution between multiple command buffers. This feature allows for more efficient management of command buffers, particularly in scenarios where the number of nodes in the graph is large.

Differential Revision: [D79575908](https://our.internmc.facebook.com/intern/diff/D79575908/)

[ghstack-poisoned]
Copy link

pytorch-bot bot commented Aug 4, 2025

🔗 Helpful Links

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

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

❌ 2 New Failures, 1 Unrelated Failure

As of commit 0710f6c with merge base 0a2bf93 (image):

NEW FAILURES - The following jobs have failed:

BROKEN TRUNK - The following job failed but were present on the merge base:

👉 Rebase onto the `viable/strict` branch to avoid these failures

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

trivedivivek added a commit that referenced this pull request Aug 4, 2025
…iff adds a config to limit the maximum number of command buffers created when splitting execution between multiple command buffers.

This diff introduces a new configuration option, `execute_max_cmds`, to limit the maximum number of command buffers created when splitting execution between multiple command buffers. This feature allows for more efficient management of command buffers, particularly in scenarios where the number of nodes in the graph is large.

Differential Revision: [D79575908](https://our.internmc.facebook.com/intern/diff/D79575908/)

ghstack-source-id: 300619367
Pull Request resolved: #13113
@meta-cla meta-cla bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Aug 4, 2025
@facebook-github-bot
Copy link
Contributor

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

…ers. This diff adds a config to limit the maximum number of command buffers created when splitting execution between multiple command buffers."

This diff introduces a new configuration option, `execute_max_cmds`, to limit the maximum number of command buffers created when splitting execution between multiple command buffers. This feature allows for more efficient management of command buffers, particularly in scenarios where the number of nodes in the graph is large.

Differential Revision: [D79575908](https://our.internmc.facebook.com/intern/diff/D79575908/)

[ghstack-poisoned]
trivedivivek added a commit that referenced this pull request Aug 6, 2025
…iff adds a config to limit the maximum number of command buffers created when splitting execution between multiple command buffers.

Pull Request resolved: #13113

This diff introduces a new configuration option, `execute_max_cmds`, to limit the maximum number of command buffers created when splitting execution between multiple command buffers. This feature allows for more efficient management of command buffers, particularly in scenarios where the number of nodes in the graph is large.
ghstack-source-id: 301208253
@exported-using-ghexport

Differential Revision: [D79575908](https://our.internmc.facebook.com/intern/diff/D79575908/)
@facebook-github-bot
Copy link
Contributor

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

…ers. This diff adds a config to limit the maximum number of command buffers created when splitting execution between multiple command buffers."

This diff introduces a new configuration option, `execute_max_cmds`, to limit the maximum number of command buffers created when splitting execution between multiple command buffers. This feature allows for more efficient management of command buffers, particularly in scenarios where the number of nodes in the graph is large.

Differential Revision: [D79575908](https://our.internmc.facebook.com/intern/diff/D79575908/)

[ghstack-poisoned]
trivedivivek added a commit that referenced this pull request Aug 7, 2025
…iff adds a config to limit the maximum number of command buffers created when splitting execution between multiple command buffers.

Pull Request resolved: #13113

This diff introduces a new configuration option, `execute_max_cmds`, to limit the maximum number of command buffers created when splitting execution between multiple command buffers. This feature allows for more efficient management of command buffers, particularly in scenarios where the number of nodes in the graph is large.
ghstack-source-id: 301393815
@exported-using-ghexport

Differential Revision: [D79575908](https://our.internmc.facebook.com/intern/diff/D79575908/)
@facebook-github-bot
Copy link
Contributor

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

@trivedivivek trivedivivek added the release notes: vulkan Changes to the Vulkan backend delegate label Aug 7, 2025
@facebook-github-bot facebook-github-bot merged commit 4439b8f into gh/trivedivivek/129/base Aug 7, 2025
101 of 105 checks passed
@facebook-github-bot facebook-github-bot deleted the gh/trivedivivek/129/head branch August 7, 2025 22:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported release notes: vulkan Changes to the Vulkan backend delegate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants