Skip to content

Arm backend: Bump amount of DDR in corstone320#18315

Closed
perheld wants to merge 1 commit intopytorch:mainfrom
perheld:change-1195001
Closed

Arm backend: Bump amount of DDR in corstone320#18315
perheld wants to merge 1 commit intopytorch:mainfrom
perheld:change-1195001

Conversation

@perheld
Copy link
Collaborator

@perheld perheld commented Mar 19, 2026

Make it the same as corstone300 to be able to run larger models.

Signed-off-by: per.held@arm.com
Change-Id: Ic180de91ec25544a35474e2850d99b0b8b925b59

cc @digantdesai @freddan80 @per @zingo @oscarandersson8218 @mansnils @Sebastian-Larsson @robell

Make it the same as corstone300 to be able to run larger models.

Signed-off-by: per.held@arm.com
Change-Id: Ic180de91ec25544a35474e2850d99b0b8b925b59
Copilot AI review requested due to automatic review settings March 19, 2026 09:04
@perheld perheld requested a review from digantdesai as a code owner March 19, 2026 09:04
@perheld perheld 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 Mar 19, 2026
@pytorch-bot
Copy link

pytorch-bot bot commented Mar 19, 2026

🔗 Helpful Links

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

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

❌ 1 New Failure, 2 Cancelled Jobs, 3 Unrelated Failures

As of commit 2848472 with merge base fb90480 (image):

NEW FAILURE - The following job has failed:

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

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

@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 Mar 19, 2026
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the Corstone-320 executor runner linker script to provide a larger DDR region (matching Corstone-300) so larger models can be linked/loaded for Arm backend bare-metal examples.

Changes:

  • Increase Corstone-320 DDR memory region length from 0x10000000 to 0x60000000.
  • Update the copyright header year range.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

@perheld
Copy link
Collaborator Author

perheld commented Mar 19, 2026

This might not be the way to solve this. I need to investigate this further.

@perheld perheld closed this Mar 19, 2026
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.

2 participants