Re: [systemd-devel] [PATCH] Try to start rescue.target if default.target is missing

2012-09-12 Thread Lennart Poettering
On Tue, 11.09.12 12:06, Tomasz Torcz (to...@pipebreaker.pl) wrote: From 0a3f3b2a46607a94efd2936ae7094a9ef7421e76 Mon Sep 17 00:00:00 2001 From: Tomasz Torcz to...@pipebreaker.pl Date: Fri, 7 Sep 2012 14:14:44 +0200 Subject: [PATCH] Try to start rescue.target if default.target is missing In

Re: [systemd-devel] (no subject)

2012-09-12 Thread Lennart Poettering
On Mon, 03.09.12 23:46, Colin Guthrie (co...@mageia.org) wrote: Hi, Heya, OK, so these are the changes I'd propose to fix the issues mentioned in this thread. I've not tested them so this is more for general feedback as to whether this approach is generally a good one or not. The

Re: [systemd-devel] (no subject)

2012-09-12 Thread Colin Guthrie
'Twas brillig, and Lennart Poettering at 12/09/12 07:34 did gyre and gimble: On Mon, 03.09.12 23:46, Colin Guthrie (co...@mageia.org) wrote: Hi, Heya, OK, so these are the changes I'd propose to fix the issues mentioned in this thread. I've not tested them so this is more for general

Re: [systemd-devel] [PATCH] shutdown: do reboot() for openvz container

2012-09-12 Thread Kir Kolyshkin
On 09/11/2012 05:24 AM, Lennart Poettering wrote: On Fri, 24.08.12 16:22, Kir Kolyshkin (k...@openvz.org) wrote: Proper handling of reboot() syscall issued from the inside of a container was always supported by OpenVZ kernels. More to say, OpenVZ relies on the fact that container calls reboot

Re: [systemd-devel] [PATCH] shutdown: do reboot() for openvz container

2012-09-12 Thread Daniel P. Berrange
On Wed, Sep 12, 2012 at 02:47:48PM +0400, Kir Kolyshkin wrote: On 09/11/2012 05:24 AM, Lennart Poettering wrote: On Fri, 24.08.12 16:22, Kir Kolyshkin (k...@openvz.org) wrote: Proper handling of reboot() syscall issued from the inside of a container was always supported by OpenVZ kernels.

[systemd-devel] [PATCH] tmpfiles: plug file descriptor leak.

2012-09-12 Thread Dave Reisner
Introduced in d4e9eb91. --- Sorry Lennart, I should have pointed this out along with my nudging last night on IRC about the prior patch. src/tmpfiles/tmpfiles.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/tmpfiles/tmpfiles.c b/src/tmpfiles/tmpfiles.c index 70de06b..ed51ec8 100644

Re: [systemd-devel] [PATCH] shutdown: do reboot() for openvz container

2012-09-12 Thread Lennart Poettering
On Wed, 12.09.12 14:47, Kir Kolyshkin (k...@openvz.org) wrote: This patch brings the reboot() back for OpenVZ container. Turns out the normal Linux containers understand reboot() just fine too. Please note though that the problem with reboot() wrt upstream containers was really nasty --

Re: [systemd-devel] [PATCH] shutdown: do reboot() for openvz container

2012-09-12 Thread Lennart Poettering
On Wed, 12.09.12 11:51, Daniel P. Berrange (berra...@redhat.com) wrote: On Wed, Sep 12, 2012 at 02:47:48PM +0400, Kir Kolyshkin wrote: On 09/11/2012 05:24 AM, Lennart Poettering wrote: On Fri, 24.08.12 16:22, Kir Kolyshkin (k...@openvz.org) wrote: Proper handling of reboot() syscall

Re: [systemd-devel] [PATCH] tmpfiles: plug file descriptor leak.

2012-09-12 Thread Lennart Poettering
On Wed, 12.09.12 16:21, Dave Reisner (dreis...@archlinux.org) wrote: Introduced in d4e9eb91. --- Sorry Lennart, I should have pointed this out along with my nudging last night on IRC about the prior patch. Applied! Thanks! src/tmpfiles/tmpfiles.c | 2 ++ 1 file changed, 2 insertions(+)

Re: [systemd-devel] [PATCH] shutdown: do reboot() for openvz container

2012-09-12 Thread Kay Sievers
On Wed, Sep 12, 2012 at 11:54 PM, Lennart Poettering lenn...@poettering.net wrote: On Wed, 12.09.12 11:51, Daniel P. Berrange (berra...@redhat.com) wrote: NB when libvirt starts an LXC container, it first checks to see whether the kernel has the container aware reboot() support. If it does

Re: [systemd-devel] [PATCH] shutdown: do reboot() for openvz container

2012-09-12 Thread Lennart Poettering
On Thu, 13.09.12 00:25, Kay Sievers (k...@vrfy.org) wrote: On Wed, Sep 12, 2012 at 11:54 PM, Lennart Poettering lenn...@poettering.net wrote: On Wed, 12.09.12 11:51, Daniel P. Berrange (berra...@redhat.com) wrote: NB when libvirt starts an LXC container, it first checks to see whether