Need API to allow shell to choose custom OOM killer scores

Bug #1579799 reported by Gerry Boland
48
This bug affects 9 people
Affects Status Importance Assigned to Milestone
Canonical System Image
Fix Released
Wishlist
Unassigned
ubuntu-app-launch (Ubuntu)
Fix Released
Wishlist
Ted Gould

Bug Description

The desire is to tune Xapps to be killed last in life cycle preferences, since they are unable to restore their state.

Also shell needs to be able to set custom OOM scores for the application spawning a trusted prompt helper, as well as that trust helper.

Instead of UAL trying to deduce what the shell is doing and decide OOM scores, it would be better to allow the shell to decide the weighting applied.

Related branches

Revision history for this message
Ted Gould (ted) wrote :

Today UAL has basically three states for applications, on, off and paused. There are then OOM scores associated with those states. And it seems that this is requesting more states. But I'm not sure I understand. For instance, for X11 apps they're never paused, so their OOM score stays the same as the focused app, do we need a state where the focused app is killed before an X11 application? I also don't believe that applications that have a trusted overlay are paused, so I'm not sure how the scoring would work there either.

So what are the states that you see applications being in, and what would their scores be relative to one another?

Changed in ubuntu-app-launch (Ubuntu):
importance: Undecided → Wishlist
status: New → Incomplete
assignee: nobody → Ted Gould (ted)
Revision history for this message
Ted Gould (ted) wrote :

Spent a bit of time going back and forth on this with tvoss. We don't want to have a ton of application states, that'll make insanity, but it does make sense for qtmir to drive policy for applications. We don't know completely what that policy is today.

Long story short, the plan is to provide an API that is an enum with known values that can be overridden (it's base type will be int). In this way qtmir can experiment and then we can start to collect the values as we find them to be useful.

Ted Gould (ted)
Changed in ubuntu-app-launch (Ubuntu):
status: Incomplete → Confirmed
dobey (dobey)
Changed in ubuntu-app-launch (Ubuntu):
status: Confirmed → In Progress
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package ubuntu-app-launch - 0.9+16.10.20160825-0ubuntu1

---------------
ubuntu-app-launch (0.9+16.10.20160825-0ubuntu1) yakkety; urgency=medium

  * Use at() instead of operator[]() so that we get an exception on
    bounds checking

 -- Ted Gould <email address hidden> Thu, 25 Aug 2016 14:43:00 +0000

Changed in ubuntu-app-launch (Ubuntu):
status: In Progress → Fix Released
Changed in canonical-devices-system-image:
status: New → Fix Committed
importance: Undecided → Wishlist
milestone: none → 13
Revision history for this message
Gerry Boland (gerboland) wrote :

Thanks for this Ted!

Changed in canonical-devices-system-image:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.