[systemd-devel] [PATCH v2] 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 only root=/dev/* I also attach patch. Thanks. v2: Rewrited with use fstab_node_to_udev_node() and canonicalize_file_name() functions. From 4d14e78977df92e010

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

2014-10-27 Thread Timofey Titovets
On 10/28/2014 03:45 AM, Ivan Shapovalov wrote: 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 b

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

2014-10-27 Thread Andrei Borzenkov
В Mon, 27 Oct 2014 14:10:47 -0700 Chris Leech пишет: > > At boot fstab-generator is picking up on the _netdev option in fstab, > and the generated mount units are ordered against remote-fs properly. > If I leave a filesystem mounted at shutdown, it will be unmounted before > the iSCSI session is

Re: [systemd-devel] [PATCH 3/3] run: introduce timer support option

2014-10-27 Thread WaLyong Cho
On 10/23/2014 12:56 AM, Lennart Poettering wrote: > On Tue, 07.10.14 14:20, WaLyong Cho (walyong@samsung.com) wrote: > >> If systemd-run is called with timer option, then systemd-run call >> NewTransientUnit with service unit. And also call StartTransientUnit >> with timer unit which has same

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

2014-10-27 Thread Zbigniew Jędrzejewski-Szmek
On Tue, Oct 28, 2014 at 02:32:37AM +0100, Lennart Poettering wrote: > On Mon, 27.10.14 23:07, Lennart Poettering (lenn...@poettering.net) wrote: > > > 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 Po

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 23:07, Lennart Poettering (lenn...@poettering.net) wrote: > 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...@

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 conn

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 : > > 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 sinc

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 Before=remote-

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

[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 mo

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 un

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 >

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 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 it in the first p

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 u

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" > > > > 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 ha

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 Lennart Poettering
On Mon, 27.10.14 18:25, Dale R. Worley (wor...@alum.mit.edu) wrote: > > From: "Lennart Poettering" > > > > 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

[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 de

[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] r8712

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 Since apparmor need to

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

2014-10-27 Thread Dale R. Worley
> From: "Lennart Poettering" > > 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 are usually not

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 > > 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 bot

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 > > > > > > Since apparmor need to access /proc to communicate wi

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_

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_ha

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

2014-10-27 Thread philippedeswert
From: Philippe De Swert 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. Fixes: CID#1249723 --- src/j

[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 o

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 : > 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

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 > > > > Since apparmor need to access /proc to communicate with the kernel, > > any unit setting / as readonly will be unable to also use

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-stand

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] 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 uns

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] [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] [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] 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 > --- a/src/cor

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" escreveu: > On Sun, 26.10.14 00:30, Timofey Titovets (nefelim...@gmail.com) wrote: > > > Completed TODO: remove multi-seat-x > > Hmm, I don'

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

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 = hash

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 kerne

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 contr

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 con

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 m

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 uns

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

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 /dev/disk/

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 wrote: > On Mon, Oct 27, 2014 at 4:48 PM, Lennart Poettering > 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 >>> wrote: >>> > On Tue, 14.10.14 16:19, Jan Synacek (jsy

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 /dev/sda2

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] [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 : > > > > 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 configurat

[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 de

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 : > > 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 misleading hen

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 : > > 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

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 : > 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 > files away from /

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] 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 > wrote: > > Network namespaces are relevant for the process that originally binds > > the sockets. In the case of socket-activated containers that would be > > the ho

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 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 jump >> to m

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 l

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 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 >> wrote: >> > On Tue, 14.10.14 16:19, Jan Synacek (jsyna...@redhat.com) wrote: >> > >> >> https://bugzilla.redhat.com/

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 /u

Re: [systemd-devel] Waiting for nspawn services

2014-10-27 Thread Rich Freeman
On Mon, Oct 27, 2014 at 11:32 AM, Lennart Poettering 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 > locked into their own

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 > 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 always adjust a

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 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 wrote: >> > On lör, 2014-10-25 at 13:45 +0200, Kay Sievers wrote: >> >> > Kay, any ideas on the udev database stability?

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 no

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] 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 : > >> 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 "v

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 rever

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 > 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.

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 : >> 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 prefer "--us

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 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

Re: [systemd-devel] Waiting for nspawn services

2014-10-27 Thread Rich Freeman
On Mon, Oct 27, 2014 at 10:49 AM, Lennart Poettering 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 > mysql container would u

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 > 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 merge though, a

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 act

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,

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

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] 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

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 > > 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 that sets

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

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

2014-10-27 Thread Colin Guthrie
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 usually get one or two a year ;) >> There are, however, two questions

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] [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 : > > 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

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 go

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 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 the run

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

2014-10-27 Thread Timofey Titovets
2014-10-27 16:00 GMT+03:00 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 cod

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 m

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] [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 docum

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 8e6c871..

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

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 b/src/shar

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

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 deleti

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 > --- a/src/shared/ca

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 >

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: > 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 changed.

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 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 > --- a/man/systemd.socket.xml >

[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 cho

  1   2   >