Skip to content
Open
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 2 additions & 6 deletions ascend-device-configmap.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -77,15 +77,11 @@ data:
aiCore: 20
aiCPU: 7
templates:
# NOTE: Names of vnpu template for 910B4-1 are fixed by Ascend runtime and must not be changed.
# The memory is used for scheduling so the correct values must be set.
# Template vir05_1c_8g actually provides 16GB memory,
- name: vir05_1c_8g
- name: vir05_1c_16g
Copy link
Contributor

Choose a reason for hiding this comment

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

there is a comment above implies Template vir05_1c_8g actually provides 16GB memory. I don't know if there are other reasons for setting it to vir05_1c_8g here. @archlitchi

Copy link
Member

Choose a reason for hiding this comment

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

looks like different driver have different 910B4-1 template name here

Copy link
Member

Choose a reason for hiding this comment

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

as the title suggests, the change of template name occurs after adopting latest driver, so we can adopt this change

Copy link
Contributor

Choose a reason for hiding this comment

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

as the title suggests, the change of template name occurs after adopting latest driver, so we can adopt this change

ok, @greatljn please remove the previous comment together, thanks

memory: 16384
aiCore: 5
aiCPU: 1
# Template vir10_3c_16g actually provides 32GB memory
- name: vir10_3c_16g
- name: vir10_3c_32g
memory: 32768
aiCore: 10
aiCPU: 3
Expand Down