Mir

Test takes minutes to complete: MediatingDisplayChangerTest.confirmed_configuration_doesnt_revert_after_timeout

Bug #1671033 reported by Daniel van Vugt
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mir
Fix Released
Medium
Daniel van Vugt
mir (Ubuntu)
Fix Released
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

Revision history for this message
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
Revision history for this message
Daniel van Vugt (vanvugt) wrote : Re: MediatingDisplayChangerTest.confirmed_configuration_doesnt_revert_after_timeout takes minutes to complete under valgrind

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
Revision history for this message
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
Revision history for this message
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  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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