update TODO

util: make it easy to initialize the crtime from the current time in fd_setcrtime()

import: also set NOCOW for gpt disk images

Given the write patterns on disk images, we better should turn COW off
for them. In particular as the file systems used inside the disk images
should do their own data integrity checks anyway and we don't need
multiple layers of it.

journald: turn off COW for journal files on btrfs

btrfs' COW logic results in heavily fragment journal files, which is
detrimental for perfomance. Hence, turn off COW for journal files as we
create them.

Turning off COW comes at the cost of data integrity guarantees, but this
should be acceptable, given that we do our own checksumming, and
generally have a pretty conservative write pattern.

Also see discussion on linux-btrfs:

http://www.spinics.net/lists/linux-btrfs/msg41001.html

util: upgrade default $TERM from vt102 to vt220 if we have no idea about the connected terminal

So far, if we had no knowledge about the correct $TERM we defaulted to
v102, as a safe, conservative choice. However, the terminfo data for
vt102 is not aware of pageup/pagedown, which makes "less" much harder
work with than necessary. Setting vt220 allows them to work correctly.

"vt220" should be a sufficiently safe choice too, given that xterm,
gnome-terminal and the linux console all strive to implement vt220 as
baseline, already to pass pageup/pagedown correctly to apps.

Effectively, with this change "journalctl -e" run inside a
"systemd-nspawn" terminal will now run a pager where pageup/pagedown
works, which is quite an improvement of usability for containers.

bus-util: fix error number output when waiting for job

update TODO

units: make resolved pull in its own .busname unit, but only on kdbus systems

The daemon requires the busname unit to operate, since it contains the
policy that allows it to acquire its service name.

service: automatically create After= dependency from services to their .busname units, if BusName= is set

conf-parse: don't accept invalid bus names as BusName= arguments in service units

conf-parse: make syntax logging functions behave more like other log functons

In particular, don't patch the error number to EINVAL if 0, and don't
negate it.

(Also, add do {} while (false) around multi-line macro)

units: improve Description= for systemd's own busname unit

hwdb: mouse DPI data, Logitech Gaming Mouse G400

busctl: exit cleanly when the bus connection is severed

sd-bus: when we synthesize messages, initialize timestamps ourselves

sd-bus: unify how we set the sender of synthetic messages

sd-bus: make use of the newly added timestamps on kdbus kernel messages

sd-bus: always catch name requests for the special names "org.freedesktop.DBus" and "org.freedesktop.DBus.Local" and refuse them

update TODO

machinectl: prettify "machinectl list" output

sysv-generator: initialize units before use to ensure correct ordering

The original loop called fix_order() on each service immediately after
loading it, but fix_order() would reference other units which were not
loaded yet.
This resulted in bogus and unnecessary orderings based on the static
start priorities.

Therefore call load_sysv() for every init script when traversing them in
enumerate_sysv(). This ensures that all units are loaded when
fix_order() is called.

Bug-Debian: https://bugs.debian.org/771118

update TODO

man: document nss-mymachines

man: add an alias to libnss_myhostname.so2 for nss-myhostname

The module appears under the name "libnss_myhostname.so.2" in the file
system, hence let's link it up under that name.

man: rework nss-myhostname man page

Remove a number of incorrect links, and explain that "localhost" and
"gateway" are now resolved by it, too.

Update TODO

ptyfwd: simplify how we handle vhangups a bit

btrfs-util: rework how we iterate through the results of the TREE_SEARCH results

Let's introduce some syntactic sugar with iteration macros, and add
correct key increment calls.

sysv-generator: handle Provides: for non-virtual facility names

The list of provided facility names as specified via Provides: in the
LSB header was originally implemented by adding those facilities to the
Names= property via unit_add_name().

In commit 95ed3294c632f5606327149f10cef1eb34422862 the internal SysV
support was replaced by a generator and support for parsing the Names=
option had been removed from the unit file parsing in v186.
As a result, Provides: for non-virtual facility was dropped when
introducing the sysv-generator.

Since quite a few SysV init scripts still use that functionality (at
least in distros like Debian which have a large body of SysV init
scripts), add back support by making those facility names available via
symlinks to the unit filename to ensure correct orderings between
SysV init scripts which use those facility names.

Bug-Debian: https://bugs.debian.org/774335

update TODO

machinectl: make sure that "machinectl login" exits immediately when the machine it is connected to dies

util: make use of kcmp() to compare fds, if it is available

util: don't fail recursive bind mounting if we cannot read the mount flags from an obstructed mounted

core: implement serialization/deserialization of fd store elements

tmpfiles: make gcc shut up

journal: consider file deletion errors a reason for rotation

journald: whenever we rotate a file, btrfs defrag it

Our write pattern is quite awful for CoW file systems (btrfs...), as we
keep updating file parts in the beginning of the file. This results in
fragmented journal files. Hence: when rotating files, defragment them,
since at that point we know that no further write accesses will be made.

Update TODO

tmpfiles: finish with EXIT_FAILURE if anything failed

Return value is successful only if everything succeeded.

tree-wide: remove unnecessary LOG_PRI

LOG_DEBUG is already a log level, there is no need to use LOG_PRI which
is for filtering out the facility.

journal-upload: enable curl debug output conditionally

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

po: Initial Hungarian translation

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

man: clarify path escaping and reference systemd-escape

This patch adds more detail to the description of how path escaping
operates and provides a pointer to the systemd-escape program. Either
would serve to answer the question raised in the bug report, so
hopefully this will allow it to be closed.

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

build-sys: add two more missing makefile links

machined: simplification

journald: allow restarting journald without losing stream connections

Making use of the fd storage capability of the previous commit, allow
restarting journald by serilizing stream state to /run, and pushing open
fds to PID 1.

core: add new logic for services to store file descriptors in PID 1

With this change it is possible to send file descriptors to PID 1, via
sd_pid_notify_with_fds() which PID 1 will store individually for each
service, and pass via the usual fd passing logic on next invocation.
This is useful for enable daemon reload schemes where daemons serialize
their state to /run, push their fds into PID 1 and terminate, restoring
their state on next start from the data in /run and passed in from PID
1.

The fds are kept by PID 1 as long as no POLLHUP or POLLERR is seen on
them, and the service they belong to are either not dead or failed, or
have a job queued.

cgroup: memory limits on / are not supported

cgroup: fix error message

systemd[1]: Failed to set memory.limit_in_bytes on : Invalid argument

journal: call connect() with dropped privileges

When systemd starts a service, it first opened /run/systemd/journal/stdout
socket, and only later switched to the right user.group (if they are
specified). Later on, journald looked at the credentials, and saw
root.root, because credentials are stored at the time the socket is
opened. As a result, all messages passed over _TRANSPORT=stdout were
logged with _UID=0, _GID=0.

Drop real uid and gid temporarily to fix the issue.

core: rework counting of running jobs

Let's unify the code that counts the running jobs a bit, in order to
make sure we are less likely to miss one.

This is related to this bug:

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

However, it probably won't fix it fully, and I cannot reproduce the issue.

The change also adds an explicit assert change when the counter is off.

po: add Swedish translation

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

path-lookup, systemctl: export lookup_paths_init_from_scope() from shared/install.c and use it

machine: add reference to machine-dbus.h to Makefile.am

Commit 003dffde2c1b93 ("machined: Move image discovery logic into src/shared,
so that we can make use of it from nspawn") moved some definitions from
machine.h to a new machine-dbus.h, but did not include it in Makefile.am

Tested that `make distcheck` works after this fix.

tmpfiles: fix 'D' lines

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

nss-myhostname: always will in canonical hostname field when resolving addresses to hostnames

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

bus-proxyd: don't allow to acquire org.freedesktop.DBus name

core: fix typo in log message

machinectl: remove spurious newline

po: update Italian translation

sd-bus: sync kdbus.h

Catch up with latest changes in kdbus.ko:

 * Signals can be sent as unicast now, hence they need to be marked as
   such with the KDBUS_MSG_SIGNAL in the message flags.

 * Follow ioctl number change for KDBUS_CMD_FREE

util: Do not clear parent mount flags when setting up namespaces

When setting up a namespace, mount flags like noexec, nosuid and
nodev are cleared, so the mounts always have exec, suid and dev
flags enabled.

Copy source directory mount flags to target mount when remounting
the bind mounts.

udevadm,..: make --help output of udev tools more like the output of the various other tools

fstab-generator: use more appropriate checks for swap and device availability

We always should use the same checks when deciding whether swap support
and mounting of devices is supported. Hence, let's make
fstab-generator's logic more similar to the usual logic we follow:

a) Look for /proc/swaps and no container support before activating
swaps.

b) Look for /sys being writable befire supporting device mounts.

libsystemd-network: fix typo in lldp

util: Fix signedness error in lines(), match implementations

Regression introduced by ed757c0cb03eef50e8d9aeb4682401c3e9486f0b

Mirror the implementation of columns(), since the fd_columns()
functions returns a negative integer for errors.

Also fix columns() to return the unsigned variable instead of the
signed intermediary (they're the same, but better to be explicit).

update TODO

journald: reuse IOVEC_TOTAL_SIZE() macros where possible

journald: when we detect the journal file we are about to write to has been deleted, rotate

https://bugzilla.redhat.com/show_bug.cgi?id=1171719

systemctl: properly iterate through service array when dispatching to sysv

https://bugzilla.redhat.com/show_bug.cgi?id=1175734

journald: add some additional checks before we divide by values read from journal file headers

Since the file headers might be replaced by zeroed pages now due to
sigbus we should make sure we don't end up dividing by zero because we
don't check values read from journal file headers for changes.

journalctl: static variables immediately configured via command line arguments should be prefixed with "arg_"

journal: install sigbus handler for journal tools too

This makes them robust regarding truncation. Ideally, we'd export this
as an API, but given how messy SIGBUS handling is, and the uncertain
ownership logic of signal handlers we should not do this (unless libc
one day invents a scheme how to sanely install SIGBUS handlers for
specific memory areas only). However, for now we can still make all our
own tools robust.

Note that external tools will only have read-access to the journal
anyway, where SIGBUS is much more unlikely, given that only writes are
subject to disk full problems.

systemctl: fix waiting for jobs when using direct connections to PID 1 for dbus

journald: constify all things

machined,bus-proxy: fix connecting to containers

journald: prefix exported calls with "server_", unexport unnecessary calls

journald: process SIGBUS for the memory maps we set up

Even though we use fallocate() it appears that file systems like btrfs
will trigger SIGBUS on certain low-disk-space situation. We should
handle that, hence catch the signal, add it to a list of invalidated
pages, and replace the page with an empty memory area. After each write
check if SIGBUS was triggered, and consider the write invalid if it was.

This should make journald a lot more robust with file systems where
fallocate() is not reliable, for example all CoW file systems
(btrfs...), where changing written data can fail with disk full errors.

https://bugzilla.redhat.com/show_bug.cgi?id=1045810

update TODO

nspawn: mount most of the cgroup tree read-only in nspawn containers except for the container's own subtree in the name=systemd hierarchy

More specifically mount all other hierarchies in their entirety and the
name=systemd above the container's subtree read-only.

cgroup: downgrade log messages when we cannot write to cgroup trees that are mounted read-only

hwdb: fix a typo

hwdb: revert Logitech Optical USB Mouse

Reporter says he incorrectly measured the data but the device is not available
anymore to correct it. We'll have to wait for someone else to submit the data.

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

machinectl: Check type instead of path before printing the type

Looks like a typo when introduced in fefdc04b38725457a91651218feb7000f6ccc1f4

po: update Russian translation

By analogy with commit 4038d7502f.

po: update French translation

mount: do not use -n when running in --user mode

-n is only allowed for root. /etc/mtab is nowadays almost always a link to /proc/,
so in practice this does not really matter too much, but should allow .mount units
to work in --user mode.

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

Type of mount(2) flags is unsigned long

udev: improve help/usage for some more programs

networkctl: avoid potential use of unitialized variables

Those values are based on a file we read from disk, so we should
verify everything we receive, and make sure everything we print
is sensible.

Also, print fractional seconds for TTL.

networkctl: remove unused variable

network: fix scanf/printf format

usec_t is defined as 64 bit wide, but long is 32 bit on many archs.

missing: add __NR_renameat2

build-sys: Fix install location of systemd.pc

systemd.pc contains "libdir" which can be architecture specific. Thus it needs
to be installed into libdir/pkgconfig/ instead of datadir/pkgconfig.

As nothing else is using pkgconfigdata any more, remove it entirely.

Note that udev.pc does not contain architecture specific values and thus can be
kept in /usr/share/pkgconfig/.

Remove "to allow" from policy messages

It carries no additional information and forces a passive sentence
structure which is longer and harder to parse.

po: update Polish translation

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

machined: fix grammar in org.freedesktop.machine1.policy.in

[zj: change "in into" to "into".]

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

build-sys: fix build issue with machines.target

When doing "make clean" the unit/machines.target file gets deleted.
This causes a build error later on when trying to rebuild systemd.

V2: The file probably belongs to dist_systemunit_DATA

man: Clarify effect when both calendar day and date are listed in timer

See bug 87859 (https://bugs.freedesktop.org/show_bug.cgi?id=87859). Bug
reporter found the language describing the effect of specifying both a
day and date unclear; hopefully the attached patch will clarify and
allow the bug to be closed.

lldp: fix sd_lldp_save()

Fix a bunch of needless memzero() calls, a bunch of use-after-free
regarding _cleanup_free_ and drop unused variables.

Hint: Do NOT use _cleanup_free_ for temporary strappend() helpers that are
freed multiple times. All you safe is the last free() call, which is
really not worth the trouble resetting it to NULL all the time.