Skip to content

[DeviceMSAN] fix origin check e2e tests#19137

Merged
aelovikov-intel merged 3 commits intointel:syclfrom
yingcong-wu:yc/0625-msan-test-fix
Jun 26, 2025
Merged

[DeviceMSAN] fix origin check e2e tests#19137
aelovikov-intel merged 3 commits intointel:syclfrom
yingcong-wu:yc/0625-msan-test-fix

Conversation

@yingcong-wu
Copy link
Contributor

@yingcong-wu yingcong-wu commented Jun 25, 2025 •
edited
Loading

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

yingcong-wu commented Jun 26, 2025 

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

yingcong-wu commented Jun 26, 2025

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

Reviewers

@AllanZyne AllanZyne AllanZyne approved these changes

@zhaomaosu zhaomaosu zhaomaosu approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@yingcong-wu @sarnex @AllanZyne @zhaomaosu @aelovikov-intel