Mir

Test takes minutes to complete: MediatingDisplayChangerTest.confirmed_configuration_doesnt_revert_after_timeout

Bug #1671033 reported by Daniel van Vugt on 2017-03-08
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mir
Fix Released
Medium
Daniel van Vugt
mir (Ubuntu)
Medium
Unassigned

Bug Description

MediatingDisplayChangerTest.confirmed_configuration_doesnt_revert_after_timeout takes 30+ seconds under valgrind. But only half a second without valgrind.

Related branches

Daniel van Vugt (vanvugt) wrote :

Tagged 'testsfail' just for convenience. I know it's not failing.

Changed in mir:
assignee: nobody → Daniel van Vugt (vanvugt)
assignee: Daniel van Vugt (vanvugt) → nobody
tags: added: testsfail
Changed in mir:
assignee: nobody → Daniel van Vugt (vanvugt)
milestone: none → 1.0.0

Here are the run times from a recent branch of mine in CI. Looks like over a minute to complete this test case is pretty common...

https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=amd64,compiler=clang,platform=mesa,release=zesty/4235/consoleFull
09:51:55 19: [ OK ] MediatingDisplayChangerTest.confirmed_configuration_doesnt_revert_after_timeout (73195 ms)

https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=amd64,compiler=gcc,platform=mesa,release=xenial+overlay/4235/consoleFull
09:43:46 21: [ OK ] MediatingDisplayChangerTest.confirmed_configuration_doesnt_revert_after_timeout (49675 ms)

https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=amd64,compiler=gcc,platform=mesa,release=zesty/4235/consoleFull
09:50:04 21: [ OK ] MediatingDisplayChangerTest.confirmed_configuration_doesnt_revert_after_timeout (67826 ms)

https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=i386,compiler=gcc,platform=android,release=vivid+overlay/4235/consoleFull
09:33:42 11: [ OK ] MediatingDisplayChangerTest.confirmed_configuration_doesnt_revert_after_timeout (65554 ms)

https://mir-jenkins.ubuntu.com/job/build-2-binpkg-mir/arch=i386,compiler=gcc,platform=mesa,release=xenial+overlay/4235/consoleFull
MediatingDisplayChangerTest.confirmed_configuration_doesnt_revert_after_timeout (77161 ms)
09:46:50 21: [ RUN ]

https://mir-jenkins.ubuntu.com/job/device-runtests-mir/3235/device_type=krillin/consoleFull
09:51:00 [ OK ] MediatingDisplayChangerTest.confirmed_configuration_doesnt_revert_after_timeout (6717 ms)

Changed in mir:
status: Triaged → In Progress
summary: MediatingDisplayChangerTest.confirmed_configuration_doesnt_revert_after_timeout
- takes 30+ seconds under valgrind
+ takes minutes to complete under valgrind
Changed in mir:
importance: Low → Medium
summary: + Test takes minutes to complete:
MediatingDisplayChangerTest.confirmed_configuration_doesnt_revert_after_timeout
- takes minutes to complete under valgrind
Mir CI Bot (mir-ci-bot) wrote :

Fix committed into lp:mir at revision None, scheduled for release in mir, milestone 1.0.0

Changed in mir:
status: In Progress → Fix Committed
Changed in mir (Ubuntu):
status: New → Triaged
importance: Undecided → Medium
Launchpad Janitor (janitor) wrote :
Download full text (8.3 KiB)

This bug was fixed in the package mir - 0.27.0+17.10.20170630-0ubuntu1

---------------
mir (0.27.0+17.10.20170630-0ubuntu1) artful; urgency=medium

  [ Daniel van Vugt ]
  * New upstream release 0.27.0 (https://launchpad.net/mir/+milestone/0.27.0)
    - ABI summary:
      . mirclient ABI unchanged at 9
      . mirserver ABI bumped to 44
      . mircommon ABI unchanged at 7
      . mirplatform ABI bumped to 61
      . mirprotobuf ABI unchanged at 3
      . mirplatformgraphics ABI bumped to 13
      . mirclientplatform ABI unchanged at 5
      . mirinputplatform ABI bumped to 7
      . mircore ABI unchanged at 1
    - Enhancements:
      . Mostly groundwork required to support major enhancements coming in
        future Mir versions.
      . Removed android-input and eliminated the entire "3rd_party/" subtree.
        Now the Mir source tree contains original code only.
      . Added mir_prompt_session_new_fds_for_prompt_providers_sync API.
      . mirout: Added load and save options for keeping display configs
        on disk.
      . mirout: Added "--" support for applying configuration changes under
        Unity8.
      . Fixed failure of DRM hardware cursor {hide(); show(image);}
      . Added server option: "--cursor software" (MIR_SERVER_CURSOR=software)
      . Added letterboxing/black bars support to the GL renderer in preparation
        for generic output cloning.
      . Added client API for getting the logical size of an output.
      . Migrated MirCookie to use SHA-256.
      . Ensure RealKMSOutputConfiguration stays in sync with actual hardware
        state.
      . Added support for drag-and-drop.
      . Lots of other client API enhancements.
      . Minor clean-ups, optimizations and dead code removal.
      . Added support for building on Ubuntu 17.10 artful.
      . Update example code to use undeprecated API.
      . mesa-kms: Support hardware cursors in hybrid setups.
      . Rework and publish the graphics platform APIs
    - Bugs fixed:
      . [enhancement] Make able to get version information from client /
        server APIs (LP: #1195540)
      . Touch screen coordinates don't rotate with the screen (LP: #1349660)
      . Subpixel order not included in Mir display information (LP: #1393578)
      . [enhancement] Missing client API for relative surface movement (e.g.
        dragging client-decorated windows) (LP: #1420334) . Mir does not reset
        key states when paused or resumed (modifiers get stuck after VT
        switching) (LP: #1536279)
      . NBS never uses mc::MultiMonitorMode::single_monitor_fast, even when
        only a single monitor is plugged in (LP: #1561418)
      . Inconsistent behaviour of Num Lock (LP: #1588237)
      . A scaled (not panned or clipped) mirror/clone mode is desired
        (LP: #1639226)
      . Rotating an output left or right without restarting the
        compositor distorts the image (LP: #1643488)
      . support display scaling slider in unity8 (LP: #1645372)
      . [ FAILED ] NestedInputWithMouse.mouse_pointer_coordinates_in_nested_
        server_are_accumulated (LP: #1646375)
      . [ FAILED ] NestedInputWithMouse.mouse_pointer_position_is_in_sync_with_
        ho...

Read more...

Changed in mir (Ubuntu):
status: Triaged → Fix Released
Changed in mir:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers