Skip to content
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

TEST 'MemorySanitizer-Unit :: ./Msan-x86_64-Test/MemorySanitizer.fstatat' FAILED #49831

Open
neilnelson mannequin opened this issue May 26, 2021 · 12 comments
Open

TEST 'MemorySanitizer-Unit :: ./Msan-x86_64-Test/MemorySanitizer.fstatat' FAILED #49831

neilnelson mannequin opened this issue May 26, 2021 · 12 comments
Assignees
Labels
bugzilla Issues migrated from bugzilla

Comments

@neilnelson
Copy link
Mannequin

neilnelson mannequin commented May 26, 2021

Bugzilla Link 50487
Version 12.0
OS Linux
Blocks #51489
CC @tstellar

Extended Description

Obtained this error on Ubuntu 21.04 12.0.1 rc1 release run

******************** TEST 'MemorySanitizer-Unit :: ./Msan-x86_64-Test/MemorySanitizer.fstatat' FAILED ********************
Note: Google Test filter = MemorySanitizer.fstatat
[==========] Running 1 test from 1 test case.
[----------] Global test environment set-up.
[----------] 1 test from MemorySanitizer
[ RUN ] MemorySanitizer.fstatat
/home/nnelson/Documents/llvm-project/llvm/utils/release/rc1/llvm-project/compiler-rt/lib/msan/tests/msan_test.cpp:738: Failure
Expected: true
To be equal to: TestForNotPoisoned((st->st_dev))
Which is: false
/home/nnelson/Documents/llvm-project/llvm/utils/release/rc1/llvm-project/compiler-rt/lib/msan/tests/msan_test.cpp:739: Failure
Expected: true
To be equal to: TestForNotPoisoned((st->st_mode))
Which is: false
/home/nnelson/Documents/llvm-project/llvm/utils/release/rc1/llvm-project/compiler-rt/lib/msan/tests/msan_test.cpp:740: Failure
Expected: true
To be equal to: TestForNotPoisoned((st->st_size))
Which is: false
[ FAILED ] MemorySanitizer.fstatat (1 ms)
[----------] 1 test from MemorySanitizer (1 ms total)

[----------] Global test environment tear-down
[==========] 1 test from 1 test case ran. (1 ms total)
[ PASSED ] 0 tests.
[ FAILED ] 1 test, listed below:
[ FAILED ] MemorySanitizer.fstatat

1 FAILED TEST

@neilnelson
Copy link
Mannequin Author

neilnelson mannequin commented May 26, 2021

assigned to @vitalybuka

@neilnelson
Copy link
Mannequin Author

neilnelson mannequin commented Aug 18, 2021

Added release-13.0.0 to Blocks field. This bug was seen on the 13.0.0-rc1 release run.

@tstellar
Copy link
Collaborator

tstellar commented Sep 2, 2021

Evgenii, There are several failures like this reported on Ubuntu. Do you have any idea what the problem might be?

Other similar failures:
#49831
llvm/llvm-bugzilla-archive#50489
llvm/llvm-bugzilla-archive#50490
llvm/llvm-bugzilla-archive#50491
llvm/llvm-bugzilla-archive#50492

@eugenis
Copy link
Contributor

eugenis commented Sep 2, 2021

My guess would be something has changed around here:

https:/llvm/llvm-project/blob/main/compiler-rt/lib/msan/msan_interceptors.cpp#L709

we depend on libc internals by intercepting __fxstatat.

@tstellar
Copy link
Collaborator

tstellar commented Sep 3, 2021

Neil, are you able to reproduce this on other versions of Ubuntu?

@neilnelson
Copy link
Mannequin Author

neilnelson mannequin commented Sep 3, 2021

I will try Ubuntu 20.10 and let you know. After that I can build whatever you think may work well such as Ubuntu 20.04, Xubuntu 20.04.

@neilnelson
Copy link
Mannequin Author

neilnelson mannequin commented Sep 3, 2021

Output from Ubuntu 20.10. Looks much better.

Failed Tests (6):
SanitizerCommon-asan-i386-Linux :: Linux/crypt_r.cpp
SanitizerCommon-asan-i386-Linux :: Posix/crypt.cpp
SanitizerCommon-lsan-i386-Linux :: Linux/crypt_r.cpp
SanitizerCommon-lsan-i386-Linux :: Posix/crypt.cpp
SanitizerCommon-ubsan-i386-Linux :: Linux/crypt_r.cpp
SanitizerCommon-ubsan-i386-Linux :: Posix/crypt.cpp

Testing Time: 743.91s
Skipped : 39
Unsupported : 4859
Passed : 96180
Expectedly Failed: 355
Failed : 6
make[3]: *** [CMakeFiles/check-all.dir/build.make:58: CMakeFiles/check-all] Error 1
make[3]: Target 'CMakeFiles/check-all.dir/build' not remade because of errors.
make[3]: Leaving directory '/home/nnelson/Documents/llvm-project/llvm/utils/release/rc2/Phase3/Release+Asserts/llvmCore-13.0.0-rc2.obj'
make[2]: *** [CMakeFiles/Makefile2:30926: CMakeFiles/check-all.dir/all] Error 2
make[2]: Leaving directory '/home/nnelson/Documents/llvm-project/llvm/utils/release/rc2/Phase3/Release+Asserts/llvmCore-13.0.0-rc2.obj'
make[1]: *** [CMakeFiles/Makefile2:30933: CMakeFiles/check-all.dir/rule] Error 2
make[1]: Target 'check-all' not remade because of errors.
make[1]: Leaving directory '/home/nnelson/Documents/llvm-project/llvm/utils/release/rc2/Phase3/Release+Asserts/llvmCore-13.0.0-rc2.obj'
make: *** [Makefile:264: check-all] Error 2
[Release+Asserts Phase3] check-all failed

@neilnelson
Copy link
Mannequin Author

neilnelson mannequin commented Sep 3, 2021

I was just comparing the output between the two runs.

Ubuntu 20.10
Skipped : 39
Unsupported : 4859
Passed : 96180
Expectedly Failed: 355
Failed : 6

Ubuntu 21.04
Skipped : 39
Unsupported : 4135
Passed : 96933
Expectedly Failed: 355
Failed : 15

It looks like there are more programs checked in 21.04: 96933 - 96180 = 753. Total programs 101477 - 101439 = 38.

@tstellar
Copy link
Collaborator

tstellar commented Sep 4, 2021

I wonder if there was some glibc change from 20.10 to 21.04 that caused these failures.

@tstellar
Copy link
Collaborator

tstellar commented Sep 4, 2021

@tstellar
Copy link
Collaborator

mentioned in issue #51489

@llvmbot llvmbot transferred this issue from llvm/llvm-bugzilla-archive Dec 11, 2021
@asl asl added this to the LLVM 13.0.1 release milestone Dec 12, 2021
@tstellar
Copy link
Collaborator

The deadline for requesting fixes for the release has passed. This bug is being removed from the LLVM 13.0.1 release milestone. If you have a fix or think this bug is important enough to block the release, please explain why in a comment and add the bug back to the LLVM 13.0.1 release milestone.

@tstellar tstellar removed this from the LLVM 13.0.1 release milestone Dec 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugzilla Issues migrated from bugzilla
Projects
None yet
Development

No branches or pull requests

3 participants