Merge pull request #1841 from keszybz/remove-snapshot

Remove snapshot unit type

Merge pull request #1655 from ssahani/net

network: port to extract_first_word

Merge pull request #1838 from keszybz/trivialities

Trivialities

Remove snapshot unit type

Snapshots were never useful or used for anything. Many systemd
developers that I spoke to at systemd.conf2015, didn't even know they
existed, so it is fairly safe to assume that this type can be deleted
without harm.

The fundamental problem with snapshots is that the state of the system
is dynamic, devices come and go, users log in and out, timers fire...
and restoring all units to some state from the past would "undo"
those changes, which isn't really possible.

Tested by creating a snapshot, running the new binary, and checking
that the transition did not cause errors, and the snapshot is gone,
and snapshots cannot be created anymore.

New systemctl says:
Unknown operation snapshot.
Old systemctl says:
Failed to create snapshot: Support for snapshots has been removed.

IgnoreOnSnaphost settings are warned about and ignored:
Support for option IgnoreOnSnapshot= has been removed and it is ignored

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

TODO: remove SYSTEMD_PAGER

This is already done in Fedora rawhide.

man: normalize indentation in sd-detect-virt(1)

Merge pull request #1835 from poettering/grabbag-of-stuff

Lots of small fixes

Merge pull request #1785 from ssahani/word

core: unit deps port to extract_first_word

Merge pull request #1834 from filbranden/testexecute2

test-execute: Fix systemd escaping and shell issues (v2)

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

core: unit deps port to extract_first_word

networkctl: lldp port to extract_first_word

bond: port to extract_first_word

Merge pull request #1823 from whot/hwdb-updates

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

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

test-execute: Fix systemd escaping and shell issues

In most cases, systemd requires escaping $ (for systemd variable
substitution) and % (for specifiers) by doubling them. This was somewhat
of an issue in tests like exec-environment*.service where systemd was
doing the substitutions and we were not really checking that those were
available in the actual environment of the command. Fix that.

Expressions such as `exit $(test ...)` are incorrect. They only work
because $(test ...) will produce no output, so the command will become a
bare "exit" which will exit with the status of the latest executed
command which turns out to be the test... The direct approach is simply
calling "test" as the last command, for which the shell will propagate
the exit status.

One situation where this was breaking tests was on `exit $(test ...) &&
$(test ...) && $(test ...)` where the second and third tests were not
really executing, since the first command is actually `exit` so && was
doing nothing there. Fixed it by just using `test ... && test ... &&
test ...` as it was initially intended.

Pass -x to all shell executions for them to produce useful debugging
output to stderr. Consequently, removed most of the explicit `echo`s
that are no longer needed.

Mark all units as Type=oneshot explicitly.

Also made sure all shell variables are properly quoted.

v2: Added an explicit LC_ALL=C to ionice invocations since some locales
(such as French) will add a space before the colon in the output.

Tested by running `sudo ./test-execute` and confirming all tests enabled
on my system (essentially all of them except for the s390 one) passed.
Tweaked the variables or options or expected values and confirmed the
tests do indeed fail when the values are not exactly the expected ones.

v2: Also tested with `LANG=fr_FR.UTF-8 sudo ./test-execute` to confirm
it still works in a different locale.

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