-
Notifications
You must be signed in to change notification settings - Fork 17
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
An issue about Multi-Socket sample code #33
Comments
Hi @Sean58238, could you please clarify, have you commented out this portion of the code for all of setup1-4 runs with Config 2? |
While we're waiting for the clarification, I'll try to provide some guidance, maybe this would help to resolve misunderstanding. Here is the documentation chapter w.r.t NUMA support in DML. It says:
In the example, by default, If Hope this helps, let me know. |
hi @Sean58238 does this clarifies things for you? |
yes, setup1-4 runs with Config 2, this issue happened on this config |
Issue descript:
Multi-Socket sample code default set socket number = 4 , running on different config/SKU has some different results
#define SOCKET_COUNT 4u
Config 1:
CPU: Intel(R) Xeon(R) Platinum 8490H
Socket : 2
DSA device per Socket: 4
Enable 1 device: dsa0 (on socket0)
Both SOCKET_COUNT equal to 1~4 can running successful.
Config 2:
CPU: Intel(R) Xeon(R) Platinum 8470
Socket : 2
DSA device per Socket: 1
setup1: // error failed to submit to node0
Enable 1 device: dsa0 (on socket0)
SOCKET_COUNT=4
setup2: // error failed to submit to node1
Enable 1 device: dsa0 (on socket0)
Enable 1 device: dsa1 (on socket1)
SOCKET_COUNT=4
setup3: // successful
Enable 1 device: dsa0 (on socket0)
Enable 1 device: dsa1 (on socket1)
SOCKET_COUNT=2
setup4: // successful
Enable 1 device: dsa0 (on socket0)
SOCKET_COUNT=4
Commented out code: current_job->numa_id = i
Why has these different results , does any logic issue about numa node check of DML?
The text was updated successfully, but these errors were encountered: