Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
[SYCL][CUDA] Implement sycl_ext_oneapi_peer_access extension #8303
[SYCL][CUDA] Implement sycl_ext_oneapi_peer_access extension #8303
Changes from 30 commits
5d788c6
8685475
de16f88
b5f9481
64ecf25
15d4bf6
a35294f
df55a69
ddca3c3
c3a2009
f0f448d
1855367
644c880
e5b421e
6f45d53
1849619
aa7a7eb
3380230
5f6360c
e33a578
32d714a
7d12cd7
4cf5999
63f23e5
aaec286
76245f2
55a9b6a
80dfb3a
76f6772
85949f7
65aa452
ea7e855
eae9b12
c825b9c
ae358bb
ce81826
72aca2e
c2579aa
0c87405
dfcb01c
b84da4d
9e5408b
57c5747
029c1fe
8d41240
b86954d
a6f7602
cc5eb63
716bf30
484cf25
333b7d6
308f45a
8b8cbc8
b2b6fcb
2bea80b
04b8610
d0b138c
3e488c4
c1905c6
19d2a7d
f2a2026
37aecb1
e80ff9b
8956f50
b987dba
6724a40
69a6506
11ca2a6
8b9c2ab
0a09023
f1eb0f0
40e93d0
e5e967f
92e1420
f0993b9
28028ac
6ed3213
512d2a8
1e088a1
80dd26a
9dc099c
b191914
5d83aeb
c389980
8bd6b60
5e7d821
ab3ac25
47acd23
4ab6215
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we need to define the returned value (and type) for these queries, is it pi_bool aka uint32_t?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See the definition of
cuDeviceGetP2PAttribute
here : https://docs.nvidia.com/cuda/cuda-driver-api/group__CUDA__PEER__ACCESS.html#group__CUDA__PEER__ACCESS_1g4c55c60508f8eba4546b51f2ee545393This function is used to get the query result. Although currently we only support queries which can be represented as booleans there is e.g.
CU_DEVICE_P2P_ATTRIBUTE_PERFORMANCE_RANK
which returns a relative integer value. This is why the return type is integer and not bool for future proofing.L0 backend should decide whether it also will require such attributes at some point, and then whether SYCL (and in turn UR) will want to support them.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am fine with it being integer, but ask this to be documented (comments in pi.h)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OK I forgot the to document the return type. I will do this now. Note that this is a comment so won't affect the test status of this PR that is all green (everything passing.)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Rather than
die
, shouldn't we return some sort of "false" status, indicating that P2P isn't available (yet)? That way we can document this extension as "supported", and we can enable end-to-end tests on all backends.Same for the other backends.