Re: [systemd-devel] [PATCH v3] Add FDB support

2014-12-17 Thread Jóhann B. Guðmundsson
On 12/17/2014 05:46 PM, "Jóhann B. Guðmundsson" wrote: On 12/17/2014 03:35 PM, Alin Rauta wrote: Hi Tom, I've formatted the patch based on our previous discussions. This one has support only for adding FDB entries. An example configuration is like below: [Match] Name=em1 [Network] DHCP=v4

Re: [systemd-devel] How to create a systemd service?

2014-12-17 Thread jenia.ivlev
That's really nice information. Thanks a lot Mantas. ___ systemd-devel mailing list systemd-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/systemd-devel

[systemd-devel] [PATCH] systemctl: print unit package in status

2014-12-17 Thread Spencer Baugh
When printing the status of a unit, open a connection to the session bus and query PackageKit for the package that the unit file belongs to. Print it if PackageKit knows. --- src/systemctl/systemctl.c | 32 1 file changed, 32 insertions(+) diff --git a/src/systemc

Re: [systemd-devel] [PATCH] path: follow symbolic link for parent path

2014-12-17 Thread Zbigniew Jędrzejewski-Szmek
On Wed, Dec 17, 2014 at 08:59:15AM +0100, Umut Tezduyar Lindskog wrote: > Thanks Zbigniew, > > I have only fixed the fallback: case and missed to fix the real case > where we use the name_to_handle. Could you please change that too. It > will look something like this: > > @@ -489,7 +489,7 @@ int

Re: [systemd-devel] [PATCH] path: follow symbolic link for parent path

2014-12-17 Thread Zbigniew Jędrzejewski-Szmek
On Wed, Dec 17, 2014 at 10:48:55AM +0100, Umut Tezduyar Lindskog wrote: > On Wed, Dec 17, 2014 at 10:21 AM, "Jóhann B. Guðmundsson" > wrote: > > > > On 12/17/2014 02:20 AM, Zbigniew Jędrzejewski-Szmek wrote: > >> > >> On Tue, Dec 16, 2014 at 09:58:39PM +0100, Umut Tezduyar Lindskog wrote: > >>> >

Re: [systemd-devel] name_to_handle_at() with UBIFS

2014-12-17 Thread Lennart Poettering
On Wed, 17.12.14 20:48, Umut Tezduyar Lindskog (u...@tezduyar.com) wrote: > On Wed, Dec 17, 2014 at 7:54 PM, Lennart Poettering > wrote: > > On Wed, 17.12.14 10:45, Umut Tezduyar Lindskog (u...@tezduyar.com) wrote: > > > >> For the record: > >> > >> It is not enough to have CONFIG_FHANDLE to get

Re: [systemd-devel] journald syslog forwarding

2014-12-17 Thread Zbigniew Jędrzejewski-Szmek
On Wed, Dec 17, 2014 at 08:30:41PM +, Holger Winkelmann [TP] wrote: > Hi, sorry for delay in reply.. > > >> Journal carries messages from the initramfs. We cannot send them from > >> the initramfs, unless we bring up the network then, which we don't want > >> to do just for this purpose. But t

Re: [systemd-devel] journald syslog forwarding

2014-12-17 Thread Holger Winkelmann [TP]
Hi, sorry for delay in reply.. >> Journal carries messages from the initramfs. We cannot send them from >> the initramfs, unless we bring up the network then, which we don't want >> to do just for this purpose. But those messages are stored in /run/log, >> and then flushed to /var/log, and the upl

Re: [systemd-devel] name_to_handle_at() with UBIFS

2014-12-17 Thread Umut Tezduyar Lindskog
On Wed, Dec 17, 2014 at 7:54 PM, Lennart Poettering wrote: > On Wed, 17.12.14 10:45, Umut Tezduyar Lindskog (u...@tezduyar.com) wrote: > >> For the record: >> >> It is not enough to have CONFIG_FHANDLE to get the support for >> name_to_handle_with(). The FS also needs to support it and UBIFS >> do

Re: [systemd-devel] [PATCH v2 5/5] mount: auto-detect iSCSI and FCoE as requiring network

2014-12-17 Thread Lennart Poettering
On Wed, 17.12.14 13:13, Karel Zak (k...@redhat.com) wrote: > On Fri, Dec 12, 2014 at 08:11:54PM +0100, Lennart Poettering wrote: > > > I guess it's enough to add the 'fd' to systmed sd_event_add_io() and > > > call mnt_table_parse_mtab() when a change is detected. (As already > > > implemeted in t

Re: [systemd-devel] name_to_handle_at() with UBIFS

2014-12-17 Thread Lennart Poettering
On Wed, 17.12.14 10:45, Umut Tezduyar Lindskog (u...@tezduyar.com) wrote: > For the record: > > It is not enough to have CONFIG_FHANDLE to get the support for > name_to_handle_with(). The FS also needs to support it and UBIFS > doesn't have support for it at the moment. For what precisely? The m

Re: [systemd-devel] [PATCH v3] Add FDB support

2014-12-17 Thread Jóhann B. Guðmundsson
On 12/17/2014 03:35 PM, Alin Rauta wrote: Hi Tom, I've formatted the patch based on our previous discussions. This one has support only for adding FDB entries. An example configuration is like below: [Match] Name=em1 [Network] DHCP=v4 [BridgeFDB] MACAddress=44:44:12:34:56:71 VLANId=9 [Brid

[systemd-devel] [PATCH v3] Add FDB support

2014-12-17 Thread Alin Rauta
Signed-off-by: Alin Rauta --- Makefile.am | 1 + man/systemd.network.xml | 22 +++ src/libsystemd/sd-rtnl/rtnl-message.c| 56 ++- src/libsystemd/sd-rtnl/rtnl-types.c | 15 +- src/network/networkd-fdb.c | 252 +++

[systemd-devel] [PATCH v3] Add FDB support

2014-12-17 Thread Alin Rauta
Hi Tom, I've formatted the patch based on our previous discussions. This one has support only for adding FDB entries. An example configuration is like below: [Match] Name=em1 [Network] DHCP=v4 [BridgeFDB] MACAddress=44:44:12:34:56:71 VLANId=9 [BridgeFDB] MACAddress=44:44:12:34:56:70 VLANId=2

Re: [systemd-devel] Using `systemctl edit` on "invalid" unit names

2014-12-17 Thread Zbigniew Jędrzejewski-Szmek
On Wed, Dec 17, 2014 at 11:29:38AM +0300, Ivan Shapovalov wrote: > On Tuesday, December 16, 2014 at 06:35:09 AM, Zbigniew Jędrzejewski-Szmek > wrote: > > On Sun, Dec 14, 2014 at 04:21:32PM +0300, Ivan Shapovalov wrote: > > > On Saturday 13 December 2014 at 15:34:01, Ronny Chevalier wrote:

Re: [systemd-devel] [PATCH v2 5/5] mount: auto-detect iSCSI and FCoE as requiring network

2014-12-17 Thread Karel Zak
On Fri, Dec 12, 2014 at 08:11:54PM +0100, Lennart Poettering wrote: > > I guess it's enough to add the 'fd' to systmed sd_event_add_io() and > > call mnt_table_parse_mtab() when a change is detected. (As already > > implemeted in the original Chris' patch.) > > Karel, if I got this right, then the

Re: [systemd-devel] [dm-devel] multipath breaks with recent udev/systemd

2014-12-17 Thread Hannes Reinecke
On 12/16/2014 11:18 PM, Benjamin Marzinski wrote: > On Tue, Dec 16, 2014 at 04:10:44PM -0600, Benjamin Marzinski wrote: >> On Mon, Dec 15, 2014 at 10:31:44AM +0100, Hannes Reinecke wrote: [ .. ] >>> So during bootup it's anyone's guess who's first, multipath or udev. >>> And depending on the timing

Re: [systemd-devel] [PATCH] path: follow symbolic link for parent path

2014-12-17 Thread Umut Tezduyar Lindskog
On Wed, Dec 17, 2014 at 10:21 AM, "Jóhann B. Guðmundsson" wrote: > > On 12/17/2014 02:20 AM, Zbigniew Jędrzejewski-Szmek wrote: >> >> On Tue, Dec 16, 2014 at 09:58:39PM +0100, Umut Tezduyar Lindskog wrote: >>> >>> --- >>> src/shared/path-util.c | 2 +- >>> 1 file changed, 1 insertion(+), 1 dele

[systemd-devel] name_to_handle_at() with UBIFS

2014-12-17 Thread Umut Tezduyar Lindskog
For the record: It is not enough to have CONFIG_FHANDLE to get the support for name_to_handle_with(). The FS also needs to support it and UBIFS doesn't have support for it at the moment. Umut ___ systemd-devel mailing list systemd-devel@lists.freedeskto

Re: [systemd-devel] [PATCH] path: follow symbolic link for parent path

2014-12-17 Thread Jóhann B. Guðmundsson
On 12/17/2014 02:20 AM, Zbigniew Jędrzejewski-Szmek wrote: On Tue, Dec 16, 2014 at 09:58:39PM +0100, Umut Tezduyar Lindskog wrote: --- src/shared/path-util.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/shared/path-util.c b/src/shared/path-util.c index dcc8321..304

Re: [systemd-devel] Using `systemctl edit` on "invalid" unit names

2014-12-17 Thread Ivan Shapovalov
On Tuesday, December 16, 2014 at 06:35:09 AM, Zbigniew Jędrzejewski-Szmek wrote: > On Sun, Dec 14, 2014 at 04:21:32PM +0300, Ivan Shapovalov wrote: > > On Saturday 13 December 2014 at 15:34:01, Ronny Chevalier wrote: > > > 2014-12-13 11:33 GMT+01:00 Ivan Shapovalov : > > > > Hello all,

Re: [systemd-devel] [PATCH] path: follow symbolic link for parent path

2014-12-17 Thread Umut Tezduyar Lindskog
Thanks Zbigniew, I have only fixed the fallback: case and missed to fix the real case where we use the name_to_handle. Could you please change that too. It will look something like this: @@ -489,7 +489,7 @@ int path_is_mount_point(const char *t, bool allow_symlink) { -r = name_to_handle_a