Merge pull request #1793 from filbranden/extract1

More refactorings in extract_first_word

Merge pull request #1783 from vcaputo/still_make_progress_when_throttling

core: still make progress when throttling the manager loop

Merge pull request #1812 from evverx/test-fixes

tests: various fixes

tests: various fixes

* remove journal flushing (systemd-journal-flush.service runs journalctl --flush on boot)
* use sh -c and PATH instead of @SYSTEMCTL@ expansion
* remove unnecessary semicolons etc

Merge pull request #1823 from whot/hwdb-updates

Updated axis ranges/resolutions for Lenovo T510 and Dell Inspiron N5040

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

hwdb: add axis range corrections for Lenovo T510

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

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

hwdb: add Dell Inspiron N5040 touchpad corrections

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

hwdb: sort the brand names alphabetically

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

extract-word: Skip coalesced separators in place

Just skip them in place, instead of setting separator=true. We only do
that in a single place (while finding a separator outside of quote or
backslash states) so we don't really need a separate state for it.

Tested that no regressions were introduced in test-extract-word. Ran a
full `make check` and also installed the binaries on a test system and
did not see any issues related to parsing unit files or starting units
after a reboot.

extract-word: Do not re-evaluate the state on each parsed character

Use inner loops to keep processing the same state, except when there is
a state change, then break back to the outer loop so that the correct
branch can be selected again.

Tested that no regressions were introduced in test-extract-word.

extract-word: increment pointer p and keep c in sync in for loop

This will make it easier to use inner loops to keep looping in the same
state, by just updating p and c in the same way in the inner loops.

Tested that no regressions were created in test-extract-word.

extract-word: Check for early bail out before inspecting separators

It's a pretty small optimization but doesn't hurt...

Tested with test-extract-word.

extract-word: replace an use of `goto` with structured code

Using `goto` might be appropriate for the "finish" cases but it was
really not necessary at this point of the code... Just use if/else
blocks to accomplish the same.

Confirmed that the test cases in test-extract-word keep working as
expected.

extract-word: move start block outside the for loop

This block runs once before all the other handling, so move it outside
the main loop and put it in its own loop until it's finished doing its
job.

Tested by confirming `make check` (and particularly test-extract-word)
still passes and by booting a system with binaries including this
commit.

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

core: still make progress when throttling the manager loop

Don't simply continue after sleeping, it potentially puts us in a state
of spinning doing nothing slowly, if the ratelimit_test() keeps
detecting the need for limiting.

Observed in vms after the host had been suspended for a while, on resume
systemd entered a loop of making zero progress spamming the console
with:
[431942.850090] systemd[1]: Looping too fast. Throttling execution a
little.

I see no reason to have a continue here, the intention should be to
throttle execution, not circumvent it altogether.

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