nexus4 & nexus7 hotplug survival

[kevin gunn]

with latest vivid+o
doing some hotplugging the phone seems robust, always changing modes from full screen to windowed mode, and no rebooting. However, the result of the hotplug (in) doesn't always seem to result in a frame on the monitor screen
Once in this state, I have to reboot the phone
it may be my specific monitor combination - but it seems much better if i am using power on my monitor to "connect", if the monitor is already on is when I believe i experience this issue, even tho the issue seems tied to the phone (since rebooting cures)

[kevin gunn (kgunn72)]

it would seem survival is less than 50%
i've tested with both nexus4 & 7 and frequently seeing a reboot on disconnect - it's usually preceeded by the billboard being display & some combination of screen blanking off/on
I have no crash filess present.
attached is the unity8.log of the run that rebooted (on nexus 4)

[kevin gunn (kgunn72)]

unity8 log from a crash run on Nexus7

[kevin gunn (kgunn72)]

...and the actual crash on Nexus7 (hard to say if the crash and the reboot are related per se)

[kevin gunn (kgunn72)]

unity8 log for the reboot scenario on Nexus4, i think is dubious...i pulled it again, attached here

[Launchpad Janitor (janitor)]

This bug was fixed in the package qtmir - 0.4.6+15.10.20151021-0ubuntu1

---------------
qtmir (0.4.6+15.10.20151021-0ubuntu1) wily; urgency=medium

  [ Alan Griffiths ]
  * Opaquify MirWindowManager to control visibility of upcoming Window
    Management work

  [ Albert Astals Cid ]
  * Don't search for the element again

  [ CI Train Bot ]
  * New rebuild forced.

  [ Daniel d'Andrada ]
  * Improve multimonitor support
  * MirSurfaceItem: Survive holding a surface with an empty texture
  * Shell draws its own cursor using the new Cursor QML element

  [ Gerry Boland ]
  * Initial multimonitor support - react correctly to Mir
    DisplayConfiguration changes. (LP: #1488831, #1488863, #1436735)
  * Workaround for AutoPilot input coordinate positioning being outside
    screen geometry

  [ Lukáš Tinkl ]
  * Implement support for mouse wheel events; correctly pass around
    buttons (LP: #1497091)
  * React to surface modifications (window caption)

  [ Michał Sawicz ]
  * Improve multimonitor support
  * MirSurfaceItem: Survive holding a surface with an empty texture
  * React to surface modifications (window caption)

  [ Nick Dedekind ]
  * Added touch performance tracing and test.
  * Removed the manipulation of the CMAKE_INSTALL_PREFIX from
    debian/rules

 -- Michał Sawicz <email address hidden> Wed, 21 Oct 2015 11:47:16 +0000

[Gerry Boland (gerboland)]

Doing some plug/unplug with Nexus7, I got a black screen situation. dmesg logs:

[ 4330.578460] kgsl kgsl-3d0: |kgsl_iommu_fault_handler| GPU PAGE FAULT: addr = C1D59A00 pid = 3052
[ 4330.578613] kgsl kgsl-3d0: |kgsl_iommu_fault_handler| context = 0 FSR = 4001000A FSYNR0 = E000008 FSYNR1 = 443542(read fault)
[ 4330.578735] kgsl kgsl-3d0: ---- premature free ----
[ 4330.578826] kgsl kgsl-3d0: [C1700000-C1F70000] (egl_surface) was already freed by pid 3052
[ 4330.578948] kgsl kgsl-3d0: ---- nearby memory ----
[ 4330.579132] kgsl kgsl-3d0: [C1700000 - C1D58000] (+guard) (pid = 3052) (egl_surface)
[ 4330.579223] kgsl kgsl-3d0: <- fault @ C1D59A00
[ 4330.579345] kgsl kgsl-3d0: *EMPTY*

[ 4330.821563] kgsl kgsl-3d0: unity8-dash[3052]: gpu fault ctx 4 ts 2755 status F5C7C003 rb 0ab0/0c03 ib1 a746194c/02e7 ib2 a7fe3000/05e0
[ 4330.859558] kgsl kgsl-3d0: |adreno_snapshot| GPU snapshot froze 1096Kb of GPU buffers
[ 4330.859771] kgsl kgsl-3d0: |kgsl_device_snapshot| snapshot created at pa af380000 size 148328
[ 4330.860046] kgsl kgsl-3d0: |kgsl_iommu_clk_disable_event| IOMMU disable clock event being cancelled, iommu_last_cmd_ts: 130d8, retired ts: 130d3
[ 4330.870727] kgsl kgsl-3d0: |kgsl_snapshot_save_frozen_objs| Allocated memory for snapshot objects at address f47ff000, size 24eb0c
[ 4330.961547] kgsl kgsl-3d0: unity8-dash[3052]: gpu failed ctx 4 ts 2755

[ 4350.961486] kgsl kgsl-3d0: |adreno_reset| Device soft reset failed
[ 4350.961700] kgsl kgsl-3d0: |kgsl_iommu_clk_disable_event| IOMMU disable clock event being cancelled, iommu_last_cmd_ts: 130e9, retired ts: 130db

[Gerry Boland (gerboland)]

USC log:
ERROR: /build/mir-U8WSgm/mir-0.17.0+15.04.20151008.2/src/platforms/android/server/gl_context.cpp(191): Throw in function virtual void mir::graphics::android::FramebufferGLContext::swap_buffers() const
Dynamic exception type: N5boost16exception_detail10clone_implINS0_19error_info_injectorISt12system_errorEEEE
std::exception::what: eglSwapBuffers failure: EGL_CONTEXT_LOST (0x300e)

Failed to connect to server socket: No such file or directory

[Launchpad Janitor (janitor)]

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

[Daniel van Vugt (vanvugt)]

No change ever applied for Mir. Also 0.18.0 is not release yet :)

[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
   ...