man: improve and fix documentation for RemainAfterElapse=

NEWS: add in missing NEWS entry for 228 feature RemainAfterElapse=

Merge pull request #1943 from teg/networkd-fixes

networkd: fixes

networkd: dhcp4 - dirty link when the lease changes

Without this commit, we risk not picking up DNS addresses
as soon as they are available.

networkd: ndisc - revert to letting the kernel handle NDisc

There is still a bug in the userspace version, so temporarily revert this.

This is a work-around for issue #1866.

Merge pull request #1940 from evverx/man-journald-syslog-forwarding

man: remove section about syslog forwarding

Merge pull request #1912 from mustrumr/calendarspec-sub-second-v3

Calendarspec sub second v3

Merge pull request #1889 from ssahani/socket-proto

socket: Add support for socket protcol

build: bump version numbers

Prepare for v228 release and bump version numbers.

Merge pull request #1936 from poettering/transient-timers

Fix transient timers and other fixes

Socket: socket protocol add to dbus properties

socket: Add support for socket protcol

Now we don't support the socket protocol like
sctp and udplite .

This patch add a new config param
SocketProtocol: udplite/sctp

With this now we can configure the protocol as

udplite = IPPROTO_UDPLITE
sctp = IPPROTO_SCTP

Tested with nspawn:

man: remove section about syslog forwarding

journald: turn ForwardToSyslog= off by default
After all, rsyslog and friends nowadays read their data directly from
the journal, hence the forwarding is unnecessary in most cases.

see 46b131574fdd7d77c15a0919ca9010c

man: calendarspec sub-second support

test: calendarspec sub-second tests

calendarspec: sub-second support, v3

run: automatically clean up transient timers created by systemd-run

core: add RemainAfterElapse= setting to timer units

Previously, after a timer unit elapsed we'd leave it around for good,
which has the nice benefit that starting a timer that shall trigger at a
specific point in time multiple times will only result in one trigger
instead of possibly many. With this change a new option
RemainAfterElapse= is added. It defaults to "true", to mimic the old
behaviour. If set to "false" timer units will be unloaded after they
elapsed. This is specifically useful for transient timer units.

core: Minor cleaning up of unit/log status and log logic

We only reorder a few things and modernize some constructs. No
functional changes.

- Move some if checks from the caller to the callee of a few functions.

- Use IN_SE() where we can

- Move status printing functions together

run: when automatically generating names for transient units, use unique bus ID, fallback to random

Previously we used the process ID to generate transient unit names.
However, that is problematic as PIDs get reused easily, and applying
them to remote systems makes little sense.

Fortunately, each bus peer gets a unique, non-reusable ID assigned when
attaching to a bus, hence let's use that, if we can. In some cases we
cannot however, because we connect directly to PID's private socket, and
thus are not a proper bus peer with a unique ID. In that case generate a
random UUID to name the unit after.

core: make unit_make_transient() more thorough

Let's reset more stuff that does not apply to transient units. Also,
let's readd the unito to all queues, because it's identity now changed.

core: dispatch load queue each time we set up a transient units

manager_load_unit() will dispatch the load queue anyway, but let's make
sure we also dispatch it immediately, after truning a unit into a
transient one and loading the properties from the message. That way the
know about the validity of the unit before we begin processing the next
auxiliary unit.

core: move check whether a unit is suitable to become transient into unit.c

Lets introduce unit_is_pristine() that verifies whether a unit is
suitable to become a transient unit, by checking that it is no
referenced yet and has no data on disk assigned.

core: generate nice error messages for auxiliary transient units, too

Let's move the validation checks into the loop that sets up the main and
auxiliary transient units, so that we can generate pretty error messages
for all units a transient unit transaction generates, not just for the
main unit.

sysctl: use %P instead of %p in core pattern

That way we'll get the PID on the host, rather than the one in a PID
namespace. Which should make the coredump handler less confusing.

Fixes #1930.

Merge pull request #1920 from teg/networkd-fixes

networkd fixes

Merge pull request #1932 from teg/networkd-ndisc-timeout

networkd: ndisc timeout

networkd: ndisc - consider configured on timeout

Don't block indefinitely, when control has been passed on from NDisc to DHCPv6.
In this case there is likely no IPv6 support on the local link, so otherwise
this would block indefinitely.

sd-dhcp6-client: bind to link-local address

This ensures that several DHCPv6 clients can run on separate interfaces
simultaneously.

networkd: ndisc - fix token support

Fixes CID#1338680. Thanks to Thomas Andersen.

sd-dhcp6-client: allow multiple clients on host

We need to enable SO_REUSEADDR in order for several sockets to be allowed
to bind to the same port (even on different links).

networkd: ndisc - always configure dhcp6 client

The ndisc client may trigger the dhcpv6 client to be started (this is the common case),
so we should allocate the dhcpv6 client whenever we allocate the ndisc one.

Merge pull request #1909 from keszybz/filetriggers-v2

Move daemon-reload from package %post scripts to file triggers

Merge pull request #1925 from evverx/fix-pam-systemd-user

tests: copy /etc/pam.d/systemd-user from the host on Debian, Ubuntu

Merge pull request #1926 from phomes/include-order-libudev

tree-wide: group include of libudev.h with sd-*

Merge pull request #1927 from evverx/no-entries-in-quiet-mode

journalctl: don't print -- No entries -- in quiet mode

Merge pull request #1924 from poettering/some-fixes-2

casting fixes, another siphash24 alignment fix, and more

journalctl: don't print -- No entries -- in quiet mode

tree-wide: group include of libudev.h with sd-*

Some additions to NEWS

tests: copy /etc/pam.d/systemd-user from the host on Debian, Ubuntu

Fixes:
systemd-testsuite systemd[34]: PAM _pam_load_conf_file: unable to open /etc/pam.d/system-auth
systemd-testsuite systemd[34]: PAM _pam_load_conf_file: unable to open /etc/pam.d/system-auth
systemd-testsuite systemd[34]: user@0.service: Failed at step PAM spawning /lib/systemd/systemd: Operation not permitted
...
on Debian, Ubuntu

siphash: minor coding style fixes and modernizations

Only cosmetics really, doesn't change any actual logic.

siphash: fix another alignment issue

tree-wide: make macros for converting fds to pointers and back generic and use them everywhere

remount-fs: modernize coding style a bit

a) Use _cleanup_ where it makes sense

b) Uniformly use negative errno-style errors internally, convert to
   EXIT_FAILURE/EXIT_SUCCESS only when actually exiting.

c) Use log_oom() where appropriate

d) Fix minor memory leak in hashmap addition error path.

e) Don't pretend we could continue sensibly on OOM or fork() failure

f) Use PR_SET_PDEATHSIG to make sure clients we don't kill on error are
   cleaned up.

g) Make use of STRV_MAKE() where it's pretty to do so.

h) Simplify error paths.

sd-ipv4ll: fix error path if sd-ipv4acd allocation fails

Let's make sure the destructor cannot hit the n_ref == 0 case.

journald: trivial simplification

coredump: modernize error logging a bit

core: don't generate warnings when write access to the cgroup fs fails in --user due to EACCES

After all, in the classic hierarchy that's pretty much the default case.

tree-wide: use right cast macros for UIDs, GIDs and PIDs

Merge pull request #1923 from zonque/siphash

siphash24: let siphash24_finalize() and siphash24() return the result…

Merge pull request #1922 from phomes/sort-includes

Sort includes

siphash24: let siphash24_finalize() and siphash24() return the result directly

Rather than passing a pointer to return the result, return it directly
from the function calls.

Also, return the result in native endianess, and let the callers care
about the conversion. For hash tables and bloom filters, we don't care,
but in order to keep MAC addresses and DHCP client IDs stable, we
explicitly convert to LE.

tree-wide: sort includes

Sort the includes accoding to the new coding style.

tree-wide: add missing includes

Add a few includes that we rely on to be include already.

networkd: ndisc/dhcpv6 - handle starting running clients

The clients may be triggered to be started repeatedly without being stopped first,
simply swallow the error rather than failing the link.

Merge pull request #1919 from martinpitt/master

siphash42: add tests with unaligned input pointers

Merge pull request #1918 from dvdhrm/user3

login: allow re-using users (v3)

siphash42: add tests with unaligned input pointers

Add test case for calling siphash24 with unaligned input pointers, as we
commonly get with calling it on the result on basename() or similar.

This provides a test for PR #1916, rescued from the superseded PR #1911.

Thanks to Steve Langasek for the test!

login: ignore JobRemoved of old jobs

If we requeue jobs, we are no longer interested in old jobs. Hence, we
better ignore any JobRemoved signals for old jobs and concentrate on our
replacements.

Merge pull request #1916 from zonque/align

siphash: alignment

Merge pull request #1915 from poettering/btrfs-root-subvol

tmpfiles: create subvolumes for "v", "q", and "Q" only if / is a subv…

login: make sure to replace existing units

When queuing unit jobs, we should rather replace existing units than
fail. This is especially important when we queued a user-shutdown and a
new login is encountered. In this case, we better raplce the shutdown
jobs. systemd takes care of everything else.

login: fix re-use of users

If the last reference to a user is released, we queue stop-jobs for the
user-service and slice. Only once those are finished, we drop the
user-object. However, if a new session is opened before the user object is
fully dropped, we currently incorrectly re-use the object. This has the
effect, that we get stale sessions without a valid "systemd --user"
instance.

Fix this by properly allowing user_start() to be called, even if
user->stopping is true.

login: group static fields in "struct User"

Make sure to put static fields together in "struct User". This makes it
easier to figure out the lifetime of each field.

login: make user->service static

Just like user->slice, there is no reason to store the unit name in /run,
nor should we allocate it dynamically on job instantiation/removal. Just
keep it statically around at all times and rely on user->started ||
user->stopping to figure out whether the unit exists or not.

login: make user_new() and user_free() follow coding-style

Few changes to user_new() and user_free():
 - Use _cleanup_(user_freep) in constructor
 - return 'int' from user_new()
 - make user_free() deal with partially initialized objects
 - keep reverse-order in user_free() compared to user_new()
 - make user_free() return NULL
 - make user_free() accept NULL as no-op

login: keep user->slice constant

Currently, we allocate user->slice when starting a slice, but we never
release it. This is incompatible if we want to re-use a user object once
it was stopped. Hence, make sure user->slice is allocated statically on
the user object and use "u->started || u->stopping" as an indication
whether the slice is actually available on pid1 or not.

login: simply XDG_RUNTIME_DIR management

Lets not pretend we support changing XDG_RUNTIME_DIR via logind state
files. There is no reason to ever write the string into /run, as we
allocate it statically based on the UID, anyway. Lets stop that and just
allocate the runtime_path in "struct User" at all times.

We keep writing it into the /run state to make sure pam_systemd of
previous installs can still read it. However, pam_systemd is now fixed to
allocate it statically as well, so we can safely remove that some time in
the future.

Last but not least: If software depends on systemd, they're more than free
to assume /run/user/$uid is their runtime dir. Lets not require sane
applications to query the environment to get their runtime dir. As long as
applications know their login-UID, they should be safe to deduce the
runtime dir.

tmpfiles: create subvolumes for "v", "q", and "Q" only if / is a subvolume

It's not a good idea to create subvolumes for parts of the OS tree (such
as /home, or /var) if the root directory is not a subvolume too. We
shouldn't assume control of "heavier" objects such as subvolumes, if the
originating object (the root directory) is a "light-weight" object, i.e.
a plain directory.

Effectively this means that chroot() environments that are run on a
plain directory do not have to deal with problems around systemd
creating subvolumes that cannot be removed with a simple "rm" anymore.
However, if the chroot manager creates a proper subvolume for such an
environment it will also get further subvolumes placed in there, under
the assumption that the manager understands the concept of subvolumes in
that case.

siphash24: change result argument to uint64_t

Change the "out" parameter from uint8_t[8] to uint64_t. On architectures which
enforce pointer alignment this fixes crashes when we previously cast an
unaligned array to uint64_t*, and on others this should at least improve
performance as the compiler now aligns these properly.

This also simplifies the code in most cases by getting rid of typecasts. The
only place which we can't change is struct duid's en.id, as that is _packed_
and public API, so we can't enforce alignment of the "id" field and have to
use memcpy instead.

siphash24: fix memory alignment

Use unaligned_read_le64() to access input buffer when reading complete
64-bit words.

This should fix memory traps on platforms with strict aliasing.

basic: add unaligned macros for little endian

Also add test code for that.

Merge pull request #1886 from poettering/tasks-max

Enable TasksMax by default for all units

TODO: sort

Merge pull request #1907 from evverx/fix-pam-install-on-debian

tests: fix pam_modules installation on Debian (and Debian-like systems)

virt: sort #includes alphabetically

nitpick from PR #1910

Merge pull request #1908 from phomes/network-memleak

network: dhcp6 - fix mem leak

Merge pull request #1906 from poettering/pid-check

machined,logind: be more careful when accepting PIDs and UIDs from clients

nspawn: set TasksMax= for containers to 8192 by default

logind: change default to UserTasksMax= to 4096

core: enable TasksMax= for all services by default, and set it to 512

Also, enable TasksAccounting= for all services by default, too.

See:

http://lists.freedesktop.org/archives/systemd-devel/2015-November/035006.html

Merge pull request #1910 from martinpitt/master

virt: add missing #includes

virt: add missing #includes

src/basic/virt.c: In function 'detect_vm_device_tree':
src/basic/virt.c:117:17: error: unknown type name '_cleanup_closedir_'
                 _cleanup_closedir_ DIR *dir = NULL;

src/basic/virt.c:128:17: error: implicit declaration of function 'FOREACH_DIRENT' [-Werror=implicit-function-declaration]
                 FOREACH_DIRENT(dent, dir, return -errno)

tests: intsall /etc/login.defs

Fixes:
systemd-testsuite login[31]: cannot open login definitions /etc/login.defs [No such file or directory]
systemd-testsuite systemd[1]: Received SIGCHLD from PID 31 (login).
systemd-testsuite systemd[1]: Child 31 (login) died (code=exited, status=1/FAILURE)
systemd-testsuite systemd[1]: console-getty.service: Child 31 belongs to console-getty.service
systemd-testsuite systemd[1]: console-getty.service: Main process exited, code=exited, status=1/FAILURE
systemd-testsuite systemd[1]: console-getty.service: Changed running -> dead

on Debian/Ubuntu

Useful on other distros

Move daemon-reload from package %post scripts to file triggers

This uses new functionality added in rpm 4.13. Instead of doing
one daemon-reload per packages, we do just one or two
(When both installing and uninstalling packages, we do
two. Unfortunately this also includes the common case of upgrades.
When only installing or when only installing, we do just one.)

New file triggers.systemd can be built, but the contents have
to be copied into the rpm spec file by hand. Using %{load} does
not seem to work. It can serve as documentation.

network: dhcp6 - fix mem leak

from 7a695d8e
CID#1338679

tests: fix pam_modules installation on Debian (and Debian-like systems)

libpam_modules installs modules into /lib/$(dpkg-architecture -qDEB_HOST_MULTIARCH)/security
on Debian

Fixes:
systemd-testsuite login[36]: PAM unable to dlopen(pam_group.so): /lib/security/pam_group.so: cannot open shared object file: No such file or directory
systemd-testsuite login[36]: PAM adding faulty module: pam_group.so
systemd-testsuite login[36]: PAM unable to dlopen(pam_limits.so): /lib/security/pam_limits.so: cannot open shared object file: No such file or directory
systemd-testsuite login[36]: PAM adding faulty module: pam_limits.so
...
etc

machined,logind: be more careful when accepting PIDs and UIDs from clients

Always validate first before we start processing the data.

Merge pull request #1903 from phomes/gitignore

gitignore: add test-install-root

gitignore: add test-install-root

Merge pull request #1901 from evverx/various-fixes

Various fixes

Merge pull request #1899 from phomes/NEWS-typo

NEWS: typo fixes

tests: install execs from user units too

Fixes:
systemd[39]: systemd-exit.service: Executing: /bin/kill -s 58 29
systemd[39]: systemd-exit.service: Failed at step EXEC spawning /bin/kill: No such file or directory
systemd[29]: Received SIGCHLD from PID 39 ((kill)).
systemd[29]: Child 39 ((kill)) died (code=exited, status=203/EXEC)

firstboot: log on take_etc_passwd_lock error too

Merge pull request #1898 from marcosfrm/journalctl-shell-completion-2

shell-completion: journalctl: add --sync

NEWS: typo fixes

shell-completion: journalctl: add --sync

From 94b6551

Merge pull request #1896 from phomes/install-wrong-enum

install: don't mix state and type enums

install: don't mix state and type enums

from 0ec0deaa