Commit 8f3eb3e
authored
Refactor Memory Allocator from Arm Executor Runner (pytorch#12353)
### Summary
Some applications desire using more minimal memory allocators when
leveraging the existing the exist arm_executor_runner.cpp, e.g.
[here](https://github.com/BujSet/zephyr/blob/executorch-module-integration/samples/modules/executorch/arm/hello_world/src/et_memory_allocator.hpp).
This change decouples a custom memory allocator wrapper from the
executor runner so that apps can choose different implementations as
needed.
### Test plan
Ran the steps of the [Arm EthosU
tutorial](https://docs.pytorch.org/executorch/main/tutorial-arm-ethos-u.html)
to ensure that existing flow still works. I.e. running:
```
./examples/arm/setup.sh --i-agree-to-the-contained-eula
source ...
./examples/arm/run.sh --model_name=add --no_quantize --target=ethos-u55-128
```
still produces the following output:
```
I [executorch:arm_executor_runner.cpp:762 main()] Model executed successfully.
I [executorch:arm_executor_runner.cpp:768 main()] model_pte_program_size: 2064 bytes.
I [executorch:arm_executor_runner.cpp:769 main()] model_pte_loaded_size: 2064 bytes.
I [executorch:arm_executor_runner.cpp:783 main()] method_allocator_used: 320 / 62914560 free: 62914240 ( used: 0 % )
I [executorch:arm_executor_runner.cpp:790 main()] method_allocator_planned: 64 bytes
I [executorch:arm_executor_runner.cpp:794 main()] method_allocator_loaded: 232 bytes
I [executorch:arm_executor_runner.cpp:798 main()] method_allocator_input: 24 bytes
I [executorch:arm_executor_runner.cpp:799 main()] method_allocator_executor: 0 bytes
I [executorch:arm_executor_runner.cpp:802 main()] peak_temp_allocator: 64 / 2097152 free: 2097152 ( used: 0 % )
I [executorch:arm_executor_runner.cpp:812 main()] 1 outputs:
Output[0][0]: (int) 2
Output[0][1]: (int) 2
Output[0][2]: (int) 2
Output[0][3]: (int) 2
Output[0][4]: (int) 2
I [executorch:arm_executor_runner.cpp:952 main()] Program complete, exiting.
I [executorch:arm_executor_runner.cpp:956 main()] ♦
Info: /OSCI/SystemC: Simulation stopped by user.
[warning ][main@0][3440 ns] Simulation stopped by user
[backends/arm/scripts/run_fvp.sh] Simulation complete, 0
Checking for problems in log:
No problems found!
+ set +x
```1 parent c41c433 commit 8f3eb3e
File tree
4 files changed
+83
-53
lines changed- examples/arm/executor_runner
4 files changed
+83
-53
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
566 | 566 | | |
567 | 567 | | |
568 | 568 | | |
569 | | - | |
| 569 | + | |
570 | 570 | | |
571 | 571 | | |
572 | 572 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
19 | 19 | | |
20 | 20 | | |
21 | 21 | | |
| 22 | + | |
22 | 23 | | |
23 | 24 | | |
24 | 25 | | |
| |||
288 | 289 | | |
289 | 290 | | |
290 | 291 | | |
291 | | - | |
292 | | - | |
293 | | - | |
294 | | - | |
295 | | - | |
296 | | - | |
297 | | - | |
298 | | - | |
299 | | - | |
300 | | - | |
301 | | - | |
302 | | - | |
303 | | - | |
304 | | - | |
305 | | - | |
306 | | - | |
307 | | - | |
308 | | - | |
309 | | - | |
310 | | - | |
311 | | - | |
312 | | - | |
313 | | - | |
314 | | - | |
315 | | - | |
316 | | - | |
317 | | - | |
318 | | - | |
319 | | - | |
320 | | - | |
321 | | - | |
322 | | - | |
323 | | - | |
324 | | - | |
325 | | - | |
326 | | - | |
327 | | - | |
328 | | - | |
329 | | - | |
330 | | - | |
331 | | - | |
332 | | - | |
333 | | - | |
334 | | - | |
335 | | - | |
336 | | - | |
337 | | - | |
338 | | - | |
339 | | - | |
340 | | - | |
341 | | - | |
342 | | - | |
343 | 292 | | |
344 | 293 | | |
345 | 294 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
| 8 | + | |
| 9 | + | |
| 10 | + | |
| 11 | + | |
| 12 | + | |
| 13 | + | |
| 14 | + | |
| 15 | + | |
| 16 | + | |
| 17 | + | |
| 18 | + | |
| 19 | + | |
| 20 | + | |
| 21 | + | |
| 22 | + | |
| 23 | + | |
| 24 | + | |
| 25 | + | |
| 26 | + | |
| 27 | + | |
| 28 | + | |
| 29 | + | |
| 30 | + | |
| 31 | + | |
| 32 | + | |
| 33 | + | |
| 34 | + | |
| 35 | + | |
| 36 | + | |
| 37 | + | |
| 38 | + | |
| 39 | + | |
| 40 | + | |
| 41 | + | |
| 42 | + | |
| 43 | + | |
| 44 | + | |
| 45 | + | |
| 46 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
| 8 | + | |
| 9 | + | |
| 10 | + | |
| 11 | + | |
| 12 | + | |
| 13 | + | |
| 14 | + | |
| 15 | + | |
| 16 | + | |
| 17 | + | |
| 18 | + | |
| 19 | + | |
| 20 | + | |
| 21 | + | |
| 22 | + | |
| 23 | + | |
| 24 | + | |
| 25 | + | |
| 26 | + | |
| 27 | + | |
| 28 | + | |
| 29 | + | |
| 30 | + | |
| 31 | + | |
| 32 | + | |
| 33 | + | |
| 34 | + | |
| 35 | + | |
0 commit comments