[systemd-devel] systemd intentional behaviour? (signals)

2012-08-23 Thread Gerardo Exequiel Pozzi
Hello Just for fun I am sending some signals to systemd (188) to see how reacts compared to sysvinit. While sysvinit ignores them or restores from "crash" after 30 seconds of sleep, systemd/journald just logs the status them freezing execution. Example kill -SEGV 1 -> freeze kill -QUIT 1 ->

Re: [systemd-devel] udev 182: response timeout for request_firmware in module_probe path

2012-08-23 Thread Alan Cox
> IMO, the driver probing path is allowed to sleep, so looks request firmware > should be allowed inside .probe(). I'm not convinced about that. It can sleep but its holding various locks in most cases, and it looks like that can end up in a complete heap. By all means *request* the firmware asyn

Re: [systemd-devel] udev 182: response timeout for request_firmware in module_probe path

2012-08-23 Thread Ming Lei
On Fri, Aug 24, 2012 at 12:46 AM, Alan Cox wrote: >> IMO, the driver probing path is allowed to sleep, so looks request firmware >> should be allowed inside .probe(). > > I'm not convinced about that. It can sleep but its holding various locks > in most cases, and it looks like that can end up in

Re: [systemd-devel] systemd security issue

2012-08-23 Thread Kay Sievers
On Thu, Aug 23, 2012 at 6:04 PM, Richard Hughes wrote: > On 23 August 2012 07:21, Damian Ivanov wrote: >> a) In the gnome-control-center==> Color Management you can play >> with the settings of a monitor attached to another seat. > > I've not done any scoping for colord and multiseat yet. I h

Re: [systemd-devel] systemd security issue

2012-08-23 Thread Richard Hughes
On 23 August 2012 07:21, Damian Ivanov wrote: > a) In the gnome-control-center==> Color Management you can play > with the settings of a monitor attached to another seat. I've not done any scoping for colord and multiseat yet. I have got a pluggable USB docking station to test here, but I'm w

Re: [systemd-devel] udev 182: response timeout for request_firmware in module_probe path

2012-08-23 Thread Ming Lei
On Thu, Aug 23, 2012 at 11:31 PM, Kay Sievers wrote: > On Thu, Aug 23, 2012 at 5:16 PM, Ming Lei wrote: >> On Tue, Aug 21, 2012 at 1:34 PM, Ming Lei wrote: > >>> I found that udev 182 doesn't response for the request_firmware in >>> module_probe path until 30sec later after the 'ADD' event of fi

Re: [systemd-devel] udev 182: response timeout for request_firmware in module_probe path

2012-08-23 Thread Kay Sievers
On Thu, Aug 23, 2012 at 5:16 PM, Ming Lei wrote: > On Tue, Aug 21, 2012 at 1:34 PM, Ming Lei wrote: >> I found that udev 182 doesn't response for the request_firmware in >> module_probe path until 30sec later after the 'ADD' event of firmware >> device, but no such problem in udev175, sounds lik

Re: [systemd-devel] udev 182: response timeout for request_firmware in module_probe path

2012-08-23 Thread Ming Lei
Cc systemd-devel Hi, On Tue, Aug 21, 2012 at 1:34 PM, Ming Lei wrote: > Hi Kay, > > I found that udev 182 doesn't response for the request_firmware in > module_probe path until 30sec later after the 'ADD' event of firmware > device, but no such problem in udev175, sounds like a regression of ude

Re: [systemd-devel] systemd coverity

2012-08-23 Thread Zbigniew Jędrzejewski-Szmek
On 08/23/2012 03:11 PM, Daniel P. Berrange wrote: > On Thu, Aug 23, 2012 at 03:04:23PM +0200, Zbigniew Jędrzejewski-Szmek wrote: >> On 08/23/2012 02:36 PM, Lennart Poettering wrote: >>> maybe we should add macros like: >>> >>> #define _cleanup_free_ __attribute__((cleanup(freep))) >>>

Re: [systemd-devel] systemd coverity

2012-08-23 Thread Daniel P. Berrange
On Thu, Aug 23, 2012 at 03:04:23PM +0200, Zbigniew Jędrzejewski-Szmek wrote: > On 08/23/2012 02:36 PM, Lennart Poettering wrote: > > maybe we should add macros like: > > > > #define _cleanup_free_ __attribute__((cleanup(freep))) > > #define _cleanup_fclose_ __attribute__((cleanup(f

Re: [systemd-devel] systemd coverity

2012-08-23 Thread Zbigniew Jędrzejewski-Szmek
On 08/23/2012 02:36 PM, Lennart Poettering wrote: > maybe we should add macros like: > > #define _cleanup_free_ __attribute__((cleanup(freep))) > #define _cleanup_fclose_ __attribute__((cleanup(fclosep))) > > What do you think? I personally think that this would be a welcome chang

Re: [systemd-devel] systemd coverity

2012-08-23 Thread Zbigniew Jędrzejewski-Szmek
On 08/23/2012 02:26 PM, Lennart Poettering wrote: > On Thu, 23.08.12 13:57, Zbigniew Jędrzejewski-Szmek ([email protected]) wrote: > >> On 08/23/2012 12:36 PM, Lukáš Nykrýn wrote: >>> Hello, >>> Two more patches from diff scan 188-189. >> Applied both. > > I think you forgot to push? Aaaargh. Pus

Re: [systemd-devel] systemd coverity

2012-08-23 Thread Lennart Poettering
On Thu, 23.08.12 09:25, Lukáš Nykrýn ([email protected]) wrote: > > diff --git a/src/readahead/readahead-analyze.c > b/src/readahead/readahead-analyze.c > index 11b2b2d..9a929c0 100644 > --- a/src/readahead/readahead-analyze.c > +++ b/src/readahead/readahead-analyze.c > @@ -144,6 +144,7 @@ int

Re: [systemd-devel] systemd coverity

2012-08-23 Thread Lennart Poettering
On Thu, 23.08.12 13:57, Zbigniew Jędrzejewski-Szmek ([email protected]) wrote: > On 08/23/2012 12:36 PM, Lukáš Nykrýn wrote: > > Hello, > > Two more patches from diff scan 188-189. > Applied both. I think you forgot to push? Lennart -- Lennart Poettering - Red Hat, Inc. ___

Re: [systemd-devel] [PATCH v9002 1/2] timedated: gather timezone from /etc/localtime sym target

2012-08-23 Thread Frederic Crozat
Le mardi 21 août 2012 à 23:11 -0700, Shawn Landden a écrit : > @@ -218,19 +248,21 @@ static int write_data_timezone(void) { > return r; > } > > -p = strappend("/usr/share/zoneinfo/", tz.zone); > +p = strappend(ZONEINFO_PATH, tz.zone); > if (!p)

Re: [systemd-devel] systemd coverity

2012-08-23 Thread Zbigniew Jędrzejewski-Szmek
On 08/23/2012 12:36 PM, Lukáš Nykrýn wrote: > Hello, > Two more patches from diff scan 188-189. Applied both. Zbyszek ___ systemd-devel mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/systemd-devel

Re: [systemd-devel] systemd coverity

2012-08-23 Thread Zbigniew Jędrzejewski-Szmek
On 08/23/2012 09:25 AM, Lukáš Nykrýn wrote: > Hello, > Coverity found some new small issues between releases 185-188. See attached > patches. Applied all, with small change in 'login: check return of parse_pid and parse_uid'. Zbyszek ___ systemd-devel m

Re: [systemd-devel] systemd coverity

2012-08-23 Thread Lukáš Nykrýn
Lukáš Nykrýn píše v Čt 23. 08. 2012 v 09:25 +0200: > Hello, > Coverity found some new small issues between releases 185-188. See attached > patches. > > Regards > Lukas > Hello, Two more patches from diff scan 188-189. Regards Lukas >From fb11041ccb2149381ebf9ddc7457b4e83dcdef41 Mon Sep 17 00:

[systemd-devel] systemd coverity

2012-08-23 Thread Lukáš Nykrýn
Hello, Coverity found some new small issues between releases 185-188. See attached patches. Regards Lukas >From b258ab8b5fec97e924ba5d3784be9b72d7966118 Mon Sep 17 00:00:00 2001 From: Lukas Nykryn Date: Mon, 20 Aug 2012 14:33:21 +0200 Subject: [PATCH 1/4] load-fragment: initialize bool invert b