Skip to content

[DeviceMSAN] fix origin check e2e tests #19137

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

Merged
merged 3 commits into from
Jun 26, 2025

Conversation

yingcong-wu
Copy link
Contributor

@yingcong-wu yingcong-wu commented Jun 25, 2025

With non -O0, the call stack is not preserved, like malloc_shared will be inlined, the call stack would be like

 #0 in int* sycl::_V1::malloc_host<int>(unsigned long, sycl::_V1::context const&, sycl::_V1::property_list const&, sycl::_V1::detail::code_location const&) /tmp/syclws/include/sycl/usm.hpp:215:27
 #1 in ?? (/lib/x86_64-linux-gnu/libc.so.6+0x757867a2a1c9)
 #2 in __libc_start_main (/lib/x86_64-linux-gnu/libc.so.6+0x757867a2a28a)

instead of

 #0 in int* sycl::_V1::malloc_host<int>(unsigned long, sycl::_V1::context const&, sycl::_V1::property_list const&, sycl::_V1::detail::code_location const&) /tmp/syclws/include/sycl/usm.hpp:215:27
 #1 in int* sycl::_V1::malloc_host<int>(unsigned long, sycl::_V1::queue const&, sycl::_V1::property_list const&, sycl::_V1::detail::code_location const&) /tmp/syclws/include/sycl/usm.hpp:223:10
 #2 in main /tmp/syclws/llvm/sycl/test-e2e/MemorySanitizer/track-origins/check_host_usm_initialized_on_host.cpp:15:17
 #3 in ?? (/lib/x86_64-linux-gnu/libc.so.6+0x7a67f842a1c9)
 #4 in __libc_start_main (/lib/x86_64-linux-gnu/libc.so.6+0x7a67f842a28a)

Also, add env to every %{run} directive to make sure they are not affected by system env.

@yingcong-wu yingcong-wu requested a review from a team as a code owner June 25, 2025 09:02
@yingcong-wu yingcong-wu changed the title fix test [DeviceMSAN] fix e2e tests Jun 25, 2025
@yingcong-wu yingcong-wu changed the title [DeviceMSAN] fix e2e tests [DeviceMSAN] fix origin check e2e tests Jun 25, 2025
@yingcong-wu
Copy link
Contributor Author

Error: No space left on device

Hi @intel/llvm-gatekeepers , the CI server seems to have no space available and it is blocking the CI checks. Please help with the issue, thanks.

@sarnex
Copy link
Contributor

sarnex commented Jun 26, 2025

Fixed it I think, thanks

@yingcong-wu
Copy link
Contributor Author

Failures look like infrastructure problems. This is a test only change.

@aelovikov-intel aelovikov-intel merged commit aaab7e9 into intel:sycl Jun 26, 2025
35 of 39 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants