Mir

CI InputEvents failure in krillin

Bug #1603145 reported by Kevin DuBois
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mir
Triaged
Medium
Unassigned
mir (Ubuntu)
Incomplete
Medium
Unassigned

Bug Description

InputEvents from mir_privileged tests is failing with a segfault.

CI logs: https://mir-jenkins.ubuntu.com/job/device-runtests-mir/device_type=krillin/1055/consoleFull

Related branches

lp://staging/~kdub/mir/fix-1603145
On hold for merging into lp://staging/mir
Mir CI Bot: Approve (continuous-integration)
Mir development team: Pending requested
Diff: 118 lines (+52/-38)
1 file modified
tests/privileged-tests/test_input_events.cpp (+52/-38)
lp://staging/~kdub/mir/fix-ci
Daniel van Vugt: Approve
Mir CI Bot: Needs Fixing (continuous-integration)
Cemil Azizoglu (community): Approve
Brandon Schaefer (community): Approve
Diff: 39 lines (+12/-1)
3 files modified
cmake/MirCommon.cmake (+1/-0)
tests/privileged-tests/test_input_events.cpp (+1/-1)
tools/valgrind_suppressions_libhybris (+10/-0)
lp://staging/~ci-train-bot/mir/mir-ubuntu-yakkety-landing-071 (Merged)
Revision history for this message
Kevin DuBois (kdub) wrote :

interestingly, it only happens when running the test in series with the other tests in the suite. Perhaps a bad-driver-shutdown bug?

Revision history for this message
Kevin DuBois (kdub) wrote :

Forking off the client code to its own process ( so the driver gets loaded there) seems to avert the problem. This was precipitated by a change to libhybris... maybe its not the driver with the problem, but libhybris?

Changed in mir:
assignee: nobody → Kevin DuBois (kdub)
status: New → In Progress
Revision history for this message
Daniel van Vugt (vanvugt) wrote :

We declared this fixed in the changelog of 0.23.5 today. I'm not sure if that's the preferred way to hand workarounds or we should keep this open since it's not yet fixed.

I'm leaning toward keeping this open and wait for the full fix. The 0.23.5 changelog has several such points of confusion that need retrospective clarification anyway.

Brandon Schaefer (brandontschaefer)
Changed in mir:
milestone: 0.25.0 → 0.26.0
Daniel van Vugt (vanvugt)
Changed in mir:
importance: High → Medium
Daniel van Vugt (vanvugt)
Changed in mir:
milestone: 0.26.0 → 1.0.0
Daniel van Vugt (vanvugt)
Changed in mir:
milestone: 1.0.0 → none
status: In Progress → Triaged
milestone: none → 1.0.0
status: Triaged → In Progress
Daniel van Vugt (vanvugt)
Changed in mir:
milestone: 0.27.0 → 0.28.0
Daniel van Vugt (vanvugt)
Changed in mir:
milestone: 0.28.0 → none
status: In Progress → Triaged
Revision history for this message
Michał Sawicz (saviq) wrote :

Syncing task from Mir.

Changed in mir (Ubuntu):
importance: Undecided → Medium
status: New → Triaged
Revision history for this message
Alan Griffiths (alan-griffiths) wrote :

I don't think this ever happened anywhere but krillin, and that's history.

Changed in mir (Ubuntu):
status: Triaged → Incomplete
Changed in mir:
assignee: Kevin DuBois (kdub) → nobody
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.