Skip to content

Support peer access DPC++ extension #2077

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.

Sign up for GitHub

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

Merged
ndgrigorian merged 29 commits into from
May 15, 2025
Merged

Support peer access DPC++ extension #2077

Changes from 1 commit
Commits
Show all changes
29 commits
Select commit Hold shift + click to select a range
59d6646
Implement dpctl.SyclDevice peer access
ndgrigorian May 3, 2025
7d2d20f
Add backend validation for peer access
ndgrigorian May 4, 2025
fc50e3b
Directly use `get_backend()` from sycl_device
ndgrigorian May 5, 2025
e08c2c9
Tweak docstrings
ndgrigorian May 6, 2025
3af91a8
Simplify logic checking backends for peer access
ndgrigorian May 6, 2025
dfca713
link method for disabling peer access to enabling method
ndgrigorian May 6, 2025
bdfee1b
Fix copy paste errors in peer device interface header
ndgrigorian May 6, 2025
4b3e9f1
Clarify which device has invalid backend in peer_access methods
ndgrigorian May 6, 2025
a21d585
Use ostringstream to output backend names instead of integers
ndgrigorian May 6, 2025
b8c899e
Use ostringstream in kernel bundle interface
ndgrigorian May 6, 2025
bb1f1d9
Adds Python tests for peer_access methods
ndgrigorian May 7, 2025
7106845
parametrize test_peer_device_arg_validation
ndgrigorian May 7, 2025
8563cb0
Factor out peer access validation
ndgrigorian May 12, 2025
9e7031e
Factor common code out of C-API peer access functions
ndgrigorian May 12, 2025
da45cdb
Add C-API tests for peer access functions
ndgrigorian May 12, 2025
ba7222f
Add check that backend types are the same in _CallPeerAccess
ndgrigorian May 12, 2025
417889d
Inline _CallPeerAccess helper function
ndgrigorian May 13, 2025
59377b4
Inline _check_peer_access helper function
ndgrigorian May 13, 2025
09a1ef6
Clean up peer access functions in libsyclinterface
ndgrigorian May 14, 2025
eff82ba
Add helper for raising more specific errors in peer access methods
ndgrigorian May 14, 2025
2fe0c96
Add tests for dpctl peer access enum helper utilities
ndgrigorian May 14, 2025
c76b521
Add an additional libsyclinterface test for peer access to self
ndgrigorian May 14, 2025
629a4b4
Adds a Python test for peer access to self raising ValueError
ndgrigorian May 14, 2025
101366f
Fix typo in peer access tests
ndgrigorian May 14, 2025
178ce96
Change peer access test description
ndgrigorian May 14, 2025
321582f
Fix a typo
ndgrigorian May 14, 2025
56bd37f
Refactor separate can_access_peer methods into a single method
ndgrigorian May 14, 2025
6a34d74
Update docstring of `can_access_peer`
ndgrigorian May 14, 2025
7317df9
Further tweak `can_access_peer` docstring
ndgrigorian May 14, 2025
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
Prev Previous commit
Next Next commit
Clean up peer access functions in libsyclinterface 
Discard unused header for sstream
  • Loading branch information
@ndgrigorian
ndgrigorian committed May 14, 2025
commit 09a1ef640d54406f9222b49d8f753b2fd1b8bced
11 changes: 4 additions & 7 deletions libsyclinterface/source/dpctl_sycl_device_interface.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,6 @@
#include "dpctl_sycl_type_casters.hpp"
#include "dpctl_utils_helper.h"
#include <algorithm>
#include <sstream>
#include <stddef.h>
#include <sycl/sycl.hpp> /* SYCL headers */
#include <utility>
Expand Down Expand Up @@ -956,9 +955,8 @@ void DPCTLDevice_EnablePeerAccess(__dpctl_keep const DPCTLSyclDeviceRef DRef,
}
}
else {
std::ostringstream os;
os << "Given devices do not support peer access";
error_handler(os.str(), __FILE__, __func__, __LINE__);
error_handler("Devices do not support peer access", __FILE__,
__func__, __LINE__);
}
}
return;
Expand All @@ -978,9 +976,8 @@ void DPCTLDevice_DisablePeerAccess(__dpctl_keep const DPCTLSyclDeviceRef DRef,
}
}
else {
std::ostringstream os;
os << "Given devices do not support peer access";
error_handler(os.str(), __FILE__, __func__, __LINE__);
error_handler("Devices do not support peer access", __FILE__,
__func__, __LINE__);
}
}
return;
Expand Down