update TODO

timesyncd: make use of floating event sources for signal handling

timesyncd: enable watchdog support

timesyncd: run timesyncd as unpriviliged user "systemd-timesync" (but still with CAP_SYS_TIME)

man: note that entire sections can now be ignored

Prefixing a section name with "X-" will cause it and all of its contents
to be silently ignored as of commit 342aea19.

networkd: log - only log about udev initalization on debug level

networkd: keep list of active addresses

networkd: IP address equality

TODO

udev: avoid use of uninitialized err

After 1ea972174baba40dbc80c51cbfc4edc49764b59b err is no longer
set unless we hit a special case. Initialize it to 0 and remove
a check that will never fail.

systemctl: more cleanup

This is followup on 05cae7f3431446236139434ee58a6275f3cb31e8

I think the intention was to use the newly introduced 'path'
variable in the inner loop instead of p.

conf-parser: silently ignore sections starting with "X-"

This allows external tools to keep additional unit information in a
separate section without scaring users with a big warning.

man: logind.conf: fix grammar issues, unclear wording, and unclear default values

hwdb: Update database of Bluetooth company identifiers

bus: fix unitialized variable access in error path

networkd: add missing files

core: let selinux_setup() load policy more than once

When you switch-root into a new root that has SELinux policy, you're
supposed to to run selinux_init_load_policy() to set up SELinux and load
policy. Normally this gets handled by selinux_setup().

But if SELinux was already initialized, selinux_setup() skips loading
policy and returns 0. So if you load policy normally, and then you
switch-root to a new root that has new policy, selinux_setup() never
loads the new policy. What gives?

As far as I can tell, this check is an artifact of how selinux_setup()
worked when it was first written (see commit c4dcdb9 / systemd v12):

  * when systemd starts, run selinux_setup()
  * if selinux_setup() loads policy OK, restart systemd

So the "if policy already loaded, skip load and return 0" check was
there to prevent an infinite re-exec loop.

Modern systemd only calls selinux_setup() on initial load and after
switch-root, and selinux_setup() no longer restarts systemd, so we don't
need that check to guard against the infinite loop anymore.

So: this patch removes the "return 0", thus allowing selinux_setup() to
actually perform SELinux setup after switch-root.

We still want to check to see if SELinux is initialized, because if
selinux_init_load_policy() fails *but* SELinux is initialized that means
we still have (old) policy active. So we don't need to halt if
enforce=1.

logind: allow suspending if there are no displays

With proprietary graphics drivers, there won't be any 'drm' devices in
sysfs, so logind will never suspend the system upon closing the lid,
even if only one (internal) display is connected. This has been reported
by multiple users so far.

IMHO, it's better to suspend the system in this case for safety reasons,
to avoid having nvidia blob users' laptops overheat, for the same reason
that sleep inhibitors are overridden (LidSwitchIgnoreInhibited=yes).

networkd: manager - read fallback DNS servers from config file

We will still use the compiled-in defaults if no DNS entry exists in the config file.

networkd: network - store DNS servers in List rather than Set

This way we preserve the order of preference.

networkd: also add IPv6 DNS servers by default

nspawn: properly format container_uuid in UUID format

http://lists.freedesktop.org/archives/systemd-devel/2014-April/018971.html

man: update journald rate limit defaults

This brings the man page back into sync with the actual code.

path-lookup: don't hardcode .config

If XDG_CONFIG_HOME is set, then we should respect that.

path-util: add path_make_relative()

In user_dirs() in path-lookup.c, I want to replace this:
        symlink("../../../.config/systemd/user", data_home);
with
        symlink(config_home, data_home);
to avoid hardcoding .config when XDG_CONFIG_HOME is set.

The problem is that config_home is an absolute path, and it's better
to make the symlink relative. path_make_relative() is an utility
function that converts an absolute path into a relative one.

fsck: Allow to specify the fsck repair option in the cmdline

Some unattended systems do not have a console attached and entering
the default rescue mode will not be too helpful. Allow to specify
the "-y" option to attempt to fix all filesystem errors.

Manually verified by downloading an image.gz of e2fsprogs, using
losetup and running systemd-fsck on the loop device and varying
the fsck.repair=preen|yes|no option.

networkd: hardcode a set of default dns servers

Similarly to NTP servers, this can be set at compile-time.

man: readahead: fix cmdline switch inconsistency between readahead.c and docs

Source code has "files-max" and XML has --max-files.

readahead: modernizations

man: fix some minor language typos

build-sys: at configure check for verifying that ln supports --relative

network: always create /run/systemd/network/links

This ways the networkd client library should work even if networkd is
not running.

http://lists.freedesktop.org/archives/systemd-devel/2014-May/019242.html

Revert "networkd: order after udev kernel socket"

This reverts commit a555350d47c4b70d716a63424933b34902c98300.

This did not fix the problem, just made it harder to hit.

networkd: log the initialization status of links

TODO

networkd: fix typo

networkd: rename Address and Route list fields

timesyncd: fix typo in comment

man: clarify that the ExecReload= command should be synchronous

http://lists.freedesktop.org/archives/systemd-devel/2014-May/019054.html

core: make sure to serialize jobs for all units

Previously we wouldn't serialize jobs for units that themselves have
nothing to serialize.

http://lists.freedesktop.org/archives/systemd-devel/2014-May/019051.html

Update TODO

timesyncd: fix error path return value

https://bugs.freedesktop.org/show_bug.cgi?id=78752

timesyncd: limit capabilities to CAP_SYS_TIME

network: fix build failure, missing KMOD_XXX flags

- Add KMOD_CFLAGS and KMOD_LIBS where appropiate
- networkd now requires kmod. make --disable-kmod --enable-networkd
to raise an error.

logind: bring polkit policy for hibernate in line with suspend/poweroff/reboot

THere's no reason why hibernate should be better protected then
suspendor poweroff, so sync the policies.

networkd: logging - align messages

networkd: log ifindices when links and netdevs are added

networkd: rename NetDev variable for consistency with Link

sd-bus: make sure we properly handle NULL callback functions

selinux: fix build for non-selinux systems

sd-bus: fix an assert when running in kdbus mode

resolve: fix copy/paste error

sd-bus: don't hit assert when installing match

update TODO

sd-bus: always keep slot reference while dispatching callback

Also, make sure we automatically destroy reply callbacks that are
floating.

sd-resolve: add "floating" resolve queries

Same story as for sd-bus and sd-event: allow passing NULL to store query
in in which case the query is freed automatically.

sd-event: introduce concept of "floating" event sources

These are the counterpart of "floating" bus slots, i.e. event sources
that are bound to the lifetime of the event object itself, and thus
don't require an explicit reference to be kept.

core: close socket fds asynchronously

http://lists.freedesktop.org/archives/systemd-devel/2014-April/018928.html

systemctl: small cleanup

readahead: add test to show fs_on_ssd() result

networkd, build-sys: spelling fix

Make sure that keys are properly removed from hashmap

This is a speculative fix for https://bugzilla.redhat.com/show_bug.cgi?id=1088865.
Even though I cannot find a code path that where this would be
an issue, for consistency, if we assume that cgroup_path might have
been set before we got to unit_deserialize, we should make sure that
the unit is removed from the hashmap before we free the key. This seems
to be the only place where the key could be prematurely freed, leading to
hashmap corruption.

Remove unnecessary casts in printfs

No functional change expected :)

Make systemctl --root look for files in the proper places

Running systemctl enable/disable/set-default/... with the --root
option under strace reveals that it accessed various files and
directories in the main fs, and not underneath the specified root.
This can lead to correct results only when the layout and
configuration in the container are identical, which often is not the
case. Fix this by adding the specified root to all file access
operations.

This patch does not handle some corner cases: symlinks which point
outside of the specified root might be interpreted differently than
they would be by the kernel if the specified root was the real root.
But systemctl does not create such symlinks by itself, and I think
this is enough of a corner case not to be worth the additional
complexity of reimplementing link chasing in systemd.

Also, simplify the code in a few places and remove an hypothetical
memory leak on error.

shared/install: do not prefix created symlink with root path

Before: /var/tmp/inst1//etc/systemd/system/default.target -> /var/tmp/inst1//usr/lib/systemd/system/graphical.target
After: /var/tmp/inst1/etc/systemd/system/default.target -> /usr/lib/systemd/system/graphical.target

Fix typos in systemctl manpage

udev: do not skip the execution of RUN when renaming a network device fails

sd-bus: introduce sd_bus_slot objects encapsulating callbacks or vtables attached to a bus connection

This makes callback behaviour more like sd-event or sd-resolve, and
creates proper object for unregistering callbacks.

Taking the refernce to the slot is optional. If not taken life time of
the slot will be bound to the underlying bus object (or in the case of
an async call until the reply has been recieved).

sd-event: reorder header slightly

hashmap: add hashmap_remove2() to remove item from hashtable and return both value and key

core: no need to pass bus object to selinux access check calls anymore

bus: add userdata API for bus name tracker objects

core: sysvcompat - avoid repeated function call

core: sysvcompat - $network should be equivalent to network-online, rather than network target

Most likely the  facility needed is actual connectivity, rather than whether or not the
network managment daemon is running.

We also need to explicitly pull in the network-online.target, as it is not active by
default.

This means {systemd-networkd,NetworkManager}-wait-online.service, can be enabled by default
as part of network-online.target, and only delay boot when some service actively pulls it in.

See: <https://bugzilla.gnome.org/show_bug.cgi?id=728965>

Cc: Pavel Šimerda <psimerda@redhat.com>
Cc: Michal Sekletar <msekleta@redhat.com>

shared: add ring buffer

New "struct ring" object that implements a basic ring buffer for arbitrary
byte-streams. A new basic runtime test is also added.

This will be needed for our pty helpers for systemd-console and friends.

shared: add ALIGN_POWER2 macro

Sounds easy, turns out to be horrible to implement: ALIGN_POWER2 returns
the next higher power of 2. clz(0) is undefined, same is true for
left-shift-overflows, yey, C rocks!

replace more dup() by F_DUPFD_CLOEXEC

pam_systemd: use F_DUPFD_CLOEXEC when dupping session fds

http://lists.freedesktop.org/archives/systemd-devel/2014-May/019034.html

udev: rename netif - properly break lines in kmsg

Before:

30,997,4553484,-;systemd-udevd[439]: renamed network interface wwan0 to wwp0s20u4i6systemd-udevd[439]: renamed network interface wlan0 to wlp3s0
30,998,1175077801,c;systemd-udevd[2345]: renamed network interface wwan0 to wwp0s20u4i6

After:

30,834,4553484,-;systemd-udevd[439]: renamed network interface wwan0 to wwp0s20u4i6
30,835,4732949,-;systemd-udevd[439]: renamed network interface wlan0 to wlp3s0
30,988,1175077801,-;systemd-udevd[2345]: renamed network interface wwan0 to wwp0s20u4i6

timesyncd: start after networkd

This is needed for the network monitor to work (as it requires networkd to set up the correct directories first).

networkd-ipip-tunnel: add support ttl

Add support for ipip tunnel ttl.

networkd: Add todo patch kernel for tunnel module alias

Add to todo :

1. Patch kernel to support module alias for tunnel device
2. Remove libkmod dependency from networkd

networkd: manager - don't leak kmod context

Also, keep the kmod_new internal to networkd-manager.c

networkd: unit - add cap to load modules

Remember to drop this when the kernel gains autoloading for all netdev kinds.

networkd: introduce ipip tunnel

This patch enables basic ipip tunnel support.
It works with kernel module ipip

example conf:

file: ipip.netdev

[NetDev]
Name=ipip-tun
Kind=ipip
MTUBytes=1480

[Tunnel]
Local=192.168.223.238
Remote=192.169.224.239
TTL=64

file: ipip.network

[Match]
Name=em1

[Network]
Tunnel=ipip-tun

[tomegun:
         - drop unused variable
         - take ref when enslaving]

job: always add waiting jobs to run queue during coldplug

commit 20a83d7bf was not equivalent to the original bug fix proposed by
Michal Sekletar <msekleta@redhat.com>. The committed version only added
the job to the run queue if the job had a timeout, which most jobs do
not have. Just re-ordering the code gets us the intended functionality

networkd: get preexiting addresses when a link is added

networkd: listen for address changes

This is proof-of-concept only, as we only log the changes but don't do anything
with it.

rtnl: message - fix check for broadcast messages

rtnl: message - read group membership of incoming messages

rtnl: message - verify that we read the pending message size from the kernel

Reuse the auth-checking for both the peek and the real read.

rtnl: message - move code around

No functional change.

rtnl: change from bitmask to enum for rtnl groups

The bitmask is deprecated in the kernel, so move to the new interface. At the moment
this does not make a difference for us, but it avoids having to change the API in the future.

networkd: link - redo flag change logging

Make the logging less verbose by only printing all the changed flags on one line,
at the same time make it more complete by supporting all flags currently supported
by the kernel.

We still fall back to printing the raw flags in case we get something we do not recognize
This may be useful when running on new kernels.

sd-rtnl: message - add support for getting prefixlen from address messages

networkd: network - fix leak

networkd: manager - initialize variables

networkd: netdev - rephrase logging message a bit

Make it fit with what is logged from the link.

networkd: manager - refactor link tracking a bit

systemctl: return an error code is status fails

This got lost in the refactoring in f74294c1dabb4.

Also make sure that the return code corresponds to the *first* failure,
not the last.

https://lists.fedoraproject.org/pipermail/devel/2014-May/199080.html