Move module-util.h to src/shared/ and load_module() to libshared

Unfortunately this needs libshared to link to libkmod. Before it was linked
into systemd-udevd, udevadm, and systemd each seperately. On most systems this
doesn't make much difference, because at least systemd would be installed, but
it might not be in small chroots. It is a small library, so I hope this is not
a big issue.

core/kmod-setup: restore comments

They were removed in 7491e6e7c5fcb3c445a0656a440bd1551adb6ba1, but the
original version is more informative. Let's add them back.

man: fix typo

Merge pull request #9490 from keszybz/document-sd_message_read

Document sd_bus_message_read

Merge pull request #9489 from keszybz/copyright-removal-3

Copyright header removal, part 3 (or 4?)

rpm: emit warning when macro with typo is used

Follow-up for a6bb550458. Suggested by @ignatenkobrain.

$ rpmbuild --eval %_environmentdir
/usr/lib/environment.d
$ rpmbuild --eval %_environmnentdir
warning: Use %_environmentdir instead
/usr/lib/environment.d

Merge pull request #9417 from Conan-Kudo/rpm-environmentdir-typo

rpm: Fix typo in %_environmentdir

conf-parser,login: logs description of error in parsing size

man: add missing <option> tag

One mention of --user was missing the surrounding <option> tag, making
the table look inconsistent (though the inconsistency is less obvious
after the reordering in 709f4c472c, since the four specifiers mentioning
the --user option are no longer immediately adjacent).

test: fix networkd-test.py rate limiting and dynamic user

 - Reset systemd-networkd.service before each test run, to avoid running
   into restart limits.
 - Our networkd-test-router.service unit needs to run as root and thus
   can't use `User=`; but networkd still insists on the
   `systemd-network` system user to exist, so create it.

man: document sd_bus_read(), add more about sd_bus_read_basic()

seccomp: add swapcontext into @process for ppc32

There are some modern programming languages use userspace context switches
to implement coroutine features. PowerPC (32-bit) needs syscall "swapcontext" to get
contexts or switch between contexts, which is special.

Adding this rule should fix #9485.

tree-wide: add a space after (void)

resolved: fix typo in log message

Follow-up for cc7d50a5714bc810af51b0c55be12b4f55acc089.

man: use the same markup for "ap" everywhere

tree-wide: drop some leftover header for Lennart Poettering

Follow-up for 0c697941389b7379c4471bc0a067ede02814bc57.

journald: drop copyright headers for Sebastian Thorarensen

For #9320.

man: drop mode line in file headers

This is already included in .dir-locals, so we don't need it
in the files themselves.

Add IPVLAN= to systemd.network man page (#9480)

Required and seems to work in 238.133

tests: up the image size a little, use twice that size with STRIP_BINARIES=no

to make sure that there's enough room for unstripped binaries.

sd-bus: publicize sd_bus_message_readv()

The D-Bus library supplies a va_list variant of
`sd_bus_message_append()` called `sd_bus_message_appendv()`,
but failed to provide a va_list variant of its opposite,
`sd_bus_message_read()`. This commit publicizes a previously static
function as `sd_bus_message_readv()`.

tree-wide: use  instead of #ifdef for HAVE_*

resolve: update log message

Follow-up for f43580f17d9977ea330deacc8931982e41a49abf.

Merge pull request #9468 from yuwata/small-cleanups

small fixes and cleanups for comments

Use #if instead of #ifdef for ENABLE_GSHADOW

ENABLE_GSHADOW is defined to be 0 or 1. So #if should be used instead of #ifdef.

Add accel mount matrix for BayTrail-based tablets by DEXP

This commit adds accel mount matrix for DEXP Ursus 10XW and Z110i tablets. In current state (*svn*DEXP*:*pn*DEXPOEM*) it will apply to many (if not almost all) BayTrail-based tablets with Windows by DEXP (with same DMI strings) but this expected to cause no (or very little) harm as this devices is mostly very similar with slight difference.

However, if correction of this quirk becomes necessary in the future, this DMI strings could be used to distinguish 10XW and Z110i from other tablets by DEXP.

DEXP Ursus 10XW: dmi:bvnTechvisionHASHVersion96AC8868:bvr5.6.5:bd12/26/2014:svnDEXP:pnDEXPOEM:pvr1.0:rvnDEXP:rnDEXPOEM:rvr1.0:cvnToBeFilledByO.E.M.:ct3:cvrToBeFilledByO.E.M.:
DEXP Ursus Z110i Apollo: dmi:bvnTechvisionHASHVersion1DE28747:bvr5.6.5:bd03/14/2015:svnDEXP:pnDEXPOEM:pvr1.0:rvnDEXP:rnDEXPOEM:rvr1.0:cvnToBeFilledByO.E.M.:ct3:cvrToBeFilledByO.E.M.:

man: add missing parentheses

bus-util: update comment

Originally, request_name_handler_may_reload_dbus() was written for
timesyncd. But now it is also called by networkd or resolved.

tree-wide: drop double newline

tree-wide: drop empty comments

tree-wide: drop empty lines in the begining of comments

network: update log message

Closes #9396.

bus-util: drop unnecessary !!

Merge pull request #9422 from yuwata/conf-files

cleanups for default conf files

tree-wide: use 'signed int' instead of 'int' for bit field variables

Suggested by LGTM: https://lgtm.com/rules/1506024027114/

Merge pull request #9452 from evverx/coverity.sh

 coverity.sh: use a comma to separate module names

resolve: warn when our stub listener is disabled but resolv.conf uses it

Closes #9450.

meson: do not compare objects of different types

This fixes the following warning:
```
meson.build:1140: WARNING: Trying to compare values of different types (DependencyHolder, list) using !=.
The result of this is undefined and will become a hard error in a future Meson release.
```

Follow-up for f02582f69fe1e7663a87ba80bd4f90d5d23ee75f(#9410).

sd-bus: stop using the result of an assignment as an operand of &&

This makes OBJECT_PATH_FOREACH_PREFIX consistent with PATH_FOREACH_PREFIX
and also fixes 7 alerts reported by LGTM at
https://lgtm.com/projects/g/systemd/systemd/snapshot/ac0a08700344a5690803df8dd80e8bb5013184a5/files/src/libsystemd/sd-bus/bus-objects.c?sort=name&dir=ASC&mode=heatmap&showExcluded=true#V1383

core: tiny comment fix

summary: update nspawn description string a bit

nspawn as it is now is a generally useful tool, hence let's drop the
comments about it being useful for debug and so on only.

The new wording just makes the first sentence of the main page also the
summary.

sleep: fix one more printf format of a fiemap field

Use PRIu64 constant to get the format right on LP-64 architectures,
cast to (uint64_t) to solve incompatibility of __u64.

This was missed in ad4bc3352285f467f4ffa03c3171b19fa0a8758d, so fix it
with this follow up.

basic: add missing comma in raw_clone assembly for sparc

Fixes: 96f64eb5741b157f26a9089816fdd992e959792e
Closes: https://github.com/systemd/systemd/issues/9444

cryptsetup: Add dependency on loopback setup to generated units

For loopback volumes, the generated unit needs to depend on
systemd-tmpfiles-setup-dev.service to ensure that loopback
support is loaded.

Fixes #9308

bus-util: make log level lower in request_name_destroy_callback()

Fixes #9442.

Merge pull request #9440 from yuwata/fix-js

journal-gateway: update browse.html

Merge pull request #9438 from yuwata/lgtm-fixes-2

small fixes suggested by LGTM

resolve: reduce number of conversions between ifname and ifindex

This also fixes minor memleak introduced in
654457e560c5723b90b419f7651b87040aade07e.

resolvconf: fixes for the compatibility interface

Also use compat_main() when called as `resolvconf`, since the interface
is closer to that of `systemd-resolve`.

Use a heap allocated string to set arg_ifname, since a stack allocated
one would be lost after the function returns. (This last one broke the
case where an interface name was suffixed with a dot, such as in
`resolvconf -a tap0.dhcp`.)

Tested:
  $ build/resolvconf -a nonexistent.abc </etc/resolv.conf
  Unknown interface 'nonexistent': No such device

Fixes #9423.

analyze: actually select longest activated-time of services

Discovered by LGTM.

journal-gateway: use localStorage["cursor"] only when it has valid value

Discovered by LGTM.

journal-gateway: explicitly declare local variables

Suggested by LGTM.

sd-bus: fix implicit downcast of bitfield reported by LGTM

sleep: fix printf format of fiemap fields

Use PRIu64 and PRIu32 constants to also get the format right on LP-64
architectures.

For the 64-bit fields, we need a cast to (uint64_t), since __u64 is
defined as a `long long unsigned` and PRIu64 expects a `long unsigned`.
In practice, both are the same, so the cast should be OK.

README.md: add an LGTM badge

Now that @yuwata has fixed a lot of alerts, the badge looks
much better than before, so let's show it :-)

rpm: Add misspelled %_environmentdir macro for temporary compatibility

This should be removed after systemd 240 is released.

Signed-off-by: Neal Gompa <ngompa13@gmail.com>

tree-wide: use localtime_r() instead of localtime()

Follow-up for e46acb7950a9f07ac60d772309de842c444ad2bd.

netdev: do not assign unused values

crypt-util: Translate libcryptsetup log level instead of using log_debug()

This makes sure that errors reported by libcryptsetup are shown to the
user instead of getting swallowed up by log_debug().

coverity.sh: fail if uploading data to Coverity Scan is forbidden

coverity.sh: use a comma to separate module names

Otherwise Python bails out with
```
Traceback (most recent call last):
  File "<string>", line 1, in <module>
NameError: name 'json' is not defined
```

See https://travis-ci.org/systemd/systemd/jobs/397490437

coverity.sh: make the script compatible with Python 3

coverity.sh: fix a couple indents

man: add missing option for system.conf

core: add missing option and drop nonexistent option in system.conf

journal: add missing option in journald.conf

journal-remote: add license and reference to man page in conf files

network: provide default networkd.conf

timedate: use gmtime_r() and localtime_r()

gmtime() and localtime() operate on a static buffer. let's avoid this,
as we never know whether some library might use these calls in some
backrgound thread.

Discovered by lgtm:

https://lgtm.com/projects/g/systemd/systemd/

Merge pull request #9411 from poettering/systemctl-is-running-fixes

tiny systemctl fixes

rpm: Fix typo in %_environmentdir

Signed-off-by: Neal Gompa <ngompa13@gmail.com>

man: add comments about MTUBytes= and MACAddress= for tuntap devices

Closes #9407.

Merge pull request #4522 from lucaswerkmeister/unescape-template

escape: support --unescape with --template

busctl: make parameter const

it's initialized from an argv[] entry, hence let's be make it
non-writable by default.

meson: check whether gnutls supports TCP fast open

Fixes #9403

systemctl: if we can't detect the system-is-running state, assume it's not running

systemctl: log errors to LOG_DEBUG rather than eating them up entirely

systemctl: add 'static' to constant state array

bls: allow firmware-supported file systems

This copies the text from Matthew Garrett's version verbatim. At
least MacOS uses HFS+ for their ESP.

[hwdb] Add support for HP ProBook 650 wifi switch key (#9260)

Add support for HP ProBook 650 wifi switch key

resolved: assert t->server is set in dns_transaction_emit_tcp.

Uncovered by Coverity. Fixes CID 1393390.

core/job: add check for return of job_type_merge_and_collapse()

Using an assertion is fine, since calls to job_merge_into_installed()
are protected by a check for job_type_is_conflicting().

Uncovered by Coverity, fixes CID 996307.

hwdb: added accelerometer quirk for Aspire Switch 10 (sw5-011)

Merge pull request #9398 from yuwata/lgtm-fixes

fixes error handlings and several cleanups suggested by LGTM

core/manager: Make sure jobs statistics are not double after daemon-reload

We add n_installed_jobs and n_failed_jobs to our inner state after
deserialization. This is fine during daemon-reexec when we start with clear
Manager (and some jobs possibly queued before deserialization), however,
daemon-reload works with the same manager and adding the values would
effectively double the counters. Reset the counters before we deserialize and
add their values again.

man: SystemMaxUse= clarification in journald.conf(5)

Similar semantics applies to {System,Runtime}MaxUse= as to
{System,Runtime}MaxFiles=, i.e. active files aren't vacuumed and the limit
is not thus strictly honored.

escape: add --instance option

Suggested by @keszybz in #4522.

boot: do not assume the last character in cmdline is a newline

Fixes #9401.

escape: support --unescape with --template

tmpfiles: specify access mode for /run/systemd/netif

This partially reverts 2af767729489f6baa98a2641b2007acab44ed353.
As the directories are certainly readable and not-writable by
non-privileged users.

network: fix log message

timesync: update log message

Fixes #9388.

timesync: changes type of drift_freq to int64_t

drift_freq is used for storing timex.freq, and is a 64bit integer.
To support x32 ABI, this changes the type of drift_freq to int64_t.

Fixes #9387.

hwdb: Fix secondary backslash on MSI GS65 laptop

The MSI GS65 Stealth Thin series laptop contains an extra physical backslash key next to the spacebar which does not yield the correct keycode.

This pull request fixes the issue

Add resolution of Trust Flex Graphics Tablet to hwdb

Product:
https://www.trust.com/en/product/21259-flex-design-tablet-black
http://trust.com/21259

Support in libinput is already in place since libinput 1.11 release (I tested it)
https://gitlab.freedesktop.org/libinput/libinput/issues/38

So only missing part is definition in hwdb.

docs: put LGTM on the list of QA tools and fix a couple typos

install: fix error handling in is_symlink_with_known_name()

portable: fix error handling

import: drop unnecessary condition

sd-dhcp: drop unnecessary condition