Re: [systemd-devel] [PATCH] core/cryptsetup: Add WantsMountFor option to enable fallback to password request for crypt mounts.

2014-05-21 Thread Lennart Poettering
On Sun, 18.05.14 19:36, Przemek Rudy (pru...@o2.pl) wrote: > diff --git a/src/core/automount.c b/src/core/automount.c > index 65e6d6f..d4359b9 100644 > --- a/src/core/automount.c > +++ b/src/core/automount.c > @@ -124,7 +124,7 @@ static int automount_add_mount_links(Automount *a) { > if (

Re: [systemd-devel] Start-up resource and prioritization control

2014-05-21 Thread Lennart Poettering
On Thu, 24.04.14 11:15, Umut Tezduyar Lindskog (u...@tezduyar.com) wrote: > b) The ongoing patch > http://lists.freedesktop.org/archives/systemd-devel/2014-March/018220.html > is promising but it seems to be stopped. Any reason? Well, I am still working on processing the backlog unmerged patches.

Re: [systemd-devel] [PATCH] Fix a few compiler warnings

2014-05-21 Thread Thomas H.P. Andersen
On Tue, May 20, 2014 at 5:43 PM, Lennart Poettering wrote: > On Mon, 19.05.14 19:52, Tom Gundersen (t...@jklm.no) wrote: > >> > _public_ int sd_peer_get_session(int fd, char **session) { >> > -struct ucred ucred; >> > +struct ucred ucred = {}; >> >> I can't reproduce this warning,

Re: [systemd-devel] Start-up resource and prioritization control

2014-05-21 Thread Lennart Poettering
On Tue, 20.05.14 11:33, Umut Tezduyar Lindskog (u...@tezduyar.com) wrote: > > Wouldn't thes easily deadlock? Imagine you have two services on your > > system A and B. Each of them needs to communicate with the other it > > would become fully active. If your limit of active jobs is 2 there is > > n

Re: [systemd-devel] Start-up resource and prioritization control

2014-05-21 Thread Lennart Poettering
On Tue, 20.05.14 15:16, Umut Tezduyar Lindskog (u...@tezduyar.com) wrote: > > Wouldn't this be solved by telling the kernel to schedule the starting > > services with high latency (or whatever the terminology is), i.e., > > give each of them a relatively large timeslice. That would decrease > > th

Re: [systemd-devel] [PATCH] Fix a few compiler warnings

2014-05-21 Thread Lennart Poettering
On Wed, 21.05.14 10:29, Thomas H.P. Andersen (pho...@gmail.com) wrote: > > On Tue, May 20, 2014 at 5:43 PM, Lennart Poettering > wrote: > > On Mon, 19.05.14 19:52, Tom Gundersen (t...@jklm.no) wrote: > > > >> > _public_ int sd_peer_get_session(int fd, char **session) { > >> > -struct uc

[systemd-devel] [PATCH 1/1] networkd: sit-tunnel add support for pmtudisc

2014-05-21 Thread Susant Sahani
V2 fixed the uninitialized variable pmtudisc. This patch adds path of mtu discovery for sit tunnel. To enable/disable DiscoverPathMTU is introduced. Example configuration file: sit.netdev [NetDev] Name=sit-tun Kind=sit MTUBytes=1480 [Tunnel] DiscoverPathMTU=1 Local=X.X.X.X Remote=X.X.X.X By d

[systemd-devel] timesyncd, dnssec

2014-05-21 Thread Jason A. Donenfeld
Hey Lennart & Folks, I've been watching the commits come in for timesyncd. I saw recently you're restoring the time from file timestamps, just so something is decently recent. Neat. I thought I'd share a problem I had with another init system with regards to NTP and DNSSEC, so that it's somewhere

Re: [systemd-devel] Start-up resource and prioritization control

2014-05-21 Thread Umut Tezduyar Lindskog
On Wed, May 21, 2014 at 10:42 AM, Lennart Poettering wrote: > On Tue, 20.05.14 15:16, Umut Tezduyar Lindskog (u...@tezduyar.com) wrote: > >> > Wouldn't this be solved by telling the kernel to schedule the starting >> > services with high latency (or whatever the terminology is), i.e., >> > give ea

Re: [systemd-devel] [PATCH] core: add startup resource control option

2014-05-21 Thread Umut Tezduyar Lindskog
Hi Cho, Do you have any technical reason why CPU shares are reverted back to the default value on "StartupFinished" but instead when the unit is in "activated" state? Through out the boot, until StartupFinished, a unit might get activated quite early but still having a higher CPU share. If the se

Re: [systemd-devel] [RFC PATCH] Fix so install will work without 'ln --relative' support

2014-05-21 Thread Umut Tezduyar Lindskog
On Tue, May 20, 2014 at 5:59 PM, Michael Biebl wrote: > 2014-05-20 10:58 GMT+02:00 Umut Tezduyar Lindskog : >> On Fri, May 16, 2014 at 4:14 PM, Lennart Poettering >> wrote: >>> If it is desirable to cross-build systemd on such an old distribution >>> I'd recommend sticking an alternative "ln" imp

Re: [systemd-devel] [PATCH] core: add startup resource control option

2014-05-21 Thread WaLyong Cho
On 05/21/2014 09:02 PM, Umut Tezduyar Lindskog wrote: > Hi Cho, > > Do you have any technical reason why CPU shares are reverted back to > the default value on "StartupFinished" but instead when the unit is in > "activated" state? Sorry, I couldn't understand what you say. Did you mean if service

Re: [systemd-devel] [PATCH] core: add startup resource control option

2014-05-21 Thread Umut Tezduyar Lindskog
On Wed, May 21, 2014 at 3:54 PM, WaLyong Cho wrote: > On 05/21/2014 09:02 PM, Umut Tezduyar Lindskog wrote: >> Hi Cho, >> >> Do you have any technical reason why CPU shares are reverted back to >> the default value on "StartupFinished" but instead when the unit is in >> "activated" state? > > Sorr

[systemd-devel] writing service file that used some /proc/cmdline variable

2014-05-21 Thread Vasiliy Tolstov
Hello. I'm need to write specific service file that using /proc/cmdline variable (not standard, my own). Does systemd already have parsed array or how can i do that? -- Vasiliy Tolstov, e-mail: v.tols...@selfip.ru jabber: v...@selfip.ru ___ systemd-deve

Re: [systemd-devel] writing service file that used some /proc/cmdline variable

2014-05-21 Thread Ran Benita
On Wed, May 21, 2014 at 09:38:43PM +0400, Vasiliy Tolstov wrote: > Hello. I'm need to write specific service file that using > /proc/cmdline variable (not standard, my own). > Does systemd already have parsed array or how can i do that? ConditionKernelCommandLine= See systemd.unit(5) _

Re: [systemd-devel] writing service file that used some /proc/cmdline variable

2014-05-21 Thread Ran Benita
On Wed, May 21, 2014 at 08:44:22PM +0300, Ran Benita wrote: > On Wed, May 21, 2014 at 09:38:43PM +0400, Vasiliy Tolstov wrote: > > Hello. I'm need to write specific service file that using > > /proc/cmdline variable (not standard, my own). > > Does systemd already have parsed array or how can i do

[systemd-devel] systemd-nspawn 208 fails to mount special filesystems

2014-05-21 Thread Leho Kraav
Hi all I've been trying to solve this for several hours over multiple days now and am at the end of my wits, so asking for help. One machine, my laptop, boots "systemd-nspawn -D /srv/canister -b" perfectly fine. All services load OK in a blink of an eye, login prompt appears, all is good. E

Re: [systemd-devel] writing service file that used some /proc/cmdline variable

2014-05-21 Thread Vasiliy Tolstov
2014-05-21 21:49 GMT+04:00 Ran Benita : > Oops, I thought you meant a condition. I don't know if there's a > a way to *use* the variables directly in the service files, but it can > be done easily enough with a shell-script wrapper. Yes, i think about wrapper, but systemd contains many useful fe

Re: [systemd-devel] Start-up resource and prioritization control

2014-05-21 Thread Uoti Urpala
On Tue, 2014-05-20 at 15:16 +0200, Umut Tezduyar Lindskog wrote: > On Tue, May 20, 2014 at 1:46 PM, Tom Gundersen wrote: > > Wouldn't this be solved by telling the kernel to schedule the starting > > services with high latency (or whatever the terminology is), i.e., > > give each of them a relativ

Re: [systemd-devel] [PATCH] Fix a few compiler warnings

2014-05-21 Thread Thomas H.P. Andersen
On Wed, May 21, 2014 at 10:46 AM, Lennart Poettering wrote: > On Wed, 21.05.14 10:29, Thomas H.P. Andersen (pho...@gmail.com) wrote: > >> >> On Tue, May 20, 2014 at 5:43 PM, Lennart Poettering >> wrote: >> > On Mon, 19.05.14 19:52, Tom Gundersen (t...@jklm.no) wrote: >> > >> >> > _public_ int sd

Re: [systemd-devel] writing service file that used some /proc/cmdline variable

2014-05-21 Thread Kirill Elagin
Deja vu ;). http://lists.freedesktop.org/archives/systemd-devel/2014-May/019136.html So, yes, wrapper seems to be the way to go. -- Кирилл Елагин On Wed, May 21, 2014 at 10:18 PM, Vasiliy Tolstov wrote: > 2014-05-21 21:49 GMT+04:00 Ran Benita : > > Oops, I thought you meant a condition. I do

Re: [systemd-devel] systemd-nspawn 208 fails to mount special filesystems

2014-05-21 Thread Lennart Poettering
On Wed, 21.05.14 20:51, Leho Kraav (l...@kraav.com) wrote: > Hi all > > > I've been trying to solve this for several hours over multiple days > now and am at the end of my wits, so asking for help. > > One machine, my laptop, boots "systemd-nspawn -D /srv/canister -b" > perfectly fine. All serv

Re: [systemd-devel] [PATCH] core: add startup resource control option

2014-05-21 Thread Lennart Poettering
On Fri, 16.05.14 00:09, WaLyong Cho (walyong@samsung.com) wrote: Heya, > Similar to CPUShares= and BlockIOWeight= respectively. However only > assign the specified weight during startup. Each control group > attribute is re-assigned as weight by CPUShares=weight and > BlockIOWeight=weight aft

Re: [systemd-devel] Start-up resource and prioritization control

2014-05-21 Thread Lennart Poettering
On Wed, 21.05.14 13:52, Umut Tezduyar Lindskog (u...@tezduyar.com) wrote: > > Not following here... Aren't you describing a best-effort system here? > > But the CPUShares= stuff is best-effort stuff anyway, and just tells the > > kernel what is more important thant other stuff. Or, to turn this >

Re: [systemd-devel] [PATCH] core: add startup resource control option

2014-05-21 Thread Lennart Poettering
On Wed, 21.05.14 14:02, Umut Tezduyar Lindskog (u...@tezduyar.com) wrote: > > Hi Cho, > > Do you have any technical reason why CPU shares are reverted back to > the default value on "StartupFinished" but instead when the unit is in > "activated" state? Well, the startup is only considered finis

Re: [systemd-devel] [PATCH 1/1] networkd: sit-tunnel add support for pmtudisc

2014-05-21 Thread Lennart Poettering
On Wed, 21.05.14 14:43, Susant Sahani (sus...@redhat.com) wrote: > diff --git a/src/network/networkd-netdev.c b/src/network/networkd-netdev.c > index c54b0c1..63f8ff7 100644 > --- a/src/network/networkd-netdev.c > +++ b/src/network/networkd-netdev.c > @@ -545,6 +545,7 @@ static int netdev_load_one

Re: [systemd-devel] [PATCH] build: put missing KMOD_CFLAGS into Makefile

2014-05-21 Thread Lennart Poettering
On Tue, 20.05.14 23:31, Jan Engelhardt (jeng...@inai.de) wrote: > The build fails if kmod is not in a default location. Thanks! Applied! Lennart -- Lennart Poettering, Red Hat ___ systemd-devel mailing list systemd-devel@lists.freedesktop.org http:/

Re: [systemd-devel] [PATCH] tests: fix test-rtnl-manual.c compile failure

2014-05-21 Thread Lennart Poettering
On Mon, 31.03.14 15:23, Cristian Rodríguez (crrodrig...@opensuse.org) wrote: > Needs KMOD_CFLAGS Is this still an issue? I assume not, right? Otherwise, please resend! > --- > Makefile.am | 4 > 1 file changed, 4 insertions(+) > > diff --git a/Makefile.am b/Makefile.am > index 6c7d6e5..77

Re: [systemd-devel] [PATCH 1/3] nspawn: move container wait logic into wait_for_container() function

2014-05-21 Thread Lennart Poettering
On Fri, 11.04.14 01:45, Djalal Harouni (tix...@opendz.org) wrote: > Move the container wait logic into its own wait_for_container() function > and add two status codes: CONTAINER_TERMINATED or CONTAINER_REBOOTED > > These status codes are used to terminate nspawn or loop again in case of > CONTAI

Re: [systemd-devel] [PATCH] build: put missing KMOD_CFLAGS into Makefile

2014-05-21 Thread Jan Engelhardt
On Thursday 2014-05-22 02:27, Lennart Poettering wrote: >On Tue, 20.05.14 23:31, Jan Engelhardt (jeng...@inai.de) wrote: > >> The build fails if kmod is not in a default location. > >Thanks! >Applied! Have you thought about just putting everything into AM_CFLAGS=, so that you do not need per-targe

Re: [systemd-devel] [RFC] nspawn: make nspawn robust to container failure

2014-05-21 Thread Lennart Poettering
On Fri, 02.05.14 16:45, Djalal Harouni (tix...@opendz.org) wrote: > nspawn and the container child use eventfd to wait and notify each other > that they are ready so the container setup can be completed. Looks good, but doesn't apply anymore, could you rebase please? Sorry for the delay in revie

[systemd-devel] bulid: linking order is irrelevant

2014-05-21 Thread Jan Engelhardt
commit ffcf82d250e95ea0a06a95d7adc72bfad6db51e3 Author: Kay Sievers Date: Thu May 22 09:41:32 2014 +0900 build-sys: fix linking order ./.libs/libsystemd-network.a(libsystemd_network_la-network-internal.o): network-internal.c:function net_get_unique_predictable_data: er

Re: [systemd-devel] [PATCH v2] nspawn: move container wait logic into wait_for_container()

2014-05-21 Thread Lennart Poettering
On Fri, 02.05.14 12:05, Djalal Harouni (tix...@opendz.org) wrote: Ah, thanks for changing the clobbering thing. Should have read the rest of the thread before replying to the earlier patch... (making the enum anonymous might still be a an option though, but I will merge it even at is). Sorry for

Re: [systemd-devel] bulid: linking order is irrelevant

2014-05-21 Thread Kay Sievers
On Thu, May 22, 2014 at 9:53 AM, Jan Engelhardt wrote: > Linking order is not a problem with .la files, as .la files track > their dependencies -- just like .so files. It is a generally a problem that the build system tools do not solve. We rely on gc-sections, platforms without support for it f

Re: [systemd-devel] simple way to crash systemd via a dangling symlink

2014-05-21 Thread Michael Biebl
Filed this as https://bugs.freedesktop.org/show_bug.cgi?id=76899 Would be great to have someone look at it. Having systemd crash due to something simple as a dangling symlink is pretty bad imho. 2014-03-27 5:18 GMT+01:00 Michael Biebl : > There is an interesting bug which can be used to crash sys

Re: [systemd-devel] [PATCH 1/1] networkd: sit-tunnel add support for pmtudisc

2014-05-21 Thread Susant Sahani
> +netdev->tunnel_pmtudisc = 1; Should be "true", not "1". If something is of type "bool" we should use "true" for setting it and "false" for unsetting it. And this is of type "bool". Ok . > static int netdev_fill_sit_rtnl_message(Link *link, sd_rtnl_message *m) { > NetDev *net

Re: [systemd-devel] [PATCH 1/1] networkd: sit-tunnel add support for pmtudisc

2014-05-21 Thread Susant Sahani
On 05/22/2014 08:32 AM, Susant Sahani wrote: static int netdev_fill_sit_rtnl_message(Link *link, sd_rtnl_message *m) { NetDev *netdev; +uint8_t pmtudisc = 1; int r; assert(link); @@ -207,6 +208,17 @@ static int netdev_fill_sit_rtnl_message(Link *link, sd

Re: [systemd-devel] timesyncd, dnssec

2014-05-21 Thread David Timothy Strauss
On Wed, May 21, 2014 at 5:48 AM, Jason A. Donenfeld wrote: > Temporary work around is to hard code IPs into NTP. It'd be neat to do the following: (1) Do a DNS lookup for NTP servers, fetching DNSSec information. (2) If a signature/clock sync issue is the only barrier to validating the response

Re: [systemd-devel] [PATCH] build: put missing KMOD_CFLAGS into Makefile

2014-05-21 Thread Lennart Poettering
On Thu, 22.05.14 02:46, Jan Engelhardt (jeng...@inai.de) wrote: > > On Thursday 2014-05-22 02:27, Lennart Poettering wrote: > >On Tue, 20.05.14 23:31, Jan Engelhardt (jeng...@inai.de) wrote: > > > >> The build fails if kmod is not in a default location. > > > >Thanks! > >Applied! > > Have you th

Re: [systemd-devel] [PATCH 3/3] nspawn: allow to bind mount journal on top of a non empty container journal dentry

2014-05-21 Thread Lennart Poettering
On Fri, 11.04.14 01:45, Djalal Harouni (tix...@opendz.org) wrote: > Currently if nspawn was called with --link-journal=host or > --link-journal=auto and the right /var/log/journal/machine-id/ exists > then the bind mount the subdirectory into the container might fail due > to the ~/mycontainer/var

Re: [systemd-devel] [PATCH 3/3] nspawn: allow to bind mount journal on top of a non empty container journal dentry

2014-05-21 Thread Lennart Poettering
On Tue, 29.04.14 00:15, Djalal Harouni (tix...@opendz.org) wrote: > > On Fri, Apr 25, 2014 at 08:30:36PM +0200, Tom Gundersen wrote: > > On Fri, Apr 11, 2014 at 2:45 AM, Djalal Harouni wrote: > > > Currently if nspawn was called with --link-journal=host or > > > --link-journal=auto and the right

Re: [systemd-devel] Start-up resource and prioritization control

2014-05-21 Thread Umut Tezduyar Lindskog
On Thu, May 22, 2014 at 2:14 AM, Lennart Poettering wrote: > On Wed, 21.05.14 13:52, Umut Tezduyar Lindskog (u...@tezduyar.com) wrote: > >> > Not following here... Aren't you describing a best-effort system here? >> > But the CPUShares= stuff is best-effort stuff anyway, and just tells the >> > ke

Re: [systemd-devel] GuessMainPID=no required to make daemon reload work

2014-05-21 Thread Alexander E. Patrakov
05.05.2014 21:14, Lennart Poettering wrote: On Mon, 05.05.14 10:00, Gerd v. Egidy (li...@egidy.de) wrote: systemd will behave as expected: once your main process terminates it will re-read PID from this file (assuming that before dying your old process writes its child's PID) and set it as MAIN

Re: [systemd-devel] Start-up resource and prioritization control

2014-05-21 Thread Umut Tezduyar Lindskog
Hi, On Wed, May 21, 2014 at 10:44 PM, Uoti Urpala wrote: > On Tue, 2014-05-20 at 15:16 +0200, Umut Tezduyar Lindskog wrote: >> On Tue, May 20, 2014 at 1:46 PM, Tom Gundersen wrote: >> > Wouldn't this be solved by telling the kernel to schedule the starting >> > services with high latency (or wha

Re: [systemd-devel] [PATCH] core: add startup resource control option

2014-05-21 Thread Umut Tezduyar Lindskog
On Thu, May 22, 2014 at 2:18 AM, Lennart Poettering wrote: > On Wed, 21.05.14 14:02, Umut Tezduyar Lindskog (u...@tezduyar.com) wrote: > >> >> Hi Cho, >> >> Do you have any technical reason why CPU shares are reverted back to >> the default value on "StartupFinished" but instead when the unit is i

[systemd-devel] Dead link in "man 7 daemon"

2014-05-21 Thread Alexander E. Patrakov
Hello. "man 7 daemon" links to Apple MacOS X Daemon Requirements: http://developer.apple.com/mac/library/documentation/MacOSX/Conceptual/BPSystemStartup/Articles/LaunchOnDemandDaemons.html#//apple_ref/doc/uid/TP40001762-104738 However, the link is dead. The correct link is: https://developer.a