Bug 1214355

Summary:	Add systemd-oomd to default installation
Product:	[openSUSE] openSUSE Tumbleweed	Reporter:	Enrico Belleri <idesmi>
Component:	Basesystem	Assignee:	systemd maintainers <systemd-maintainers>
Status:	RESOLVED WONTFIX	QA Contact:	E-mail List <qa-bugs>
Severity:	Enhancement
Priority:	P5 - None	CC:	antonio.feijoo, fbui
Version:	Current
Target Milestone:	---
Hardware:	Other
OS:	Other
Whiteboard:
Found By:	---	Services Priority:
Business Priority:		Blocker:	---
Marketing QA Status:	---	IT Deployment:	---

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.