Bug 1214355 - Add systemd-oomd to default installation
Summary: Add systemd-oomd to default installation
Status: RESOLVED WONTFIX
Alias: None
Product: openSUSE Tumbleweed
Classification: openSUSE
Component: Basesystem (show other bugs)
Version: Current
Hardware: Other Other
: P5 - None : Enhancement (vote)
Target Milestone: ---
Assignee: systemd maintainers
QA Contact: E-mail List
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2023-08-17 09:33 UTC by Enrico Belleri
Modified: 2023-08-22 12:07 UTC (History)
2 users (show)

See Also:
Found By: ---
Services Priority:
Business Priority:
Blocker: ---
Marketing QA Status: ---
IT Deployment: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Enrico Belleri 2023-08-17 09:33:05 UTC
systemd-oomd is a userspace out-of-memory killer.
It is currently made available in the systemd-experimental package, following https://bugzilla.opensuse.org/show_bug.cgi?id=1200456 which occurred already 1 year ago-

In Fedora this service is included and enabled by default.

I suggest promoting it from the systemd-experimental.

I haven't done extensive testing myself, so I would propose for now to create a new package called systed-oomd.
At the moment, it is bundled with repart, measure, pcrphase, ukify which I shouldn't need to install all simply to use oomd.
Comment 1 Enrico Belleri 2023-08-17 16:47:26 UTC
Build test here: https://build.opensuse.org/project/show/home:iDesmI:oomd
Comment 2 Franck Bui 2023-08-22 09:41:23 UTC
Last time we checked, we were not sure about oomd defaults as they could trigger false-positives in some cases. That's why it's still in the experimental package.

Antonio since you checked last time, can you drop some words ?
Comment 3 Franck Bui 2023-08-22 09:45:36 UTC
(In reply to Enrico Belleri from comment #0)
> I haven't done extensive testing myself, so I would propose for now to
> create a new package called systed-oomd.

If a feature is not tested we should keep it as part of the experimental stuff so we can still change things later even if we break backward compat.

If we move it into a dedicated package (I'm not sure it needs a new one though) we would loose this flexibility.


> At the moment, it is bundled with repart, measure, pcrphase, ukify which I
> shouldn't need to install all simply to use oomd.

That shouldn't be a big deal.
Comment 4 Antonio Feijoo 2023-08-22 10:18:16 UTC
Fedora includes systemd-oomd since 2.5 years ago, they changed the upstream default parameters back and forth and still haven't gotten to the point where this works smoothly. Apparently the default behavior is very aggressive in desktop environments, killing applications that need a lot of memory to run.

You can follow their issue here: https://pagure.io/fedora-workstation/issue/358 (still with new complaints after the latest configuration change).

BTW, the TW kernel does not enable PSI by default, it's configured with `CONFIG_PSI_DEFAULT_DISABLED=y`.
Comment 5 Enrico Belleri 2023-08-22 11:50:29 UTC
(In reply to Antonio Feijoo from comment #4)
> BTW, the TW kernel does not enable PSI by default, it's configured with
> `CONFIG_PSI_DEFAULT_DISABLED=y`.

It's enough to add `psi=1` to kernel parameters to enable it.

I understand the downsides of having oomd enabled, this was an attempt to make my usecase the default but it's okay to leave it to the avant-garde Fedora for now. I am going to follow the Fedora defaults for a while.

Thank you for the replies!
Comment 6 Franck Bui 2023-08-22 12:07:08 UTC
(In reply to Enrico Belleri from comment #5)
> I understand the downsides of having oomd enabled, this was an attempt to
> make my usecase the default but it's okay to leave it to the avant-garde
> Fedora for now. I am going to follow the Fedora defaults for a while.

Feel free to provide some feedback with these defaults.

Thanks.