update TODO

parse-util: really refuse parsing negative values as positive ones, even on x86-32

strtoull() doesn't make it particularly easy to detect passed-in
negative numbers, as it silently converts them to positive ones without
generating any error. Since we are not interested in negative values we
should hence explicitly filter them out by looking at the string
directly and returning ERANGE if we see a leading "-".

Fixes: #1829

journald: dispatch SIGTERM/SIGINT with a low priority

Let's make sure to process all queued log data before exiting, so that
we don't unnecessary lose messages when shutting down.

https://github.com/systemd/systemd/pull/1812#issuecomment-155149871

core: accept time units for time-based resource limits

Let's make sure "LimitCPU=30min" can be parsed properly, following the
usual logic how we parse time values. Similar for LimitRTTIME=.

While we are at it, extend a bit on the man page section about resource
limits.

Fixes: #1772

core: when parsing resource limits, be more careful with types and corner cases

Let's not convert RLIM_INFINITY to "unsigned long long" and then back to
rlim_t, but let's leave it in the right type right-away.

Parse resource limits as 64 bit in all cases, as according to the man
page that's what libc does anyway.

Make sure setting a resource limit to (uint64_t) -1 results in a parsing
error, and isn't implicitly converted to RLIM_INFINITY.

core: simplify parsing of capability bounding set settings

Let's generate a simple error, and that's it. Let's not try to be smart
and record the last word that failed.

Also, let's make sure we don't compare numeric values with 0 by relying
on C's downgrade-to-bool feature, as suggested in CODING_STYLE.

time-util: add parse_time(), which is like parse_sec() but allows specification of default time unit if none is specified

This is useful if we want to parse RLIMIT_RTTIME values where the common
UNIX syntax is without any units but refers to a non-second unit (µs in
this case), but where we want to allow specification of units.

defs: rework CONF_DIRS_NULSTR() macro

The macro is generically useful for putting together search paths, hence
let's make it truly generic, by dropping the implicit ".d" appending it
does, and leave that to the caller. Also rename it from
CONF_DIRS_NULSTR() to CONF_PATHS_NULSTR(), since it's not strictly about
dirs that way, but any kind of file system path.

Also, mark CONF_DIR_SPLIT_USR() as internal macro by renaming it to
_CONF_PATHS_SPLIT_USR() so that the leading underscore indicates that
it's internal.

cgls: when showing root slice, put -.slice at top of tree

cgls: suppress output of controller name, if it's the systemd one

virt: make sure that we detect unknown container managers as VIRTUALIZATION_CONTAINER_OTHER

If we don't know a container manager, we should consider it as "other"
rather than as no container manager at all, to provide a somwhat useful
upgrade path.

CODING_STYLE: elaborate on usage of C99 fixed size integer types

man: fix reference to description of time span syntax

Merge pull request #1825 from ssahani/ipv61-1

networkd: add support to configure IPv6 hop limit

Merge pull request #1827 from davidreynolds/use_extract_first_word

conf-parser: use extract_first_word

Merge pull request #1830 from jsynacek/journalctl-time

man/journalctl: mention systemd.time(7) where appropriate

Merge pull request #1831 from keszybz/todo-trimming

Todo trimming

Merge pull request #1735 from thom311/master

lldp: avoid compiler warnings in lldp_tfl_package_read* functions

man: describe the reason why runlevels are obsolete

Put it at the top of the file, where it's hard to miss.

Also add the mapping of runlevel → target because since it is now
static.

I'm not adding runlevel(7), because we do not want to make obsolete
stuff even more prominent.

man: tweak description of machinectl show

Also, machinectl status is anything but terse. Remove "terse".

TODO: update for lz4/xz stuff

No point in fixing xz now that lz4 is the default.

man/journalctl: mention systemd.time(7) where appropriate

Original bug report: https://bugzilla.redhat.com/show_bug.cgi?id=1182661

conf-parser: use extract_first_word

man: add man for IPV6 hop limit

networkd: Add support to configure IPV6 hop limit

This patch adds support to configure IPV6 hop limit.

For example:
/proc/sys/net/ipv6/conf/wlp3s0/hop_limit

Merge pull request #1790 from endocode/kayrus/fix_man_kernel_cl

Fixed kernel-command-line links

Merge pull request #1629 from ssahani/vxlan

networkd: vxlan add option to set FDB entries

Merge pull request #1821 from darkcircle/ko-catalog-translation

l10n: add a new Korean translation for the catalog

Merge pull request #1820 from michich/errno-v2

[v2] treewide: treatment of errno and other cleanups

add a new Korean translation for the catalog

Merge pull request #1762 from endocode/dongsu/l10n-ko-msgs

po: update Korean translation

treewide: apply errno.cocci

with small manual cleanups for style.

coccinelle: additional errno.cocci hunk

rewrites:
  log_error_errno(errno, ...);
  return -errno;
into:
  return log_error_errno(errno, ...);

coccinelle: errno.cocci improvements

Apply to all log_*_errno loglevels.

Merge pull request #1798 from evverx/fix-test-udev

test-udev: small fixes

Merge pull request #1799 from jengelh/doc

doc: typo and ortho fixes

Merge pull request #1807 from michich/boot-timer

core/timer: in containers OnBootSec should count from container startup

Merge pull request #1794 from karelzak/size_limit

core: support IEC suffixes for RLIMIT stuff

Merge pull request #1817 from iaguis/nspawn-custom-service-3

nspawn: support custom container service name

detect-virt: add rkt app container runtime

nspawn: support custom container service name

We were hardcoding "systemd-nspawn" as the value of the $container env
variable and "nspawn" as the service string in machined registration.

This commit allows the user to configure it by setting the
$SYSTEMD_NSPAWN_CONTAINER_SERVICE env variable when calling
systemd-nspawn.

If $SYSTEMD_NSPAWN_CONTAINER_SERVICE is not set, we use the string
"systemd-nspawn" for both, fixing the previous inconsistency.

Merge pull request #1796 from michaelolbrich/journal-size

journal: reduce minimum journal file size to 512 KiB

core/timer: in containers OnBootSec should count from container startup

In Fedora dnf-makecache.timer specifies OnBootSec=10min, but it fired
during boot in systemd-nspawn, because the clock ticks since the host's
boot.

Fix it by treating OnBootSec the same as OnStartupSec when running in
a container.

Merge pull request #1801 from chuyd/fix-typos

test: Fix typo in TEST-03-JOBS/test-jobs.sh

Merge pull request #1803 from phomes/unused-var

core: remove unused variable

core: remove unused variable

unused since 7b2313f5

test: Fix typo in TEST-03-JOBS/test-jobs.sh

Fix typo s/enqueueing/enqueuing/

Signed-off-by: Jesus Ornelas Aguayo <jesus.ornelas.aguayo@intel.com>

Merge pull request #1795 from msekletar/networkd-forward

networkd: fix option name in log message

Merge pull request #1800 from evverx/fix-test-execute

test-execute: don't use /usr/bin/uname. use sh and PATH

test-execute: don't use /usr/bin/uname. use sh and PATH

Fixed kernel-command-line links

doc: use expanded forms for written style

doc: correct orthography, word forms and missing/extraneous words

test-udev: small fixes

* print '\n' on error
* use UDEVLIBEXECDIR (udev_rules_new uses it too)

doc: correct punctuation and improve typography in documentation

journal: reduce minimum journal file size to 512 KiB

For low end embedded systems 4 MiB for each journal file is a lot of
memory. Journald will use at least 512 KiB even if JOURNAL_FILE_SIZE_MIN is
set to less than that so just use 512 KiB.

core: support IEC suffixes for RLIMIT stuff

Let's make things more user-friendly and support for example

  LimitAS=16G

rather than force users to always use LimitAS=16106127360.

The change is relevant for options:

  [Default]Limit{FSIZE,DATA,STACK,CORE,RSS,AS,MEMLOCK,MSGQUEUE}

The patch introduces config_parse_bytes_limit(), it's the same as
config_parse_limit() but uses parse_size() tu support the suffixes.

Addresses: https://github.com/systemd/systemd/issues/1772

networkd: fix option name in log message

sd-resolve: simplify pthread_join() call

POSIX says:
The pthread_join() function shall not return an error code of [EINTR].

timesync: return negative errno from manager_adjust_clock() on error

nspawn: save errno before reopening log after exec failure

nspawn: no fake errno

The S_ISREG test does not set errno, so don't use it in the error
message.

nspawn: simplify error returns

Use the "return log_error_errno(...)" idiom to have fewer curly braces.

The last hunk also fixes the return value of setup_journal(), but the
fix has no practical effect.

libsystemd-network: inet_pton does not set errno on parsing error

it would set errno only for an invalid address family.

Also fix a copy&paste error in one error string.

import: report error before losing errno

unlink() may change errno, so report the error from rename() first, then
unlink.

firstboot: fix errno sign

journal: posix_fallocate() does not set errno

manpage says:
 posix_fallocate() returns zero on success, or an error number on
 failure. Note that errno is not set.

treewide: use the negative error codes returned by our functions

Our functions return negative error codes.
Do not rely on errno being set after calling our own functions.

journal: fix incorrect errno reporting

pread() returns -1 on error and sets errno. Do not use the -1 as errno.

journal: use int64_t instead of long for catalog file size

This replaces the use of ftell() with ftello() for 64-bit size on all
archs.

Also drops a pointless check for NULL before calling strbuf_cleanup().

journal: drop unnecessary write_catalog() parameter

write_catalog() use the hashmap only to get its size. The size is
already given in parameter 'n'.

log: whitespace style fix

tty-ask-password-agent: fix typo in error message

Merge pull request #1761 from ssahani/word

core: parse socket port to extract_first_word

Merge pull request #1778 from rhdrjones/detect-virt

detect-virt: one fix and arm/aarch64 dmi detection

Merge pull request #1780 from evverx/fix-install-dbus

test-functions: fix dbus-1 installation

test-functions: fix dbus-1 installation

The basic setup for the well-known system and session buses is
now done in read-only files in ${datadir} (normally /usr/share).
See the NEWS entry for 1.9.18 for details.

http://cgit.freedesktop.org/dbus/dbus/tree/NEWS

detect-virt: dmi: look for KVM

Some guests (ARM, AArch64, x86-RHEL) have 'KVM' in the product name.
Look for that first in order to more precisely report "kvm" when
detecting a QEMU/KVM guest. Without this patch we report "qemu",
even if KVM acceleration is in use on ARM/AArch64 guests.

I've only tested a backported version of this and the previous
patch on an AArch64 guest (which worked). Of course it would be
nice to get regression testing on all guest types that depend on
dmi done.

arm/aarch64: detect-virt: check dmi

ARM/AArch64 guests now have SMBIOS tables populated (when boot
with a late enough QEMU and a late enough AAVMF is used as the
bootloader). Furthermore, when booting ARM/AArch64 guests with
ACPI, the DT detection obviously no longer works, so we need
dmi detection.

detect-virt: detect in best-heuristic order

afaict, this will fix a regression caused by commit 75f86906c5.
Where we used to report "kvm" before that patch, without this patch,
we would only report "qemu". The reason is because cpuid detection
must come before dmi detection. Also, both can safely come before
other xen heuristics. Untested.

Merge pull request #1775 from fbuihuu/set-property-completion

bash-completion: add minimal support fpr 'set-property' command

Merge pull request #1776 from marktheunissen/master

systemctl: update documentation for --plain option

Merge pull request #1777 from torstehu/fix-typo2

core: fix typo

core: fix typo

Merge pull request #1773 from evverx/dont-hide-enomem

core: small fixes to parse_namespace

systemctl: update documentation for --plain option

- Existing documentation did not mention that --plain omits the bullets and works on list-units and list-machines.

Merge pull request #1768 from vcaputo/sd-daemon-listen-fds-overflow-bis

sd-daemon: fix potential LISTEN_FDS overflow in sd_listen_fds()

bash-completion: add minimal support fpr 'set-property' command

'set-property' was missing from the list of known command.

Also a list of unit names will be proposed as next argument.

However no support on property names is provided since it would
require a hard coded list of them.

core: small fixes to parse_namespace

* don't hide ENOMEM
* log r instead of 0

sd-daemon: fix potential LISTEN_FDS overflow in sd_listen_fds()

core: parse socket port to extract_first_word

Merge pull request #1767 from poettering/journald-watchdog

restore journald watchdog

journal: restore watchdog support

sd-daemon: explicitly filter out -1 when parsing watchdog timeout

We already filter out 0, and as -1 is usually special (meaning infinity,
as in USEC_INFINITY) we should better not accept it either. Better safe
than sorry...

util-lib: move CONF_DIRS_NULSTR definition to def.h

After all, this is not some compiler or C magic, but something very
specific to how systemd works, hence let's move it into def.h, and out
of macro.h

util-lib: move character class definitions to string-util.h

Merge pull request #1764 from ssahani/jiffies-1

(V2) networkd: bridge convert to jiffies

networkd: bridge prop convert to jiffies

Time-util: introduce usec to jiffies

Merge pull request #1765 from teg/pppoe-removal

sd-pppoe: drop