[testsfail] ClientLatency failure under valgrind
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Mir |
Won't Fix
|
Medium
|
Daniel van Vugt |
Bug Description
Running the ClientLatency test suite under valgrind produces failures, which are not present on non-valgrind runs:
└>>> valgrind --trace-
==1122== Memcheck, a memory error detector
==1122== Copyright (C) 2002-2015, and GNU GPL'd, by Julian Seward et al.
==1122== Using Valgrind-3.11.0 and LibVEX; rerun with -h for copyright info
==1122== Command: bin/mir_
==1122==
MIR_CLIENT_
MIR_SERVER_
LD_LIBRARY_
exec=bin/
==1122== Memcheck, a memory error detector
==1122== Copyright (C) 2002-2015, and GNU GPL'd, by Julian Seward et al.
==1122== Using Valgrind-3.11.0 and LibVEX; rerun with -h for copyright info
==1122== Command: bin/mir_
==1122==
Running main() from main.cpp
Note: Google Test filter = ClientLatency*
[==========] Running 2 tests from 1 test case.
[----------] Global test environment set-up.
[----------] 2 tests from ClientLatency
[ RUN ] ClientLatency.
/home/kdub/
Value of: observed_latency
Expected: is > 1.9
Actual: 1.9 (of type float)
[ FAILED ] ClientLatency.
[ RUN ] ClientLatency.
/home/kdub/
Value of: observed_latency
Expected: is < 1.1
Actual: 1.6 (of type float)
[ FAILED ] ClientLatency.
[----------] 2 tests from ClientLatency (12606 ms total)
[----------] Global test environment tear-down
[==========] 2 tests from 1 test case ran. (12649 ms total)
[ PASSED ] 0 tests.
[ FAILED ] 2 tests, listed below:
[ FAILED ] ClientLatency.
[ FAILED ] ClientLatency.
Related branches
- Alberto Aguirre (community): Needs Fixing
- PS Jenkins bot (community): Approve (continuous-integration)
- Alan Griffiths: Disapprove
- Alexandros Frantzis (community): Needs Fixing
-
Diff: 42 lines (+18/-0)1 file modifiedtests/acceptance-tests/test_latency.cpp (+18/-0)
- PS Jenkins bot (community): Approve (continuous-integration)
- Alberto Aguirre (community): Approve
-
Diff: 12 lines (+1/-1)1 file modifiedcmake/MirCommon.cmake (+1/-1)
- PS Jenkins bot (community): Approve (continuous-integration)
- Cemil Azizoglu (community): Approve
- Alberto Aguirre (community): Approve
-
Diff: 98 lines (+28/-13)1 file modifiedtests/acceptance-tests/test_latency.cpp (+28/-13)
Changed in mir: | |
milestone: | none → 0.19.0 |
Changed in mir: | |
status: | New → In Progress |
Changed in mir: | |
status: | Fix Committed → Won't Fix |
This bug seems to be a tributary of lp: #1505962