Skip to content

Conversation

@zingo
Copy link
Collaborator

@zingo zingo commented Sep 4, 2025

Summary

If you use run.sh with --etdump e.g. something like this:

examples/arm/run.sh --et_build_root=arm_test/test_run --target=ethos-u55-256 --model_name=mv2 --etdump

The script will dig out the ETDump file from the FVP logs and use it to produce a cycle table automatically.

Please remember the the CorstoneFVP is not a cycle accurate simulator and the CPU cycle numbers can not be trusted, please run on real hardware or FPGA to get valid number. Running it on a FVP is mostly useful to check that the tools/flow work and not the actual CPU cycle numbers.

Test plan

This is tested by the test scripts

cc @digantdesai @freddan80 @per @oscarandersson8218

Signed-off-by: Zingo Andersen <[email protected]>
Change-Id: I97280ae9d0132744a937e6fa0e2e350568d25f4f
@zingo zingo requested a review from digantdesai as a code owner September 4, 2025 11:59
@pytorch-bot
Copy link

pytorch-bot bot commented Sep 4, 2025

🔗 Helpful Links

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

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

❌ 4 New Failures, 1 Pending

As of commit 07c0eb4 with merge base f6d90ef (image):

NEW FAILURES - The following jobs have failed:

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 4, 2025
@zingo zingo added ciflow/trunk partner: arm For backend delegation, kernels, demo, etc. from the 3rd-party partner, Arm release notes: arm Changes to the ARM backend delegate labels Sep 4, 2025
Copy link
Collaborator

@oscarandersson8218 oscarandersson8218 left a comment

Choose a reason for hiding this comment

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

Looks good to me. CI failures seem unrelated.

@oscarandersson8218 oscarandersson8218 merged commit c6a6caa into pytorch:main Sep 5, 2025
289 of 293 checks passed
@zingo zingo deleted the Arm-backend-Auto-generate-cycle-table-if-ETDump-is-used branch September 9, 2025 14:14
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: arm Changes to the ARM backend delegate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants