[systemd-devel] [PATCH] util: fix strict aliasing violations in use of struct inotify_event

2014-12-22 Thread Shawn Paul Landden
There is alot of cleanup that will have to happen to turn on -fstrict-aliasing, but I think our code should be correct to the rule. --- src/journal/sd-journal.c | 9 ++--- src/shared/util.c| 9 ++--- src/udev/udevd.c | 9 ++--- 3 files changed, 18 insertions(+), 9

[systemd-devel] I am trying to hook up docker and systemd.

2014-12-22 Thread Daniel J Walsh
I have a working version of docker which runs systemd/journald within the container and sets up the /var/log/journal/UUID inside the container to match the version outside. I also have registered the container with machinectl. Everything seems to work fine except that when I execute journalctl

Re: [systemd-devel] [PATCH] timedated: support split usr

2014-12-22 Thread Lennart Poettering
On Sun, 21.12.14 22:55, Shawn Paul Landden (sh...@churchofgit.com) wrote: The current Debian solution to this is really ugly, and I would rather have them use the correct patch even if split usr is dumb. Well, just copying the file cannot really work. We read the symlink back to figure out

Re: [systemd-devel] Improving module loading

2014-12-22 Thread Lennart Poettering
On Sat, 20.12.14 10:45, Hoyer, Marko (ADITG/SW2) (mho...@de.adit-jv.com) wrote: I had such a discussion earlier with some of the systemd guys. My intention was to introduce an additional unit for module loading for exactly the reason you mentioned. The following (reasonable) outcome was: -

Re: [systemd-devel] [PATCH] util: fix strict aliasing violations in use of struct inotify_event

2014-12-22 Thread Lennart Poettering
On Mon, 22.12.14 00:17, Shawn Paul Landden (sh...@churchofgit.com) wrote: There is alot of cleanup that will have to happen to turn on -fstrict-aliasing, but I think our code should be correct to the rule. I don#t think -fstrict-aliasing can really work given that we use some rather broken

Re: [systemd-devel] systemd log entries

2014-12-22 Thread Lennart Poettering
On Sun, 21.12.14 15:21, Jóhann B. Guðmundsson (johan...@gmail.com) wrote: Why are we spamming the journal with entries like this -- Subject: -- Defined-By: -- Support: -- Documentation: ... Dec 21 15:13:25 localhost.localdomain systemd-logind[540]: New seat seat0. -- all information

Re: [systemd-devel] systemd log entries

2014-12-22 Thread Lennart Poettering
On Sun, 21.12.14 17:35, Jóhann B. Guðmundsson (johan...@gmail.com) wrote: And there is no reference to what c1 actually means in the upstream Well, the catalog entries can certainly be improved, I'd be happy for a patch that improves them. documentation and advertising the mailinglist as an

Re: [systemd-devel] java crashes only on boot

2014-12-22 Thread Lennart Poettering
On Sun, 21.12.14 21:24, Florian Lindner (mailingli...@xgm.de) wrote: Hello, on my up to date Arch system I use Crashplan which is a java app for offsite backups. It used to work fine, until recently... No idea why it crashes, but this is probably something to contact the authors of that

[systemd-devel] [PATCH udev v4] udev: Add builtin/rule to export touchscreen/tablet size as udev properties

2014-12-22 Thread Carlos Garnacho
This rule is only run on tablet/touchscreen devices, and extracts their size in millimeters, as it can be found out through their struct input_absinfo. This may be useful to separate policy and application at the time of mapping these devices to the available outputs in windowing environments

Re: [systemd-devel] [PATCH] timedated: support split usr

2014-12-22 Thread Simon McVittie
On 22/12/14 14:33, Lennart Poettering wrote: I thought Debian would nowadays mount /usr from the initrd too? We now do that in unstable, but unfortunately this change wasn't well coordinated and caused several serious regressions, so it's unlikely to migrate into Debian 8. (A separate /usr on

Re: [systemd-devel] [PATCH] util: fix strict aliasing violations in use of struct inotify_event

2014-12-22 Thread Uoti Urpala
On Mon, 2014-12-22 at 00:17 -0800, Shawn Paul Landden wrote: There is alot of cleanup that will have to happen to turn on -fstrict-aliasing, but I think our code should be correct to the rule. -uint8_t buffer[INOTIFY_EVENT_MAX] _alignas_(struct inotify_event); +

Re: [systemd-devel] Improving module loading

2014-12-22 Thread Lucas De Marchi
On Mon, Dec 22, 2014 at 1:04 PM, Lennart Poettering lenn...@poettering.net wrote: On Sat, 20.12.14 10:45, Hoyer, Marko (ADITG/SW2) (mho...@de.adit-jv.com) wrote: I had such a discussion earlier with some of the systemd guys. My intention was to introduce an additional unit for module loading

Re: [systemd-devel] [PATCH 1/2] build-sys: do not use pkgconfig dbus-1.pc to find dbus directories

2014-12-22 Thread Filipe Brandenburger
Ping? This was discussed before and I think it's the one thing we need to remove the dependencies on dbus-devel... (See also patch 2/2 of this series which gets rid of --dbusinterfacedir which is not used anywhere. We might also want to change detection of dbus libs in configure.ac but I'd say

Re: [systemd-devel] [PATCH 0/9] capabilities: remove include of sys/capability.h where possible

2014-12-22 Thread Filipe Brandenburger
Ping? Also wondering if it makes sense to go ahead and implement our own cap_to_text and cap_from_text to generate capability strings from the bitmaps (and further remove dependency on libcap.) I think it does, considering we now already have our own list of valid capabilities and the constants

Re: [systemd-devel] [PATCH] update .gitignore to include test-lldp

2014-12-22 Thread Filipe Brandenburger
Ping? On Fri, Dec 19, 2014 at 9:45 AM, Filipe Brandenburger filbran...@google.com wrote: --- .gitignore | 1 + 1 file changed, 1 insertion(+) diff --git a/.gitignore b/.gitignore index 1b5d60f..078fd9a 100644 --- a/.gitignore +++ b/.gitignore @@ -207,6 +207,7 @@ /test-libudev-sym*

Re: [systemd-devel] [PATCHv2] test: do not use last cap from kernel in test-cap-list

2014-12-22 Thread Filipe Brandenburger
Ping? On Fri, Dec 12, 2014 at 10:47 AM, Filipe Brandenburger filbran...@google.com wrote: The new test-cap-list introduced in commit 2822da4fb7f891 uses the included table of capabilities. However, it uses cap_last_cap() which probes the kernel for the last available capability. On an older

[systemd-devel] [PATCH] gummiboot: freeing the wrong variable

2014-12-22 Thread David Härdeman
The error path in efivar_get free's what would have been the copy of the string if the strcpy had succeeded, which it did not (or we wouldn't be in the error path). Signed-off-by: David Härdeman da...@hardeman.nu --- src/efi/util.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff

[systemd-devel] [PATCH] hwdb: add rule and first entry for PS/2 mice

2014-12-22 Thread Peter Hutterer
https://bugs.freedesktop.org/show_bug.cgi?id=87037 --- some feedback on the rule would be appreciated, in case there's a better approach to matching. hwdb/70-mouse.hwdb | 4 rules/70-mouse.rules | 3 +++ 2 files changed, 7 insertions(+) diff --git a/hwdb/70-mouse.hwdb

[systemd-devel] [PATCH] util: fix strict aliasing violations in use of struct inotify_event v2

2014-12-22 Thread Shawn Paul Landden
There is alot of cleanup that will have to happen to turn on -fstrict-aliasing, but I think our code should be correct to the rule. v2: move union def to header file --- src/journal/sd-journal.c | 6 +++--- src/shared/util.c| 6 +++--- src/shared/util.h| 5 + src/udev/udevd.c

[systemd-devel] [PATCH] gitignore: add another test

2014-12-22 Thread Shawn Paul Landden
--- .gitignore | 1 + 1 file changed, 1 insertion(+) diff --git a/.gitignore b/.gitignore index 1b5d60f..00a46a5 100644 --- a/.gitignore +++ b/.gitignore @@ -206,6 +206,7 @@ /test-libudev /test-libudev-sym* /test-list +/test-lldp /test-unaligned /test-locale-util /test-local-addresses --

[systemd-devel] [PATCH] util: fix strict aliasing violations in use of struct inotify_event v3

2014-12-22 Thread Shawn Paul Landden
There is alot of cleanup that will have to happen to turn on -fstrict-aliasing, but I think our code should be correct to the rule. v2: move union def to header file v3: fix syntax --- src/journal/sd-journal.c | 6 +++--- src/shared/util.c| 7 +++ src/shared/util.h| 6 ++