[systemd-devel] [PATCH 1/1] RFC: Set the Default OOM Score from configuration file

[Lennart Poettering]

On Sun, 28.04.13 20:37, Umut Tezduyar (umut at tezduyar.com) wrote:

> Hi,
> 
> Should DefaultOOMScore= and DefaultLimit***= be really under [Manager]?
> Wouldn't it be better to have an [Exec] section. Or even have [Service],
> [Mount],.. sections that can give separate configuration per exec
> unit.

Hmm, the fields without "Default" prefix are applied to PID1 itself,
too, hence they definitely belong into [Manager] I'd say.

For the ones prefix with Default, we could probably move them to a new
section [Units] or so. But we'd probably have to do that in a compatible
way, so this doesn't break existing systems. And that's not too
nice. Not sure whether it is worth that effort?

> Also, there are many configuration options per exec units that are not in
> the system.conf. Are we open taking patches on completing missing
> configuration options?

Actually, the last time I checked them there were only very few where a
default in system.conf really makes sense I think. Which ones do you
have in mind?

I mean, we can surely add more, but I let's always come up with a useful
usecase first, please...

Lennart

-- 
Lennart Poettering - Red Hat, Inc.