[systemd-devel] [PATCH 7/7] calendar: parse months

2014-10-27 Thread Daniele Medri
--- src/shared/calendarspec.c | 4 1 file changed, 4 insertions(+) diff --git a/src/shared/calendarspec.c b/src/shared/calendarspec.c index 8e6c871..ec84d65 100644 --- a/src/shared/calendarspec.c +++ b/src/shared/calendarspec.c @@ -907,6 +907,10 @@ int calendar_spec_from_string(const char

[systemd-devel] [PATCH 1/7] calendar: add constant values

2014-10-27 Thread Daniele Medri
--- src/shared/calendarspec.c | 22 +++--- 1 file changed, 15 insertions(+), 7 deletions(-) diff --git a/src/shared/calendarspec.c b/src/shared/calendarspec.c index 64d0dec..73f87c6 100644 --- a/src/shared/calendarspec.c +++ b/src/shared/calendarspec.c @@ -24,6 +24,12 @@

[systemd-devel] [PATCH 4/7] calendar: new case 'minutely'

2014-10-27 Thread Daniele Medri
--- src/shared/calendarspec.c | 7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/src/shared/calendarspec.c b/src/shared/calendarspec.c index 196a330..6186301 100644 --- a/src/shared/calendarspec.c +++ b/src/shared/calendarspec.c @@ -833,7 +833,12 @@ int

[systemd-devel] [PATCH 5/7] calendar: re-ordering 'weekly'

2014-10-27 Thread Daniele Medri
--- src/shared/calendarspec.c | 26 +- 1 file changed, 13 insertions(+), 13 deletions(-) diff --git a/src/shared/calendarspec.c b/src/shared/calendarspec.c index 6186301..fae7dfd 100644 --- a/src/shared/calendarspec.c +++ b/src/shared/calendarspec.c @@ -857,10 +857,10 @@

[systemd-devel] [PATCH 3/7] calendar: functions and callings to handle the months's labels

2014-10-27 Thread Daniele Medri
--- src/shared/calendarspec.c | 198 ++ src/shared/calendarspec.h | 1 + 2 files changed, 199 insertions(+) diff --git a/src/shared/calendarspec.c b/src/shared/calendarspec.c index 67fd76a..196a330 100644 --- a/src/shared/calendarspec.c +++

[systemd-devel] [PATCH 6/7] calendar: no backward compatibility for erroneous label

2014-10-27 Thread Daniele Medri
--- src/shared/calendarspec.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/shared/calendarspec.c b/src/shared/calendarspec.c index fae7dfd..8e6c871 100644 --- a/src/shared/calendarspec.c +++ b/src/shared/calendarspec.c @@ -885,8 +885,7 @@ int

[systemd-devel] [PATCH 2/7] calendar: constant value for weekdays

2014-10-27 Thread Daniele Medri
--- src/shared/calendarspec.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/src/shared/calendarspec.c b/src/shared/calendarspec.c index 73f87c6..67fd76a 100644 --- a/src/shared/calendarspec.c +++ b/src/shared/calendarspec.c @@ -126,7 +126,7 @@ static void

[systemd-devel] [PATCH] man: fix typos in description of SELinuxContextFromNet=

2014-10-27 Thread Ivan Shapovalov
--- man/systemd.socket.xml | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/man/systemd.socket.xml b/man/systemd.socket.xml index ce04b0b..57f769f 100644 --- a/man/systemd.socket.xml +++ b/man/systemd.socket.xml @@ -678,7 +678,7 @@ varlistentry

[systemd-devel] systemctl preset for user units (and how presets are applied after factory reset)

2014-10-27 Thread Colin Guthrie
Hi, When implementing socket activation for PulseAudio, the question came up of how to ship the enablement when running make install. In my first incarnation I simply shipped a /usr/lib/systemd/user/socket.target.wants/pulseaudio.socket symlink. I did this because I considered this a vendor

Re: [systemd-devel] [PATCH] man: fix typos in description of SELinuxContextFromNet=

2014-10-27 Thread Tom Gundersen
Pushed. Thanks! On Mon, Oct 27, 2014 at 10:07 AM, Ivan Shapovalov intelfx...@gmail.com wrote: --- man/systemd.socket.xml | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/man/systemd.socket.xml b/man/systemd.socket.xml index ce04b0b..57f769f 100644 ---

Re: [systemd-devel] [PATCH] util: introduce sethostname_idempotent

2014-10-27 Thread Michal Sekletar
On Tue, Oct 21, 2014 at 07:29:31PM +0200, Lennart Poettering wrote: On Tue, 21.10.14 18:32, Michal Sekletar (msekl...@redhat.com) wrote: snip Go ahead and commit. Ideally with those two nitpicks fixed, but even if you don't it's OK. sethostname_idempotent now returns 1 when hostname was

Re: [systemd-devel] [PATCH 3/4] shutdown: don't do final unmounting when inside the container and running without CAP_SYS_ADMIN

2014-10-27 Thread Michal Sekletar
On Wed, Oct 08, 2014 at 04:54:59PM +0200, Lennart Poettering wrote: On Wed, 08.10.14 16:49, Michal Sekletar (msekl...@redhat.com) wrote: Hmm, I think we should just do need_umount = !in_container, like we do for the other things like loopback detaching, dm detaching or swapoff.

Re: [systemd-devel] [PATCH 1/7] calendar: add constant values

2014-10-27 Thread Lennart Poettering
On Mon, 27.10.14 08:42, Daniele Medri (dme...@gmail.com) wrote: src/shared/calendarspec.c | 22 +++--- 1 file changed, 15 insertions(+), 7 deletions(-) diff --git a/src/shared/calendarspec.c b/src/shared/calendarspec.c index 64d0dec..73f87c6 100644 ---

Re: [systemd-devel] [PATCH 2/7] calendar: constant value for weekdays

2014-10-27 Thread Lennart Poettering
On Mon, 27.10.14 08:42, Daniele Medri (dme...@gmail.com) wrote: Could you merge the #define for BITS_WEEKDAYS into this patch please, as this patch appears to be the one that starts making use of this? --- src/shared/calendarspec.c | 10 +- 1 file changed, 5 insertions(+), 5

Re: [systemd-devel] [PATCH 3/7] calendar: functions and callings to handle the months's labels

2014-10-27 Thread Lennart Poettering
On Mon, 27.10.14 08:42, Daniele Medri (dme...@gmail.com) wrote: typedef struct CalendarSpec { int weekdays_bits; + int months_bits; I see the usefulness of the functionality, and I am willing to take a patch for this, but adding a new months_bits field here is not the way to

Re: [systemd-devel] [PATCH 4/7] calendar: new case 'minutely'

2014-10-27 Thread Lennart Poettering
On Mon, 27.10.14 08:42, Daniele Medri (dme...@gmail.com) wrote: Looks good! Applied! Thanks! --- src/shared/calendarspec.c | 7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/src/shared/calendarspec.c b/src/shared/calendarspec.c index 196a330..6186301 100644 ---

Re: [systemd-devel] [PATCH 5/7] calendar: re-ordering 'weekly'

2014-10-27 Thread Lennart Poettering
On Mon, 27.10.14 08:42, Daniele Medri (dme...@gmail.com) wrote: Hmm, can you elaborate on this change and why you made it? --- src/shared/calendarspec.c | 26 +- 1 file changed, 13 insertions(+), 13 deletions(-) diff --git a/src/shared/calendarspec.c

Re: [systemd-devel] [PATCH 6/7] calendar: no backward compatibility for erroneous label

2014-10-27 Thread Lennart Poettering
On Mon, 27.10.14 08:42, Daniele Medri (dme...@gmail.com) wrote: Hmm? Why would we want to drop the backwards compatibility here? We juste added it a few days ago! --- src/shared/calendarspec.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/shared/calendarspec.c

Re: [systemd-devel] [PATCH 7/7] calendar: parse months

2014-10-27 Thread Lennart Poettering
On Mon, 27.10.14 08:42, Daniele Medri (dme...@gmail.com) wrote: This should be in the same patch that introduces parse_months()! --- src/shared/calendarspec.c | 4 1 file changed, 4 insertions(+) diff --git a/src/shared/calendarspec.c b/src/shared/calendarspec.c index

Re: [systemd-devel] [PATCH 4/7] calendar: new case 'minutely'

2014-10-27 Thread Lennart Poettering
On Mon, 27.10.14 13:42, Lennart Poettering (lenn...@poettering.net) wrote: On Mon, 27.10.14 08:42, Daniele Medri (dme...@gmail.com) wrote: Looks good! Applied! Thanks! Also made man page changes for this now. Please, when posting a patch like this always include the man page changes

Re: [systemd-devel] [PATCH] login: remove multi-seat-x

2014-10-27 Thread Lennart Poettering
On Sun, 26.10.14 00:30, Timofey Titovets (nefelim...@gmail.com) wrote: Completed TODO: remove multi-seat-x Hmm, I don't remember the details of this, did X release a new version which makes this tool unnecessary? I think they commited code that made it unnecessary, but did they actually release

Re: [systemd-devel] udev database backwards compatibility guarantees

2014-10-27 Thread Alexander Larsson
On lör, 2014-10-25 at 13:45 +0200, Kay Sievers wrote: Kay, any ideas on the udev database stability? No stability. And so far no guarantees that things will not change. The versions of the udev daemon, libudev and the runtime data must match. Any expectations about version mix and match

Re: [systemd-devel] udev database backwards compatibility guarantees

2014-10-27 Thread Kay Sievers
On Mon, Oct 27, 2014 at 2:22 PM, Alexander Larsson al...@redhat.com wrote: On lör, 2014-10-25 at 13:45 +0200, Kay Sievers wrote: Kay, any ideas on the udev database stability? No stability. And so far no guarantees that things will not change. The versions of the udev daemon, libudev and

Re: [systemd-devel] systemctl preset for user units (and how presets are applied after factory reset)

2014-10-27 Thread Zbigniew Jędrzejewski-Szmek
On Mon, Oct 27, 2014 at 10:42:56AM +, Colin Guthrie wrote: So that led me to ask myself: does systemctl preset support user units? The man page doesn't make it clear that e.g. the --global switch applies to preset as well as enable, but in testing it (happily) appears to work! So I'm going

Re: [systemd-devel] [PATCH] login: remove multi-seat-x

2014-10-27 Thread Lennart Poettering
On Mon, 27.10.14 16:23, Timofey Titovets (nefelim...@gmail.com) wrote: 2014-10-27 16:00 GMT+03:00 Lennart Poettering lenn...@poettering.net: On Sun, 26.10.14 00:30, Timofey Titovets (nefelim...@gmail.com) wrote: Completed TODO: remove multi-seat-x Hmm, I don't remember the details of

Re: [systemd-devel] [PATCH] Add timesync-wait tool

2014-10-27 Thread Lennart Poettering
On Fri, 24.10.14 23:13, Lukasz Stelmach (stl...@poczta.fm) wrote: On 24.10.2014 00:28, Lennart Poettering wrote: On Thu, 23.10.14 21:24, Łukasz Stelmach (stl...@poczta.fm) wrote: +int main(int argc, char *argv[]) { +struct timex tbuf; +int r; + +

Re: [systemd-devel] Problem with modprobe in lm_sensors.service

2014-10-27 Thread Lennart Poettering
On Mon, 06.10.14 14:41, Dale R. Worley (wor...@alum.mit.edu) wrote: I am running Fedora 16 with kernel 3.14.19-100.fc19.x86_64 and systemd-204-21.fc19.x86_64. On startup (and sometimes shutdown), I see a message like this in /var/log/messages: Oct 6 13:53:37 hobgoblin modprobe[623]:

Re: [systemd-devel] [PATCH] Move apparmor code before the namespace setup

2014-10-27 Thread Lennart Poettering
On Sat, 11.10.14 21:57, m...@zarb.org (m...@zarb.org) wrote: From: Michael Scherer m...@zarb.org Since apparmor need to access /proc to communicate with the kernel, any unit setting / as readonly will be unable to also use the AppArmorProfile setting, as found on debian bug 760526. A unit

Re: [systemd-devel] systemd in initramfs: /etc/passwd, /etc/group, emergency.service and sulogin

2014-10-27 Thread Lennart Poettering
On Sat, 25.10.14 00:52, Ivan Shapovalov (intelfx...@gmail.com) wrote: Hi, A few questions regarding usage of systemd+udev in initramfs. Before all, this is what I want to achieve (to prevent XY-problems): working emergency.service in initramfs. The questions are a bit Arch-specific and

Re: [systemd-devel] Waiting for nspawn services

2014-10-27 Thread Lennart Poettering
On Sat, 25.10.14 05:39, Rich Freeman (r-syst...@thefreemanclan.net) wrote: One of the useful options in nspawn is the ability to boot the init within the container using -b, especially if that init happens to be systemd. However, I could not find any easy way to set a dependency on a

Re: [systemd-devel] systemctl preset for user units (and how presets are applied after factory reset)

2014-10-27 Thread Zbigniew Jędrzejewski-Szmek
On Mon, Oct 27, 2014 at 02:16:38PM +, Colin Guthrie wrote: Zbigniew Jędrzejewski-Szmek wrote on 27/10/14 13:52: [As a side note, if this is the recommended approach, then we should probably add appropriate RPM macros to macros.systemd for user unit presets...] Good point. I

Re: [systemd-devel] [PATCH v4] udev hwdb: Support shipping pre-compiled database in system images

2014-10-27 Thread Lennart Poettering
On Fri, 24.10.14 16:24, Martin Pitt (martin.p...@ubuntu.com) wrote: Hey Zbyszek, Zbigniew Jędrzejewski-Szmek [2014-10-24 19:45 +0200]: This enumeration is also used below... The definition should be shared. You might want to also consider using NULSTR_FOREACH for iteration. Ah, thanks

Re: [systemd-devel] string constants vs. #defines [was: Re: [PATCH v4] udev hwdb: Support shipping pre-compiled database in system images]

2014-10-27 Thread Lennart Poettering
On Sun, 26.10.14 07:51, Martin Pitt (martin.p...@ubuntu.com) wrote: +static const char hwdb_bin_paths[] = +/etc/udev/hwdb.bin\0 +UDEVLIBEXECDIR /hwdb.bin\0; Actually we don't need to define a variable for this, a #define would be enough. Are #defines actually preferred in

Re: [systemd-devel] [PATCH] udev hwdb: Support shipping pre-compiled database in system images

2014-10-27 Thread Lennart Poettering
On Sat, 25.10.14 23:18, Kay Sievers (k...@vrfy.org) wrote: On Mon, Oct 20, 2014 at 6:35 PM, Lennart Poettering lenn...@poettering.net wrote: On Fri, 17.10.14 15:10, Martin Pitt (martin.p...@ubuntu.com) wrote: Looks generally OK (as discussed in Düsseldorf). I'll leave this for Kay to

Re: [systemd-devel] Waiting for nspawn services

2014-10-27 Thread Rich Freeman
On Mon, Oct 27, 2014 at 10:49 AM, Lennart Poettering lenn...@poettering.net wrote: In general I think making use of socket notification here would be the much better option, as it removes the entire need for ordering things here. nspawn already support socket activation just fine. If your

Re: [systemd-devel] udev database backwards compatibility guarantees

2014-10-27 Thread Lennart Poettering
On Mon, 27.10.14 14:43, Kay Sievers (k...@vrfy.org) wrote: On Mon, Oct 27, 2014 at 2:22 PM, Alexander Larsson al...@redhat.com wrote: On lör, 2014-10-25 at 13:45 +0200, Kay Sievers wrote: Kay, any ideas on the udev database stability? No stability. And so far no guarantees that things

Re: [systemd-devel] [PATCH] udev hwdb: Support shipping pre-compiled database in system images

2014-10-27 Thread Michael Biebl
2014-10-27 16:21 GMT+01:00 Lennart Poettering lenn...@poettering.net: Sounds all fine, I would prefer though to use --usr-only or --system-only, something a little bit more descriptive than just --vendor though. The hwdb data is full ov vendor strings and it sounds too confusing to me. I'd

Re: [systemd-devel] Waiting for nspawn services

2014-10-27 Thread Lennart Poettering
On Mon, 27.10.14 11:24, Rich Freeman (r-syst...@thefreemanclan.net) wrote: On Mon, Oct 27, 2014 at 10:49 AM, Lennart Poettering lenn...@poettering.net wrote: In general I think making use of socket notification here would be the much better option, as it removes the entire need for ordering

Re: [systemd-devel] Waiting for nspawn services

2014-10-27 Thread Reindl Harald
Am 27.10.2014 um 16:32 schrieb Lennart Poettering: (Note though that ymmv on this, because depending on the software you use it might want to reverse-dns lookup incomoing connections, and that would fail if the container doesn't have network access to do DNS... That said, if mysql would do

Re: [systemd-devel] [PATCH] udev hwdb: Support shipping pre-compiled database in system images

2014-10-27 Thread Lennart Poettering
On Mon, 27.10.14 16:31, Michael Biebl (mbi...@gmail.com) wrote: 2014-10-27 16:21 GMT+01:00 Lennart Poettering lenn...@poettering.net: Sounds all fine, I would prefer though to use --usr-only or --system-only, something a little bit more descriptive than just --vendor though. The hwdb data

Re: [systemd-devel] [PATCH 5/7] calendar: re-ordering 'weekly'

2014-10-27 Thread Daniele Medri
Hmm, can you elaborate on this change and why you made it? Readability, cases sorted by time (daily weekly monthly). ___ systemd-devel mailing list systemd-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/systemd-devel

Re: [systemd-devel] [PATCH 5/7] calendar: re-ordering 'weekly'

2014-10-27 Thread Lennart Poettering
On Mon, 27.10.14 16:40, Daniele Medri (dme...@gmail.com) wrote: Hmm, can you elaborate on this change and why you made it? Readability, cases sorted by time (daily weekly monthly). Well, I think I disagree. We mostly have ordered these if() blocks by the order of appearance in the

Re: [systemd-devel] udev database backwards compatibility guarantees

2014-10-27 Thread Kay Sievers
On Mon, Oct 27, 2014 at 4:25 PM, Lennart Poettering lenn...@poettering.net wrote: On Mon, 27.10.14 14:43, Kay Sievers (k...@vrfy.org) wrote: On Mon, Oct 27, 2014 at 2:22 PM, Alexander Larsson al...@redhat.com wrote: On lör, 2014-10-25 at 13:45 +0200, Kay Sievers wrote: Kay, any ideas on

Re: [systemd-devel] [PATCH] tmpfiles: only change device permissions if mknod succeeded

2014-10-27 Thread Lennart Poettering
On Sat, 25.10.14 01:36, Tom Gundersen (t...@jklm.no) wrote: On Mon, Oct 20, 2014 at 9:32 PM, Lennart Poettering lenn...@poettering.net wrote: On Tue, 14.10.14 16:19, Jan Synacek (jsyna...@redhat.com) wrote: https://bugzilla.redhat.com/show_bug.cgi?id=1147248 Hmm, so far tmpfiles

Re: [systemd-devel] Waiting for nspawn services

2014-10-27 Thread Rich Freeman
On Mon, Oct 27, 2014 at 11:32 AM, Lennart Poettering lenn...@poettering.net wrote: Network namespaces are relevant for the process that originally binds the sockets. In the case of socket-activated containers that would be the host. If you then pass the fds into the containers and those are

Re: [systemd-devel] [PATCH] tmpfiles: only change device permissions if mknod succeeded

2014-10-27 Thread Tom Gundersen
On Mon, Oct 27, 2014 at 4:48 PM, Lennart Poettering mzerq...@0pointer.de wrote: On Sat, 25.10.14 01:36, Tom Gundersen (t...@jklm.no) wrote: On Mon, Oct 20, 2014 at 9:32 PM, Lennart Poettering lenn...@poettering.net wrote: On Tue, 14.10.14 16:19, Jan Synacek (jsyna...@redhat.com) wrote:

Re: [systemd-devel] [PATCH] udev hwdb: Support shipping pre-compiled database in system images

2014-10-27 Thread Martin Pitt
Lennart Poettering [2014-10-27 16:37 +0100]: After all, moving those files away from /etc doesn't really make sense on split-/usr systems anyway, as it wouldn't help monopolizing vendor data in /usr really... Or at least I think the --usr stuff is really about monopolizing vendor data in /usr,

Re: [systemd-devel] [PATCH] udev hwdb: Support shipping pre-compiled database in system images

2014-10-27 Thread Lennart Poettering
On Mon, 27.10.14 16:53, Martin Pitt (martin.p...@ubuntu.com) wrote: Lennart Poettering [2014-10-27 16:37 +0100]: After all, moving those files away from /etc doesn't really make sense on split-/usr systems anyway, as it wouldn't help monopolizing vendor data in /usr really... Or at least I

Re: [systemd-devel] Starting configurable set of services first

2014-10-27 Thread Umut Tezduyar Lindskog
On Wed, Oct 22, 2014 at 7:44 PM, Lennart Poettering lenn...@poettering.net wrote: On Tue, 02.09.14 10:06, Umut Tezduyar Lindskog (u...@tezduyar.com) wrote: Hi, I would like to start a configurable set of services first and the services are wanted by multi-user.target. I am using a service to

Re: [systemd-devel] Waiting for nspawn services

2014-10-27 Thread Lennart Poettering
On Mon, 27.10.14 11:49, Rich Freeman (r-syst...@thefreemanclan.net) wrote: On Mon, Oct 27, 2014 at 11:32 AM, Lennart Poettering lenn...@poettering.net wrote: Network namespaces are relevant for the process that originally binds the sockets. In the case of socket-activated containers that

Re: [systemd-devel] Starting configurable set of services first

2014-10-27 Thread Lennart Poettering
On Mon, 27.10.14 17:06, Umut Tezduyar Lindskog (u...@tezduyar.com) wrote: Should this be something we should support natively by systemd? As discussed at th systemd hackfest: I am a bit conservative about this as it introduces plenty chance for deadlocks, where services might

Re: [systemd-devel] [PATCH] udev hwdb: Support shipping pre-compiled database in system images

2014-10-27 Thread Michael Biebl
2014-10-27 16:37 GMT+01:00 Lennart Poettering lenn...@poettering.net: Well, I think that's OK really. We want to stay compatible with split-usr distros, but that kind of setup is really not at the focus of what we design for. I still think this should be named --usr. After all, moving those

Re: [systemd-devel] [PATCH] udev hwdb: Support shipping pre-compiled database in system images

2014-10-27 Thread Lennart Poettering
On Mon, 27.10.14 17:25, Michael Biebl (mbi...@gmail.com) wrote: 2014-10-27 16:37 GMT+01:00 Lennart Poettering lenn...@poettering.net: Well, I think that's OK really. We want to stay compatible with split-usr distros, but that kind of setup is really not at the focus of what we design for.

Re: [systemd-devel] [PATCH] udev hwdb: Support shipping pre-compiled database in system images

2014-10-27 Thread Michael Biebl
2014-10-27 17:31 GMT+01:00 Lennart Poettering mzerq...@0pointer.de: Now, the distinction udevadm is supposed to make here though is very different, it is a distinction between /etc and /usr, i.e. between vendor data and local configuration. Using the word system to distuingish this is highly

[systemd-devel] On /dev/disk/by-id/ata-...-part2 missing, again

2014-10-27 Thread Alexander E. Patrakov
Some time ago, I have complained that some boots are unsuccessful, because the /dev/disk/by-id/ata-OCZ-VECTOR_OCZ-Z5CB4KC20X0ZG7F8-part2 symlink (which should point to /dev/sda2) is not created by udev in the initramfs (which uses dracut). Thankfully, people on IRC have suggested some useful

Re: [systemd-devel] [PATCH] udev hwdb: Support shipping pre-compiled database in system images

2014-10-27 Thread Lennart Poettering
On Mon, 27.10.14 17:44, Michael Biebl (mbi...@gmail.com) wrote: 2014-10-27 17:31 GMT+01:00 Lennart Poettering mzerq...@0pointer.de: Now, the distinction udevadm is supposed to make here though is very different, it is a distinction between /etc and /usr, i.e. between vendor data and

Re: [systemd-devel] On /dev/disk/by-id/ata-...-part2 missing, again

2014-10-27 Thread Lennart Poettering
On Mon, 27.10.14 21:46, Alexander E. Patrakov (patra...@gmail.com) wrote: Some time ago, I have complained that some boots are unsuccessful, because the /dev/disk/by-id/ata-OCZ-VECTOR_OCZ-Z5CB4KC20X0ZG7F8-part2 symlink (which should point to /dev/sda2) is not created by udev in the initramfs

Re: [systemd-devel] On /dev/disk/by-id/ata-...-part2 missing, again

2014-10-27 Thread Alexander E. Patrakov
27.10.2014 21:49, Lennart Poettering wrote: On Mon, 27.10.14 21:46, Alexander E. Patrakov (patra...@gmail.com) wrote: Some time ago, I have complained that some boots are unsuccessful, because the /dev/disk/by-id/ata-OCZ-VECTOR_OCZ-Z5CB4KC20X0ZG7F8-part2 symlink (which should point to

Re: [systemd-devel] [PATCH] tmpfiles: only change device permissions if mknod succeeded

2014-10-27 Thread Tom Gundersen
On Mon, Oct 27, 2014 at 4:53 PM, Tom Gundersen t...@jklm.no wrote: On Mon, Oct 27, 2014 at 4:48 PM, Lennart Poettering mzerq...@0pointer.de wrote: On Sat, 25.10.14 01:36, Tom Gundersen (t...@jklm.no) wrote: On Mon, Oct 20, 2014 at 9:32 PM, Lennart Poettering lenn...@poettering.net wrote:

Re: [systemd-devel] On /dev/disk/by-id/ata-...-part2 missing, again

2014-10-27 Thread Lennart Poettering
On Mon, 27.10.14 21:57, Alexander E. Patrakov (patra...@gmail.com) wrote: 27.10.2014 21:49, Lennart Poettering wrote: On Mon, 27.10.14 21:46, Alexander E. Patrakov (patra...@gmail.com) wrote: Some time ago, I have complained that some boots are unsuccessful, because the

Re: [systemd-devel] How soon after login can I rely on systemd --user having reached sockets.target?

2014-10-27 Thread Lennart Poettering
On Sun, 26.10.14 20:47, Damien Robert (damien.olivier.rob...@gmail.com) wrote: From Lennart Poettering, Fri 24 Oct 2014 at 01:09:56 (+0200) : So, I really would prefer if this logic wasn't just a hook, but actually the primary action of logging in graphically via a display manager. Ok,

Re: [systemd-devel] On /dev/disk/by-id/ata-...-part2 missing, again

2014-10-27 Thread Alexander E. Patrakov
27.10.2014 22:13, Lennart Poettering wrote: On Mon, 27.10.14 21:57, Alexander E. Patrakov (patra...@gmail.com) wrote: 27.10.2014 21:49, Lennart Poettering wrote: On Mon, 27.10.14 21:46, Alexander E. Patrakov (patra...@gmail.com) wrote: Some time ago, I have complained that some boots are

Re: [systemd-devel] On /dev/disk/by-id/ata-...-part2 missing, again

2014-10-27 Thread Lennart Poettering
On Mon, 27.10.14 22:22, Alexander E. Patrakov (patra...@gmail.com) wrote: Thanks for the pointer. It is good to know that the information is available in the kernel. Unfortunately, it is not possible to run the lslocks program manually or see the contents of /proc/locks exactly at the

Re: [systemd-devel] On /dev/disk/by-id/ata-...-part2 missing, again

2014-10-27 Thread Alexander E. Patrakov
27.10.2014 22:28, Lennart Poettering wrote: On Mon, 27.10.14 22:22, Alexander E. Patrakov (patra...@gmail.com) wrote: Thanks for the pointer. It is good to know that the information is available in the kernel. Unfortunately, it is not possible to run the lslocks program manually or see the

Re: [systemd-devel] [PATCH] manager: do not print anything while passwords are being queried

2014-10-27 Thread Lennart Poettering
On Sun, 26.10.14 05:37, Zbigniew Jędrzejewski-Szmek (zbys...@in.waw.pl) wrote: https://bugs.freedesktop.org/show_bug.cgi?id=73942 So in really old systemd versions I had a concept in place of never printing to the console if there was a controlling process on it. The kernel's concept of a

Re: [systemd-devel] On /dev/disk/by-id/ata-...-part2 missing, again

2014-10-27 Thread Lennart Poettering
On Mon, 27.10.14 22:35, Alexander E. Patrakov (patra...@gmail.com) wrote: 27.10.2014 22:28, Lennart Poettering wrote: On Mon, 27.10.14 22:22, Alexander E. Patrakov (patra...@gmail.com) wrote: Thanks for the pointer. It is good to know that the information is available in the kernel.

Re: [systemd-devel] [PATCH 1/4] manager: print a warning after 2/3 of the start timeout

2014-10-27 Thread Lennart Poettering
On Sat, 25.10.14 21:43, Zbigniew Jędrzejewski-Szmek (zbys...@in.waw.pl) wrote: --- This could use some more polishing, but is useful as is I think... I love the idea! +static int manager_log_running_jobs(Manager *m) { +unsigned n; + +assert(m); + +n =

Re: [systemd-devel] [PATCH 3/4] manager: stop start timeout when sysinit.target is reached

2014-10-27 Thread Lennart Poettering
On Sat, 25.10.14 21:43, Zbigniew Jędrzejewski-Szmek (zbys...@in.waw.pl) wrote: +if (result == JOB_DONE unit_has_name(u, SPECIAL_SYSINIT_TARGET)) +manager_cancel_start_timeout(u-manager); + I'd really prefer to hook this up with manager_state(), so that we don't

Re: [systemd-devel] [PATCH] login: remove multi-seat-x

2014-10-27 Thread Laércio de Sousa
Lennart, I've told you some time ago that Xorg-server since release 1.16 no longer requires multi-seat-x wrapper. Em 27/10/2014 11:00, Lennart Poettering lenn...@poettering.net escreveu: On Sun, 26.10.14 00:30, Timofey Titovets (nefelim...@gmail.com) wrote: Completed TODO: remove

Re: [systemd-devel] [PATCH] core: send sigabrt on watchdog timeout

2014-10-27 Thread Lennart Poettering
On Sat, 25.10.14 23:58, Umut Tezduyar Lindskog (umut.tezdu...@axis.com) wrote: if sigabrt doesn't do the job, follow regular shutdown routine, sigterm sigkill. I like it, with one exception. diff --git a/src/core/unit.c b/src/core/unit.c index e40e6f2..66804c9 100644 ---

Re: [systemd-devel] [PATCH] login: remove multi-seat-x

2014-10-27 Thread Lennart Poettering
On Mon, 27.10.14 16:00, Laércio de Sousa (laercioso...@sme-mogidascruzes.sp.gov.br) wrote: Lennart, I've told you some time ago that Xorg-server since release 1.16 no longer requires multi-seat-x wrapper. Ah, OK! Hmm, can't get the patch to apply though, seems corrupted by the mailer.

Re: [systemd-devel] [PATCH] bash-completion: fix systemctl isolate

2014-10-27 Thread Lennart Poettering
On Thu, 09.10.14 12:03, Jan Synacek (jsyna...@redhat.com) wrote: Patch looks good, but I'll leave this for Zbigniew to merge, as my own expertise is bash completion stuff is pretty limited. --- shell-completion/bash/systemctl.in | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-)

Re: [systemd-devel] How soon after login can I rely on systemd --user having reached sockets.target?

2014-10-27 Thread Lennart Poettering
On Thu, 23.10.14 17:26, Zbigniew Jędrzejewski-Szmek (zbys...@in.waw.pl) wrote: order it after basic.target (which things are by default anyway)... My proposal now, (which is the same Damien's as I understood him): 1. pam_systemd should sync on default.target 2. by default

Re: [systemd-devel] On /dev/disk/by-id/ata-...-part2 missing, again

2014-10-27 Thread Alexander E. Patrakov
27.10.2014 22:13, Lennart Poettering wrote: On Mon, 27.10.14 21:57, Alexander E. Patrakov (patra...@gmail.com) wrote: 27.10.2014 21:49, Lennart Poettering wrote: On Mon, 27.10.14 21:46, Alexander E. Patrakov (patra...@gmail.com) wrote: Some time ago, I have complained that some boots are

Re: [systemd-devel] [PATCH 3/4] manager: stop start timeout when sysinit.target is reached

2014-10-27 Thread Zbigniew Jędrzejewski-Szmek
On Mon, Oct 27, 2014 at 06:51:08PM +0100, Lennart Poettering wrote: On Sat, 25.10.14 21:43, Zbigniew Jędrzejewski-Szmek (zbys...@in.waw.pl) wrote: +if (result == JOB_DONE unit_has_name(u, SPECIAL_SYSINIT_TARGET)) +manager_cancel_start_timeout(u-manager); +

Re: [systemd-devel] [PATCH 1/4] manager: print a warning after 2/3 of the start timeout

2014-10-27 Thread Zbigniew Jędrzejewski-Szmek
On Mon, Oct 27, 2014 at 06:49:22PM +0100, Lennart Poettering wrote: +log_info(Active jobs (%u running):\n\t%s., m-n_running_jobs, all); +} else +log_info(No jobs seem to be running.); + +return 0; +} A long-standing TODO list item

Re: [systemd-devel] [PATCH] Move apparmor code before the namespace setup

2014-10-27 Thread Michael Scherer
On Mon, Oct 27, 2014 at 03:38:37PM +0100, Lennart Poettering wrote: On Sat, 11.10.14 21:57, m...@zarb.org (m...@zarb.org) wrote: From: Michael Scherer m...@zarb.org Since apparmor need to access /proc to communicate with the kernel, any unit setting / as readonly will be unable to also

Re: [systemd-devel] [PATCH] login: remove multi-seat-x

2014-10-27 Thread Timofey Titovets
2014-10-27 21:05 GMT+03:00 Lennart Poettering mzerq...@0pointer.de: On Mon, 27.10.14 16:00, Laércio de Sousa (laercioso...@sme-mogidascruzes.sp.gov.br) wrote: Lennart, I've told you some time ago that Xorg-server since release 1.16 no longer requires multi-seat-x wrapper. Ah, OK! Hmm,

[systemd-devel] remote-fs ordering, iSCSI and _netdev

2014-10-27 Thread Chris Leech
Hi, I was hoping someone could help me make sure I'm not overlooking something with trying to manage mounts on iSCSI disks. I have an iscsi.service which starts and stops sessions to iSCSI targets. It's set with Before=remote-fs-pre.target and Wants=remote-fs-pre.target to ensure that remote fs

[systemd-devel] [PATCH] journal: avoid NULL pointer dereference in chain_cache_put

2014-10-27 Thread philippedeswert
From: Philippe De Swert philippedesw...@gmail.com ordered_hashmap_steal_first() can return NULL (and this is usually checked except here), so there is a potential NULL pointer reference. Thus moving the NULL check to cover both possible allocations of the ChainCacheItem. Found with coverity.

Re: [systemd-devel] [PATCH 3/4] manager: stop start timeout when sysinit.target is reached

2014-10-27 Thread Lennart Poettering
On Mon, 27.10.14 20:03, Zbigniew Jędrzejewski-Szmek (zbys...@in.waw.pl) wrote: On Mon, Oct 27, 2014 at 06:51:08PM +0100, Lennart Poettering wrote: On Sat, 25.10.14 21:43, Zbigniew Jędrzejewski-Szmek (zbys...@in.waw.pl) wrote: +if (result == JOB_DONE unit_has_name(u,

Re: [systemd-devel] [PATCH 1/4] manager: print a warning after 2/3 of the start timeout

2014-10-27 Thread Lennart Poettering
On Mon, 27.10.14 20:07, Zbigniew Jędrzejewski-Szmek (zbys...@in.waw.pl) wrote: On Mon, Oct 27, 2014 at 06:49:22PM +0100, Lennart Poettering wrote: +log_info(Active jobs (%u running):\n\t%s., m-n_running_jobs, all); +} else +log_info(No jobs

Re: [systemd-devel] [PATCH] Move apparmor code before the namespace setup

2014-10-27 Thread Lennart Poettering
On Mon, 27.10.14 20:16, Michael Scherer (m...@zarb.org) wrote: On Mon, Oct 27, 2014 at 03:38:37PM +0100, Lennart Poettering wrote: On Sat, 11.10.14 21:57, m...@zarb.org (m...@zarb.org) wrote: From: Michael Scherer m...@zarb.org Since apparmor need to access /proc to communicate

Re: [systemd-devel] [PATCH] journal: avoid NULL pointer dereference in chain_cache_put

2014-10-27 Thread Lennart Poettering
On Mon, 27.10.14 23:36, philippedesw...@gmail.com (philippedesw...@gmail.com) wrote: From: Philippe De Swert philippedesw...@gmail.com ordered_hashmap_steal_first() can return NULL (and this is usually checked except here), so there is a potential NULL pointer reference. Thus moving the

Re: [systemd-devel] I wonder… why systemd provokes this amount of polarity and resistance

2014-10-27 Thread Dale R. Worley
From: Lennart Poettering lenn...@poettering.net Please, let's discuss this elsewhere. Let's keep a strict technical focus on this ML! I believe that you mean that outsiders are welcome here to provide assistance to systemd as it has already been implemented. One difficulty is that outsiders

Re: [systemd-devel] [PATCH] Move apparmor code before the namespace setup

2014-10-27 Thread Reindl Harald
Am 27.10.2014 um 23:20 schrieb Lennart Poettering: On Mon, 27.10.14 20:16, Michael Scherer (m...@zarb.org) wrote: On Mon, Oct 27, 2014 at 03:38:37PM +0100, Lennart Poettering wrote: On Sat, 11.10.14 21:57, m...@zarb.org (m...@zarb.org) wrote: From: Michael Scherer m...@zarb.org Since

[systemd-devel] Kernel module(r8712u) reload - suspend/resume - systemd

2014-10-27 Thread poma
It is tested and works, # systemctl suspend [ 144.218876] PM: Entering mem sleep [ 144.219249] r8712u 2-3:1.0 wlp0s4f1u3: Suspending... [ 144.219255] r8712u 2-3:1.0 wlp0s4f1u3: Unable to suspend RESUME [ 146.844240] r8712u 2-3:1.0 wlp0s4f1u3: Resuming... [ 146.844241]

[systemd-devel] Writing a udev rule for U2F security tokens?

2014-10-27 Thread Andy Lutomirski
Hi- I'd like to write a generic udev rule for U2F security tokens and to possibly get it integrated into systemd / udev, but I'm not sure how to write it in the first place. U2F tokens are USB HID devices that have a usage page 0xF1D0 that contains usage 0x01. The rule should match any hidraw

Re: [systemd-devel] I wonder… why systemd provokes this amount of polarity and resistance

2014-10-27 Thread Lennart Poettering
On Mon, 27.10.14 18:25, Dale R. Worley (wor...@alum.mit.edu) wrote: From: Lennart Poettering lenn...@poettering.net Please, let's discuss this elsewhere. Let's keep a strict technical focus on this ML! I believe that you mean that outsiders are welcome here to provide assistance to

Re: [systemd-devel] Kernel module(r8712u) reload - suspend/resume - systemd

2014-10-27 Thread Lennart Poettering
On Mon, 27.10.14 23:43, poma (pomidorabelis...@gmail.com) wrote: but is there a better way to do it? This appears to be a kernel driver bug. Please report this issue against the kernel driver in question, systemd is not the right place to work around that. Lennart -- Lennart Poettering, Red

Re: [systemd-devel] I wonder… why systemd provokes this amount of polarity and resistance

2014-10-27 Thread Zbigniew Jędrzejewski-Szmek
On Mon, Oct 27, 2014 at 06:25:56PM -0400, Dale R. Worley wrote: From: Lennart Poettering lenn...@poettering.net Please, let's discuss this elsewhere. Let's keep a strict technical focus on this ML! I believe that you mean that outsiders are welcome here to provide assistance to

Re: [systemd-devel] Writing a udev rule for U2F security tokens?

2014-10-27 Thread Greg KH
On Mon, Oct 27, 2014 at 04:12:30PM -0700, Andy Lutomirski wrote: Hi- I'd like to write a generic udev rule for U2F security tokens and to possibly get it integrated into systemd / udev, but I'm not sure how to write it in the first place. U2F tokens are USB HID devices that have a usage

Re: [systemd-devel] Writing a udev rule for U2F security tokens?

2014-10-27 Thread Andy Lutomirski
On Mon, Oct 27, 2014 at 4:32 PM, Greg KH gre...@linuxfoundation.org wrote: On Mon, Oct 27, 2014 at 04:12:30PM -0700, Andy Lutomirski wrote: Hi- I'd like to write a generic udev rule for U2F security tokens and to possibly get it integrated into systemd / udev, but I'm not sure how to write

Re: [systemd-devel] Kernel module(r8712u) reload - suspend/resume - systemd

2014-10-27 Thread poma
On 28.10.2014 00:15, Lennart Poettering wrote: On Mon, 27.10.14 23:43, poma (pomidorabelis...@gmail.com) wrote: but is there a better way to do it? This appears to be a kernel driver bug. Please report this issue against the kernel driver in question, systemd is not the right place to

Re: [systemd-devel] [PATCH] core: collapse JOB_RELOAD on an inactive unit into JOB_NOP

2014-10-27 Thread Uoti Urpala
[Resending to the list, as it seems recipients were wrong in the first attempt] The discussion on this died down. I'm bringing this back up as it's IMO quite a significant problem. To recap: The core issue is that if a start job is queued for foo.service, systemctl reload foo.service blocks

[systemd-devel] [PATCH] Bootchart: allow parse LABEL, UUID, PARTUUID for svg info

2014-10-27 Thread Timofey Titovets
Good time of day, list. I try to fix Fixme in svg.c: /* FIXME: this works only in the simple case */ By default function try to get root=/dev/* I write small function to determine block device name by specified LABEL, UUID, PARTUUID. Please check code, its working, but i think it can look

Re: [systemd-devel] [PATCH] Bootchart: allow parse LABEL, UUID, PARTUUID for svg info

2014-10-27 Thread Ivan Shapovalov
On Tuesday 28 October 2014 at 03:30:13, Timofey Titovets wrote: Good time of day, list. I try to fix Fixme in svg.c: /* FIXME: this works only in the simple case */ By default function try to get root=/dev/* I write small function to determine block device name by specified LABEL, UUID,

Re: [systemd-devel] remote-fs ordering, iSCSI and _netdev

2014-10-27 Thread Zbigniew Jędrzejewski-Szmek
On Mon, Oct 27, 2014 at 02:10:47PM -0700, Chris Leech wrote: Hi, I was hoping someone could help me make sure I'm not overlooking something with trying to manage mounts on iSCSI disks. I have an iscsi.service which starts and stops sessions to iSCSI targets. It's set with

Re: [systemd-devel] [PATCH] login: remove multi-seat-x

2014-10-27 Thread Lennart Poettering
On Mon, 27.10.14 23:10, Timofey Titovets (nefelim...@gmail.com) wrote: 2014-10-27 21:05 GMT+03:00 Lennart Poettering mzerq...@0pointer.de: On Mon, 27.10.14 16:00, Laércio de Sousa (laercioso...@sme-mogidascruzes.sp.gov.br) wrote: Lennart, I've told you some time ago that Xorg-server

Re: [systemd-devel] remote-fs ordering, iSCSI and _netdev

2014-10-27 Thread Lennart Poettering
On Mon, 27.10.14 14:10, Chris Leech (cle...@redhat.com) wrote: So for any mounts to remote block devices (unlike remote file system protocols which are detected by the fs name), unless there is an fstab entry at the time fstab-generator is run they get treated like local fs mounts and

  1   2   >