udev: build by-path identifiers for ATA devices.

/dev/disk/by-path

total 0
lrwxrwxrwx. 1 root root  9 Sep  4 10:02 pci-0000:00:1f.2-ata-2 -> ../../sr0
lrwxrwxrwx. 1 root root  9 Sep  4 10:02 pci-0000:00:1f.2-ata-3 -> ../../sdd
lrwxrwxrwx. 1 root root 10 Sep  4 10:02 pci-0000:00:1f.2-ata-3-part1 -> ../../sdd1
lrwxrwxrwx. 1 root root 10 Sep  4 10:02 pci-0000:00:1f.2-ata-3-part2 -> ../../sdd2
lrwxrwxrwx. 1 root root 10 Sep  4 10:02 pci-0000:00:1f.2-ata-3-part3 -> ../../sdd3
lrwxrwxrwx. 1 root root  9 Sep  4 10:02 pci-0000:03:00.0-ata-4 -> ../../sda
lrwxrwxrwx. 1 root root 10 Sep  4 10:02 pci-0000:03:00.0-ata-4-part1 -> ../../sda1
lrwxrwxrwx. 1 root root 10 Sep  4 10:02 pci-0000:03:00.0-ata-4-part2 -> ../../sda2
lrwxrwxrwx. 1 root root  9 Sep  4 10:02 pci-0000:08:00.0-ata-1 -> ../../sdc
lrwxrwxrwx. 1 root root 10 Sep  4 10:02 pci-0000:08:00.0-ata-1-part1 -> ../../sdc1
lrwxrwxrwx. 1 root root 10 Sep  4 10:02 pci-0000:08:00.0-ata-1-part2 -> ../../sdc2

Merge pull request #1190 from poettering/rework-virt

basic: rework virtualization detection API

sd-login: fix sd_seat_get_active() to return ENODATA

This seems to be an oversight from:
    707b66c66381c899d7ef640e158ffdd5bcff4deb

We have to return ENODATA instead of ENOENT if a requested entry is
non-present. Also fix the call-site in udev to check for these errors.

build: prepare for v226

Bump version info and update NEWS for the upcoming release.

Merge pull request #1201 from torstehu/fix-typo

treewide: fix typos

Merge pull request #1192 from poettering/sd-bus-container

sd-bus: pass container bus errors up to calling process

treewide: fix typos

Merge pull request #1196 from evverx/systemctl-add-consists-of

systemctl: add ConsistsOf as the inverse of PartOf

Merge pull request #1198 from martinpitt/master

cgroup-util: fix devices controller

cgroup-util: fix devices controller

Commit efdb0237 accidentally changed the name of the "devices" cgroup
controller to "device".

Merge pull request #1195 from poettering/nspawn-fixes

Various nspawn fixes

systemctl: add ConsistsOf as the inverse of PartOf

update TODO

NEWS: update contributors list

nspawn: also close uid shift socket in the parent

We should really close all parent sides of our child/parent socket
pairs.

nspawn: short reads do not set errno, hence don't try to print it

inspawn: switch from SOCK_DGRAM to SOCK_SEQPACKET for internal socketpairs

SOCK_DGRAM and SOCK_SEQPACKET have very similar semantics when used with
socketpair(). However, SOCK_SEQPACKET has the advantage of knowing a
hangup concept, since it is inherently connection-oriented.

Since we use socket pairs to communicate between the nspawn main process
and the nspawn child process, where the child might die abnormally it's
interesting to us to learn about this via hangups if the child side of
the pair is closed. Hence, let's switch to SOCK_SEQPACKET for these
internal communication sockets.

Fixes #956.

nspawn: properly propagate errors when we fail to set soemthing up

Merge pull request #1193 from phomes/typos

man: typo fixes

man: typo fixes

machined: improve error message when trying to get a bus in bus-less containers

Now that we get useful error messages from sd-bus for container
connections, let's make use of this and report better errors back to
machined clients.

Fixes #685.

sd-bus: when connecting to a container AF_UNIX bus, return error

When forking of a child process for connecting to a container, pass
the preicse connection error to the calling process.

We already did this correctly for kdbus busses, let's do so for dbus1
busses, too.

Merge pull request #1191 from poettering/nspawn-split

nspawn: split up nspawn.c into multiple smaller .c files

nspawn: sort and clean up included header list

Let's remove unnecessary inclusions, and order the list alphabetically
as suggested in CODING_STYLE now.

nspawn: remove nspawn.h, it's empty now

nspawn: split out --uid= logic into nspawn-setuid.[ch]

nspawn: split out machined registration code to nspawn-register.[ch]

nspawn: split out cgroup related calls into nspawn-cgroup.[ch]

nspawn: split out network related code to nspawn-network.[ch]

nspawn: split all port exposure code into nspawn-expose-port.[ch]

nspawn: split out mount related functions into a new nspawn-mount.c file

Merge pull request #1189 from poettering/unit-start

unit: move "not supported" check after condition check in unit_start()

unit: move "not supported" check after condition check in unit_start()

Make sure we always check conditions before checking whether the unit
type is supported in unit_start(), since condition checks are "clean
errors", while "not supported" errors are fatal.

This cleans up the boot output of systemd in containers, where a lot of
NOTSUPP lines were shown befor this fix.

This partially reverts 8ff4d2ab0d4758e914aea6d86154d85f2b2c787f which
reorder the checks.

basic: rework virtualization detection API

Introduce a proper enum, and don't pass around string ids anymore. This
simplifies things quite a bit, and makes virtualization detection more
similar to architecture detection.

NEWS: fix typos

update NEWS

Merge pull request #1178 from poettering/gpt-auto-fixes

handle LUKS root partitions better in gpt-auto, plus other fixes

Merge pull request #1183 from dvdhrm/cpename

man: clarify wording of os-release.CPE_NAME

Merge pull request #1182 from martinpitt/master

tests: Skip test-cgroup-util test_mask_supported() when not running under systemd

man: clarify wording of os-release.CPE_NAME

We expect the CPE_NAME to be formatted in URI binding syntax. Make that
clear in the documentation. Furthermore, the CPE-spec has been taken over
by NIST, so adjust the links as well.

Reported by: Ben Harris <bjh21@cam.ac.uk>

tests: Skip test-cgroup-util test_mask_supported() when not running under systemd

Commit 5f4c5fef6 introduced this new test case, but this does not work in
build chroots where cgroupfs is not mounted. So skip the test if systemd is not
running.

Merge pull request #1181 from evverx/update-systemd-analyze-completion

shell-completion: update systemd-analyze bash-completion

shell-completion: update systemd-analyze bash-completion

* Change --no-man to --man (see dad29df)
* --{from,to}-pattern require arg

Merge pull request #1179 from poettering/sd-event-sigchld-fix

sd-event: make sure to create a signal queue for the right signal

Merge pull request #1165 from poettering/nspawn-files

various fixes to the core, logind, machined, nspawn

sd-event: make sure to create a signal queue for the right signal

We should never access the "signal" part of the event source unless the
event source is actually for a signal. In this case it's a child pid
handler however, hence make sure to use the right signal.

This is a fix for PR #1177, which in turn was a fix for
9da4cb2be260ed123f2676cb85cb350c527b1492.

systemctl: a number of cleanups regarding error handling in systemctl

gpt-auto: minor simplificatin handling the no-auto GPT flag

Let's query the flags only once, and document why we ignore it for the
ESP.

gpt-auto: try to handle LUKS root partitions better

If the root file system is located on an encrypted root disk, we'll not
find the GPT partition table for it. Let's fix that by following the
slaves/ symlinks in /sys for the device. We only handle devices having
exactly one backing device.

Also see: #1167

Merge pull request #1168 from poettering/readme-kernel-version

README: bump minimal required kernel version

Merge pull request #1177 from phomes/sd-event-bug-fix

sd-event: fix call to event_make_signal_data

NEWS: add more content to both the 225 and 226 NEWS section

sd-event: fix call to event_make_signal_data

This looks like a typo from commit 9da4cb2b where it was added.

Merge pull request #1174 from mbiebl/remove-sphinx-leftover

build-sys: remove sphinx binary from configure summary

Merge pull request #1176 from piotrdrag/master

Updated Polish translation

Updated Polish translation

build-sys: remove sphinx binary from configure summary

We no longer use sphinx as part of the build process so remove it from
the configure summary as well.
This is a leftover from commit 2799e519cabb6dfa99341b9a56ebd4dc2a4ec22a.

README: bump minimal required kernel version

We generally try to support 2y old kernels, which allows us bump the
minimal required version to 3.11 now.

Also, clarify that support for the unified cgroup hierarchy requires 4.2
or newer.

Merge pull request #1153 from evverx/dot-alias-handling

analyze: add "alias" handling to dot subcommand

analyze: add "alias" handling to dot subcommand

`systemd-analyze dot default.target` works fine

Merge pull request #1159 from AnchorCat/polkit-details/v2

Provide unit name and operation in manage-units polkit checks (v2)

Merge pull request #1162 from dvdhrm/bus-recursive-nodes

sd-bus: make introspection data non-recursive

units: make sure that .nspawn files override the default settings in systemd-nspawn@.service

nspawn: add new .nspawn files for container settings

.nspawn fiels are simple settings files that may accompany container
images and directories and contain settings otherwise passed on the
nspawn command line. This provides an efficient way to attach execution
data directly to containers.

Merge pull request #1161 from dvdhrm/include-order2

CODING_STYLE: mandate alphabetical include order (v2)

Merge pull request #1160 from dvdhrm/bus-user

sd-bus: derive uid from cgroup if possible

machine: make sure to call unlockpt() even for local host pty connections

This fixes breakage for local host pty handling, introduced in
395745ba533ac91fe118f43ec83f13a752c0b473.

Fixes #1139

logind: make scope of wall message handling smaller

unit: make unit_can_start() more accurate

This funciton is exposed via CanStart on the bus, and should be as
accurate as possible. Hence: make sure to return false for units of unit
types not supported on the system, and for unit types where
configuration failed to load.

Also see #1105.

core: don't use uninitialized errno

machine: clarify that /var/lib/containers is legacy

sd-bus: make introspection data non-recursive

Currently, our introspection data looks like this:

        <node>
         <interface name="org.freedesktop.DBus.Peer">
           ...
         </interface>
         <interface name="org.freedesktop.DBus.Introspectable">
           ...
         </interface>
         <interface name="org.freedesktop.DBus.Properties">
           ...
         </interface>
         <node name="org"/>
         <node name="org/freedesktop"/>
         <node name="org/freedesktop/login1"/>
         <node name="org/freedesktop/login1/user"/>
         <node name="org/freedesktop/login1/user/self"/>
         <node name="org/freedesktop/login1/user/_1000"/>
         <node name="org/freedesktop/login1/seat"/>
         <node name="org/freedesktop/login1/seat/self"/>
         <node name="org/freedesktop/login1/seat/seat0"/>
         <node name="org/freedesktop/login1/session"/>
         <node name="org/freedesktop/login1/session/self"/>
         <node name="org/freedesktop/login1/session/c1"/>
        </node>

(ordered alphabetically for better visibility)

This is grossly incorrect. The spec says that we're allowed to return
non-directed children, however, it does not allow us to return data
recursively in multiple parents. If we return "org", then we must not
return anything else that starts with "org/".

It is unclear, whether we can include child-nodes as a tree. Moreover, it
is usually not what the caller wants. Hence, this patch changes sd-bus to
never return introspection data recursively. Instead, only a single
child-layer is returned.

This patch relies on enumerators to never return hierarchies. If someone
registers an enumerator via sd_bus_add_enumerator, they better register
sub-enumerators if they support *TRUE* hierarchies. Each enumerator is
treated as a single layer and not filtered.
Enumerators are still allowed to return nested data. However, that data
is still required to be a single hierarchy. For instance, returning
"/org/foo" and "/com/bar" is fine, but including "/com" or "/org" in that
dataset is not.
This should be the default for enumerators and I see no reason to filter
in sd-bus. Moreover, filtering that data-set would require to sort the
strv by path and then do prefix-filtering. This is O(n log n), which
would be fine, but still better to avoid.

Fixes #664.

TODO: update networkd section

Remove two freshly implemented features, and add TSO support as a new
one.

CODING_STYLE: mandate alphabetical include order

systemd-internal headers must not rely on include order. That means, they
either must contain forward-declarations of used types/functions, or they
must include all dependencies on their own. Therefore, there is no reason
to mandate an include order on the call-side.

However, global includes should always be ordered first. We don't want
local definitions to leak into global includes, possible changing their
behavior. Apparently, namespacing is a complex problem that people are
incapable of implementing properly..

Apart from "global before local", there is no reason to mandate a random
include order (which we happen to do right now). Instead, mandate
alphabetical ordering. The current rules do not have any benefit at all.
They neither reduce include-complexity, nor allow easy auditing of
include files. But with alphabetical ordering, we get duplicate-detection
for free, it gets *much much* easier to figure out whether a header is
already included, and it is trivial to add new headers.

sd-bus: derive uid from cgroup if possible

Whenever we run in a user context, sd_bus_{default_user,open_user}() and
friends should always connect to the user-bus of the current context,
instead of deriving the uid from getuid(). This allows us running
programs via sudo/su, without the nasty side-effect of accidentally
connecting to the root user-bus.

This patch enforces the idea of making su/sudo *not* opening sessions by
default. That is, all they do is raising privileges, but keeping
everything set as before. You can still use su/sudo to open real sessions
by requesting a login-session (or loading pam_systemd otherwise).
However, in this case XDG_RUNTIME_DIR= will not be set (as usual in these
cases), hence, you will not be able to connect to *any* user-bus.

Long story short: With this patch applied, both:
        - ./busctl --user
        - sudo ./busctl --user
..will successfully connect to the user-bus of the local user.

Fixes #390.

core: pass details to polkit for some unit actions

The following details are passed:

- unit: the primary name of the unit upon which the action was
        invoked (i.e. after resolving any aliases);
- verb: one of 'start', 'stop', 'reload', 'restart', 'try-restart',
        'reload-or-restart', 'reload-or-try-restart', 'kill',
        'reset-failed', or 'set-property', corresponding to the
        systemctl verb used to invoke the action.

Typical use of these details in a polkit policy rule might be:

  // Allow alice to manage example.service;
  // fall back to implicit authorization otherwise.
  polkit.addRule(function(action, subject) {
      if (action.id == "org.freedesktop.systemd1.manage-units" &&
          action.lookup("unit") == "example.service" &&
          subject.user == "alice") {
          return polkit.Result.YES;
      }
  });

We also supply a custom polkit message that includes the unit's name and
the requested operation.

bus-util: support details in CheckAuthorization calls

Extra details for an action can be supplied when calling polkit's
CheckAuthorization method. Details are a list of key/value string pairs.
Custom policy can use these details when making authorization decisions.

Merge pull request #1140 from poettering/sd-event-signals

A variety of sd-event, sd-login and cgroup fixes

sd-login: minor header commenting improvements

sd-login: add new sd_pid_get_cgroup() API

This adds a new sd_pid_get_cgroup() call to sd-login which may be used
to query the control path of a process. This is useful for programs when
making use of delegation units, in order to figure out which subtree has
been delegated.

In light of the unified control group hierarchy this is finally safe to
do, hence let's add a proper API for it, to make it easier to use this.

Merge pull request #1157 from dvdhrm/logind-wallfix

login: fix NULL-deref on wall_message

Merge pull request #1145 from systemd-mailing-devs/1441372815-12195-1-git-send-email-hdegoede@redhat.com

hwdb: Add Thinkpad T550 / W550s to 70-pointingstick.hwdb

login: fix NULL-deref on wall_message

We treat an empty wall-message equal to a NULL wall-message since:

        commit 5744f59a3ee883ef3a78214bd5236157acdc35ba
        Author: Lennart Poettering <lennart@poettering.net>
        Date:   Fri Sep 4 10:34:47 2015 +0200

            logind: treat an empty wall message like a NULL one

Fix the shutdown scheduler to not deref a NULL pointer, but properly
check for an empty wall-message.

Fixes: #1120

Merge pull request #1150 from evverx/update-systemctl-completion

shell-completion: update systemctl bash completion

Merge pull request #1149 from martinpitt/fix-dhcp-error-codes

networkd: adjust error codes for nonexisting DHCP data

Merge pull request #1146 from martinpitt/master

tests: Skip tests which need to access /sys/fs/cgroup if that is not …

NEWS: add entries for v226

Initial set of features for the upcoming v226 release next week. This is
mostly about the unified cgroup hierarchy and DHCP.

shell-completion: update systemctl bash completion

Many new options have been added since the bash completion was last
updated.

networkd: adjust error codes for nonexisting DHCP data

Commit 0339cd770 changed libsystemd-network's error code for missing DHCP lease
data from ENOENT to ENODATA. Adjust networkd accordingly.

This fixes interfaces being stuck in "degraded/configuring" mode forever.

https://github.com/systemd/systemd/issues/1147

tests: Skip tests which need to access /sys/fs/cgroup if that is not available

Commit efdb023 ("core: unified cgroup hierarchy support") introduced a new
error ENOEXEC in cg_unified() if /sys/fs/cgroup/ is not available. Adjust the
"skip" checks in various tests accordingly.

Add a corresponding "skip" check to test-bus-creds as well, as
sd_bus_creds_new_from_pid() now calls cg_unified() as well.

This re-fixes "make check" in build chroots without /sys/fs/cgroup.

https://github.com/systemd/systemd/issues/1132

hwdb: Add Thinkpad T550 / W550s to 70-pointingstick.hwdb

Like many other recent thinkpads the factory default pointingstick
sensitivity on these devices is quite low, making the pointingstick
very slow in moving the cursor.

This extends the existing hwdb rules for tweaking the sensitivity to
also apply to the T550 / W550s models.

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

Merge pull request #1142 from dvdhrm/proxy-nofile

bus-proxy: increase NOFILE limit

Merge pull request #1141 from poettering/logind-fixes

Various logind fixes

bus-proxy: increase NOFILE limit

The bus-proxy manages the kdbus connections of all users on the system
(regarding the system bus), hence, it needs an elevated NOFILE.
Otherwise, a single user can trigger ENFILE by opening NOFILE connections
to the bus-proxy.

Note that the bus-proxy still does per-user accounting, indirectly via
the proxy/fake API of kdbus. Hence, the effective per-user limit is not
raised by this. However, we now prevent one user from consuming the whole
FD limit of the shared proxy.

Also note that there is no *perfect* way to set this. The proxy is a
shared object, so it needs a larger NOFILE limit than the highest limit
of all users. This limit can be changed dynamically, though. Hence, we
cannot protect against it. However, a raised NOFILE limit is a privilege,
so we just treat it as such and basically allow these privileged users to
be able to consume more resources than normal users (and, maybe, cause
some limits to be exceeded by this).

Right now, kdbus hard-codes 1024 max connections per user on each bus.
However, we *must not* rely on this. This limits could be easily dropped
entirely, as the NOFILE limit is a suitable limit on its on.

logind: when parsing a boolean via sd-bus the type must be "int"

And not bool.

logind: treat an empty wall message like a NULL one

cgroups: delegation to unprivileged services is safe in the unified hierarchy

Delegation to unpriviliged processes is safe in the unified hierarchy,
hence allow it. This has the benefit of permitting "systemd --user"
instances to further partition their resources between user services.

sd-login: rework error handling

Makre sure we always return sensible errors for the various, following
the same rules, and document them in a comment in sd-login.c. Also,
update all relevant man pages accordingly.

sd-event: make sure RT signals are not dropped

RT signals operate in a queue, and we should be careful to never merge
two queued signals into one. Hence, makes sure we only ever dequeue a
single signal at a time and leave the remaining ones queued in the
signalfd. In order to implement correct priorities for the signals
introduce one signalfd per priority, so that we only process the highest
priority signal at a time.

test: add one more test case for parse_pid()