Intermittent test failure in ClientSurfaceEvents.client_can_query_current_orientation
Bug #1335741 reported by
Alexandros Frantzis
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Mir |
Fix Released
|
High
|
Alan Griffiths | ||
mir (Ubuntu) |
Fix Released
|
High
|
Unassigned |
Bug Description
ClientSurfaceEv
I can reproduce locally (although not easily) with:
valgrind bin/mir_
or without valgrind if the system is under sufficient load.
[1] https:/
Related branches
lp:~alan-griffiths/mir/fix-1335741
- PS Jenkins bot (community): Approve (continuous-integration)
- Alberto Aguirre (community): Approve
- Alexandros Frantzis (community): Approve
-
Diff: 71 lines (+17/-4)1 file modifiedtests/acceptance-tests/test_client_surface_events.cpp (+17/-4)
Changed in mir: | |
assignee: | Alexandros Frantzis (afrantzis) → Alan Griffiths (alan-griffiths) |
Changed in mir: | |
status: | New → In Progress |
Changed in mir: | |
milestone: | none → 0.4.0 |
Changed in mir: | |
milestone: | 0.4.0 → 0.5.0 |
Changed in mir (Ubuntu): | |
importance: | Undecided → High |
status: | New → Triaged |
Changed in mir: | |
status: | Fix Committed → Fix Released |
To post a comment you must log in.
Trying to reproduce with the above command line I don't see the above failure mode. I do see an "Invalid write". I won't file a separate bug until we've investigated further:
Repeating all tests (iteration 12) . . .
Note: Google Test filter = ClientSurfaceEv ents.client_ can_query_ current_ orientation ents.client_ can_query_ current_ orientation display_ server/ mir1/tests/ acceptance- tests/test_ client_ surface_ events. cpp:254: Failure event(mir_ event_type_ orientation, std::chrono: :seconds( 1)) :UnitTest: :AddTestPartRes ult(testing: :TestPartResult ::Type, char const*, int, std::string const&, std::string const&) (in /home/alan/ display_ server/ mir1/build/ bin/mir_ acceptance_ tests) :internal: :AssertHelper: :operator= (testing: :Message const&) const (in /home/alan/ display_ server/ mir1/build/ bin/mir_ acceptance_ tests) ents_client_ can_query_ current_ orientation_ Test::TestBody( ) (test_client_ surface_ events. cpp:254) :internal: :HandleSehExcep tionsInMethodIf Supported< testing: :Test, void>(testing: :Test*, void (testing: :Test:: *)(), char const*) (in /home/alan/ display_ server/ mir1/build/ bin/mir_ acceptance_ tests) :internal: :HandleExceptio nsInMethodIfSup ported< testing: :Test, void>(testing: :Test*, void (testing: :Test:: *)(), char const*) (in /home/alan/ display_ server/ mir1/build/ bin/mir_ acceptance_ tests) :Test:: Run() (in /home/alan/ display_ server/ mir1/build/ bin/mir_ acceptance_ tests) :TestInfo: :Run() (in /home/alan/ display_ server/ mir1/build/ bin/mir_ acceptance_ tests) :TestCase: :Run() (in /home/alan/ display_ server/ mir1/build/ bin/mir_ acceptance_ tests) :internal: :UnitTestImpl: :RunAllTests( ) (in /home/alan/ display_ server/ mir1/build/ bin/mir_ acceptance_ tests) :internal: :HandleSehExcep tionsInMethodIf Supported< testing: :internal: :UnitTestImpl, bool>(testing: :internal: :UnitTestImpl* , bool (testing: :internal: :UnitTestImpl: :*)(), char const*) (in /home/alan/ display_ server/ mir1/build/ bin/mir_ acceptance_ tests) :internal: :HandleExceptio nsInMethodIfSup ported< testing: :internal: :UnitTestImpl, bool>(testing: :internal: :UnitTestImpl* , bool (testing: :internal: :UnitTestImpl: :*)(), char const*) (in /home/alan/ display_ server/ mir1/build/ bin/mir_ acceptance_ tests) :UnitTest: :Run() (in /home/alan/ display_ server/ mir1/build/ bin/mir_ acceptance_ tests)
[==========] Running 1 test from 1 test case.
[----------] Global test environment set-up.
[----------] 1 test from ClientSurfaceEvents
[ RUN ] ClientSurfaceEv
/home/alan/
Value of: wait_for_
Actual: false
Expected: true
==3769== Invalid write of size 4
==3769== at 0x77D288: testing:
==3769== by 0x77212C: testing:
==3769== by 0x601A77: ClientSurfaceEv
==3769== by 0x794335: void testing:
==3769== by 0x78F58B: void testing:
==3769== by 0x7772E2: testing:
==3769== by 0x777ABD: testing:
==3769== by 0x77814D: testing:
==3769== by 0x77E797: testing:
==3769== by 0x795713: bool testing:
==3769== by 0x7903E5: bool testing:
==3769== by 0x77D456: testing:
==3769== Address 0x0 is not stack'd, malloc'd or (recently) free'd
==3769==
==3769==
==3769== HEAP SUMMARY:
==3769== in use at exit: 296,994 bytes in 4,389 blocks
==3769== total heap usage: 154,055 allocs, 149,666 frees, 8,123,620 bytes allocated
==3769==
==3769== LEAK SUMMARY:
==3769== ...