test-network: check link before starting dnsmasq

test-network: show command outputs

test-network: sort imported modules

test-network: sort links and units

test-network: fix typo in class name

Merge pull request #10963 from poettering/bus-force-state-change-signal

force PropertiesChanged bus signal on all unit state changes

fd-util: Fix error handling in safe_fclose

Function fclose_nointr returns negative value on error.

Merge pull request #10994 from poettering/sd-bus-tweaks

small sd-bus tweaks

Merge pull request #10988 from ssahani/IFLA_BRPORT_MCAST_TO_UCAST

networkd: bridge add support to configure multicast_to_unicast

Merge pull request #11032 from ssahani/invert-rule

networkd: RPDB rule - add support to configure inverted rule.

test-network: add test for bridge MulticastToUnicast

test-network: Add test for invert RPDB rule

networkd: RPDB rule - add support to configure inverted rule.

Closes #10706

Merge pull request #11021 from ssahani/isatap

networkd: Add support to configure ISATAP tunnel

Merge pull request #11053 from poettering/rc-local-tweak

rc-local-generator message tweak

rc-local-generator: tweak log message generation

Let's be more explicit, and not use LOG_WARNING log level when there's
nothing really wrong.

Fixes: #11044

Follow-up for: #10955

rc-local-generator: small shortification

README: embed a CentOS CI badge into README

Plus be more specific about where the badges point to.

systemctl: drop my copyright

Signed-off-by: Marc-Antoine Perennou <Marc-Antoine@Perennou.com>

Merge pull request #11025 from evverx/clang-asan

travis: also run the tests built with clang under ASan+UBsan

Merge pull request #11051 from poettering/sock-addr-un-len

trick out ubsan

socket-util: Let's trick out ubsan when it validate struct sockaddr_un.sun_path[]

Linux is stupid and sometimes returns a "struct sockaddr_un" that is
longer than its fields, as it NUL terminates .sun_path[] even if it has
full length. ubsan detects this, rightfully. Since this is a Linux
misdesign let's trick out ubsan a bit.

Fixes: #11024

socket-util: break lines in sockaddr_pretty() signature

It's soo long, let's make this more readable.

socket-util: shorten code a bit

man/daemon.xml: use AM_DISTCHECK_CONFIGURE_FLAGS

Change the man page to recommend setting AM_DISTCHECK_CONFIGURE_FLAGS
instead of DISTCHECK_CONFIGURE_FLAGS. This is the recommended way from
the automake manual, as otherwise the flag can't be overridden by the
user with something like:
  DISTCHECK_CONFIGURE_FLAGS=--some-additional-flags make distcheck

References:
https://www.gnu.org/software/automake/manual/html_node/Checking-the-Distribution.html
https://osmocom.org/issues/3718

rules: watch metadata changes on DASD devices

To make sure the change event is emitted and udev db is updated
after metadata changes.

Merge pull request #11041 from yuwata/update-missing-v2

missing: separate missing.h more

Merge pull request #11042 from yuwata/tiny-coding-style-fixes

Tiny coding style fixes

cgroup: Use varname for cgroup_disable documentation

The current use of literal + replaceable is pretty ugly as it usually
ends up with cgroup_disable= rendered in quotes, which looks really
weird, and this doesn't conform with others of a similar type (for
example, the earlier `DefaultDependencies=no` discussion in the same
file.

Merge pull request #11037 from poettering/json-table

optionally output tables as JSON

Merge pull request #11009 from poettering/root-cgroup-again

tweak root cgroup attribute fiddling for cgroupsv1 again

util: drop missing.h from util.h

tree-wide: add whitespace between type and variable name

l10n: update Czech Translation

ip-protocol-list: sort headers

missing: move char{16,32}_t definitions to missing_type.h

missing: move capabilities to missing_capability.h

missing: move prctl related entries to missing_prctl.h

missing: move statx related definitions to missing_stat.h

missing: move entries defined in magic.h to missing_magic.h

missing: move input related entries to missing_input.h

missing: rename securebits.h to missing_securebits.h

missing: move btrfs related entries to missing_btrfs.h and missing_btrfs_tree.h

missing: move missing entries in keyctl.h to missing_keyctl.h

meson: add missing missing_*.h

analyze: Fix if not HAVE_SECCOMP

The new systemd analyze code dependend on routines provided by
libseccomp even if HAVE_SECCOMP is false. This expands the code disabled
in analyze-security to make sure we don't depend on seccomp.

Merge pull request #11039 from yuwata/fix-11036

missing: re-add mistakenly dropped entries

travis: install llvm to get llvm-symbolizer

See https://clang.llvm.org/docs/AddressSanitizer.html#symbolizing-the-reports

missing: rearrange missing_if_link.h to support centos

missing: re-add mistakenly dropped entries

These entries are mistakenly dropped by 9714c020fc4cda1823c2a77e3fd08aefa7d78b25.

Fixes #11036.

travis: also run the tests built with clang under ASan+UBsan

format-table: never try to reuse cells that have color/url/uppercase set

The table cell reusing code is supposed to be an internal memory
optimization, and not more. This means behaviour should be the same as
if we wouldn't reuse cells.

tree-wide: specify all table headers in lower-case

format-table: add option to uppercase cells on display

This adds a per-cell option for uppercasing displayed strings.
Implicitly turn this on for the header row. The fact that we format the
table header in uppercase is a formatting thing after all, hence should
be applied by the formatter, i.e. the table display code.

Moreover, this provides us with the benefit that we can more nicely
reuse the specified table headers as JSON field names, like we already
do: json field names are usually not uppercase.

machinectl: optionally output table in JSON

loginctl: optionally output table in json

This works now:

  $ loginctl list-session --output=json-pretty

format-table: optionally covert/output table to JSON

format-table: use right field for display

Since .timespan and .timestamp are unionized on top of each other this
doesn't actually matter, but it is still more correct to address it
under it's correct name.

output-mode: add new helper OUTPUT_MODE_IS_JSON()

Just a pretty way to determine whether the selected output mode is a
JSON output mode.

output-mode: add generic helper to pick right JsonFormatFlags for given OutputMode

json: add new display flag JSON_FORMAT_COLOR_AUTO

Typically we want to enable color when colors_enabled() says so, hence
let's automatize this.

json: introduce explicit type for json formatting flags

Merge pull request #11031 from poettering/gcc-attr-cleanup

various gcc attribute clean-ups

analyze: tweak wording of description for ProtectSystem= (#11035)

networkd: bridge add support to configure multicast_to_unicast

closes #10649

Merge pull request #10567 from cdown/disable_controller

cgroup: Add DisableControllers= directive to disable controller in subtree

Merge pull request #10920 from yuwata/hashmap-destructor

hashmap: make hashmap_free() call destructors of key or value

Merge pull request #10976 from yuwata/typesafe-netlink-call

netlink: introduce typesafe netlink functions

tree-wide: use new macro HAS_FEATURE_ADDRESS_SANITIZER everywhere

sd-bus: use _variable_no_sanitize_address_ also for BUS_ERROR_MAP_ELF_REGISTER

static-destruct: turn on new _variable_no_sanitize_address_ variable for STATIC_DESTRUCTOR_REGISTER

macro: add macro for llvm no_sanitize_address attribute

We want it for global variables, which LLVM supports and GCC currently
does not (GCC does support it for functions, but we care about global
variables here).

Why is this relevant? When asan is used global variables are padded with
hotzones before and after. But we can't have that for the registration
variables we place in special ELF sections: we want them tightly packed
so that we can iterate through them.

Note that for gcc this isn't an issue, as it will pack stuff in
non-standard sections anyway, even if asan is used.

macro: define HAS_FEATURE_ADDRESS_SANITIZER also on gcc

Let's make differences between compilers more minimal.

util-lib: introduce HAS_FEATURE_ADDRESS_SANITIZER

https://clang.llvm.org/docs/AddressSanitizer.html#conditional-compilation-with-has-feature-address-sanitizer

test: add test for static destructor

cgroup: Add DisableControllers= directive to disable controller in subtree

Some controllers (like the CPU controller) have a performance cost that
is non-trivial on certain workloads. While this can be mitigated and
improved to an extent, there will for some controllers always be some
overheads associated with the benefits gained from the controller.
Inside Facebook, the fix applied has been to disable the CPU controller
forcibly with `cgroup_disable=cpu` on the kernel command line.

This presents a problem: to disable or reenable the controller, a reboot
is required, but this is quite cumbersome and slow to do for many
thousands of machines, especially machines where disabling/enabling a
stateful service on a machine is a matter of several minutes.

Currently systemd provides some configuration knobs for these in the
form of `[Default]CPUAccounting`, `[Default]MemoryAccounting`, and the
like. The limitation of these is that Default*Accounting is overrideable
by individual services, of which any one could decide to reenable a
controller within the hierarchy at any point just by using a controller
feature implicitly (eg. `CPUWeight`), even if the use of that CPU
feature could just be opportunistic. Since many services are provided by
the distribution, or by upstream teams at a particular organisation,
it's not a sustainable solution to simply try to find and remove
offending directives from these units.

This commit presents a more direct solution -- a DisableControllers=
directive that forcibly disallows a controller from being enabled within
a subtree.

cgroup: Traverse leaves to realised cgroup to release controllers

This adds a depth-first version of unit_realize_cgroup_now which can
only do depth-first disabling of controllers, in preparation for the
DisableController= directive.

cgroup: Rework unit_realize_cgroup_now to explicitly be breadth-first

systemd currently doesn't really expend much effort in disabling
controllers. unit_realize_cgroup_now *may* be able to disable a
controller in the basic case when using cgroup v2, but generally won't
manage as downstream dependents may still use it.

This code doesn't add any logic to fix that, but it starts the process
of moving to have a breadth-first version of unit_realize_cgroup_now for
enabling, and a depth-first version of unit_realize_cgroup_now for
disabling.

cgroup: Move attribute application into unit_create_cgroup

We always end up doing these together, so just colocate them and require
manager state for unit_create_cgroup.

sd-bus,static-destruct: clean up how we do our ELF section magic

This cleans up a bit how we set up things for the ELF section magic:

1. Let's always use our gcc macros, instead of __attribute__ directly

2. Align our structures to sizeof(void*), i.e. the pointer size, rather
   than a fixed 8 or __BIGGEST_ALIGNMENT__. The former is unnecessarily
   high for 32bit systems, the latter too high for 64bit systems. gcc
   seems to use ptr alignment for static variables itself, hence this
   should be good enough for us too. Moreover, the Linux kernel also
   uses pointer alginment for all its ELF section registration magic,
   hence this should be good enough for us too.

3. Let's always prefix the sections we create ourself with SYSTEMD_,
   just to make clear where they come from.

4. Always align the pointer we start from when iterating through these
   lists. This should be unnecessary, but makes things nicely
   systematic, as we'll align all pointers we use to access these
   sections properly.

json: use new _align_() macro

tree-wide: drop redundant space between __attribute__ and ((

We follow no general rule, but in most cases we do not place a space
outside of macro.h. Hence let's stick to that, and adapt macro.h too,
and follow the rule systematically that there shall not be a space
between __attribute__ and ((...

Yes, this does not matter at all, and is purely OCD cosmetics. But then
again, the uses of __attribute__ are very local only, hence the changes
cleaning this up are small and are unlikely to have to be repeated too
often...

macro: add macros for a couple of more gcc attributes we use

tree-wide: use gcc attribute macros where appropriate

We have these macros already, hence use them.

network: drop unnecessary buffers

Merge pull request #10992 from yuwata/follow-up-10948

network: make fib rule accept arbitrary ip protocol

missing: split network related entries

Also adds comments which kernel version added the entries.

Closes #10553.

sd-boot: remove Stall() call (#11029)

This is left-over from debugging.

test-network: add test for ISATAP

networkd: Add support to configure ISATAP tunnel

Let's just reuse the code of sit tunnel to create a ISATAP tunnel.
Matter of turning a flag

Please see https://elixir.bootlin.com/linux/v4.19.6/source/net/ipv6/sit.c#L208

vxlan: minor coding style fixes.

Merge pull request #11011 from poettering/tmpfile-util

split up fileio.c a bit

resolved: sort headers again

util-lib: split out env file parsing code into env-file.c

It's quite complex, let's split this out.

No code changes, just some file rearranging.

util-lib: move open_serialization_fd() to serialize.c

It definitely fits better there.

No code changes, just some rearranging.

resolved: remove duplicate #include

test: remove duplicate #include

util-lib: split out all temporary file related calls into tmpfiles-util.c

This splits out a bunch of functions from fileio.c that have to do with
temporary files. Simply to make the header files a bit shorter, and to
group things more nicely.

No code changes, just some rearranging of source files.

util-lib: don't include fileio.h from fileio-label.h

There's no reason for doing that, hence simply don't.

test: remove unnecessary include