Skip to content

Conversation

gggekov
Copy link
Collaborator

@gggekov gggekov commented Sep 11, 2025

If running multiple inferences on the same Ethos-U custom delegate, we need to reset the temp_allocator after every inference. Otherwise, we keep allocating more memory within the temp_allocator rather than reuse the temp_allocator between inferences.

cc @digantdesai @freddan80 @per @zingo @oscarandersson8218

If running multiple inferences on the same Ethos-U custom delegate,
we need to reset the temp_allocator after every inference. Otherwise,
we keep allocating more memory within the temp_allocator rather than
reuse the temp_allocator between inferences.

Change-Id: Id3233bb8b635b83868751c7c619c6d418ada58ec
Copy link

pytorch-bot bot commented Sep 11, 2025

🔗 Helpful Links

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

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

❌ 11 New Failures, 7 Cancelled Jobs, 3 Unrelated Failures

As of commit 8abb465 with merge base 4d6209b (image):

NEW FAILURES - The following jobs have failed:

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

FLAKY - The following job failed but was likely due to flakiness present on trunk:

BROKEN TRUNK - The following jobs failed but was 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.

@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 Sep 11, 2025
@gggekov gggekov added partner: arm For backend delegation, kernels, demo, etc. from the 3rd-party partner, Arm ciflow/trunk release notes: none Do not include this in the release notes labels Sep 11, 2025
@zingo zingo added this to the 1.0.0 milestone Sep 11, 2025
@facebook-github-bot
Copy link
Contributor

@mergennachin has imported this pull request. If you are a Meta employee, you can view this in D82220111.

@facebook-github-bot facebook-github-bot merged commit 8358516 into pytorch:main Sep 11, 2025
290 of 319 checks passed
StrycekSimon pushed a commit to nxp-upstream/executorch that referenced this pull request Sep 23, 2025
Differential Revision: D82220111

Pull Request resolved: pytorch#14210
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ciflow/trunk CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. partner: arm For backend delegation, kernels, demo, etc. from the 3rd-party partner, Arm release notes: none Do not include this in the release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants