nexus4 & nexus7 hotplug survival

Bug #1488863 reported by kevin gunn
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Canonical Pocket Desktop
Fix Released
Critical
kevin gunn
Mir
Fix Released
Critical
Unassigned
mir (Ubuntu)
Fix Released
Undecided
Unassigned
qtmir (Ubuntu)
Fix Released
Critical
Gerry Boland

Bug Description

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)

Related branches

kevin gunn (kgunn72)
tags: added: pd
tags: added: android multimonitor
kevin gunn (kgunn72)
Changed in mir:
importance: Undecided → High
Changed in qtmir:
importance: Undecided → High
Changed in canonical-pocket-desktop:
importance: Undecided → High
Changed in qtmir (Ubuntu):
importance: Undecided → High
Gerry Boland (gerboland)
Changed in mir:
status: New → Invalid
Changed in qtmir:
status: New → In Progress
Changed in canonical-pocket-desktop:
status: New → In Progress
Changed in qtmir:
assignee: nobody → Gerry Boland (gerboland)
Changed in canonical-pocket-desktop:
assignee: nobody → Gerry Boland (gerboland)
Changed in qtmir (Ubuntu):
status: New → In Progress
assignee: nobody → Gerry Boland (gerboland)
Michał Sawicz (saviq)
Changed in qtmir:
status: In Progress → Fix Released
Changed in canonical-pocket-desktop:
status: In Progress → Fix Committed
Changed in qtmir (Ubuntu):
status: In Progress → Fix Released
kevin gunn (kgunn72)
Changed in canonical-pocket-desktop:
assignee: Gerry Boland (gerboland) → kevin gunn (kgunn72)
status: Fix Committed → New
Changed in mir:
status: Invalid → New
Changed in qtmir:
status: Fix Released → New
Changed in qtmir (Ubuntu):
status: Fix Released → New
Revision history for this message
kevin gunn (kgunn72) wrote :

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)
summary: - nexus4 hotplug survival
+ nexus4 & nexus7 hotplug survival
Revision history for this message
kevin gunn (kgunn72) wrote :

unity8 log from a crash run on Nexus7

Revision history for this message
kevin gunn (kgunn72) wrote :

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

Revision history for this message
kevin gunn (kgunn72) wrote :

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

Changed in qtmir (Ubuntu):
importance: High → Critical
Changed in canonical-pocket-desktop:
importance: High → Critical
Changed in qtmir:
importance: High → Critical
Changed in mir:
importance: High → Critical
Revision history for this message
Launchpad Janitor (janitor) wrote :

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

Changed in qtmir (Ubuntu):
status: New → Fix Released
kevin gunn (kgunn72)
Changed in qtmir (Ubuntu):
status: Fix Released → New
Gerry Boland (gerboland)
description: updated
Revision history for this message
Gerry Boland (gerboland) wrote :

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

Revision history for this message
Gerry Boland (gerboland) wrote :

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

Changed in mir:
milestone: none → 0.18.0
Revision history for this message
Launchpad Janitor (janitor) wrote :

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

Changed in mir (Ubuntu):
status: New → Confirmed
Changed in qtmir (Ubuntu):
status: New → Confirmed
kevin gunn (kgunn72)
Changed in mir:
status: New → Fix Released
Changed in qtmir:
status: New → Fix Released
Changed in canonical-pocket-desktop:
status: New → Fix Released
Changed in mir (Ubuntu):
status: Confirmed → Fix Released
Changed in qtmir (Ubuntu):
status: Confirmed → Fix Released
Revision history for this message
Daniel van Vugt (vanvugt) wrote :

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

Changed in mir:
status: Fix Released → Incomplete
milestone: 0.18.0 → none
Changed in mir (Ubuntu):
status: Fix Released → Incomplete
Revision history for this message
Launchpad Janitor (janitor) wrote :
Download full text (6.3 KiB)

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

Read more...

Changed in mir (Ubuntu):
status: Incomplete → Fix Released
Changed in mir:
status: Incomplete → Fix Released
Michał Sawicz (saviq)
no longer affects: qtmir
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.