-
Notifications
You must be signed in to change notification settings - Fork 112
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
[DeviceSanitizer] Support detecting out-of-bounds error on CPU Device & Static Local Memory #1210
Merged
Merged
Changes from 1 commit
Commits
Show all changes
185 commits
Select commit
Hold shift + click to select a range
4c9fe03
Add UR Sanitizer Layer
AllanZyne d6e589b
UR Sanitizer: Add shadow memory alloc & memset
AllanZyne 67fcd5e
[UR][CUDA][L0][HIP] Add virtual memory adapter implementations
steffenlarsen 1096b0f
Remove duplicate CUDA UR_DEVICE_INFO_VIRTUAL_MEMORY_SUPPORT case
steffenlarsen c36bd72
Fix return of check error
steffenlarsen 6ba94ac
Fix common include in cuda device.hpp
steffenlarsen 9ae8a99
Zero-initialize CUmemAccessDesc
steffenlarsen 9f90e43
Add missing newline
steffenlarsen 336ce89
Fix wrongful use of UR_ASSERT in cuda implementation
steffenlarsen 84a3afa
an -> a
steffenlarsen a8dadc1
UR Sanitizer: fix bug
AllanZyne 4d63350
UR Sanitizer: Add membuffer
AllanZyne 25b1183
UR Sanitizer: Add membuffer red zone marker
AllanZyne e6bc306
UR Sanitizer: add membuffer check
AllanZyne 23fb1b3
UR log
AllanZyne cb8a761
UR change directory
AllanZyne 5c44b1d
UR refactor code
AllanZyne a377c94
UR refactor code
AllanZyne 96eea7a
UR: WIP partition membuffer
AllanZyne 0a72ea9
UR: clean code
AllanZyne 8fa86dd
UR: clean code
AllanZyne d9a51a7
Revert "an -> a"
AllanZyne 730bf45
Revert "Fix wrongful use of UR_ASSERT in cuda implementation"
AllanZyne fed541a
UR: refactor
AllanZyne ad2b1a7
Revert "Add missing newline"
AllanZyne dec5f32
Revert "Zero-initialize CUmemAccessDesc"
AllanZyne ca58886
Revert "Fix common include in cuda device.hpp"
AllanZyne 11cca13
Revert "Fix return of check error"
AllanZyne 3e6194e
Revert "Remove duplicate CUDA UR_DEVICE_INFO_VIRTUAL_MEMORY_SUPPORT c…
AllanZyne 89febf0
Revert "[UR][CUDA][L0][HIP] Add virtual memory adapter implementations"
AllanZyne 1b81df9
UR: revert gitignore
AllanZyne 030400d
UR: cmake option
AllanZyne 7208518
UR: clean code
AllanZyne 668a11e
UR: clean code
AllanZyne f623135
UR: clean code
AllanZyne 7be8667
UR: clean code
AllanZyne e4a47ac
UR: clean code
AllanZyne 61c0ca9
UR: llvm license
AllanZyne 0ad1bb1
UR: llvm license
AllanZyne cc4402a
UR: once
AllanZyne bf358c1
UR: clean code
AllanZyne d5e7946
UR: clean includes
AllanZyne 50666b7
UR: clean comment
AllanZyne de03096
UR: rename files
AllanZyne b47ca29
UR: update readme
AllanZyne 1e2b6a0
UR: fix according to PR comments
AllanZyne 42f6755
UR: UR_ENABLE_SANITIZER=ON default
AllanZyne 5aaba09
ignore VS clangd files
AllanZyne 8dc90fd
fix comments
AllanZyne 3b819f6
resolve comments
AllanZyne 54220e9
resolve comments
AllanZyne c814c68
Add UR Sanitizer Layer
AllanZyne a5f31d9
UR Sanitizer: Add shadow memory alloc & memset
AllanZyne 96eea47
[UR][CUDA][L0][HIP] Add virtual memory adapter implementations
steffenlarsen 51f7ac7
Remove duplicate CUDA UR_DEVICE_INFO_VIRTUAL_MEMORY_SUPPORT case
steffenlarsen d15f020
Fix return of check error
steffenlarsen 9bb9a73
Fix common include in cuda device.hpp
steffenlarsen 55a57aa
Zero-initialize CUmemAccessDesc
steffenlarsen 1bfbcbd
Add missing newline
steffenlarsen ef1337f
Fix wrongful use of UR_ASSERT in cuda implementation
steffenlarsen 2215b83
an -> a
steffenlarsen 20a0a78
UR Sanitizer: fix bug
AllanZyne cde417d
UR Sanitizer: Add membuffer
AllanZyne 4cd06d0
UR Sanitizer: Add membuffer red zone marker
AllanZyne 1ce4227
UR Sanitizer: add membuffer check
AllanZyne f9165a7
UR log
AllanZyne 20fb98f
UR change directory
AllanZyne 521b6af
UR refactor code
AllanZyne 07a2a3d
UR refactor code
AllanZyne 5b33e48
UR: WIP partition membuffer
AllanZyne a7a204f
UR: clean code
AllanZyne e42471b
UR: clean code
AllanZyne d2a337f
Revert "an -> a"
AllanZyne 255203e
Revert "Fix wrongful use of UR_ASSERT in cuda implementation"
AllanZyne 9309e1b
UR: refactor
AllanZyne 3e3e070
Revert "Add missing newline"
AllanZyne 1e20c4d
Revert "Zero-initialize CUmemAccessDesc"
AllanZyne 23d90b5
Revert "Fix common include in cuda device.hpp"
AllanZyne d28acd7
Revert "Fix return of check error"
AllanZyne 541c7d2
Revert "Remove duplicate CUDA UR_DEVICE_INFO_VIRTUAL_MEMORY_SUPPORT c…
AllanZyne 12fc508
Revert "[UR][CUDA][L0][HIP] Add virtual memory adapter implementations"
AllanZyne 301547f
UR: revert gitignore
AllanZyne c3114b1
UR: cmake option
AllanZyne 041d50f
UR: clean code
AllanZyne 4734669
UR: clean code
AllanZyne 2fd17ce
UR: clean code
AllanZyne c6d9e08
UR: clean code
AllanZyne 987882d
UR: clean code
AllanZyne 8442d4d
UR: llvm license
AllanZyne 4f35b66
UR: llvm license
AllanZyne 21089cc
UR: once
AllanZyne 69407d4
UR: clean code
AllanZyne 0b767bb
UR: clean includes
AllanZyne 232ae72
UR: clean comment
AllanZyne b9716f4
UR: rename files
AllanZyne d830a18
UR: update readme
AllanZyne 2c221e9
UR: fix according to PR comments
AllanZyne 3fe33fb
UR: UR_ENABLE_SANITIZER=ON default
AllanZyne 3d58e4b
ignore VS clangd files
AllanZyne 892e85f
fix comments
AllanZyne 6cab044
resolve comments
AllanZyne 0315011
resolve comments
AllanZyne 6ea75ea
Revert "Revert "[UR][CUDA][L0][HIP] Add virtual memory adapter implem…
AllanZyne cca4e45
Revert "Revert "Remove duplicate CUDA UR_DEVICE_INFO_VIRTUAL_MEMORY_S…
AllanZyne 848c0cf
Revert "Revert "Fix return of check error""
AllanZyne e53f8d3
Revert "Revert "Fix common include in cuda device.hpp""
AllanZyne 536aaae
Revert "Revert "Zero-initialize CUmemAccessDesc""
AllanZyne 6f8ccdb
Revert "Revert "Add missing newline""
AllanZyne 118190c
Revert "Revert "Fix wrongful use of UR_ASSERT in cuda implementation""
AllanZyne c1e7b7e
Revert "Revert "an -> a""
AllanZyne 79c3c11
Revert "UR: clean code"
AllanZyne 06bf2aa
add sanitizer test
AllanZyne 9ab8705
refine log
AllanZyne 9b63c18
fix build tests
AllanZyne 2328b48
Merge branch 'main' into sanitizer-pr
AllanZyne e59d7c7
fix build errors
AllanZyne bba6f82
add basic test
AllanZyne f82d196
Merge branch 'sanitizer-pr' into sanitizer-pr-vir
AllanZyne a084650
fix warning & remove tests
AllanZyne 1148ccb
fix warning
AllanZyne fa92f72
Merge branch 'main' into sanitizer-pr
AllanZyne 524a83d
fix warning
AllanZyne c859186
Merge branch 'sanitizer-pr' into sanitizer-pr-vir
AllanZyne b91e450
fix ur api
AllanZyne e9a2093
fix fprintf
AllanZyne a1c4ddc
add some comments
AllanZyne 71d8657
Merge branch 'main' into sanitizer-pr
AllanZyne 02cab8e
Merge branch 'sanitizer-pr' into sanitizer-pr-vir
AllanZyne 621b47e
support local memory
AllanZyne 9fb1eaf
support local memory
AllanZyne 40b5ef6
fix logger
AllanZyne 0b00798
improve logger
AllanZyne b29930b
Merge branch 'main' into sanitizer-pr
AllanZyne 90e59f9
Merge branch 'sanitizer-pr' into sanitizer-pr-vir
AllanZyne b6e2776
support cpu
AllanZyne eacd339
update log
AllanZyne 80bc61b
fix print
AllanZyne c42bd9a
don't print prefix when logger::Level::QUIET
AllanZyne bc8cdb2
small fix
AllanZyne 3bede09
small fix
AllanZyne 5dc2d5c
fix KernelNameBuf
AllanZyne 77859ec
Merge branch 'sanitizer-pr' into sanitizer-pr-vir
AllanZyne d30062e
add usm host/shared
AllanZyne fa35c5e
fix tests
AllanZyne 916e37e
Add check asan context
AllanZyne c641b5b
fix local
AllanZyne c137b68
fix gpu local
AllanZyne 962b08a
fix bug
AllanZyne c3af871
fix bug
AllanZyne 4f7d2e6
fix local
AllanZyne 8fc8644
fix usm memory
AllanZyne 923352f
check pfnGetKernelSuggestedLocalWorkSizeExp
AllanZyne 5231a5d
clean code format
AllanZyne 17cace2
fix comments
AllanZyne 7dbe46e
clean code
AllanZyne 30c0bc1
follow the review comments
AllanZyne 86d4056
fix build
AllanZyne 49e2f4f
follow the review comments
AllanZyne 79998b9
follow the review comments
AllanZyne 6cc4913
fix urEnqueueKernelLaunch & remove host deviceinfo
AllanZyne 577ddd3
fix build
AllanZyne 4ee0e10
Merge branch 'main' into sanitizer-pr
AllanZyne 90270f8
Merge branch 'sanitizer-pr' into sanitizer-buffer
AllanZyne 6193dbd
fix merge
AllanZyne dc127d5
fix build
AllanZyne 2f8af13
Merge branch 'sanitizer-pr' into sanitizer-layer
AllanZyne a1ddc40
fix crash at HostInfo (#9)
AllanZyne 4846d5b
[Sanitizer] Support only device compilation on CPU device (#10)
zhaomaosu 1b57aa6
Merge branch 'main' into sanitizer-layer
AllanZyne 904fb83
fix build
AllanZyne 0255f33
remove urGetKernelSuggestedLocalWorkSize
AllanZyne 7fa003b
remove sanitizer tests
AllanZyne 7cac7c8
add license
AllanZyne b54b988
clean code
AllanZyne 34d1a9a
clean code
AllanZyne b00c401
remove Windows and macOS build
AllanZyne 5e3aa7d
fix format
AllanZyne 270e762
refactor cmake
AllanZyne 255f7fa
Add a FIXME message for urKernelSuggestGroupSize
AllanZyne 920a968
Merge branch 'main' into sanitizer-pr-cpu-local
AllanZyne 8e84eec
Poison shadow memory on CPU side by using memset
zhaomaosu 1b688aa
Put linux specific code into linux/san_utils.cpp
zhaomaosu 42d5d10
add check MemSet
AllanZyne 3910b5b
Merge branch 'main' into sanitizer-pr-cpu-local
AllanZyne 0631330
Using LIBC_SO macro
AllanZyne File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
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.
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.
should we hardcode the library version number?
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.
Makes sense. I changed it to
LIBC_SO
macro.