Bug #1304257 “[enhancement] There should be a quit signal sent t...” : Bugs : unity-mir package : Ubuntu

Revision history for this message

Michał Sawicz (saviq) wrote on 2014-04-08:

#1

Dependencies.txt Edit (7.1 KiB, text/plain; charset="utf-8")
ProcEnviron.txt Edit (317 bytes, text/plain; charset="utf-8")

Revision history for this message

Robert Carr (robertcarr) wrote on 2014-04-08:

#2

I wonder if it should be lifecycle?

Gerry Boland (gerboland) on 2014-04-08

Changed in unity-mir (Ubuntu):
status:	New → Confirmed
importance:	Undecided → Medium
Changed in unity-mir:
status:	New → Confirmed
importance:	Undecided → Medium

Revision history for this message

Daniel van Vugt (vanvugt) wrote on 2014-04-08:

#3

Signals are very different to messages. The former involves just per-app/client work. The latter requires some enhancement to the server and MirEvent. I think we need both.

Changed in mir:
status:	New → Triaged
importance:	Undecided → Medium
summary:	- There should be a quit signal sent to sessions instead of killing them - directly + [enhancement] There should be a quit signal sent to sessions instead of + killing them directly
tags:	added: enhancement

Revision history for this message

Launchpad Janitor (janitor) wrote on 2014-04-08:

#4

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

Changed in mir (Ubuntu):
status:	New → Confirmed
Changed in ubuntu-system-settings (Ubuntu):
status:	New → Confirmed

Sebastien Bacher (seb128) on 2014-06-27

affects:

ubuntu-system-settings (Ubuntu) → system-image (Ubuntu)

Revision history for this message

Ted Gould (ted) wrote on 2014-06-27:

#6

Currently upstart to stop a job sends SIGTERM and then waits 5 seconds and sends SIGKILL. I think the issue is that we need to send SIGCONT first for it to get the TERM.

Daniel van Vugt (vanvugt) on 2014-12-01

Changed in mir:
assignee:	nobody → Alexandros Frantzis (afrantzis)
milestone:	none → 0.10.0
status:	Triaged → Fix Committed

Revision history for this message

Launchpad Janitor (janitor) wrote on 2015-01-08:

#7

Download full text (5.2 KiB)

This bug was fixed in the package mir - 0.10.0+15.04.20150107.2-0ubuntu1

---------------
mir (0.10.0+15.04.20150107.2-0ubuntu1) vivid; urgency=medium

  [ Daniel van Vugt ]
  * New upstream release 0.10.0 (https://launchpad.net/mir/+milestone/0.10.0)
    - Enhancements:
      . Added support for Android HWC 1.3 devices.
      . Plumbing/preparation to support external displays on Android devices.
      . Reduced build dependencies.
      . Client API: Added version macros.
      . Began work on automatic driver probing, to intelligently choose the
        best driver for you.
      . Demo shell (mir_proving_server): Added desktop zoom feature using
        Super + mouse wheel.
      . Demo renamed: mir_demo_server_shell -> mir_proving_server
      . Other demo servers merged into -> mir_demo_server
      . Wider support for display buffer pixel formats in the mesa driver, for
        wider hardware support.
      . Performance: On mesa/desktop at least; only hold compositor buffers
        for the duration of the render, instead of the duration of the frame.
        Following this change the compositor report can now finally report
        render time instead of frame time.
      . Mir now starts reliably when a TV is connected by HDMI, and up to
        4K resolution (2160p) is known to work.
      . Plenty more enhancements logged in the bugs list below.
    - ABI summary: Servers need rebuilding, but clients do not;
      . Mirclient ABI unchanged at 8
      . Mircommon ABI unchanged at 3
      . Mirplatform ABI bumped to 5
      . Mirserver ABI bumped to 28
    - Bug fixes:
      . [regression] Mir servers (since 0.9) randomly crash in malloc due to
        heap corruption (LP: #1401488)
      . USC - mouse cursor on AMD graphics is drawing incorrectly
        (LP: #1391975)
      . Mir fails to start when a TV is connected by HDMI
        [std::exception::what: Invalid or inconsistent display configuration]
        (LP: #1395405)
      . Input/event driven clients may freeze indefinitely (LP: #1396006)
      . Mir server crashes with "std::exception::what: Failed to get front
        buffer object" when trying to fullscreen a surface (LP: #1398296)
      . Switching windows with a Trusted Prompt Session active loses the
        trusted prompt session (LP: #1355173)
      . CI test failure in multiple tests (LP: #1401364)
      . dh_install: usr/bin/mir_demo_server exists in debian/tmp but is not
        installed to anywhere (LP: #1401365)
      . [regression] demo-shell: Instead of moving surfaces they now fly
        off-screen (LP: #1403702)
      . [regression] Binaries are no longer runnable on other machines (or in
        other directories) (LP: #1406073)
      . [i865] unity-system-compositor fails to start: Failed to choose ARGB
        EGL config (LP: #1212753)
      . Mir's compositor holds buffers (blocking clients) for the duration of
        the frame, even when not necessary. (LP: #1264934)
      . Screen goes blank (black) briefly during display config changes which
        don't affect the display mode (LP: #1274359)
      . [enhancement] There should be a quit signal sent to sessions instead
        of killing them dir...

This bug was fixed in the package mir - 0.10.0+15.04.20150107.2-0ubuntu1

---------------
mir (0.10.0+15.04.20150107.2-0ubuntu1) vivid; urgency=medium

[ Daniel van Vugt ]
  * New upstream release 0.10.0 (https://launchpad.net/mir/+milestone/0.10.0)
    - Enhancements:
      . Added support for Android HWC 1.3 devices.
      . Plumbing/preparation to support external displays on Android devices.
      . Reduced build dependencies.
      . Client API: Added version macros.
      . Began work on automatic driver probing, to intelligently choose the
        best driver for you.
      . Demo shell (mir_proving_server): Added desktop zoom feature using
        Super + mouse wheel.
      . Demo renamed: mir_demo_server_shell -> mir_proving_server
      . Other demo servers merged into -> mir_demo_server
      . Wider support for display buffer pixel formats in the mesa driver, for
        wider hardware support.
      . Performance: On mesa/desktop at least; only hold compositor buffers
        for the duration of the render, instead of the duration of the frame.
        Following this change the compositor report can now finally report
        render time instead of frame time.
      . Mir now starts reliably when a TV is connected by HDMI, and up to
        4K resolution (2160p) is known to work.
      . Plenty more enhancements logged in the bugs list below.
    - ABI summary: Servers need rebuilding, but clients do not;
      . Mirclient ABI unchanged at 8
      . Mircommon ABI unchanged at 3
      . Mirplatform ABI bumped to 5
      . Mirserver ABI bumped to 28
    - Bug fixes:
      . [regression] Mir servers (since 0.9) randomly crash in malloc due to
        heap corruption (LP: #1401488)
      . USC - mouse cursor on AMD graphics is drawing incorrectly
        (LP: #1391975)
      . Mir fails to start when a TV is connected by HDMI
        [std::exception::what: Invalid or inconsistent display configuration]
        (LP: #1395405)
      . Input/event driven clients may freeze indefinitely (LP: #1396006)
      . Mir server crashes with "std::exception::what: Failed to get front
        buffer object" when trying to fullscreen a surface (LP: #1398296)
      . Switching windows with a Trusted Prompt Session active loses the
        trusted prompt session (LP: #1355173)
      . CI test failure in multiple tests (LP: #1401364)
      . dh_install: usr/bin/mir_demo_server exists in debian/tmp but is not
        installed to anywhere (LP: #1401365)
      . [regression] demo-shell: Instead of moving surfaces they now fly
        off-screen (LP: #1403702)
      . [regression] Binaries are no longer runnable on other machines (or in
        other directories) (LP: #1406073)
      . [i865] unity-system-compositor fails to start: Failed to choose ARGB
        EGL config (LP: #1212753)
      . Mir's compositor holds buffers (blocking clients) for the duration of
        the frame, even when not necessary. (LP: #1264934)
      . Screen goes blank (black) briefly during display config changes which
        don't affect the display mode (LP: #1274359)
      . [enhancement] There should be a quit signal sent to sessions instead
        of killing them directly (LP: #1304257)
      . MirMotionEvent.action needs stronger typing (to MirMotionAction etc)
        (LP: #1311699)
      . CompositorReport as used by DefaultDisplayBufferCompositor can't
        measure render time (LP: #1350716)
      . Full screen (bypassed) surfaces (e.g. GLMark2Test) are missing frames
        and appear to freeze or judder with swap interval 0 (LP: #1379685)
      . Trusted prompts need to be part of the lifecycle (LP: #1384950)
      . [testfail] BasicThreadPool.recycles_threads in CI (LP: #1391488)
      . acceptance_tests are too chatty (LP: #1394221)
      . mir_connection_create_surface callback is sometimes called twice on
        error (LP: #1394873)
      . File descriptor leaks in tests using UsingStubClientPlatform
        (LP: #1395762)
      . DisplayLayout resizes a surface to 1x1 if you ask it to fullscreen a
        surface that's partially offscreen (LP: #1398294)
      . Surfaces can consume input events before they're visible.
        (LP: #1400218)
      . dpkg-shlibdeps: Lots of warnings about libmirplatformstub.so
        (LP: #1401373)
      . Leaks in death tests can cause subsequent tests in the same process to
        fail (LP: #1402160)
      . [regression] lintian: E: mir-demos: binary-or-shlib-defines-rpath ...
        (LP: #1406098)
      . [regression] Mir utils can't run from the build tree any more
        (LP: #1407557)
      . fd reception code is not exeception-safe when unexpected numbers of
        fds are received (LP: #1394362)
      . Mir reports vertical refresh rates slightly inaccurately (LP: #1407558)
      . [Enhancement] Add an API to lock surface orientation (LP: #1382209)
      . Bootloop with system language Turkish on the Nexus 4 (LP: #1398984)
      . Remove the implicit assumption that there every surface can be mapped
        to an input handle. (LP: #1216727)
      . When revealing hidden surfaces wait for them to become exposed before
        sending events which we expect them to receive (LP: #1407783)

[ Ubuntu daily release ]
  * New rebuild forced
 -- Ubuntu daily release <ps-jenkins@lists.canonical.com>   Wed, 07 Jan 2015 23:50:16 +0000

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

Daniel van Vugt (vanvugt) on 2015-01-09

Changed in mir:
status:	Fix Committed → Fix Released

Revision history for this message

Barry Warsaw (barry) wrote on 2015-05-11:

#8

This shouldn't be an issue for system-image. it should respond properly to SIGTERM.

no longer affects:

system-image (Ubuntu)

Revision history for this message

Daniel van Vugt (vanvugt) wrote on 2016-05-23:

#9

Please don't make this bug a duplicate. It had an explicit fix committed already back in 2014.

The QtMir part is bug 1434584.

Changed in qtmir:
status:	New → Invalid
Changed in unity-mir:
status:	Confirmed → Invalid
Changed in unity-mir (Ubuntu):
status:	Confirmed → Invalid

Michał Sawicz (saviq) on 2017-03-13

affects:

qtmir → qtmir (Ubuntu)

Ubuntu
unity-mir package

[enhancement] There should be a quit signal sent to sessions instead of killing them directly

Bug Description

Related branches

Other bug subscribers

Bug attachments

Remote bug watches

	Status	Importance	Assigned to	Milestone
Mir	Fix Released	Medium	Alexandros Frantzis	Mir 0.10.0
unity-mir	Invalid	Medium	Unassigned
mir (Ubuntu)	Fix Released	Undecided	Unassigned
qtmir (Ubuntu)	Invalid	Undecided	Unassigned
unity-mir (Ubuntu)	Invalid	Medium	Unassigned

Ubuntuunity-mir package

[enhancement] There should be a quit signal sent to sessions instead of killing them directly

Bug Description

Related branches

Other bug subscribers

Bug attachments

Remote bug watches

Ubuntu
unity-mir package