tree-wide: indentation fixes

Merge pull request #2671 from 0xAX/move-pager-open-to-one-place

tree-wide: merge pager_open_if_enabled() to the pager_open()

tree-wide: merge pager_open_if_enabled() to the pager_open()

Many subsystems define own pager_open_if_enabled() function which
checks '--no-pager' command line argument and open pager depends
on its value. All implementations of pager_open_if_enabled() are
the same. Let's merger this function with pager_open() from the
shared/pager.c and remove pager_open_if_enabled() from all subsytems
to prevent code duplication.

Merge pull request #2743 from pfl/dhcp4_server

sd-dhcp-server: Send replies to BOOTP relay server port

sd-dhcp-server: Send replies to BOOTP relay server port

RFC 2131 Section 4.1 says that

 "If the ’giaddr’ field in a DHCP message from a client is non-zero,
  the server sends any return messages to the ’DHCP server’ port on the
  BOOTP relay agent whose address appears in ’giaddr’."

Fix this by adding a destination port when sending unicast UDP packets
and provide the server port when a BOOTP relay agent is being used.

Merge pull request #2722 from torstehu/fix-typo2

treewide: fix typos and then/that use

treewide: fix typos and then/that use

Merge pull request #2719 from evverx/add-test-to-makefile

tests: various fixes

Merge pull request #2726 from ssiloti/master

dnssec: only run tests that require gcrypt if we have it

Merge pull request #2727 from ian-kelling/man-pr-v3

man: clarify unit ordering language

man: clarify unit ordering language

dnssec: only run tests that require gcrypt if we have it

Merge pull request #2724 from vcaputo/minor-formatting-cleanups

tree-wide: minor formatting inconsistency cleanups

tree-wide: minor formatting inconsistency cleanups

Merge pull request #2639 from nafets227/master

Bugfix set virt=none when running on XEN Dom0

basic: Bugfix Detect XEN Dom0 as no virtualization

When running in XEN Dom0 the virtualization check:
1) detect_xen returns HYPERVISOR_NONE so next checks are executed
2) /proc/sys/hypervisor detects a XEN hypervisor
   it is lacking the special Dom0 detection as in detect_xen

With this patch, at the end of all virtualization checks we double-check if running in XEN Dom0 or DomU.

basic: Debug-logging of Virtualisation detection

print out every single detection executed and its result.

Merge pull request #2725 from phomes/misc-cleanup

Misc cleanup

networkctl: fix const

tree-wide: remove unused variables

basic: mark unused variable as such

Merge pull request #2664 from zonque/bootchart-removal

Remove systemd-bootchart

Merge pull request #2723 from poettering/importd-man

man: style fixes

man: change recommended order of NSS modules in /etc/nsswitch.conf

So far we recommended placing "nss-mymachines" after "nss-resolve" in the order
of preference in /etc/nsswitch.conf. This change reverse this order.

Rationale: single-label names are resolved via LLMNR by resolved, which has to
time out if no peer by that name exists. By placing "nss-mymachines" first
(which always responds immediately) we avoid running into this timeout for most
containers. Both modules should return the same data if LLMNR is used by the
container anyway.

While we are at it, improve the man pages of the three NSS modules in other
ways a bit.

tests: add exec-capabilityambientset* to Makefile.am

This is a follow-up to 70d7aea5c

tests: add exec-runtimedirectory* to Makefile.am

This is a follow-up to cc3ddc851

tests: fix x86 personality tests to only on x86

tests: add exec-spec-interpolation.service to Makefile.am

This is a follow-up to 2525499

Merge pull request #2708 from vcaputo/journal-restore-offline-state-on-error

journal: restore offline state on error

Merge pull request #2717 from keszybz/networkctl-prettification

Networkctl prettification

man: link some unit files to their online bus API documentation

man: add minimal man page for systemd-importd.service

networkctl: only print lldp legend for capabilities that were actually seen

basic: simplify ether_addr_is_null

Fix two typos

man: update link to Fedora Cloud images

Better to use the mirror manager and https.

architecture.h: remove PROC_CPUINFO_MODEL

This was only needed for bootchart, so it can go now.

Remove systemd-bootchart

This commit rips out systemd-bootchart. It will be given a new home, outside
of the systemd repository. The code itself isn't actually specific to
systemd and can be used without systemd even, so let's put it somewhere
else.

Merge pull request #2709 from vcaputo/normalize_inc_dec_usage

tree-wide: make ++/-- usage consistent WRT spacing

Merge pull request #2707 from teg/man-network

network: minor manpage clarifications

Merge pull request #2698 from poettering/personality

Personality fixes

tree-wide: make ++/-- usage consistent WRT spacing

Throughout the tree there's spurious use of spaces separating ++ and --
operators from their respective operands.  Make ++ and -- operator
consistent with the majority of existing uses; discard the spaces.

journal: restore offline state on error

If we fail to create the thread, technically we should leave the
offline_state as OFFLINE_JOINED, not OFFLINE_SYNCING.

Merge pull request #2706 from whot/hwdb-updates

Lenovo T460 and P50 touchpad axis corrections

man: link - add suggestion for debugging

udevadm test-builtin can be very useful for testing .link files, so add a reference.

Addresses issue #2406.

man: network - clarify BindCarrier documentation

Make it clear that we are talking of links (in the sense used in networkd) and not ports.

Addresses issue #964.

man: network - improve IPv6Token documentation

Enabling router advertisement may even trigger SLAAC or DHCPv6 to be used to configure
IPv6 addresses on the link. It may not be obvious that only in the SLAAC case will the
Token have an effect. Clarify this in the man page.

Addresses issue #863.

hwdb: reshuffle Lenovo entries to sort alphabetically by model

hwdb: add axis range corrections for the Lenovo T460 and P50

Merge pull request #2701 from keszybz/udev-rules

udev-rules cleanup fix

github: CONTRIBUTING.md moved, let's fix the link to it

tests: add personality tests

util-lib: simplify personality() string matching

virt: make sure we don't ignore some errors

man: extend the Personality= documentation

Among other fixes, add information about more architectures that are supported
these days.

util-lib: support various ppc archs in personality logic

util-lib: use the architecture ids from architecture.h for personalities

We have this ids, hence let's use them universally.

build-sys: move shared/architecture.[ch] into basic/

After all, it is pretty generic, has no external deps besides libc, and is very
similar to virt.[ch] which is also in basic/

Merge pull request #2704 from poettering/lldp-internal-fix

build-sys: add missing header file to Makefile.am

build-sys: add missing header file to Makefile.am

Fixes: #2700

udev-rules: fix querying of attributes

faccessat returns 0 on success.

A cosmetic fix is also included: the slash was doubled unnecessarily.

udev-test.pl: add --strace option

It's useful when trying to see what the tests are doing.
I hardcoded '-efile' as the option to strace, but in the future
it might be useful to make this configurable.

Merge pull request #2685 from poettering/lldp-fixes2

lldp fixes, second iteration

Merge pull request #2695 from mustrumr/fix-calendarspec-skip

calendarspec: fix find_next skipping times

Merge pull request #2692 from ssahani/tunnel

networkd: tunnel fix tunnel address

Merge pull request #2694 from zonque/proxyarp-missing

missing.h: Explicitly check for IFLA_BRPORT_PROXYARP

calendarspec: fix find_next skipping times

reset usec when bumping hours/minutes

missing.h: Explicitly check for IFLA_BRPORT_PROXYARP

RHEL explicitly disables IFLA_BRPORT_PROXYARP by renaming the enum value.

In order to support unpatched builds, we have two options:

a) redefine the enum value through missing.h and ignore the fact that it
   is really unsupported, or

b) omit that enum value on rtnl_prot_info_bridge_port_types[]

As we are not actually using this netlink type anywhere, and because it
is only hooked up for the sake of completeness, this patch opts for the
former.

Merge pull request #2687 from poettering/resolved-fix-2683

networkd: make sure we allocate the NTA set before we add items to it

Merge pull request #2686 from poettering/github-templates

add github issue template

Merge pull request #2688 from poettering/calendar-fix-2678

A fix for #2678

util-lib: fix returned error code

Make sure we propagate errors properly.

util-lib: add (void) cast to indicate that we don't care about the normalization success

After all, we verify that every calendar part is not out of bounds later on,
and it's fully OK if the normalization has no effect.

networkd: make sure we allocate the NTA set before we add items to it

See: #2683

Merge pull request #2650 from vcaputo/async_fsync

Perform journal offlines asynchronously when possible

Merge pull request #2681 from keszybz/udev-rules

udev-rules cleanup and coverity warning silencing

github: add a basic issue template

As documented here:

https://help.github.com/articles/creating-an-issue-template-for-your-repository/

github: move CONTRIBUTING.md into .github subdirectory

As suggested by:

https://github.com/blog/2111-issue-and-pull-request-templates

man: document the new EmitLLDP= .network setting

Also, beef up the LLDP documentation a bit.

update TODO

sd-lldp: beef up callback logic

Instead of just notifying about the fact that something changed in the
database, actually inform the callback what precisely changed. This is useful,
so that the LLDP tx logic can be put into "fast" mode as soon as a previously
unknown peer appears, as suggested by the LLDP spec.

networkctl: ellipsize long LLDP fields in table output

networkctl: add missing newline to printf() format string

network: turn on LLDP tx+rx for the default container network configuration

Containers are relatively trusted and the veth tunnels are small networks,
hence let's turn on LLDP both ways for them.

networkd: add basic LLDP transmission support

Let's add some minimalistic LLDP sender support. The idea is that this is
either on or off, and all fields determined automatically rather than
configured explicitly.

sd-lldp: filter out LLDP messages coming from our own MAC address

Let's not get confused should we be connected to some bridge that mirrors back
our packets.

import: don't claim we had copied a settings file if we didn't

man: update references to fedora cloud image

We are now at F23, hence let's use that as example.

networkd: make a couple of functions static

These functions are nowadays used only within networkd-link.c, hence ther's no
point in littering our public namespace with them.

networkd: rework when LLDP reception is enabled

Being on the link-layer LLDP is nothing we should turn on only when there's a
link beat. Instead, turn it on, whenever the iface is UP regardless if there's
a link beat or not. This closes the race between a link beat being available
and us subscribing to LLDP as a result.

networkctl: print a nice warning when networkd isn't running

networkctl: if there's no data from networkd about an iface show as "unmanaged"

After all, if we know that an interface exists but networkd did not store any
info about it, then it's definitely unmanaged by it.

(Note that we add this fix-up to networkctl, and not to sd-network, simply
because a missing file might also be result of the interface not existing.)

sd-network: use xsprintf() instead of asprintf() where we can

networkctl: add a couple of (void) casts

Where we knowingly ignore possible error results, let's cast to void.

networkd: rework how carrier bindings are serialized

Instead of serializing the interface name, expose the interface index, since
that's the only stable identifier.

networkd: turn on LLDP reception by default, in "routers-only" mode

This way "networkctl status" becomes a bit more useful by default, as router
information is just visible, without any further configuration.

LLDP reception is fully passive and relatively low simple and low traffic,
hence this should be safe to enable by default.

networkctl: extend "networkctl status" per-interface output to include LLDP info

This adds a small and useful field to the "systemctl status" output: the
router(s) the interface is connected to as reported via LLDP. Example output:

● 2: enp0s25
       Link File: /usr/lib/systemd/network/99-default.link
            Type: ether
           State: degraded (configured)
            Path: pci-0000:00:19.0
          Driver: e1000e
    Connected To: GS1900 on port 2 (foobar)

i.e. the last line is the relevant one.

networkctl: make use of xsprintf() where we can

networkctl: simplify networkctl

Move decode_and_sort_links() into acquire_info_all() which is the only place
this is used. The result is then nicely symmetric to acquire_info_strv().