-
Notifications
You must be signed in to change notification settings - Fork 40
Clean up: remove remaining multi-orchestrator scaffolding #438
Copy link
Copy link
Open
Description
Context
PR #437 hardcoded orchestrators[0] and removed the thread_local pto2_current_orch_idx / pto2_set_orch_thread_idx mechanism. However, several pieces of multi-orchestrator scaffolding remain in the codebase:
Remaining code to remove
PTO2_MAX_ORCH_THREADSconstant (currently 4) in allpto_runtime2.hheadersPTO2Runtime::orchestrators[]array — can become a singlePTO2OrchestratorState orchestrators[1](or a plain member)PTO2Runtime::orch_countfield and its validation logic inpto2_runtime_create_from_smorch_idxparameter passed toorch_func_(args, orch_thread_num_, orch_idx)in executor — always 0 noworch_thread_num_/sched_thread_num_split logic inaicpu_executor.cpp(orchestrator vs scheduler thread roles)- Multi-orchestrator docs references in
SUBMIT_BY_CLUSTER.mdmentioningpto2_current_orch_idx perf_aicpu_set_orch_thread_idx(static __thread) inperformance_collector_aicpu.cpp— same pattern, also uses thread-local
Files
src/a2a3/runtime/aicpu_build_graph/runtime/pto_runtime2.hsrc/a2a3/runtime/tensormap_and_ringbuffer/runtime/pto_runtime2.hsrc/a5/runtime/tensormap_and_ringbuffer/runtime/pto_runtime2.hsrc/a2a3/runtime/aicpu_build_graph/aicpu/aicpu_executor.cppsrc/a2a3/runtime/tensormap_and_ringbuffer/aicpu/aicpu_executor.cppsrc/a5/runtime/tensormap_and_ringbuffer/aicpu/aicpu_executor.cppsrc/{a2a3,a5}/platform/src/aicpu/performance_collector_aicpu.cppsrc/{a2a3,a5}/runtime/tensormap_and_ringbuffer/docs/SUBMIT_BY_CLUSTER.md
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels
Type
Projects
Status
Todo