Bug #1591328 “Pointer/cursor input lag in unity8 session” : Bugs : unity8 package : Ubuntu

Andrea Cimitan (cimi) on 2016-06-10

summary:

- Pointer input lag in unity8 desktop session
+ Pointer input lag in unity8 session

Andrea Cimitan (cimi) on 2016-06-10

description:

updated

Revision history for this message

Daniel van Vugt (vanvugt) wrote on 2016-06-10: Re: Pointer input lag in unity8 session

#1

Lots of history here: https://trello.com/c/sDnxldwa

Revision history for this message

Launchpad Janitor (janitor) wrote on 2016-06-10:

#2

Status changed to 'Confirmed' because the bug affects multiple users.

Changed in qtmir (Ubuntu):
status:	New → Confirmed
Changed in unity8 (Ubuntu):
status:	New → Confirmed

Revision history for this message

Daniel van Vugt (vanvugt) wrote on 2016-06-10:

#4

Actually everything in Unity8 lags the same due to our nested architecture. We're just prioritising improving the cursor responsiveness first.

tags:	added: performance
summary:	- Pointer input lag in unity8 session + Pointer/cursor input lag in unity8 session

Revision history for this message

Daniel van Vugt (vanvugt) wrote on 2016-06-10:

#5

Here's a quick workaround (which also shows which Unity8 animations are buggy and not based on real time):

/etc/environment:
MIR_CLIENT_FORCE_SWAP_INTERVAL=0

That will remove most of the buffer lag.

Revision history for this message

Andrea Cimitan (cimi) wrote on 2016-06-13:

#6

The workaround improves, but it on my laptop it is still miles slower than unity 7...

Revision history for this message

Daniel van Vugt (vanvugt) wrote on 2016-07-20:

#7

Well it's nice to hear you care about this stuff :)

Short term: anpok and unity8 guys are working on lag-free mouse cursor pass-through -> bug 1600220

Longer term: I will need to finish my frame notification work to finally kill all nesting and buffering lag.

Daniel van Vugt (vanvugt) on 2016-07-22

tags:

added: unity8-desktop

Revision history for this message

Daniel van Vugt (vanvugt) wrote on 2016-08-17:

#8

Even shorter term, I think fixing bug 1607677 might resolve the feeling of lag for lots of people.

Certainly the libinput acceleration curve has been bugging me for most of a year. It feels awful.

Revision history for this message

Daniel van Vugt (vanvugt) wrote on 2016-09-27:

#9

Work in progress on two separate fronts:
Short term: Pass through cursor position (bug 1600220)
Long term: Nesting eventually won't affect lag (started with https://code.launchpad.net/~vanvugt/mir/physical-frame/+merge/306199)

Changed in mir:
status:	New → In Progress
importance:	Undecided → High
assignee:	nobody → Daniel van Vugt (vanvugt)
milestone:	none → 0.25.0
assignee:	Daniel van Vugt (vanvugt) → Mir development team (mir-team)
assignee:	Mir development team (mir-team) → nobody
Changed in mir (Ubuntu):
status:	New → Triaged
importance:	Undecided → High
Changed in canonical-devices-system-image:
status:	New → In Progress
milestone:	none → 14
importance:	Undecided → High
Changed in mir:
assignee:	nobody → Mir development team (mir-team)

Brandon Schaefer (brandontschaefer) on 2016-11-07

Changed in mir:
milestone:	0.25.0 → 0.26.0

kevin gunn (kgunn72) on 2016-11-08

Changed in canonical-devices-system-image:
milestone:	14 → backlog

Revision history for this message

Daniel van Vugt (vanvugt) wrote on 2017-01-06:

#10

This bug will be fully resolved by the attached branches (needs all of them to land)

Changed in mir:
assignee:	Mir development team (mir-team) → Daniel van Vugt (vanvugt)
Changed in qtmir (Ubuntu):
status:	Confirmed → Invalid
Changed in unity8 (Ubuntu):
status:	Confirmed → In Progress
importance:	Undecided → High
assignee:	nobody → Daniel van Vugt (vanvugt)

Revision history for this message

Launchpad Janitor (janitor) wrote on 2017-01-15:

#11

This bug was fixed in the package unity8 - 8.15+17.04.20170110.4-0ubuntu1

---------------
unity8 (8.15+17.04.20170110.4-0ubuntu1) zesty; urgency=medium

  [ Albert Astals Cid ]
  * Give focus to one of the buttons of the dialog
  * tst_WindowResizeArea: Use default values for mouseFlick speed and
    iterations (LP: #1651580)
  * Require Qt 5.6 & misc fixes

  [ Daniel van Vugt ]
  * Deprecate usage of Mir's input resampling, instead opting for: (LP:
    #1497105, #1591328)

  [ Josh Arenson ]
  * Allow the scopes list to automatically scroll when a scope is being
    dragged past the bounds of the screen. (LP: #1575319)

  [ Lukáš Tinkl ]
  * Fix touch window controls being unreachable when the overlay is
    being displayed (LP: #1648167)
  * Fixup paths for window state storage in snappy environment
  * Add Unity.Platform mock for our tests (LP: #1655336)

  [ Michael Zanetti ]
  * PreviewRatingInput: Use displayText instead of text to
    enable/disable the Send button (LP: #1595910)
  * Add a D-Bus interface to control some debug facilities on the fly
  * some launcher workarounds for the snapping

[ Michał Sawicz ]
* Nuke leftover Platform in IndicatorsManager

[ Nick Dedekind ]
* Added registry for application menus

[ Michael Terry, Nick Dedekind ]
* Run the qmluitests.sh autopkg test against the installed package.

[ Michał Sawicz, Nick Dedekind ]
* Application menus

  [ Rodney Dawes ]
  * Remove the payments widget and dependency on libpay as no longer
    needed.

-- Michał Sawicz <email address hidden> Tue, 10 Jan 2017 14:48:42 +0000

Changed in unity8 (Ubuntu):
status:	In Progress → Fix Released

Daniel van Vugt (vanvugt) on 2017-01-15

Changed in mir:
milestone:	0.26.0 → 1.0.0

Revision history for this message

Daniel van Vugt (vanvugt) wrote on 2017-01-16:

#12

Per comment #10, this isn't fixed until all of the branches land. The Unity8 branch by itself doesn't really qualify as a fix that should get announced for this bug.

Changed in unity8 (Ubuntu):
status:	Fix Released → Triaged
status:	Triaged → Fix Released

Revision history for this message

Mir CI Bot (mir-ci-bot) wrote on 2017-01-17:

#13

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

Changed in mir:
status:	In Progress → Fix Committed

Daniel van Vugt (vanvugt) on 2017-01-20

Changed in mir:
milestone:	1.0.0 → 0.26.0

Daniel van Vugt (vanvugt) on 2017-01-20

Changed in canonical-devices-system-image:
status:	In Progress → Fix Committed

Revision history for this message

Launchpad Janitor (janitor) wrote on 2017-02-01:

#14

Download full text (7.2 KiB)

This bug was fixed in the package mir - 0.26.0+17.04.20170126.3-0ubuntu1

---------------
mir (0.26.0+17.04.20170126.3-0ubuntu1) zesty; urgency=medium

  [ Daniel van Vugt ]
  * New upstream release 0.26.0 (https://launchpad.net/mir/+milestone/0.26.0)
    - ABI summary:
      . mirclient ABI unchanged at 9
      . mirserver ABI bumped to 43
      . mircommon ABI unchanged at 7
      . mirplatform ABI unchanged at 14
      . mirprotobuf ABI unchanged at 3
      . mirplatformgraphics ABI unchanged at 11
      . mirclientplatform ABI unchanged at 5
      . mirinputplatform ABI unchanged at 6
      . mircore ABI unchanged at 1
    - Enhancements:
      . New/improved toolkit APIs: MirInputConfig and related functions,
        MirWindow and related functions, DisplayConfig and related functions,
        MirScreencastSpec and related functions,
      . Support for configuring input configuration
      . Introduce toolkit extension mechanism for platform specific APIs.
      . Toolkit extensions for: mesa_drm_auth, set_gbm_device and
        window_coordinate_translation
      . Screencasting to a specific MirBuffer.
      . Add DisplayConfigurationController::base_configuration() so downstreams
        can get the base configuration. (Weirdly they can already set it.).
      . x11 platform: allow adjustable scale parameter.
      . Added EDID support: client API, server logging and in mirout.
      . mirout: Add newer attributes only available in the new display
        config API: scaling factor, subpixel arrangement and form factor.
      . mirout: Log the orientation and logical size of each output too.
      . Replace the mir::Server-overridable Reports with Observers.
      . Add xkbcommon to mirclient.pc Requires.private.
      . Deprecate legacy toolkit APIs that will be removed in Mir 1.0
      . Introduced 'client-side vsync', which dramatically reduces latency
        from the client to the screen (particularly for nested servers like
        Unity8).
    - Bugs fixed:
      . [performance] Restore support for better-than-triple buffering by
        default. (LP: #1240909)
      . Frame rate is artificially low on Diamondville Intel Atom systems due
        to aggressive power management (LP: #1388490)
      . [testsfail] failure in CI in
        AndroidInputReceiverSetup.slow_raw_input_doesnt_cause_frameskipping
        (LP: #1394369)
      . [trusted prompt sessions] Can't open two prompt sessions at the same
        time (LP: #1494197)
      . Changing scale, formFactor or DPI in display configuration causes
        renderer teardown/recreate unnecessarily (LP: #1556142)
      . [testsfail] ApplicationNotRespondingDetection.failure_to_pong_is_
        noticed (LP: #1570327)
      . CI failure in TestClientInput.receives_one_touch_event_per_frame
        (LP: #1570698)
      . Mir-on-X mouse input is jerky/stuttery compared to Mir-on-KMS
        (LP: #1576600)
      . [regression] Two fingers in mir_proving_server now resizes/moves app
        windows (two finger apps unusable) (LP: #1586311)
      . Pointer/cursor input lag in unity8 session (LP: #1591328)
      . PointerConfinement.test_we_update_our_confined_region_on_a_resize
      ...

This bug was fixed in the package mir - 0.26.0+17.04.20170126.3-0ubuntu1

---------------
mir (0.26.0+17.04.20170126.3-0ubuntu1) zesty; urgency=medium

[ Daniel van Vugt ]
  * New upstream release 0.26.0 (https://launchpad.net/mir/+milestone/0.26.0)
    - ABI summary:
      . mirclient ABI unchanged at 9
      . mirserver ABI bumped to 43
      . mircommon ABI unchanged at 7
      . mirplatform ABI unchanged at 14
      . mirprotobuf ABI unchanged at 3
      . mirplatformgraphics ABI unchanged at 11
      . mirclientplatform ABI unchanged at 5
      . mirinputplatform ABI unchanged at 6
      . mircore ABI unchanged at 1
    - Enhancements:
      . New/improved toolkit APIs: MirInputConfig and related functions,
        MirWindow and related functions, DisplayConfig and related functions,
        MirScreencastSpec and related functions,
      . Support for configuring input configuration
      . Introduce toolkit extension mechanism for platform specific APIs.
      . Toolkit extensions for: mesa_drm_auth, set_gbm_device and
        window_coordinate_translation
      . Screencasting to a specific MirBuffer.
      . Add DisplayConfigurationController::base_configuration() so downstreams
        can get the base configuration. (Weirdly they can already set it.).
      . x11 platform: allow adjustable scale parameter.
      . Added EDID support: client API, server logging and in mirout.
      . mirout: Add newer attributes only available in the new display
        config API: scaling factor, subpixel arrangement and form factor.
      . mirout: Log the orientation and logical size of each output too.
      . Replace the mir::Server-overridable Reports with Observers.
      . Add xkbcommon to mirclient.pc Requires.private.
      . Deprecate legacy toolkit APIs that will be removed in Mir 1.0
      . Introduced 'client-side vsync', which dramatically reduces latency
        from the client to the screen (particularly for nested servers like
        Unity8).
    - Bugs fixed:
      . [performance] Restore support for better-than-triple buffering by
        default. (LP: #1240909)
      . Frame rate is artificially low on Diamondville Intel Atom systems due
        to aggressive power management (LP: #1388490)
      . [testsfail] failure in CI in
        AndroidInputReceiverSetup.slow_raw_input_doesnt_cause_frameskipping
        (LP: #1394369)
      . [trusted prompt sessions] Can't open two prompt sessions at the same
        time (LP: #1494197)
      . Changing scale, formFactor or DPI in display configuration causes
        renderer teardown/recreate unnecessarily (LP: #1556142)
      . [testsfail] ApplicationNotRespondingDetection.failure_to_pong_is_
        noticed (LP: #1570327)
      . CI failure in TestClientInput.receives_one_touch_event_per_frame
        (LP: #1570698)
      . Mir-on-X mouse input is jerky/stuttery compared to Mir-on-KMS
        (LP: #1576600)
      . [regression] Two fingers in mir_proving_server now resizes/moves app
        windows (two finger apps unusable) (LP: #1586311)
      . Pointer/cursor input lag in unity8 session (LP: #1591328)
      . PointerConfinement.test_we_update_our_confined_region_on_a_resize
        (LP: #1606418)
      . [ FAILED ] ClientLatency.throttled_input_rate_yields_lower_latency
        (LP: #1624188)
      . Valgrind failure on mir_acceptance_tests - leak on incoming buffer in
        Requests::free_buffer(int) (LP: #1628794)
      . Add support for returning monitor EDIDs (LP: #1634868)
      . When pointer motion crosses window borders SurfaceInputDispatcher
        injected events increase accumulated relative motion (LP: #1639749)
      . [ FAILED ] ClientLatency.dropping_latency_is_limited_to_one (AKA
        dropping_latency_is_closer_to_zero_than_one) (LP: #1640366)
      . Server::override_the_coordinate_translator() cannot be effectively used
        downstream (LP: #1641166)
      . [regression] FTBFS when cross compiling to yakkety from zesty:
        test_client_extensions.cpp: error: ‘Ne’ was not declared in this scope
        (LP: #1642473)
      . [regression] mir_proving_server crashes with std::exception::what:
        add_options() must be called before the_options() (LP: #1642504)
      . [ FAILED ] NestedServer.nested_platform_connects_and_disconnects
        (LP: #1643432)
      . googletest 1.8.0-2 (on zesty) breaks existing builds [add_library
        cannot create target "gmock" ...] (LP: #1644062)
      . [regression] Those mir_demo_client_* which default to fullscreen now
        ignore the size parameter (-s WIDTHxHEIGHT) (LP: #1644075)
      . Shell doesn't know when base display config has changed (LP: #1644189)
      . miral::ActiveOutputsMonitor (and therefore miral-shell --window-manager
        tiling) are broken by lp:mir (LP: #1645284)
      . mir_proving_server's four finger swipe to switch apps leaks touch
        events to the clients (LP: #1646700)
      . [regression] Cross compiling from zesty to yakkety fails with *** No
        rule to make target 'gmock/libs/googlemock/gtest/libgtest.a' (LP:
        #1646704)
      . [ FAILED ] ThreadedDispatcherDeathTest.destroying_dispatcher_from_a_
        callback_is_an_error (LP: #1647573)
      . [regression] eglapps now all have window title "default" (LP: #1647575)
      . Package mirtest-dev is missing a dependency on mir-renderer-gl-dev
        (LP: #1651391)
      . EDID data is missing in nested servers (LP: #1651633)
      . [regression] Latency for fullscreen interval 0 clients (e.g. games and
        benchmarks) increased with the introduction of nested passthrough
        (LP: #1651638)
      . Race between MirConnection::released() and
        MirConnection::~MirConnection() causes deadlocks, possible crashes and
        memory corruption (LP: #1653658)
      . DisplayConfigurationTest.configure_session_removed_display failure
        (LP: #1653789)
      . [regression] Development headers reference unknown header
        "mir_toolkit/mir_input_device_types.h" so some clients can't build at
        all any more (LP: #1654023)
      . [regression] mir_acceptance_tests' death tests leave behind core files
        (LP: #1654478)
      . Setting the event handler after the surface has been created does
        nothing (LP: #1654612)
      . [ FAILED ] RaiseSurfaces.key_event_with_cookie (LP: #1655293)
      . Black screen with Raspberry Pi 3 VC4 Mesa driver (LP: #1656164)
      . i386 FTBFS:
        src/platforms/eglstream-kms/server/kms_display_configuration.cpp:88:47:
        error: narrowing conversion of ‘..._drmModeConnector::connector_id’
        from ‘uint32_t {aka unsigned int}’ to ‘EGLAttrib {aka int}’ inside { }
        [-Werror=narrowing] (LP: #1656633)
      . android: overlays no longer are activated (LP: #1657755)
      . Mir FTBFS with MIR_USE_PRECOMPILED_HEADERS=OFF: error: ISO C++11
        requires at least one argument for the "..." in a variadic macro
        [-Werror] (LP: #1658604)
      . [regression] Mir cross-compile to vivid/armhf FTBFS with:
        mock_input_device_hub.h:33:18: error: ‘add_observer’ is not a type
        (LP: #1658605)
      . Caught exception at Mir/EGL driver boundary (in setSwapInterval) in
        clients and tests (LP: #1659298)
      . Support display scaling slider in unity8 (LP: #1645372)

-- Cemil Azizoglu <cemil.azizoglu@canonical.com>  Thu, 26 Jan 2017 23:29:33 +0000

Changed in mir (Ubuntu):
status:	Triaged → Fix Released

Daniel van Vugt (vanvugt) on 2017-02-02

Changed in mir:
status:	Fix Committed → Fix Released

Ubuntu
unity8 package

Pointer/cursor input lag in unity8 session

Bug Description

Related branches

Other bug subscribers

Remote bug watches

	Status	Importance	Assigned to	Milestone
Canonical System Image	Fix Committed	High	Unassigned	Canonical System Image backlog
Mir	Fix Released	High	Daniel van Vugt	Mir 0.26.0
mir (Ubuntu)	Fix Released	High	Unassigned
qtmir (Ubuntu)	Invalid	Undecided	Unassigned
unity8 (Ubuntu)	Fix Released	High	Daniel van Vugt

Ubuntuunity8 package

Pointer/cursor input lag in unity8 session

Bug Description

Related branches

Other bug subscribers

Remote bug watches

Ubuntu
unity8 package