nspawn: be verbose about interface names

Allowed interface name is relatively small. Lets not make
users go in to the source code to figure out what happened.

--machine=debian-tree conflicts with
--machine=debian-tree2

ex: Failed to add new veth \
         interfaces (host0, vb-debian-tree): File exists

udevd: event - fix event queue in daemenozied mode

We would enforce that events could only be added to the queue from the
main process, but that brake in daemonized mode. Relax the restriction
to only allow one process to add events to the queue.

Reported by Mantas Mikulėnas.

udevd: worker - modernize a bit

Initialize structs when declaring rather than using memzero().

udevd: worker - fully clean up unnecessary fds

These are only ever used in the parent process, so close them early in the worker.

bootctl: fix an error check

fix typos in systemd-nspawn man page

bootctl: ferror must be called before FILE is closed

Otherwise it will not show any error stored

udev/net_id: Only read the first 64 bytes of PCI config space

The original code used fread(), which on some libc implementions
(ie glibc 2.17) would pre-read a full 4K (PAGE_SIZE) of the
PCI config space, when only 64 bytes were requested.

I have recently come across PCIe hardware which responds with
Completion Timeouts when accesses above 256 bytes are attempted.

This can cause server systems with GHES/AEPI support to cause
and immediate kernel panic due to the failed PCI transaction.

This change replaces the buffered fread() with an explict
unbuffered read() of 64 bytes, which corrects this issue by
only reading the guaranteed first 64 bytes of PCIe config space.

sd-bus,sd-event: make public APIs

With the v221 release these APIs should be public, stable APIs, hence
let's install their headers by default now, and add their symbols to the
.sym file.

NEWS: fix date

Ah, bummer, it's tagged an now I realize I didn't fix the date. Let's do
so now, post-commit.

json: minor style fixes

test.json: fix build on x86-32 where int and intmax_t differ

update hardware databases

build-sys: fix build with seperate builddir

Let's make sure we can build rpms with this

manager: fix finish_timestamp calculation

http://lists.freedesktop.org/archives/systemd-devel/2015-May/032100.html

update TODO

bus-proxy: fix GetConnectionSELinuxSecurityContext() return value

nspawn: prohibit access to the kernel log buffer by default

Unless CAP_SYSLOG is explicitly passed block all access to kmg

device: never act on mount events for devices if device support is not available

core: properly handle jobs that are suppressed to JOB_NOPs when propagating restarts

NEWS: add note about gudev

gudev has been extracted into a separate repository managed by the gnome
project. See the announcement thread on systemd-devel for more:
  http://lists.freedesktop.org/archives/systemd-devel/2015-May/032070.html

NEWS: fix sd-boot to systemd-boot

We renamed sd-boot to systemd-boot directly after merging it, to stay
conform to all the other systemd binaries.

NEWS: fix typo and expand abbreviation

NEWS: document the IPForward= confusion

tmpfiles: create /etc/resolv.conf symlink only on boot

We will create the symlink on boot as a fallback to provide name
resolution. But if the symlink was removed afterwards, it most likely
should not be recreated. Creating it only on boot also solves the
issue where it would be created prematurely during installation,
before the system was actually booted.

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

build-sys: bump library and package versions

update TODO

util: an array with one entry is always ordered

util: introduce PERSONALITY_INVALID as macro for 0xffffffffLU

NEWS: start collecting items for v220

nspawn: finish user namespace support

update TODO

udev: link-config - fix corruption

The parser used for MTU and Speed expects them to be size_t, not unsigned int.

This caused a corruption in the rest of the structure.

Reported by David O Neill <david.m.oneill@intel.com>.

hwdb: add MS Arc Touch Mouse SE

https://bugs.freedesktop.org/show_bug.cgi?id=90540

sd-device: fix return codes on error

asprintf() does not set errno.

json: avoid cleanup of unitialized variable

units: conditionalize audit multicast socket on CAP_AUDIT_READ

The multicast logic can only work if the capability is available, hence
require it.

util: introduce reset_uid_gid() for resetting all uids and gids to 0

journald: handle more gracefully when bind() fails on audit sockets

core,nspawn: unify code that moves the root dir

TODO: add --merge support for sd-j-remote

bootctl: try to catch all errors

Do not print garbage on non-EFI installations.

bootctl: modernization

Use strjoina to avoid error handling, and openat to simplify things.

Some fixes on the way:
- ferror does not set errno, so the return value was wrong in some cases
- errors are propagated in more cases
- EFI/systemd was created, but EFI/systemd-boot was deleted
- something is always printed on error
- when checking the version, comparison was done against "systemd-bo" for some reason
- return value was converted from negative to EXIT_SUCCESS/EXIT_FAILURE twice,
  resulting in EXIT_SUCCESS all the time

bus-socket: simplify initalization of struct msghdr

hwdb: add Logitech M570 trackball

http://www.logitech.com/en-us/product/wireless-trackball-m570#specs
https://bugzilla.redhat.com/show_bug.cgi?id=1217881

json: fix a mem leak

update TODO

units: make sure systemd-nspawn@.slice instances are actually located in machine.slice

https://plus.google.com/112206451048767236518/posts/SYAueyXHeEX

core: don't consider boot-up finished if we are still reloading

http://lists.freedesktop.org/archives/systemd-devel/2015-May/032025.html

pull-dkr: fix memleak

import/pull-dkr: V2 image specification and manifest support

The maximum number of layers changed to 127, as in Dkr.

import/pull: Tag replaced with reference

Default pull version set to V2

test/test-json: Tests for the tokenizer bugfix and the DOM parser

The DOM parser tests are accompanied with structure and element analysis

shared/json: Added DOM-like JSON parser

This makes working with complexly structured documents easy
and more reliable as the parser is not susceptible to
element re-ordering.

Also fixes a bug when the tokenizer would choke after reading
a number.

shared/import-util: Tag renamed to reference

Added (sha256) digest validation function

core: when propagating restart requests due to deps, downgrade restart to try-restart

Previously, if a service A depended on a service B via Requires=, and A
was not running and B restarted this would trigger a start of A as well,
since the restart was propagated as restart independently of the state
of A.

This patch ensures that a restart of B would be propagated as a
try-restart to A, thus not changing its state if it isn't up.

http://lists.freedesktop.org/archives/systemd-devel/2015-May/032061.html

dhcp-identifier: fix for unaligned write

Reported by Michael Olbrich.

core: sd_bus_error() already checks for NULL, no need to duplicate check

core: reinstate propagation of stop/restart jobs via RequsiteOf dependencies

This reverts the primary effect of be7d9ff730cb88d7c6a869dd5c47754c78ceaef2.

After all Requisite= should be close to Requires=, without the one
exception that it doesn't pull in dependencies on start. However,
reverse deps on stop/restart should be treated the same way as for
Restart=, and this is already documented in the man page, hence stick to
it.

http://lists.freedesktop.org/archives/systemd-devel/2015-May/032049.html

util: fix typo

mount: properly check for mounts currently in /proc/self/mountinfo

http://lists.freedesktop.org/archives/systemd-devel/2015-May/032059.html

Update TODO

unit: drop support for pre-v44 job serialization

No distro ships that old systemd versions anyway, hence let's drop
support for live-upgrades for them. Offline updates are still supported.
And live-upgrades will only lose the job queue, hence basically still
work...

core: also enforce ratelimiter if we stop a unit due to BindsTo=

This extends on bea355dac94e82697aa98e25d80ee4248263bf92, and extends
the ratelimiter to not only be used for StopWhenUnneeded=1 units but
also for units that have BindsTo= on a unit that is dead.

http://lists.freedesktop.org/archives/systemd-devel/2015-April/030224.html

core: use bitfield where possible

update TODO

core: enforce a ratelimiter when stopping units due to StopWhenUnneeded=1

Otherwise we might end up in an endless stop loop.

http://lists.freedesktop.org/archives/systemd-devel/2015-April/030224.html

unit: fix unit_check_unneeded() dependency iteration

Fixes a regression introduced in be7d9ff730cb88d7c6a869dd5c47754c78ceaef2.

mount: don't claim a device is gone from /proc/self/mountinfo before it is gone from *all* lines

Devices might be referenced by multiple mount points in
/proc/self/mountinfo, hence we should consider them unmounted only after
they disappeared from all lines, not just from one.

http://lists.freedesktop.org/archives/systemd-devel/2015-May/032026.html

device: create units with intended "found" value

Change device_found_node() to also create a .device unit if a device is not
known by udev; this is the case for "tentative" devices picked up by mountinfo
(DEVICE_FOUND_MOUNT).  With that we can record the "found" attribute on the
unit.

Change device_setup_unit() to also accept a NULL udev_device, and don't
add the extra udev information in that case.

Previously device_found_node() would not create a .device unit, and
unit_add_node_link() would then create a "dead" stub one via
manager_load_unit(), so we lost the "found" attribute and unmounted everything
from that device.

https://launchpad.net/bugs/1444402
http://lists.freedesktop.org/archives/systemd-devel/2015-May/031658.html

hostname: Allow comments in /etc/hostname

The hostname(1) tool allows comments in /etc/hostname. Introduce a new
read_hostname_config() in hostname-util which reads a hostname configuration
file like /etc/hostname, strips out comments, whitespace, and cleans the
hostname. Use it in hostname-setup.c and hostnamed and remove duplicated code.

Update hostname manpage. Add tests.

https://launchpad.net/bugs/1053048

core: fix OOM checks in dbus-unit

CID#1299014

test-hashmap: fix an assert

CID#1299016

man: explain UseDomains a bit more

https://bugs.freedesktop.org/show_bug.cgi?id=85397

update TODO

update TODO

core: introduce seperate reverse dependencies for Requires= and Requisite=

This allows us to ensure that Requisite= dependencies never cause
propagation between units, while Requires= dependencies might.

http://lists.freedesktop.org/archives/systemd-devel/2015-May/031742.html

journalctl: unify how we free boot id lists a bit

Instead of use LIST_FOREACH_SAFE, just use the same, seperate destructor
everywhere.

journalctl: clean up how we log errors

All functions should either log the errors they run into, or only return
them in which case the caller should log them.

Make sure this rule is followed, so that each error is logged precisely
once, and neither never, nor more than once.

journalctl: rename boot_id_t to BootId

So far we tried to reserve the _t suffix to types we use like a value in
contrast to types we use as objects, hence let's do this in journalctl
too.

journalctl: lstat() should suffice if we call canonicalize_file_name() first

journalctl: free all command line argument objects

let's try to be valgrind clean

journalctl: only have a single exit path from main()

That way we can be sure we execute the destructors properly, and can be
valgrind-clean.

journalctl: Improve boot ID lookup

This method should greatly improve offset based lookup, by simply jumping
from one boot to the next boot. It starts at the journal head to get the
a boot ID, makes a _BOOT_ID match and then comes from the opposite
journal direction (tail) to get to the end that boot. After flushing the matches
and advancing the journal from that exact position, we arrive at the start
of next boot. Rinse and repeat.

This is faster than the old method of aggregating the full boot listing just
so we can jump to a specific boot, which can be a real pain on big journals
just for a mere "-b -1" case.

As an additional benefit --list-boots should improve slightly too, because
it does less seeking.

Note that there can be a change in boot order with this lookup method
because it will use the order of boots in the journal, not the realtime stamp
stored in them. That's arguably better, though.
Another deficiency is that it will get confused with boots interleaving in the
journal, therefore, it will refuse operation in --merge, --file and --directory mode.

https://bugs.freedesktop.org/show_bug.cgi?id=72601

udevd: main - use _exit() when daemonizing

Makes it a bit clearer what is going on, rather than jumping to the end of main().

No functional change.

udevd: main - reshuffle

First parse config, then sanitize environment before donig any further setup.

No functional change.

udevd: rename on_reque_{exit,reload} to on_sig{term,chld}

No functional change.

udevd: introduce on_event_timeout{,_warning}

This uses kill_and_sigcont() instead of kill(), otherwise no functional change.

udevd: process all SIGCHLD events every time the handler is invoked

We were returning rather than continuing in some cases. The intention
was always to fully process all pending events before returning
from the SIGCHLD handler. Restore this behaviour.

udevd: init/exit the builtins in manager_new/free

No functional change.

udevd: move file descriptors to Manager

No functional change.

resolved: allow DnsAnswer objects with no space for RRs

They might be created as result of merged answer sets, hence accept
them.

http://lists.freedesktop.org/archives/systemd-devel/2015-April/030834.html

resolved: fix crash when shutting down

Reported by Cristian Rodríguez

http://lists.freedesktop.org/archives/systemd-devel/2015-May/031626.html

nspawn: close extra fds before execing init

When systemd-nspawn gets exec*()ed, it inherits the followings file
descriptors:
- 0, 1, 2: stdin, stdout, stderr
- SD_LISTEN_FDS_START, ... SD_LISTEN_FDS_START+LISTEN_FDS: file
  descriptors passed by the system manager (useful for socket
  activation). They are passed to the child process (process leader).
- extra lock fd: rkt passes a locked directory as an extra fd, so the
  directory remains locked as long as the container is alive.

systemd-nspawn used to close all open fds except 0, 1, 2 and the
SD_LISTEN_FDS_START..SD_LISTEN_FDS_START+LISTEN_FDS. This patch delays
the close just before the exec so the nspawn process (parent) keeps the
extra fds open.

This patch supersedes the previous attempt ("cloexec extraneous fds"):
http://lists.freedesktop.org/archives/systemd-devel/2015-May/031608.html

ata_id: drop spurious space

CODING_STYLE: document that we should avoid kernel types like u32

core: Private*/Protect* options with RootDirectory

When a service is chrooted with the option RootDirectory=/opt/..., then
the options PrivateDevices, PrivateTmp, ProtectHome, ProtectSystem must
mount the directories under $RootDirectory/{dev,tmp,home,usr,boot}.

The test-ns tool can test setup_namespace() with and without chroot:
 $ sudo TEST_NS_PROJECTS=/home/lennart/projects ./test-ns
 $ sudo TEST_NS_CHROOT=/home/alban/debian-tree TEST_NS_PROJECTS=/home/alban/debian-tree/home/alban/Documents ./test-ns

kmod-setup: conditionalize kmod autoloading properly

Load kdbus.ko only if we are built with kdbus, and load ip_tables.ko
only if we are built with iptables support.

util: split all hostname related calls into hostname-util.c

README: document that we prefer if CONFIG_CHECKPOINT_RESTORE is turned on, for the kcmp() syscall