After exiting Mir (by Ctrl+Alt+BkSp) VT switching by Ctrl+Alt+Fn is broken

[Alan Griffiths]

$ sudo bin/mir_demo_server --vt 1

Exit using Ctrl+Alt+BkSp

Now try to switch back to desktop session with Ctrl+Alt+F7

Expect: switches back to desktop session
Actual: nothing happens

A switch by "sudo chvt 7" gets things back to normal

[Andreas Pokorny (andreas-pokorny)]

I can reproduce on a mobile i7 49xxx with the i915 driver on 4.2.0, same kernel on amd A10 works fine.
The problem on i915 disappears when use a ssh connection to launch mir. But then still the vt switch ioctl returns -1 and does nothing.

[Alberto Aguirre (albaguirre)]

Apparently this doesn't occur if the server is started from within a VT or from an ssh session at least on my laptop with i915 driver.

[Andreas Pokorny (andreas-pokorny)]

Ok right the problem occurs on an Amd A10 too when started from unity7.
I guess because we take the terminal parameter and settings before we switch?
So we fail to restore the terminal..

[Alan Griffiths (alan-griffiths)]

Another data point: it doesn't happen with 0.17:

$ sudo mir_demo_server --vt 1 --platform-graphics-lib /usr/lib/x86_64-linux-gnu/mir/server-platform/graphics-mesa-kms.so.6

[Andreas Pokorny (andreas-pokorny)]

The difference between launching from X and from vt or remote is that there is a DISPLAY env var, and the new input platform probing will find that server-x11.so would be capable to provide input events. Hence it also loads it. This affects the shutdown in some way that the mgm::LinuxVirtualTerminal destructor is not executed.

[Andreas Pokorny (andreas-pokorny)]

Will prepare a bandaid and then look further into the problem

[PS Jenkins bot (ps-jenkins)]

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

[Daniel van Vugt (vanvugt)]

The fix is in lp:mir (0.19 series) only right now.

[Alan Griffiths (alan-griffiths)]

OK, but it is one of the fixes that 0.18 is held for

[Launchpad Janitor (janitor)]

This bug was fixed in the package mir - 0.18.0+16.04.20151216.1-0ubuntu1

---------------
mir (0.18.0+16.04.20151216.1-0ubuntu1) xenial; urgency=medium

  [ Kevin DuBois ]
  * New upstream release 0.18.0 (https://launchpad.net/mir/+milestone/0.18.0)
    - ABI summary: Only servers need rebuilding;
      . Mirclient ABI unchanged at 9
      . Mirserver ABI bumped to 36
      . Mircommon ABI unchanged at 5
      . Mirplatform ABI unchanged at 11
      . Mirprotobuf ABI unchanged at 3
      . Mirplatformgraphics ABI bumped to 7
      . Mirclientplatform ABI unchanged at 3
      . Mirinputplatform ABI added. Current version is 4
    - Enhancements:
      . Use libinput by default, and remove the android input stack
      . Add x11 input probing
      . Add alternative buffer swapping mechanism internally, available with
        --nbuffers 0
      . Automatic searching and selection of input platforms
      . Better support for themed cursors
      . Add demo client that uses multiple buffer streams in one surface
      . Improve fingerpaint demo to use touch pressure
      . Allow for configuring cursor acceleration, scroll speed and left or
        right handed mice
      . Allow for setting a base display configuration via client api
      . Various nested server multimonitor fixes and stability improvements
      . Remove DepthId from the SurfaceStack
    - Bug fixes:
      . Unit test failures in Display.* on Android (LP: #1519276)
      . Build failure due to missing dependency of client rpc code on mir
        protobuf (LP: #1518372)
      . Test failure in
        NestedServer.display_configuration_reset_when_application_exits
        (LP: #1517990)
      . CI test failures in various NesterServer tests (LP: #1517781)
      . FTBFS with -DMIR_PLATFORM=android (LP: #1517532)
      . Nesting Mir servers with assorted display configs causes lockup
        (LP: #1516670)
      . [testsfail] RaiseSurfaces.motion_events_dont_prevent_raise
        (LP: #1515931)
      . CI test failures in GLMark2Test (LP: #1515660)
      . Shells that inject user input events need to agree with the system
        compositor on the clock to use (LP: #1515515)
      . mircookie-dev is missing nettle-dev dependency (LP: #1514391)
      . Segmentation fault on server shutdown with mesa-kms (LP: #1513901)
      . mircookie requires nettle but libmircookie-dev doesn't depend on it
        (LP: #1513792)
      . libmircookie1 package does not list libnettle as dependency
        (LP: #1513225)
      . display configuration not reset when application exits (LP: #1511798)
      . unplugging external monitor causes nested server to throttle client
        (LP: #1511723)
      . 1/2 screen on external monitor (LP: #1511538)
      . unity-system-compositor crash, no interaction on windowed mode
        (LP: #1511095)
      . [regression] arm64/powerpc cross compile doesn't build any more
        (LP: #1510778)
      . mir_connection_get_egl_pixel_format() crashes if libEGL is loaded
        RTLD_LAZY (LP: #1510218)
      . [multimonitor] nested server surface positioning incorrect
        (LP: #1506846)
      . unity-system-compositor fails to build against lp:mir r3027
   ...