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 fails: AlignedStorageTest.cpp:124: Failure Expected: (initial) != (FAILED), actual: 18446744073709551615 vs 18446744073709551615 #347

Closed
yurivict opened this issue Sep 11, 2020 · 4 comments

Comments

@yurivict
Copy link

Note: Google Test filter = AlignedStorageTest.AdviseUnused
[==========] Running 1 test from 1 test case.
[----------] Global test environment set-up.
[----------] 1 test from AlignedStorageTest
[ RUN      ] AlignedStorageTest.AdviseUnused
/usr/ports/lang/hermes/work/hermes-0.5.2-rc1/unittests/VMRuntime/AlignedStorageTest.cpp:124: Failure
Expected: (initial) != (FAILED), actual: 18446744073709551615 vs 18446744073709551615
[  FAILED  ] AlignedStorageTest.AdviseUnused (0 ms)
[----------] 1 test from AlignedStorageTest (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  ] AlignedStorageTest.AdviseUnused

 1 FAILED TEST

Version 0.5.2-rc1
clang-10
FreeBSD 12.2

@tmikov
Copy link
Contributor

tmikov commented Sep 11, 2020

Looks like regionFootprint() is failing, which isn't very surprising: it is trying to use the Linux implementation, which attempts to read /proc/self/smaps, which I suspect is Linux-specific. I think a FreeBSD implementation has to be added.

@yurivict
Copy link
Author

Please note that you open the file std::ifstream smaps("/proc/self/smaps"); and don't check for failure.

@tmikov
Copy link
Contributor

tmikov commented Sep 11, 2020

We don't really need to check for failure - it just naturally falls through to the error exit at the bottom. I think.

@tmikov
Copy link
Contributor

tmikov commented Sep 15, 2020

Closing this, since we can't provide first party support for FreeBSD. Happy to incorporate community PRs for it though.

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

No branches or pull requests

2 participants